On Fri 21 Feb 00:16 PST 2014, Maxime Ripard wrote:

Hi Bjorn,

On Thu, Feb 20, 2014 at 04:38:10PM -0800, Bjorn Andersson wrote:

quoted

+static int qup_i2c_probe(struct platform_device *pdev)
+{

[ snip ]

quoted

+
+	qup_i2c_enable_clocks(qup);
+

[ snip ]

quoted

+
+	pm_runtime_set_autosuspend_delay(qup->dev, MSEC_PER_SEC);
+	pm_runtime_use_autosuspend(qup->dev);
+	pm_runtime_enable(qup->dev);

Since the device is already woken up, you probably need to call
pm_runtime_set_active here.

Thanks.

Maxime

Regards,
Bjorn