On 2 May 2014 16:58, Geert Uytterhoeven wrote:
> Hi Ulf, Tomasz,
>
> On Fri, May 2, 2014 at 10:13 AM, Ulf Hansson wrote:
> +static int of_clk_register(struct device *dev, struct clk *clk)
> +{
> + int error;
> +
> + if (!dev->pm_domain) {
> +
On 2 May 2014 16:35, Geert Uytterhoeven wrote:
> Hi Ulf,
>
> On Fri, May 2, 2014 at 10:56 AM, Ulf Hansson wrote:
>>> +static int of_clk_pm_runtime_suspend(struct device *dev)
>>> +{
>>> + int ret;
>>> +
>>> + ret = pm_generic_runtime_suspend(dev);
>>> + if (ret)
>>> +
On 2 May 2014 16:35, Geert Uytterhoeven ge...@linux-m68k.org wrote:
Hi Ulf,
On Fri, May 2, 2014 at 10:56 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
+static int of_clk_pm_runtime_suspend(struct device *dev)
+{
+ int ret;
+
+ ret = pm_generic_runtime_suspend(dev);
+ if
On 2 May 2014 16:58, Geert Uytterhoeven ge...@linux-m68k.org wrote:
Hi Ulf, Tomasz,
On Fri, May 2, 2014 at 10:13 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
+static int of_clk_register(struct device *dev, struct clk *clk)
+{
+ int error;
+
+ if (!dev-pm_domain) {
+
Hi Ulf, Tomasz,
On Fri, May 2, 2014 at 10:13 AM, Ulf Hansson wrote:
+static int of_clk_register(struct device *dev, struct clk *clk)
+{
+ int error;
+
+ if (!dev->pm_domain) {
+ error = pm_clk_create(dev);
+ if (error)
Hi Ulf,
On Fri, May 2, 2014 at 10:56 AM, Ulf Hansson wrote:
>> +static int of_clk_pm_runtime_suspend(struct device *dev)
>> +{
>> + int ret;
>> +
>> + ret = pm_generic_runtime_suspend(dev);
>> + if (ret)
>> + return ret;
>> +
>> + ret = pm_clk_suspend(dev);
Hi Geert,
Some more review comments.
> +
> +
> +#ifdef CONFIG_PM_RUNTIME
> +
> +static int of_clk_pm_runtime_suspend(struct device *dev)
> +{
> + int ret;
> +
> + ret = pm_generic_runtime_suspend(dev);
> + if (ret)
> + return ret;
> +
> + ret =
>>
>>
>> Normally I don't think it's a good idea to "automatically" manage
>> clocks from PM core or any other place but from the driver (and
>> possibly the subsystem).
>>
>> The reason is simply that we hide things that normally is supposed to
>> be handled by the driver. Typically a cross SOC
Normally I don't think it's a good idea to automatically manage
clocks from PM core or any other place but from the driver (and
possibly the subsystem).
The reason is simply that we hide things that normally is supposed to
be handled by the driver. Typically a cross SOC driver should work
Hi Geert,
Some more review comments.
+
+
+#ifdef CONFIG_PM_RUNTIME
+
+static int of_clk_pm_runtime_suspend(struct device *dev)
+{
+ int ret;
+
+ ret = pm_generic_runtime_suspend(dev);
+ if (ret)
+ return ret;
+
+ ret = pm_clk_suspend(dev);
Hi Ulf,
On Fri, May 2, 2014 at 10:56 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
+static int of_clk_pm_runtime_suspend(struct device *dev)
+{
+ int ret;
+
+ ret = pm_generic_runtime_suspend(dev);
+ if (ret)
+ return ret;
+
+ ret =
Hi Ulf, Tomasz,
On Fri, May 2, 2014 at 10:13 AM, Ulf Hansson ulf.hans...@linaro.org wrote:
+static int of_clk_register(struct device *dev, struct clk *clk)
+{
+ int error;
+
+ if (!dev-pm_domain) {
+ error = pm_clk_create(dev);
+ if (error)
+
Hi Grant,
On Thu, May 1, 2014 at 3:56 PM, Grant Likely wrote:
> On Thu, May 1, 2014 at 2:41 PM, Geert Uytterhoeven
> wrote:
>> On Thu, May 1, 2014 at 10:03 AM, Grant Likely
>> wrote:
>>> On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven
>>> wrote:
On Tue, Apr 29, 2014 at 3:16 PM,
On Thu, May 1, 2014 at 2:41 PM, Geert Uytterhoeven wrote:
> Hi Grant,
>
> On Thu, May 1, 2014 at 10:03 AM, Grant Likely
> wrote:
>> On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven
>> wrote:
>>> On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely
>>> wrote:
>>> > I also don't like that it
Hi Grant,
On Thu, May 1, 2014 at 10:03 AM, Grant Likely wrote:
> On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven
> wrote:
>> On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely
>> wrote:
>> > I also don't like that it tries to set up every clock, but there is no
>> > guarantee that the
On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven
wrote:
> Hi Grant,
>
> On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely
> wrote:
> > On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman wrote:
> >> Geert Uytterhoeven writes:
> >>
> >> > When adding a device from DT, check if its clocks are
On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven ge...@linux-m68k.org
wrote:
Hi Grant,
On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely grant.lik...@secretlab.ca
wrote:
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven
Hi Grant,
On Thu, May 1, 2014 at 10:03 AM, Grant Likely grant.lik...@secretlab.ca wrote:
On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven ge...@linux-m68k.org
wrote:
On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely grant.lik...@secretlab.ca
wrote:
I also don't like that it tries to set
On Thu, May 1, 2014 at 2:41 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote:
Hi Grant,
On Thu, May 1, 2014 at 10:03 AM, Grant Likely grant.lik...@secretlab.ca
wrote:
On Wed, 30 Apr 2014 23:54:37 +0200, Geert Uytterhoeven
ge...@linux-m68k.org wrote:
On Tue, Apr 29, 2014 at 3:16 PM, Grant
Hi Grant,
On Thu, May 1, 2014 at 3:56 PM, Grant Likely grant.lik...@secretlab.ca wrote:
On Thu, May 1, 2014 at 2:41 PM, Geert Uytterhoeven ge...@linux-m68k.org
wrote:
On Thu, May 1, 2014 at 10:03 AM, Grant Likely grant.lik...@secretlab.ca
wrote:
On Wed, 30 Apr 2014 23:54:37 +0200, Geert
Hi Laurent,
On Wed, Apr 30, 2014 at 11:23 PM, Laurent Pinchart
wrote:
> On Thursday 24 April 2014 15:11:24 Ulf Hansson wrote:
>> On 24 April 2014 12:13, Geert Uytterhoeven wrote:
>> > When adding a device from DT, check if its clocks are suitable for Runtime
>> > PM, and register them with the
Hi Grant,
On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely wrote:
> On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman wrote:
>> Geert Uytterhoeven writes:
>>
>> > When adding a device from DT, check if its clocks are suitable for Runtime
>> > PM, and register them with the PM core.
>> > If Runtime
Hi Kevin,
On Sat, Apr 26, 2014 at 1:44 AM, Kevin Hilman wrote:
> Geert Uytterhoeven writes:
>> When adding a device from DT, check if its clocks are suitable for Runtime
>> PM, and register them with the PM core.
>> If Runtime PM is disabled, just enable the clock.
>>
>> This allows the PM core
On 30/04/14 14:25, Laurent Pinchart wrote:
On Tuesday 29 April 2014 14:16:10 Grant Likely wrote:
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman wrote:
Geert Uytterhoeven writes:
When adding a device from DT, check if its clocks are suitable for
Runtime PM, and register them with the PM
On Tuesday 29 April 2014 14:16:10 Grant Likely wrote:
> On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman wrote:
> > Geert Uytterhoeven writes:
> > > When adding a device from DT, check if its clocks are suitable for
> > > Runtime PM, and register them with the PM core.
> > > If Runtime PM is
Hi Ulf and Geert,
On Thursday 24 April 2014 15:11:24 Ulf Hansson wrote:
> On 24 April 2014 12:13, Geert Uytterhoeven wrote:
> > When adding a device from DT, check if its clocks are suitable for Runtime
> > PM, and register them with the PM core.
> > If Runtime PM is disabled, just enable the
Hi Ulf and Geert,
On Thursday 24 April 2014 15:11:24 Ulf Hansson wrote:
On 24 April 2014 12:13, Geert Uytterhoeven geert+rene...@glider.be wrote:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled,
On Tuesday 29 April 2014 14:16:10 Grant Likely wrote:
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for
Runtime PM, and register them with the PM
On 30/04/14 14:25, Laurent Pinchart wrote:
On Tuesday 29 April 2014 14:16:10 Grant Likely wrote:
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for
Hi Kevin,
On Sat, Apr 26, 2014 at 1:44 AM, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the
Hi Grant,
On Tue, Apr 29, 2014 at 3:16 PM, Grant Likely grant.lik...@secretlab.ca wrote:
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for Runtime
Hi Laurent,
On Wed, Apr 30, 2014 at 11:23 PM, Laurent Pinchart
laurent.pinch...@ideasonboard.com wrote:
On Thursday 24 April 2014 15:11:24 Ulf Hansson wrote:
On 24 April 2014 12:13, Geert Uytterhoeven geert+rene...@glider.be wrote:
When adding a device from DT, check if its clocks are
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman wrote:
> Geert Uytterhoeven writes:
>
> > When adding a device from DT, check if its clocks are suitable for Runtime
> > PM, and register them with the PM core.
> > If Runtime PM is disabled, just enable the clock.
> >
> > This allows the PM core
On Fri, 25 Apr 2014 16:44:58 -0700, Kevin Hilman khil...@linaro.org wrote:
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the
On 24.04.2014 15:11, Ulf Hansson wrote:
On 24 April 2014 12:13, Geert Uytterhoeven wrote:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows the PM core to
Geert Uytterhoeven writes:
> When adding a device from DT, check if its clocks are suitable for Runtime
> PM, and register them with the PM core.
> If Runtime PM is disabled, just enable the clock.
>
> This allows the PM core to automatically manage gate clocks of devices for
> Runtime PM.
Geert Uytterhoeven geert+rene...@glider.be writes:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows the PM core to automatically manage gate clocks of devices for
On 24.04.2014 15:11, Ulf Hansson wrote:
On 24 April 2014 12:13, Geert Uytterhoeven geert+rene...@glider.be wrote:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows
Hi Ulf,
On Thu, Apr 24, 2014 at 3:11 PM, Ulf Hansson wrote:
>> +static int of_clk_register(struct device *dev, struct clk *clk)
>> +{
>> + int error;
>> +
>> + if (!dev->pm_domain) {
>> + error = pm_clk_create(dev);
>> + if (error)
>> +
On 24 April 2014 12:13, Geert Uytterhoeven wrote:
> When adding a device from DT, check if its clocks are suitable for Runtime
> PM, and register them with the PM core.
> If Runtime PM is disabled, just enable the clock.
>
> This allows the PM core to automatically manage gate clocks of devices
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows the PM core to automatically manage gate clocks of devices for
Runtime PM.
Signed-off-by: Geert Uytterhoeven
---
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows the PM core to automatically manage gate clocks of devices for
Runtime PM.
Signed-off-by: Geert Uytterhoeven
On 24 April 2014 12:13, Geert Uytterhoeven geert+rene...@glider.be wrote:
When adding a device from DT, check if its clocks are suitable for Runtime
PM, and register them with the PM core.
If Runtime PM is disabled, just enable the clock.
This allows the PM core to automatically manage gate
Hi Ulf,
On Thu, Apr 24, 2014 at 3:11 PM, Ulf Hansson ulf.hans...@linaro.org wrote:
+static int of_clk_register(struct device *dev, struct clk *clk)
+{
+ int error;
+
+ if (!dev-pm_domain) {
+ error = pm_clk_create(dev);
+ if (error)
+
44 matches
Mail list logo