On Fri, Jul 13, 2018 at 11:03:04AM +0000, Corentin Labbe wrote:
> This patch document the new optional port-supply
>
> Signed-off-by: Corentin Labbe <clabbe@xxxxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/ata/ahci-platform.txt | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.txt b/Documentation/devicetree/bindings/ata/ahci-platform.txt
> index 47652521a6ed..40a636b9851e 100644
> --- a/Documentation/devicetree/bindings/ata/ahci-platform.txt
> +++ b/Documentation/devicetree/bindings/ata/ahci-platform.txt
> @@ -31,6 +31,7 @@ Optional properties:
> - dma-coherent : Present if dma operations are coherent
> - clocks : a list of phandle + clock specifier pairs
> - target-supply : regulator for SATA target power
> +- port-supply : regulator for SATA port
Humm, I'm pretty sure that's what target-supply was supposed to be. If
anything, need to clearly define what is what.
How does this work with multiple ports?
> - phys : reference to the SATA PHY node
> - phy-names : must be "sata-phy"
> - ahci-supply : regulator for AHCI controller
Also, please group all the supplies together.
> @@ -51,6 +52,7 @@ Sub-nodes required properties:
> And at least one of the following properties:
> - phys : reference to the SATA PHY node
> - target-supply : regulator for SATA target power
> +- port-supply : regulator for SATA port
>
> Examples:
> sata@ffe08000 {
> --
> 2.16.4
>
--
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
