2017-11-15 15:24:15

by Ben Hutchings

[permalink] [raw]
Subject: Re: [PATCH 4.4 06/56] dt-bindings: clockgen: Add compatible string for LS1012A

On Wed, 2017-11-15 at 16:16 +0100, Greg Kroah-Hartman wrote:
> On Tue, Nov 14, 2017 at 03:46:10PM +0000, Ben Hutchings wrote:
> > On Mon, 2017-11-13 at 13:55 +0100, Greg Kroah-Hartman wrote:
> > > 4.4-stable review patch.  If anyone has any objections, please
> > > let me
> > > know.
> > >
> > > ------------------
> > >
> > > From: Harninder Rai <[email protected]>
> > >
> > >
> > > [ Upstream commit 73447f68d7b2bc1df870da88b0e21d2bc1afc025 ]
> > >
> > > Signed-off-by: Harninder Rai <[email protected]>
> > > Signed-off-by: Bhaskar Upadhaya <[email protected]>
> > > Acked-by: Rob Herring <[email protected]>
> > > Signed-off-by: Shawn Guo <[email protected]>
> > > Signed-off-by: Sasha Levin <[email protected]>
> > > Signed-off-by: Greg Kroah-Hartman <[email protected]>
> > > ---
> > >  Documentation/devicetree/bindings/clock/qoriq-clock.txt |    1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > @@ -31,6 +31,7 @@ Required properties:
> > >   * "fsl,t4240-clockgen"
> > >   * "fsl,b4420-clockgen"
> > >   * "fsl,b4860-clockgen"
> > > + * "fsl,ls1012a-clockgen"
> >
> > This isn't supported or used anywhere in 4.4, so it makes sense to
> > document it.
>
> It "does" or "does not" make sense?  Should I drop this patch?

Sorry, it does not make sense to document it so please drop this.

Ben.

--
Ben Hutchings
Software Developer, Codethink Ltd.


From 1584146000784240912@xxx Wed Nov 15 15:22:34 +0000 2017
X-GM-THRID: 1583955807972418357
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread