[PATCH 5/6] ARM: dts: imx27 cpufreq-cpu0 frequencies

Shawn Guo shawn.guo at linaro.org
Fri Jun 21 01:23:46 EDT 2013


On Fri, Jun 21, 2013 at 08:54:53AM +0400, Alexander Shiyan wrote:
> > On Thu, Jun 20, 2013 at 04:50:14PM +0200, Markus Pargmann wrote:
> > > +	cpus {
> > > +		#size-cells = <0>;
> > > +		#address-cells = <1>;
> > > +
> > > +		cpu at 0 {
> > > +			device_type = "cpu";
> > > +			compatible = "fsl,imx27", "arm,arm926ejs";
> > 
> > From what Documentation/devicetree/bindings/arm/cpus.txt tells, it
> > should be "arm,arm926".  Also, why do you put "fsl,imx27" there?
> > "imx27" is a SoC name not cpu core.
> 
> I think Markus take this ARM property from one of existing DTS.
> 
> shc at shc /home/git/linux-mx27/arch/arm/boot/dts $ grep arm926 *.dtsi
> at91sam9260.dtsi:                       compatible = "arm,arm926ejs";
> at91sam9263.dtsi:                       compatible = "arm,arm926ejs";
> at91sam9g45.dtsi:                       compatible = "arm,arm926ejs";
> at91sam9n12.dtsi:                       compatible = "arm,arm926ejs";
> at91sam9x5.dtsi:                        compatible = "arm,arm926ejs";
> imx23.dtsi:                     compatible = "arm,arm926ejs";
> imx28.dtsi:                     compatible = "arm,arm926ejs";
> lpc32xx.dtsi:                   compatible = "arm,arm926ejs";
> s3c2416.dtsi:                   compatible = "arm,arm926ejs";
> spear3xx.dtsi:                  compatible = "arm,arm926ejs";
> spear600.dtsi:                  compatible = "arm,arm926ejs";
> wm8505.dtsi:                    compatible = "arm,arm926ejs";
> 
> So, documentation need to be updated or these values should be fixed.
> Another solution  is specify compatible string as:
> compatible = "arm,arm926ejs", "arm,arm926";
> What you think about this?

I assume that the compatible string in the binding doc has been reviewed
and agreed by people, so we should fix the existing users before kernel
starts using it to for matching something.

Lorenzo, comment?

Shawn




More information about the linux-arm-kernel mailing list