From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tero Kristo Subject: Re: [PATCH 1/3] Documentation: dt: Add TI SCI clock driver Date: Fri, 2 Dec 2016 23:07:54 +0200 Message-ID: <76fca1ab-fbae-07ed-7585-cd9ba1230a19@ti.com> References: <1477053961-27128-1-git-send-email-t-kristo@ti.com> <1477053961-27128-2-git-send-email-t-kristo@ti.com> <20161030204121.qvb5d33dh65awwzx@rob-hp-laptop> <41c58712-bc00-ed05-9d1d-42e31397a70c@ti.com> <0fe81866-8bfd-f3a7-d808-9cb23841f504@ti.com> <8579b123-f214-22f1-0236-e5b98ab51597@ti.com> <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Boyd , Rob Herring Cc: linux-clk , Michael Turquette , Santosh Shilimkar , Nishanth Menon , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On 02/12/16 20:58, Stephen Boyd wrote: > On 12/02/2016 10:45 AM, Rob Herring wrote: >> On Fri, Dec 2, 2016 at 2:19 AM, Tero Kristo wrote: >>> >>> Rob, any response on this one? I see you have acked the reset part of the >>> bindings which is doing pretty much the same thing as the clock part is >>> doing here, namely adding child node under the main SCI node. Is it okay to >>> do this same for other parts of the TI SCI? >> Yes. It would be silly to allow for one and not others... >> > > I'm expecting a respin for the node name (clocks or clock-controller). > I'll also make a review pass on patch 3 today so please don't respin > until after that. Yeah, I need to fix that and re-send. This series will be most likely delayed until 4.11 though seeing its very late in 4.9-rc already (and the genpd part dependency is still not ready either) so we are not in a rush right now. -Tero -- 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH 1/3] Documentation: dt: Add TI SCI clock driver To: Stephen Boyd , Rob Herring References: <1477053961-27128-1-git-send-email-t-kristo@ti.com> <1477053961-27128-2-git-send-email-t-kristo@ti.com> <20161030204121.qvb5d33dh65awwzx@rob-hp-laptop> <41c58712-bc00-ed05-9d1d-42e31397a70c@ti.com> <0fe81866-8bfd-f3a7-d808-9cb23841f504@ti.com> <8579b123-f214-22f1-0236-e5b98ab51597@ti.com> <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74@codeaurora.org> CC: linux-clk , Michael Turquette , Santosh Shilimkar , Nishanth Menon , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" From: Tero Kristo Message-ID: <76fca1ab-fbae-07ed-7585-cd9ba1230a19@ti.com> Date: Fri, 2 Dec 2016 23:07:54 +0200 MIME-Version: 1.0 In-Reply-To: <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74@codeaurora.org> Content-Type: text/plain; charset="utf-8"; format=flowed List-ID: On 02/12/16 20:58, Stephen Boyd wrote: > On 12/02/2016 10:45 AM, Rob Herring wrote: >> On Fri, Dec 2, 2016 at 2:19 AM, Tero Kristo wrote: >>> >>> Rob, any response on this one? I see you have acked the reset part of the >>> bindings which is doing pretty much the same thing as the clock part is >>> doing here, namely adding child node under the main SCI node. Is it okay to >>> do this same for other parts of the TI SCI? >> Yes. It would be silly to allow for one and not others... >> > > I'm expecting a respin for the node name (clocks or clock-controller). > I'll also make a review pass on patch 3 today so please don't respin > until after that. Yeah, I need to fix that and re-send. This series will be most likely delayed until 4.11 though seeing its very late in 4.9-rc already (and the genpd part dependency is still not ready either) so we are not in a rush right now. -Tero From mboxrd@z Thu Jan 1 00:00:00 1970 From: t-kristo@ti.com (Tero Kristo) Date: Fri, 2 Dec 2016 23:07:54 +0200 Subject: [PATCH 1/3] Documentation: dt: Add TI SCI clock driver In-Reply-To: <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74@codeaurora.org> References: <1477053961-27128-1-git-send-email-t-kristo@ti.com> <1477053961-27128-2-git-send-email-t-kristo@ti.com> <20161030204121.qvb5d33dh65awwzx@rob-hp-laptop> <41c58712-bc00-ed05-9d1d-42e31397a70c@ti.com> <0fe81866-8bfd-f3a7-d808-9cb23841f504@ti.com> <8579b123-f214-22f1-0236-e5b98ab51597@ti.com> <5f146fb6-ec88-b7ee-ef5b-a5ad32c54a74@codeaurora.org> Message-ID: <76fca1ab-fbae-07ed-7585-cd9ba1230a19@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 02/12/16 20:58, Stephen Boyd wrote: > On 12/02/2016 10:45 AM, Rob Herring wrote: >> On Fri, Dec 2, 2016 at 2:19 AM, Tero Kristo wrote: >>> >>> Rob, any response on this one? I see you have acked the reset part of the >>> bindings which is doing pretty much the same thing as the clock part is >>> doing here, namely adding child node under the main SCI node. Is it okay to >>> do this same for other parts of the TI SCI? >> Yes. It would be silly to allow for one and not others... >> > > I'm expecting a respin for the node name (clocks or clock-controller). > I'll also make a review pass on patch 3 today so please don't respin > until after that. Yeah, I need to fix that and re-send. This series will be most likely delayed until 4.11 though seeing its very late in 4.9-rc already (and the genpd part dependency is still not ready either) so we are not in a rush right now. -Tero