Any comments?
Thanks.
Sonic
On Wed, Dec 2, 2009 at 5:08 PM, sonic zhang <sonic.adi@xxxxxxxxx> wrote:
> Struct dev_pm_ops is not configured in current i2c bus type. i2c drivers
> only depends on suspend/resume entries in struct dev_pm_ops are not
> informed of PM suspend and resume events by i2c framework.
>
> Signed-off-by: Sonic Zhang <sonic.zhang@xxxxxxxxxx>
> ---
> drivers/i2c/i2c-core.c | 39 +++++++++++++++++++++++++++++++++++++++
> 1 files changed, 39 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
> index 2965043..9713c0d 100644
> --- a/drivers/i2c/i2c-core.c
> +++ b/drivers/i2c/i2c-core.c
> @@ -155,6 +155,39 @@ static void i2c_device_shutdown(struct device *dev)
> driver->shutdown(client);
> }
>
> +#ifdef CONFIG_SUSPEND
> +static int i2c_device_pm_suspend(struct device *dev)
> +{
> + struct i2c_driver *driver;
> + struct dev_pm_ops *pm;
> +
> + if (!dev->driver)
> + return 0;
> + driver = to_i2c_driver(dev->driver);
> + pm = driver->driver.pm;
> + if (!pm || !pm->suspend)
> + return 0;
> + return pm->suspend(dev);
> +}
> +
> +static int i2c_device_pm_resume(struct device *dev)
> +{
> + struct i2c_driver *driver;
> + struct dev_pm_ops *pm;
> +
> + if (!dev->driver)
> + return 0;
> + driver = to_i2c_driver(dev->driver);
> + pm = driver->driver.pm;
> + if (!pm || !pm->resume)
> + return 0;
> + return pm->resume(dev);
> +}
> +#else
> +# define i2c_device_pm_suspend NULL
> +# define i2c_device_pm_resume NULL
> +#endif
> +
> static int i2c_device_suspend(struct device *dev, pm_message_t mesg)
> {
> struct i2c_client *client = i2c_verify_client(dev);
> @@ -219,6 +252,11 @@ static const struct attribute_group *i2c_dev_attr_groups[] = {
> NULL
> };
>
> +static struct dev_pm_ops i2c_device_pm_ops = {
> + .suspend = i2c_device_pm_suspend,
> + .resume = i2c_device_pm_resume,
> +};
> +
> struct bus_type i2c_bus_type = {
> .name = "i2c",
> .match = i2c_device_match,
> @@ -227,6 +265,7 @@ struct bus_type i2c_bus_type = {
> .shutdown = i2c_device_shutdown,
> .suspend = i2c_device_suspend,
> .resume = i2c_device_resume,
> + .pm = &i2c_device_pm_ops,
> };
> EXPORT_SYMBOL_GPL(i2c_bus_type);
>
> --
> 1.6.0
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
