Le 27/03/2016 21:48, Anup Patel a écrit : > Required properties: > - compatible: should be one or more of > "brcm,bcm7425-sata-phy" > "brcm,bcm7445-sata-phy" > + "brcm,iproc-ns2-sata-phy" > "brcm,phy-sata3" > - address-cells: should be 1 > - size-cells: should be 0 > -- reg: register range for the PHY PCB interface > -- reg-names: should be "phy" > +- reg: register ranges for the PHY PCB interface > +- reg-names: should be "phy" and "phy-ctrl" > + The "phy-ctrl" registers are only required for > + "brcm,iproc-ns2-sata-phy". I objected to adding another register during the internal review, which, in the case of the STB chips is dealt with at the AHCI controller level as was requested when Brian got the driver sent, but if that is the only point here, then let's get your patches merged as-is. The rest of the changes look fine so: Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx> -- Florian -- 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