From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH 1/3] Documentation: dt: Add TI SCI clock driver Date: Fri, 2 Dec 2016 10:58:22 -0800 Message-ID: <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> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-clk-owner@vger.kernel.org To: Rob Herring , Tero Kristo Cc: linux-clk , Michael Turquette , Santosh Shilimkar , Nishanth Menon , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" List-Id: devicetree@vger.kernel.org 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. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project