All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Gerlach <d-gerlach@ti.com>
To: Santosh Shilimkar <ssantosh@kernel.org>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Kevin Hilman <khilman@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <linux-pm@vger.kernel.org>,
	<devicetree@vger.kernel.org>, Nishanth Menon <nm@ti.com>,
	Keerthy <j-keerthy@ti.com>,
	Russell King <rmk+kernel@armlinux.org.uk>,
	Tero Kristo <t-kristo@ti.com>,
	Sudeep Holla <sudeep.holla@arm.com>
Subject: Re: [PATCH v4 0/5] ARM: K2G: Add support for TI-SCI Generic PM Domains
Date: Thu, 16 Mar 2017 13:31:17 -0500	[thread overview]
Message-ID: <58CAD9F5.7030904@ti.com> (raw)
In-Reply-To: <2133897.t9KqoeriS0@aspire.rjw.lan>

Santosh,
On 03/12/2017 12:02 PM, Rafael J. Wysocki wrote:
> On Tuesday, March 07, 2017 04:22:29 AM Dave Gerlach wrote:
>> Hi,
>> This is v4 of the series to add support for TI-SCI Generic PM Domains.
>> Previous versions can be found here:
>>
>> v3: https://www.spinics.net/lists/kernel/msg2413975.html
>> v2: https://www.spinics.net/lists/kernel/msg2364612.html
>> v1: http://www.spinics.net/lists/arm-kernel/msg525204.html
>>
>> After much debate I have returned to using the phandle cell to pass the SCI ID
>> rather than the separate "ti,sci-id" property that was not very popular. In
>> order to do this I needed to make a change to the genpd framework which can be
>> seen in patch 2 and should benefit others that have client . Rather than
>> checking for zero phandle args and failing if any are present the
>> of_genpd_add_provider_simple call does not check at all and instead leaves
>> parsing and interpretation up to the platform genpd driver.
>>
>> This allows the ti_sci_pm_domains driver to parse the phandle and use the first
>> phandle cell as the sci-id rather than getting it from a separate property.
>>
>> Besides that none of the original patches have changed apart from updating the
>> year to 2017 in patches 3 and 4 and of course a small update to patch 4 to let
>> the ti_sci_pm_domains parse the phandle and extract the sci-id rather than
>> looking for the rejected ti,sci-id property.
>>
>> I did not update the "power-domain" binding document because in regards to
>> #power-domain-cells it already states "can be any value as specified by device
>> tree binding documentation of particular provider" which I think already
>> describes the change in patch 2.
> 
> I'm assuming that this will go in through the arm-soc tree.

Assuming that Rob is OK with the v5 of patch 3 that I just sent in response to
the v4 patch email, can this series go through you? We should be good to go with
v4 of the series + the small v5 update to patch 3 if Rob is ok with it.

Regards,
Dave

> 
> Thanks,
> Rafael
> 

WARNING: multiple messages have this Message-ID (diff)
From: Dave Gerlach <d-gerlach-l0cyMroinI0@public.gmane.org>
To: Santosh Shilimkar <ssantosh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: "Rafael J. Wysocki" <rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org>,
	Ulf Hansson <ulf.hansson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Kevin Hilman <khilman-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Nishanth Menon <nm-l0cyMroinI0@public.gmane.org>,
	Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>,
	Russell King <rmk+kernel-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>,
	Tero Kristo <t-kristo-l0cyMroinI0@public.gmane.org>,
	Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>
Subject: Re: [PATCH v4 0/5] ARM: K2G: Add support for TI-SCI Generic PM Domains
Date: Thu, 16 Mar 2017 13:31:17 -0500	[thread overview]
Message-ID: <58CAD9F5.7030904@ti.com> (raw)
In-Reply-To: <2133897.t9KqoeriS0-yvgW3jdyMHm1GS7QM15AGw@public.gmane.org>

Santosh,
On 03/12/2017 12:02 PM, Rafael J. Wysocki wrote:
> On Tuesday, March 07, 2017 04:22:29 AM Dave Gerlach wrote:
>> Hi,
>> This is v4 of the series to add support for TI-SCI Generic PM Domains.
>> Previous versions can be found here:
>>
>> v3: https://www.spinics.net/lists/kernel/msg2413975.html
>> v2: https://www.spinics.net/lists/kernel/msg2364612.html
>> v1: http://www.spinics.net/lists/arm-kernel/msg525204.html
>>
>> After much debate I have returned to using the phandle cell to pass the SCI ID
>> rather than the separate "ti,sci-id" property that was not very popular. In
>> order to do this I needed to make a change to the genpd framework which can be
>> seen in patch 2 and should benefit others that have client . Rather than
>> checking for zero phandle args and failing if any are present the
>> of_genpd_add_provider_simple call does not check at all and instead leaves
>> parsing and interpretation up to the platform genpd driver.
>>
>> This allows the ti_sci_pm_domains driver to parse the phandle and use the first
>> phandle cell as the sci-id rather than getting it from a separate property.
>>
>> Besides that none of the original patches have changed apart from updating the
>> year to 2017 in patches 3 and 4 and of course a small update to patch 4 to let
>> the ti_sci_pm_domains parse the phandle and extract the sci-id rather than
>> looking for the rejected ti,sci-id property.
>>
>> I did not update the "power-domain" binding document because in regards to
>> #power-domain-cells it already states "can be any value as specified by device
>> tree binding documentation of particular provider" which I think already
>> describes the change in patch 2.
> 
> I'm assuming that this will go in through the arm-soc tree.

Assuming that Rob is OK with the v5 of patch 3 that I just sent in response to
the v4 patch email, can this series go through you? We should be good to go with
v4 of the series + the small v5 update to patch 3 if Rob is ok with it.

Regards,
Dave

> 
> Thanks,
> Rafael
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: d-gerlach@ti.com (Dave Gerlach)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/5] ARM: K2G: Add support for TI-SCI Generic PM Domains
Date: Thu, 16 Mar 2017 13:31:17 -0500	[thread overview]
Message-ID: <58CAD9F5.7030904@ti.com> (raw)
In-Reply-To: <2133897.t9KqoeriS0@aspire.rjw.lan>

Santosh,
On 03/12/2017 12:02 PM, Rafael J. Wysocki wrote:
> On Tuesday, March 07, 2017 04:22:29 AM Dave Gerlach wrote:
>> Hi,
>> This is v4 of the series to add support for TI-SCI Generic PM Domains.
>> Previous versions can be found here:
>>
>> v3: https://www.spinics.net/lists/kernel/msg2413975.html
>> v2: https://www.spinics.net/lists/kernel/msg2364612.html
>> v1: http://www.spinics.net/lists/arm-kernel/msg525204.html
>>
>> After much debate I have returned to using the phandle cell to pass the SCI ID
>> rather than the separate "ti,sci-id" property that was not very popular. In
>> order to do this I needed to make a change to the genpd framework which can be
>> seen in patch 2 and should benefit others that have client . Rather than
>> checking for zero phandle args and failing if any are present the
>> of_genpd_add_provider_simple call does not check at all and instead leaves
>> parsing and interpretation up to the platform genpd driver.
>>
>> This allows the ti_sci_pm_domains driver to parse the phandle and use the first
>> phandle cell as the sci-id rather than getting it from a separate property.
>>
>> Besides that none of the original patches have changed apart from updating the
>> year to 2017 in patches 3 and 4 and of course a small update to patch 4 to let
>> the ti_sci_pm_domains parse the phandle and extract the sci-id rather than
>> looking for the rejected ti,sci-id property.
>>
>> I did not update the "power-domain" binding document because in regards to
>> #power-domain-cells it already states "can be any value as specified by device
>> tree binding documentation of particular provider" which I think already
>> describes the change in patch 2.
> 
> I'm assuming that this will go in through the arm-soc tree.

Assuming that Rob is OK with the v5 of patch 3 that I just sent in response to
the v4 patch email, can this series go through you? We should be good to go with
v4 of the series + the small v5 update to patch 3 if Rob is ok with it.

Regards,
Dave

> 
> Thanks,
> Rafael
> 

  reply	other threads:[~2017-03-16 18:32 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-07 10:22 [PATCH v4 0/5] ARM: K2G: Add support for TI-SCI Generic PM Domains Dave Gerlach
2017-03-07 10:22 ` Dave Gerlach
2017-03-07 10:22 ` Dave Gerlach
2017-03-07 10:22 ` [PATCH v4 1/5] PM / Domains: Add generic data pointer to genpd data struct Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-10 21:19   ` Kevin Hilman
2017-03-10 21:19     ` Kevin Hilman
2017-03-10 21:19     ` Kevin Hilman
2017-03-07 10:22 ` [PATCH v4 2/5] PM / Domains: Do not check if simple providers have phandle cells Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-10 21:18   ` Kevin Hilman
2017-03-10 21:18     ` Kevin Hilman
2017-03-10 21:18     ` Kevin Hilman
2017-03-13  8:20   ` Ulf Hansson
2017-03-13  8:20     ` Ulf Hansson
2017-03-13  8:20     ` Ulf Hansson
2017-03-07 10:22 ` [PATCH v4 3/5] dt-bindings: Add TI SCI PM Domains Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-12 17:21   ` santosh.shilimkar
2017-03-12 17:21     ` santosh.shilimkar at oracle.com
2017-03-12 17:21     ` santosh.shilimkar-QHcLZuEGTsvQT0dZR+AlfA
2017-03-13  8:21   ` Ulf Hansson
2017-03-13  8:21     ` Ulf Hansson
2017-03-13  8:21     ` Ulf Hansson
2017-03-15 17:53   ` Rob Herring
2017-03-15 17:53     ` Rob Herring
2017-03-15 20:14     ` Dave Gerlach
2017-03-15 20:14       ` Dave Gerlach
2017-03-15 20:14       ` Dave Gerlach
2017-03-16 18:27   ` [PATCH v5 " Dave Gerlach
2017-03-16 18:27     ` Dave Gerlach
2017-03-16 18:27     ` Dave Gerlach
2017-03-18 20:38     ` Rob Herring
2017-03-18 20:38       ` Rob Herring
2017-03-18 20:38       ` Rob Herring
2017-03-07 10:22 ` [PATCH v4 4/5] soc: ti: Add ti_sci_pm_domains driver Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-10 21:22   ` Kevin Hilman
2017-03-10 21:22     ` Kevin Hilman
2017-03-10 21:22     ` Kevin Hilman
2017-03-12 17:20   ` santosh.shilimkar
2017-03-12 17:20     ` santosh.shilimkar at oracle.com
2017-03-13  8:21   ` Ulf Hansson
2017-03-13  8:21     ` Ulf Hansson
2017-03-13  8:21     ` Ulf Hansson
2017-03-07 10:22 ` [PATCH v4 5/5] ARM: keystone: Drop PM domain support for k2g Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-07 10:22   ` Dave Gerlach
2017-03-12 17:20   ` santosh.shilimkar
2017-03-12 17:20     ` santosh.shilimkar at oracle.com
2017-03-12 17:20     ` santosh.shilimkar-QHcLZuEGTsvQT0dZR+AlfA
2017-03-12 17:02 ` [PATCH v4 0/5] ARM: K2G: Add support for TI-SCI Generic PM Domains Rafael J. Wysocki
2017-03-12 17:02   ` Rafael J. Wysocki
2017-03-12 17:02   ` Rafael J. Wysocki
2017-03-16 18:31   ` Dave Gerlach [this message]
2017-03-16 18:31     ` Dave Gerlach
2017-03-16 18:31     ` Dave Gerlach
2017-04-03 16:58     ` santosh.shilimkar
2017-04-03 16:58       ` santosh.shilimkar at oracle.com
2017-04-03 16:58       ` santosh.shilimkar

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=58CAD9F5.7030904@ti.com \
    --to=d-gerlach@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=j-keerthy@ti.com \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=rjw@rjwysocki.net \
    --cc=rmk+kernel@armlinux.org.uk \
    --cc=robh+dt@kernel.org \
    --cc=ssantosh@kernel.org \
    --cc=sudeep.holla@arm.com \
    --cc=t-kristo@ti.com \
    --cc=ulf.hansson@linaro.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: link
Be 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.