Currently the i2c driver calls the pm_runtime_enable and never the disable. This may cause a warning when pm_runtime_enable checks for the count match.Attempting to fix the same by calling pm_runtime_disable in the error and the remove path.
Cc: Kevin Hilman <khil...@ti.com> Cc: Rajendra Nayak <rna...@ti.com> Cc: Ben Dooks <ben-li...@fluff.org> Signed-off-by: Shubhrajyoti D <shubhrajy...@ti.com> --- previous discurssion http://marc.info/?l=linux-arm-kernel&m=132628828411133&w=2 drivers/i2c/busses/i2c-omap.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c index c3ca7d9..cb9d4f8 100644 --- a/drivers/i2c/busses/i2c-omap.c +++ b/drivers/i2c/busses/i2c-omap.c @@ -1122,6 +1122,7 @@ err_unuse_clocks: omap_i2c_write_reg(dev, OMAP_I2C_CON_REG, 0); pm_runtime_put(dev->dev); iounmap(dev->base); + pm_runtime_disable(&pdev->dev); err_free_mem: platform_set_drvdata(pdev, NULL); kfree(dev); @@ -1137,6 +1138,7 @@ omap_i2c_remove(struct platform_device *pdev) struct omap_i2c_dev *dev = platform_get_drvdata(pdev); struct resource *mem; + pm_runtime_disable(&pdev->dev); platform_set_drvdata(pdev, NULL); free_irq(dev->irq, dev); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
