From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [PATCH 2/3] ARM: sunxi: Add an ahci-platform compatible AHCI driver for the Allwinner SUNXi series of SoCs Date: Wed, 4 Dec 2013 12:26:02 +0000 Message-ID: <20131204122602.GN16025@e106331-lin.cambridge.arm.com> References: <1386159055-10264-1-git-send-email-oliver@schinagl.nl> <1386159055-10264-3-git-send-email-oliver@schinagl.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1386159055-10264-3-git-send-email-oliver-dxLnbx3+1qmEVqv0pETR8A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "oliver-dxLnbx3+1qmEVqv0pETR8A@public.gmane.org" Cc: "tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" , "grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , "ob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org" , "linux-ide-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "dev-3kdeTeqwOZ9EV1b7eY7vFQ@public.gmane.org" , "maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org" , "ijc-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org" , "hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org" , "oliver+list-dxLnbx3+1qmEVqv0pETR8A@public.gmane.org" List-Id: devicetree@vger.kernel.org On Wed, Dec 04, 2013 at 12:10:54PM +0000, oliver-dxLnbx3+1qmEVqv0pETR8A@public.gmane.org wrote: > From: Oliver Schinagl > > This patch adds support for the sunxi series of SoC's by allwinner. It > plugs into the ahci-platform framework. > > Note: Currently it uses a somewhat hackish approach that probably needs > a lot more work, but does the same as the IMX SoC's. > > Signed-off-by: Olliver Schinagl > --- > .../devicetree/bindings/ata/ahci-sunxi.txt | 24 ++ > drivers/ata/Kconfig | 9 + > drivers/ata/Makefile | 1 + > drivers/ata/ahci_platform.c | 12 + > drivers/ata/ahci_sunxi.c | 305 +++++++++++++++++++++ > 5 files changed, 351 insertions(+) > create mode 100644 Documentation/devicetree/bindings/ata/ahci-sunxi.txt > create mode 100644 drivers/ata/ahci_sunxi.c > > diff --git a/Documentation/devicetree/bindings/ata/ahci-sunxi.txt b/Documentation/devicetree/bindings/ata/ahci-sunxi.txt > new file mode 100644 > index 0000000..0792fa5 > --- /dev/null > +++ b/Documentation/devicetree/bindings/ata/ahci-sunxi.txt > @@ -0,0 +1,24 @@ > +Allwinner SUNXI AHCI SATA Controller > + > +SATA nodes are defined to describe on-chip Serial ATA controllers. > +Each SATA controller should have its own node. > + > +Required properties: > +- compatible : compatible list, contains "allwinner,sun4i-a10-ahci" - compatible: Should contain "allwinner,sun4i-a10-ahci" > +- reg : - reg: The offset and length of the MMIO registers. > +- interrupts : - interrupts: An interrupt-specifier for the ACHI interrupt > +- clocks : clocks for ACHI > +- clock-names : clock names for AHCI Please _define_ the set of clock-names you expect. This binding is meaningless without it. If you require clock-names, define the clocks property in terms of it. Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html