Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1560902

Re: [PATCH v2 01/14] devicetree: bindings: add bindings for ahci-da850

From David Lechner <david@lechnology.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 01/14] devicetree: bindings: add bindings for ahci-da850
Date 2017-01-17 19:40 +0100
Message-ID <t0GPM-25I-31@gated-at.bofh.it> (permalink)
References <t0B3H-72K-5@gated-at.bofh.it> <t0B3I-72K-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 01/17/2017 06:26 AM, Bartosz Golaszewski wrote:
> Add DT bindings for the TI DA850 AHCI SATA controller.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
>  Documentation/devicetree/bindings/ata/ahci-da850.txt | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/ata/ahci-da850.txt
>
> diff --git a/Documentation/devicetree/bindings/ata/ahci-da850.txt b/Documentation/devicetree/bindings/ata/ahci-da850.txt
> new file mode 100644
> index 0000000..e7111b4
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/ata/ahci-da850.txt
> @@ -0,0 +1,18 @@
> +Device tree binding for the TI DA850 AHCI SATA Controller
> +---------------------------------------------------------
> +
> +Required properties:
> +  - compatible: must be "ti,da850-ahci"
> +  - reg: physical base addresses and sizes of the controller's register areas
> +  - interrupts: interrupt specifier (refer to the interrupt binding)
> +
> +Optional properties:
> +  - clocks: clock specifier (refer to the common clock binding)

Won't you also need a clock-names property since there are two clocks?

> +
> +Example:
> +
> +	sata: ahci@218000 {
> +		compatible = "ti,da850-ahci";
> +		reg = <0x218000 0x2000>, <0x22c018 0x4>;
> +		interrupts = <67>;
> +	};
>

It would be nice to have clocks and clock-names in the example as well.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 00/14] ARM: da850-lcdk: add SATA support Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
  [PATCH v2 01/14] devicetree: bindings: add bindings for ahci-da850 Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
    Re: [PATCH v2 01/14] devicetree: bindings: add bindings for  ahci-da850 David Lechner <david@lechnology.com> - 2017-01-17 19:40 +0100
      Re: [PATCH v2 01/14] devicetree: bindings: add bindings for  ahci-da850 Sekhar Nori <nsekhar@ti.com> - 2017-01-18 10:10 +0100
  [PATCH v2 03/14] ARM: davinci: add a clock lookup entry for the SATA clock Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
  [PATCH v2 06/14] ARM: davinci: da850: model the SATA refclk Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
    Re: [PATCH v2 06/14] ARM: davinci: da850: model the SATA refclk David Lechner <david@lechnology.com> - 2017-01-17 21:40 +0100
    Re: [PATCH v2 06/14] ARM: davinci: da850: model the SATA refclk Sekhar Nori <nsekhar@ti.com> - 2017-01-18 10:40 +0100
  [PATCH v2 05/14] ARM: davinci: da850: add con_id for the SATA clock Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
  [PATCH v2 02/14] ARM: davinci_all_defconfig: enable SATA modules Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
  [PATCH v2 07/14] sata: ahci-da850: add device tree match table Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100
  [PATCH v2 09/14] sata: ahci: export ahci_do_hardreset() locally Bartosz Golaszewski <bgolaszewski@baylibre.com> - 2017-01-17 13:30 +0100

csiph-web