linux-kernel - Re: [PATCH 4.4 06/56] dt-bindings: clockgen: Add compatible string for LS1012A

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20171115155215.GC30206@kroah.com>
Date:   Wed, 15 Nov 2017 16:52:15 +0100
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Bhaskar Upadhaya <bhaskar.upadhaya@....com>
Cc:     Ben Hutchings <ben.hutchings@...ethink.co.uk>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>,
        Harninder Rai <harninder.rai@....com>,
        Rob Herring <robh@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
        Sasha Levin <alexander.levin@...izon.com>
Subject: Re: [PATCH 4.4 06/56] dt-bindings: clockgen: Add compatible string
 for LS1012A

On Wed, Nov 15, 2017 at 03:25:04PM +0000, Bhaskar Upadhaya wrote:
> 
> 
> -----Original Message-----
> From: Greg Kroah-Hartman [mailto:gregkh@...uxfoundation.org] 
> Sent: Wednesday, November 15, 2017 8:46 PM
> To: Ben Hutchings <ben.hutchings@...ethink.co.uk>
> Cc: linux-kernel@...r.kernel.org; stable@...r.kernel.org; Harninder Rai <harninder.rai@....com>; Bhaskar Upadhaya <bhaskar.upadhaya@....com>; Rob Herring <robh@...nel.org>; Shawn Guo <shawnguo@...nel.org>; Sasha Levin <alexander.levin@...izon.com>
> Subject: Re: [PATCH 4.4 06/56] dt-bindings: clockgen: Add compatible string for LS1012A
> 
> 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 <harninder.rai@....com>
> > > 
> > > 
> > > [ Upstream commit 73447f68d7b2bc1df870da88b0e21d2bc1afc025 ]
> > > 
> > > Signed-off-by: Harninder Rai <harninder.rai@....com>
> > > Signed-off-by: Bhaskar Upadhaya <Bhaskar.Upadhaya@....com>
> > > Acked-by: Rob Herring <robh@...nel.org>
> > > Signed-off-by: Shawn Guo <shawnguo@...nel.org>
> > > Signed-off-by: Sasha Levin <alexander.levin@...izon.com>
> > > Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > > ---
> > >  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?
> [Bhaskar]
> 	Hi Greg,
> 	"fsl,ls1012a-clockgen" is getting used in the file drivers/clk/clk-qoriq.c(kernel v4.14) as below.
> 	{
> 		.compat = "fsl,ls1012a-clockgen",
> 		.cmux_groups = {
> 			&ls1012a_cmux
> 		},
> 		.cmux_to_group = {
> 			0, -1
> 		},
> 		.pll_mask = 0x03,
> 	},

4.14 is great, but this is the 4.4-stable queue :)

thanks,

greg k-h