[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Thu, 10 Oct 2019 11:22:17 +1030
From: "Andrew Jeffery" <andrew@...id.au>
To: "Joel Stanley" <joel@....id.au>
Cc: linux-clk@...r.kernel.org,
"Michael Turquette" <mturquette@...libre.com>,
"Stephen Boyd" <sboyd@...nel.org>,
"Rob Herring" <robh+dt@...nel.org>,
"Mark Rutland" <mark.rutland@....com>,
"Linux ARM" <linux-arm-kernel@...ts.infradead.org>,
linux-aspeed <linux-aspeed@...ts.ozlabs.org>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
devicetree <devicetree@...r.kernel.org>
Subject: Re: [PATCH 1/2] dt-bindings: clock: Add AST2500 RMII RCLK definitions
On Tue, 8 Oct 2019, at 23:07, Joel Stanley wrote:
> On Tue, 8 Oct 2019 at 11:34, Andrew Jeffery <andrew@...id.au> wrote:
> >
> > The AST2500 has an explicit gate for the RMII RCLK for each of the two
> > MACs.
> >
> > Signed-off-by: Andrew Jeffery <andrew@...id.au>
> > ---
> > include/dt-bindings/clock/aspeed-clock.h | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/include/dt-bindings/clock/aspeed-clock.h b/include/dt-bindings/clock/aspeed-clock.h
> > index f43738607d77..64e245fb113f 100644
> > --- a/include/dt-bindings/clock/aspeed-clock.h
> > +++ b/include/dt-bindings/clock/aspeed-clock.h
> > @@ -39,6 +39,8 @@
> > #define ASPEED_CLK_BCLK 33
> > #define ASPEED_CLK_MPLL 34
> > #define ASPEED_CLK_24M 35
> > +#define ASPEED_CLK_GATE_MAC1RCLK 36
> > +#define ASPEED_CLK_GATE_MAC2RCLK 37
>
> Calling these ASPEED_CLK_GATE breaks the pattern the rest of the
> driver has in using that name for the clocks that are registered as
> struct aspeed_clk_gate clocks.
>
> Do you think we should drop the GATE_ to match the existing clocks?
I named them that way because the bits in question do just gate the
clocks, but I've renamed them to keep the pattern. Will send a v2.
Andrew
Powered by blists - more mailing lists