From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Wed, 15 Jan 2014 21:10:33 +0100 Subject: [PATCH v9 2/4] Documentation: Add documentation for APM X-Gene SoC SATA host controller DTS binding In-Reply-To: References: <1389769910-15505-1-git-send-email-lho@apm.com> <20140115111839.GB25824@e106331-lin.cambridge.arm.com> Message-ID: <5905127.tqJ3nFt5cj@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 15 January 2014 12:04:02 Loc Ho wrote: > > >> +- clocks : Reference to the clock entry. > >> +- phys : PHY reference with parameter 0. > > > > The specific value of the phy-specifier shouldn't matter to this > > binding. What should matter is what it logically corresponds to. > > I not quite following this. Are you suggest that I drop the value 0. > In the binding, one needs to specify the mode of operation - 0 is for > SATA. Can you explain more? The SATA device should not care what the argument for the PHY device is. You could connect the same device to another PHY that has a different set of arguments, which is the whole point of abstracting it. Arnd