From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v9 2/4] Documentation: Add documentation for APM X-Gene SoC SATA host controller DTS binding Date: Wed, 15 Jan 2014 21:10:33 +0100 Message-ID: <5905127.tqJ3nFt5cj@wuerfel> References: <1389769910-15505-1-git-send-email-lho@apm.com> <20140115111839.GB25824@e106331-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: linux-scsi-owner@vger.kernel.org To: Loc Ho Cc: Mark Rutland , "olof@lixom.net" , "tj@kernel.org" , "linux-scsi@vger.kernel.org" , "linux-ide@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "dmilburn@redhat.com" , "jcm@redhat.com" , "patches@apm.com" , Tuan Phan , Suman Tripathi List-Id: linux-ide@vger.kernel.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