On Mon, Feb 12, 2018 at 10:56:40PM +0530, Preetham Chandru Ramchandra wrote: > From: Preetham Ramchandra <pchandru@xxxxxxxxxx> > > This adds bindings documentation for the > AHCI controller on Tegra210 Please avoid lines that are too short. They shouldn't exceed 72 characters, but you're supposed to make fully use of those. Also the above is a sentence, so should end with a '.'. > diff --git a/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt b/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt > index 66c83c3e8915..0f4520a00716 100644 > --- a/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt > +++ b/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt > @@ -1,20 +1,19 @@ > -Tegra124 SoC SATA AHCI controller > +Tegra SoC SATA AHCI controller > > Required properties : > -- compatible : For Tegra124, must contain "nvidia,tegra124-ahci". Otherwise, > - must contain '"nvidia,<chip>-ahci", "nvidia,tegra124-ahci"', where <chip> > - is tegra132. > -- reg : Should contain 2 entries: > +- compatible : Must be one of: > + - Tegra124 : "nvidia,tegra124-ahci" > + - Tegra210 : "nvidia,tegra210-ahci" > +- reg : Should contain 3 entries: > - AHCI register set (SATA BAR5) > - SATA register set > + - AUX register set > - interrupts : Defines the interrupt used by SATA > - clocks : Must contain an entry for each entry in clock-names. > See ../clocks/clock-bindings.txt for details. > - clock-names : Must include the following entries: > - sata > - sata-oob > - - cml1 > - - pll_e > - resets : Must contain an entry for each entry in reset-names. > See ../reset/reset.txt for details. > - reset-names : Must include the following entries: > @@ -24,9 +23,19 @@ Required properties : > - phys : Must contain an entry for each entry in phy-names. > See ../phy/phy-bindings.txt for details. > - phy-names : Must include the following entries: > - - sata-phy : XUSB PADCTL SATA PHY > -- hvdd-supply : Defines the SATA HVDD regulator > -- vddio-supply : Defines the SATA VDDIO regulator > -- avdd-supply : Defines the SATA AVDD regulator > -- target-5v-supply : Defines the SATA 5V power regulator > -- target-12v-supply : Defines the SATA 12V power regulator > + - For T124: > + - sata-phy : XUSB PADCTL SATA PHY > + - For T210: > + - sata-0 > +- For T124: Can we please use TegraXYZ here? I find it useful to be consistent here because those will show up if I grep for "tegra", whereas TXYZ won't. Thierry
Attachment:
signature.asc
Description: PGP signature
