Re: [PATCH v1 2/4] dt-bindings: clock: mediatek: add g3dsys bindings

From: Stephen Boyd
Date: Thu Apr 19 2018 - 13:25:48 EST


Quoting Sean Wang (2018-04-18 20:50:02)
> On Wed, 2018-04-18 at 20:18 -0700, Stephen Boyd wrote:
> > Quoting sean.wang@xxxxxxxxxxxx (2018-04-18 03:24:54)
> > > From: Sean Wang <sean.wang@xxxxxxxxxxxx>
> > >
> > > Add bindings to g3dsys providing necessary clock and reset control to
> > > Mali-450.
> > >
> > > Signed-off-by: Sean Wang <sean.wang@xxxxxxxxxxxx>
> > > ---
> > > .../bindings/arm/mediatek/mediatek,g3dsys.txt | 30 ++++++++++++++++++++++
> >
> > Why isn't this under bindings/clock/ ?
> >
>
> Tons of bindings for clock controller have been present at
> binding/arm/mediatek. g3dsys just have a follow-up to them.
>
> It's worth another patch moving them all from bindings/arm/mediatek/ to
> bindings/clock/mediatek.
>
> what's your opinion for either doing it prior to g3dsys binding being
> added or doing it in the future ?
>

Doing it at a later time is fine. Just wondering why they're all getting
dumped into the arm/ directory.