Re: [PATCH v4 1/2] ARM: dts: imx: Add basic dtsi file for imx6sll

[Rob Herring <robh@kernel.org>]

On Sun, Mar 11, 2018 at 11:43 PM, Jacky Bai <ping.bai@nxp.com> wrote:
>> > diff --git a/arch/arm/boot/dts/imx6sll.dtsi b/arch/arm/boot/dts/imx6sll.dtsi
>> > +           spi3 = &ecspi3;
>> > +           spi4 = &ecspi4;
>> > +           usbphy0 = &usbphy1;
>> > +           usbphy1 = &usbphy2;
>>
>> Why do you need a alias for phys?
>
> The alias of usbphy seems used by drivers/usb/phy/phy-mxs-usb.c. So we need to add alias for usbphy.

That use should be fixed. The correct way to handle this is add a cell
to "fsl,anatop" with 0 or 1 to distinguish each phy.

>> > +   };
>> > +
>> > +   cpus {
>> > +           #address-cells = <1>;
>> > +           #size-cells = <0>;
>> > +
>> > +           cpu0: cpu@0 {
>> > +                   compatible = "arm,cortex-a9";
>> > +                   device_type = "cpu";
>> > +                   reg = <0>;
>> > +                   next-level-cache = <&L2>;
>> > +                   operating-points = <
>> > +                           /* kHz    uV */
>> > +                           996000  1225000
>> > +                           792000  1175000
>> > +                           396000  1075000
>> > +                           198000  975000
>> > +                   >;
>> > +                   fsl,soc-operating-points = <
>>
>> This is not documented.
>
> This is same as we used on other imx6 SOC. I don't know where to add doc for this property. Please give me some suggestion.

Along side other OPP binding docs. The real question is why you need
this and can't use the original OPP binding (which you have too) or
move to the v2 binding.

Rob