| Message ID | 20180903100201.23131-1-clabbe.montjoie@gmail.com |
|---|---|
| Headers | show |
| Series | ata: ahci_platform: support allwinner R40 AHCI | expand |
On Mon, Sep 03, 2018 at 12:01:52PM +0200, Corentin Labbe wrote: > This patchset add support for allwinner R40 AHCI controller. > > The whole patchset is tested on sun8i-r40-bananapi-m2-ultra and > on sun7i-a20-cubieboard2 which doesnt have any of the ressources added > by this serie, so no regression should come with it. > > Patchs 1-7 should go via the ata tree > Patchs 8,9 should go via the sunxi tree > Acked-by: Maxime Ripard <maxime.ripard@bootlin.com> Thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
Hi, On 03-09-18 14:42, Maxime Ripard wrote: > On Mon, Sep 03, 2018 at 12:01:52PM +0200, Corentin Labbe wrote: >> This patchset add support for allwinner R40 AHCI controller. >> >> The whole patchset is tested on sun8i-r40-bananapi-m2-ultra and >> on sun7i-a20-cubieboard2 which doesnt have any of the ressources added >> by this serie, so no regression should come with it. >> >> Patchs 1-7 should go via the ata tree >> Patchs 8,9 should go via the sunxi tree >> > > Acked-by: Maxime Ripard <maxime.ripard@bootlin.com> The entire series looks good to me too: Reviewed-by: Hans de Goede <hdegoede@redhat.com> Regards, Hans
On 9/3/18 4:01 AM, Corentin Labbe wrote: > This patchset add support for allwinner R40 AHCI controller. > > The whole patchset is tested on sun8i-r40-bananapi-m2-ultra and > on sun7i-a20-cubieboard2 which doesnt have any of the ressources added > by this serie, so no regression should come with it. > > Patchs 1-7 should go via the ata tree > Patchs 8,9 should go via the sunxi tree Applied 1-7 for 4.20, thanks everyone. -- Jens Axboe
On Mon, Sep 03, 2018 at 08:26:22AM -0600, Jens Axboe wrote: > On 9/3/18 4:01 AM, Corentin Labbe wrote: > > This patchset add support for allwinner R40 AHCI controller. > > > > The whole patchset is tested on sun8i-r40-bananapi-m2-ultra and > > on sun7i-a20-cubieboard2 which doesnt have any of the ressources added > > by this serie, so no regression should come with it. > > > > Patchs 1-7 should go via the ata tree > > Patchs 8,9 should go via the sunxi tree > > Applied 1-7 for 4.20, thanks everyone. Applied 8 and 9, thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
On Mon, Sep 03, 2018 at 12:01:53PM +0200, Corentin Labbe wrote: > This patch fix the indentation of target-supply's ':'. > > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com> > --- > Documentation/devicetree/bindings/ata/ahci-platform.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Please add acks/reviewed-bys when posting new versions. Rob
On Mon, Sep 03, 2018 at 12:01:57PM +0200, Corentin Labbe wrote: > This patch document the new optional phy-supply. > > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com> > --- > Documentation/devicetree/bindings/ata/ahci-platform.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.txt b/Documentation/devicetree/bindings/ata/ahci-platform.txt > index f495774c8af9..45b451961612 100644 > --- a/Documentation/devicetree/bindings/ata/ahci-platform.txt > +++ b/Documentation/devicetree/bindings/ata/ahci-platform.txt > @@ -31,6 +31,7 @@ Optional properties: > - clocks : a list of phandle + clock specifier pairs > - resets : a list of phandle + reset specifier pairs > - target-supply : regulator for SATA target power > +- phy-supply : regulator for PHY power This should be mutually exclusive with the phys property. If you have a phy node, then its supply belongs there. > - phys : reference to the SATA PHY node > - phy-names : must be "sata-phy" > - ahci-supply : regulator for AHCI controller > -- > 2.16.4 >
On Mon, 3 Sep 2018 12:01:59 +0200, Corentin Labbe wrote: > This patch update binding with the new R40 compatible. > > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com> > --- > Documentation/devicetree/bindings/ata/ahci-platform.txt | 2 ++ > 1 file changed, 2 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>