[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20131204122602.GN16025@e106331-lin.cambridge.arm.com>
Date: Wed, 4 Dec 2013 12:26:02 +0000
From: Mark Rutland <mark.rutland@....com>
To: "oliver@...inagl.nl" <oliver@...inagl.nl>
Cc: "tj@...nel.org" <tj@...nel.org>,
"grant.likely@...aro.org" <grant.likely@...aro.org>,
"ob.herring@...xeda.com" <ob.herring@...xeda.com>,
"linux-ide@...r.kernel.org" <linux-ide@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"dev@...ux-sunxi.org" <dev@...ux-sunxi.org>,
"maxime.ripard@...e-electrons.com" <maxime.ripard@...e-electrons.com>,
"ijc@...lion.org.uk" <ijc@...lion.org.uk>,
"hdegoede@...hat.com" <hdegoede@...hat.com>,
"oliver+list@...inagl.nl" <oliver+list@...inagl.nl>
Subject: Re: [PATCH 2/3] ARM: sunxi: Add an ahci-platform compatible AHCI
driver for the Allwinner SUNXi series of SoCs
On Wed, Dec 04, 2013 at 12:10:54PM +0000, oliver@...inagl.nl wrote:
> From: Oliver Schinagl <oliver@...inagl.nl>
>
> 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 <oliver@...inagl.nl>
> ---
> .../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 : <registers mapping>
- reg: The offset and length of the MMIO registers.
> +- interrupts : <interrupt mapping for AHCI IRQ>
- 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 linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists