On Friday 27 July 2012 01:10 AM, Stephen Warren wrote:
From: Gyungoh Yoo<jack....@maxim-ic.com>


+static int max8907_irq_set_wake(struct irq_data *data, unsigned int on)
+{
+       /* Everything happens in max8907_irq_sync_unlock */
+

Probably you need to call the wake_enable of irq which goes to the cpu here.


+       if (device_may_wakeup(chip->dev))
+               enable_irq_wake(i2c->irq);
+       else
+               disable_irq(i2c->irq);

I think lets sub devices decide the wakeups rather than it is done by the core.


+       max8907->i2c_rtc = i2c_new_dummy(i2c->adapter, MAX8907_RTC_I2C_ADDR);
+       if (!max8907->i2c_rtc) {
+               ret = -ENOMEM;
+               goto err_dummy_rtc;
+       }
+       i2c_set_clientdata(max8907->i2c_rtc, max8907);
+       max8907->regmap_rtc = devm_regmap_init_i2c(i2c,
+&max8907_regmap_rtc_config);
the argiment should be  max8907->i2c_rtc rather than i2c here.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to