Re: [PATCH] clk: tegra210: Add SLCG override gate clocks

[Thierry Reding <thierry.reding@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2246 bytes --]

On Fri, Mar 25, 2016 at 02:21:51PM -0400, Rhyland Klein wrote:
> On 3/15/2016 4:40 AM, Jon Hunter wrote:
> > 
> > On 14/03/16 16:08, Rhyland Klein wrote:
> >> On 3/14/2016 12:05 PM, Thierry Reding wrote:
> >>> * PGP Signed by an unknown key
> >>>
> >>> On Thu, Mar 10, 2016 at 02:38:05PM -0500, Rhyland Klein wrote:
> >>>> From: Bill Huang <bilhuang@nvidia.com>
> >>>>
> >>>> Add some SLCG (Second Level Clock Gating) override clocks to control
> >>>> gating and un-gating their logics.
> >>>>
> >>>> Signed-off-by: Bill Huang <bilhuang@nvidia.com>
> >>>> Signed-off-by: Rhyland Klein <rklein@nvidia.com>
> >>>> ---
> >>>>  drivers/clk/tegra/clk-id.h               | 16 ++++++
> >>>>  drivers/clk/tegra/clk-tegra210.c         | 91 ++++++++++++++++++++++++++++++++
> >>>>  include/dt-bindings/clock/tegra210-car.h | 32 +++++------
> >>>>  3 files changed, 123 insertions(+), 16 deletions(-)
> >>>
> >>> There's no rationale given here about why we need this. What will these
> >>> second level clock gates be used for? Why do we need these (seemingly)
> >>> duplicate clock entries.
> >>>
> >>
> >> These are going to be used in the to-be posted patchset around
> >> powergating. As of now they are unused, which is why I hadn't added them
> >> previously. I just wanted to try to get this dependency in before the
> >> powergate series was posted.
> > 
> > Yes we are using these on the Pixel C (aka. Smaug) and I suggested to
> > Rhyland that we upstream them. Eventually we will use them but only
> > after the core GenPD changes for Tegra are merged. From my perspective I
> > was thinking it is better to reduce the changes between the chromeos
> > 3.18 kernel and mainline. However, if you wish to wait until we need
> > them I guess we can. Otherwise ...
> > 
> > Acked-by: Jon Hunter <jonathanh@nvidia.com>
> > 
> > Cheers
> > Jon
> > 
> 
> Thierry do you think we should hold off on this until Jon's patches are
> ready or merge this sooner?

Jon, I think it would make sense for you to pick this up into your tree
along with the rest of the patches that make use of them. Even if they
end up being applied to different trees I'd like to see all of the work
as a whole first.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]