On Sun, Mar 11, 2018 at 11:43 PM, Jacky Bai <ping.bai@xxxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
