From: Wolfram Sang <wsa@the-dreams.de> To: Bibby Hsieh <bibby.hsieh@mediatek.com> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>, linux-i2c@vger.kernel.org, tfiga@chromium.org, drinkcat@chromium.org, srv_heupstream@mediatek.com, robh+dt@kernel.org, mark.rutland@arm.com, devicetree@vger.kernel.org Subject: Re: [PATCH v10 4/4] i2c: core: support bus regulator controlling in adapter Date: Tue, 28 Jan 2020 07:36:33 +0100 [thread overview] Message-ID: <20200128063633.GA1088@ninjato> (raw) In-Reply-To: <20200116025637.3524-5-bibby.hsieh@mediatek.com> [-- Attachment #1: Type: text/plain, Size: 3962 bytes --] On Thu, Jan 16, 2020 at 10:56:37AM +0800, Bibby Hsieh wrote: > Although in the most platforms, the bus power of i2c > are alway on, some platforms disable the i2c bus power > in order to meet low power request. > > We get and enable bulk regulator in i2c adapter device. > > Signed-off-by: Bibby Hsieh <bibby.hsieh@mediatek.com> ... > +#ifdef CONFIG_PM_SLEEP > +static int i2c_resume(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + if (pm_runtime_status_suspended(&adap->dev)) { > + err = regulator_enable(adap->bus_reg); > + if (err) > + return err; > + } > + > + return pm_generic_resume(dev); > +} > + > +static int i2c_suspend(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + if (!pm_runtime_status_suspended(&adap->dev)) { > + err = regulator_disable(adap->bus_reg); > + if (err) > + return err; > + } > + > + return pm_generic_suspend(dev); > +} > +#endif > + > +#ifdef CONFIG_PM > +static int i2c_runtime_resume(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + err = regulator_enable(adap->bus_reg); > + if (err) > + return err; > + > + return pm_generic_runtime_resume(dev); > +} > + > +static int i2c_runtime_suspend(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + err = pm_generic_runtime_suspend(dev); > + if (err) > + return err; > + > + return regulator_disable(adap->bus_reg); > +} > +#endif > + > +static const struct dev_pm_ops i2c_device_pm = { > + SET_SYSTEM_SLEEP_PM_OPS(i2c_suspend, i2c_resume) > + SET_RUNTIME_PM_OPS(i2c_runtime_suspend, i2c_runtime_resume, NULL) > +}; > + It looks good to me, yet I am not a PM expert. An ack from someone more into that topic would be great, if possible. > static void i2c_device_shutdown(struct device *dev) > { > struct i2c_client *client = i2c_verify_client(dev); > @@ -488,6 +563,7 @@ struct bus_type i2c_bus_type = { > .probe = i2c_device_probe, > .remove = i2c_device_remove, > .shutdown = i2c_device_shutdown, > + .pm = &i2c_device_pm, > }; > EXPORT_SYMBOL_GPL(i2c_bus_type); > > @@ -1351,6 +1427,11 @@ static int i2c_register_adapter(struct i2c_adapter *adap) > goto out_reg; > > dev_dbg(&adap->dev, "adapter [%s] registered\n", adap->name); > + adap->bus_reg = devm_regulator_get(&adap->dev, "bus"); > + if (IS_ERR(adap->bus_reg)) { > + res = PTR_ERR(adap->bus_reg); > + goto out_reg; > + } > > pm_runtime_no_callbacks(&adap->dev); > pm_suspend_ignore_children(&adap->dev, true); > diff --git a/include/linux/i2c.h b/include/linux/i2c.h > index d2f786706657..833b81a680da 100644 > --- a/include/linux/i2c.h > +++ b/include/linux/i2c.h > @@ -15,6 +15,7 @@ > #include <linux/device.h> /* for struct device */ > #include <linux/sched.h> /* for completion */ > #include <linux/mutex.h> > +#include <linux/regulator/consumer.h> > #include <linux/rtmutex.h> > #include <linux/irqdomain.h> /* for Host Notify IRQ */ > #include <linux/of.h> /* for struct device_node */ > @@ -330,6 +331,7 @@ struct i2c_client { > int init_irq; /* irq set at initialization */ > int irq; /* irq issued by device */ > struct list_head detected; > + > #if IS_ENABLED(CONFIG_I2C_SLAVE) > i2c_slave_cb_t slave_cb; /* callback for slave mode */ > #endif > @@ -723,6 +725,7 @@ struct i2c_adapter { > const struct i2c_adapter_quirks *quirks; > > struct irq_domain *host_notify_domain; > + struct regulator *bus_reg; "bus_regulator" please. "reg" is ambigious with "register". And what Tomasz said, of course. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org> To: Bibby Hsieh <bibby.hsieh-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> Cc: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>, linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, tfiga-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, drinkcat-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Subject: Re: [PATCH v10 4/4] i2c: core: support bus regulator controlling in adapter Date: Tue, 28 Jan 2020 07:36:33 +0100 [thread overview] Message-ID: <20200128063633.GA1088@ninjato> (raw) In-Reply-To: <20200116025637.3524-5-bibby.hsieh-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> [-- Attachment #1: Type: text/plain, Size: 3989 bytes --] On Thu, Jan 16, 2020 at 10:56:37AM +0800, Bibby Hsieh wrote: > Although in the most platforms, the bus power of i2c > are alway on, some platforms disable the i2c bus power > in order to meet low power request. > > We get and enable bulk regulator in i2c adapter device. > > Signed-off-by: Bibby Hsieh <bibby.hsieh-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org> ... > +#ifdef CONFIG_PM_SLEEP > +static int i2c_resume(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + if (pm_runtime_status_suspended(&adap->dev)) { > + err = regulator_enable(adap->bus_reg); > + if (err) > + return err; > + } > + > + return pm_generic_resume(dev); > +} > + > +static int i2c_suspend(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + if (!pm_runtime_status_suspended(&adap->dev)) { > + err = regulator_disable(adap->bus_reg); > + if (err) > + return err; > + } > + > + return pm_generic_suspend(dev); > +} > +#endif > + > +#ifdef CONFIG_PM > +static int i2c_runtime_resume(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + err = regulator_enable(adap->bus_reg); > + if (err) > + return err; > + > + return pm_generic_runtime_resume(dev); > +} > + > +static int i2c_runtime_suspend(struct device *dev) > +{ > + struct i2c_client *client = i2c_verify_client(dev); > + struct i2c_adapter *adap = client->adapter; > + int err; > + > + err = pm_generic_runtime_suspend(dev); > + if (err) > + return err; > + > + return regulator_disable(adap->bus_reg); > +} > +#endif > + > +static const struct dev_pm_ops i2c_device_pm = { > + SET_SYSTEM_SLEEP_PM_OPS(i2c_suspend, i2c_resume) > + SET_RUNTIME_PM_OPS(i2c_runtime_suspend, i2c_runtime_resume, NULL) > +}; > + It looks good to me, yet I am not a PM expert. An ack from someone more into that topic would be great, if possible. > static void i2c_device_shutdown(struct device *dev) > { > struct i2c_client *client = i2c_verify_client(dev); > @@ -488,6 +563,7 @@ struct bus_type i2c_bus_type = { > .probe = i2c_device_probe, > .remove = i2c_device_remove, > .shutdown = i2c_device_shutdown, > + .pm = &i2c_device_pm, > }; > EXPORT_SYMBOL_GPL(i2c_bus_type); > > @@ -1351,6 +1427,11 @@ static int i2c_register_adapter(struct i2c_adapter *adap) > goto out_reg; > > dev_dbg(&adap->dev, "adapter [%s] registered\n", adap->name); > + adap->bus_reg = devm_regulator_get(&adap->dev, "bus"); > + if (IS_ERR(adap->bus_reg)) { > + res = PTR_ERR(adap->bus_reg); > + goto out_reg; > + } > > pm_runtime_no_callbacks(&adap->dev); > pm_suspend_ignore_children(&adap->dev, true); > diff --git a/include/linux/i2c.h b/include/linux/i2c.h > index d2f786706657..833b81a680da 100644 > --- a/include/linux/i2c.h > +++ b/include/linux/i2c.h > @@ -15,6 +15,7 @@ > #include <linux/device.h> /* for struct device */ > #include <linux/sched.h> /* for completion */ > #include <linux/mutex.h> > +#include <linux/regulator/consumer.h> > #include <linux/rtmutex.h> > #include <linux/irqdomain.h> /* for Host Notify IRQ */ > #include <linux/of.h> /* for struct device_node */ > @@ -330,6 +331,7 @@ struct i2c_client { > int init_irq; /* irq set at initialization */ > int irq; /* irq issued by device */ > struct list_head detected; > + > #if IS_ENABLED(CONFIG_I2C_SLAVE) > i2c_slave_cb_t slave_cb; /* callback for slave mode */ > #endif > @@ -723,6 +725,7 @@ struct i2c_adapter { > const struct i2c_adapter_quirks *quirks; > > struct irq_domain *host_notify_domain; > + struct regulator *bus_reg; "bus_regulator" please. "reg" is ambigious with "register". And what Tomasz said, of course. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2020-01-28 6:36 UTC|newest] Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-16 2:56 [PATCH v10 0/4] add power control in i2c and at24 Bibby Hsieh 2020-01-16 2:56 ` Bibby Hsieh 2020-01-16 2:56 ` [PATCH v10 1/4] dt-binding: eeprom: at24: add vcc-supply property Bibby Hsieh 2020-01-16 2:56 ` Bibby Hsieh 2020-01-23 11:56 ` Bartosz Golaszewski 2020-01-23 11:56 ` Bartosz Golaszewski 2020-01-16 2:56 ` [PATCH v10 2/4] dt-binding: i2c: add bus-supply property Bibby Hsieh 2020-01-16 2:56 ` Bibby Hsieh 2020-01-16 2:56 ` [PATCH v10 3/4] misc: eeprom: at24: support pm_runtime control Bibby Hsieh 2020-01-16 2:56 ` Bibby Hsieh 2020-01-23 11:56 ` Bartosz Golaszewski 2020-01-16 2:56 ` [PATCH v10 4/4] i2c: core: support bus regulator controlling in adapter Bibby Hsieh 2020-01-16 2:56 ` Bibby Hsieh 2020-01-28 3:57 ` Tomasz Figa 2020-01-28 3:57 ` Tomasz Figa 2020-01-28 6:36 ` Wolfram Sang [this message] 2020-01-28 6:36 ` Wolfram Sang 2020-01-28 7:32 ` Tomasz Figa 2020-01-28 7:32 ` Tomasz Figa 2020-01-28 7:31 ` Tomasz Figa 2020-02-17 9:45 ` Tomasz Figa 2020-02-17 9:45 ` Tomasz Figa 2020-02-21 3:22 ` Bibby Hsieh 2020-02-21 3:22 ` Bibby Hsieh 2020-02-24 22:28 ` Rafael J. Wysocki
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200128063633.GA1088@ninjato \ --to=wsa@the-dreams.de \ --cc=bgolaszewski@baylibre.com \ --cc=bibby.hsieh@mediatek.com \ --cc=devicetree@vger.kernel.org \ --cc=drinkcat@chromium.org \ --cc=linux-i2c@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=robh+dt@kernel.org \ --cc=srv_heupstream@mediatek.com \ --cc=tfiga@chromium.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.