Re: [PATCH v5 2/4] Documentation: Add documentation for APM X-Gene SoC SATA host controller DTS binding

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi,

>> +- interrupt-parent   : Interrupt controller.
>
> I don't think that's strictly required.

Okay, got rip of it.

>> +Optional properties:
>> +- status             : Shall be "ok" if enabled or "na" if disabled.
>> +                       Default is "ok".
>
> Why not use the more standard "disabled" string, as I asked you to
> previously?

Missed this one. Fixed.

>> +             sata2: sata@1a400000 {
>> +                     compatible = "apm,xgene-ahci-sgmii";
>> +                     reg = <0x0 0x1a400000 0x0 0x1000>,
>> +                           <0x0 0x1f220000 0x0 0x10000>;
>> +                     interrupt-parent = <&gic>;
>> +                     interrupts = <0x0 0x87 0x4>;
>> +                     status = "ok";
>> +                     clocks = <&sata23clk 0>;
>> +                     phys = <&phy2 0>;
>> +                     phy-names = "sata-6g";
>> +             };
>> +
>> +             sata3: sata@1a800000 {
>> +                     compatible = "apm,xgene-ahci-pcie";
>> +                     reg = <0x0 0x1a800000 0x0 0x1000>,
>> +                           <0x0 0x1f230000 0x0 0x10000>;
>> +                     interrupt-parent = <&gic>;
>> +                     interrupts = <0x0 0x88 0x4>;
>> +                     status = "ok";
>> +                     clocks = <&sata45clk 0>;
>> +                     phys = <&phy3 0>;
>> +                     phy-names = "sata-6g";
>> +             };
>
> The example is pointlessly huge. You could trim it down two these last
> two nodes (with a phy node and some fixed clocks).

I will use dummy fixed clock and added two PHY nodes.

-Loc
--
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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux