From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: linux-sh@vger.kernel.org, Olof Johansson <olof@lixom.net>,
Simon Horman <horms@verge.net.au>, Arnd Bergmann <arnd@arndb.de>,
Magnus Damm <magnus.damm@gmail.com>,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
Stephen Warren <swarren@wwwdotorg.org>,
linux-mmc@vger.kernel.org, Chris Ball <cjb@laptop.org>
Subject: Re: [PATCH] ARM: shmobile: update SDHI DT compatibility string to the <unit>-<soc> format
Date: Fri, 30 Aug 2013 01:47:21 +0200 [thread overview]
Message-ID: <1621893.Re68yLczyK@avalon> (raw)
In-Reply-To: <Pine.LNX.4.64.1308291650120.3786@axis700.grange>
Hi Guennadi,
Thank you for the patch.
On Thursday 29 August 2013 17:14:49 Guennadi Liakhovetski wrote:
> Currently DT compatibility strings of both types can be found in the kernel
> sources: <unit>-<soc> and <soc>-<unit>, whereas a unique format should be
> followed and the former one is preferred. This patch converts the SDHI
> MMC driver and its users to the common standard. This is safe for now, since
> ATM no real products are using this driver with DT.
>
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>
> Ok, the "no real products" is an assumption, that Laurent and Simon kind
> of agreed upon, so, I'm using it here to avoid having to carry old
> compatibility strings with us around. If anyone has other information -
> please shout. Otherwise, I think, it would be the easiest for Chris to ack
> this and for Simon to pull this via the ARM tree for late 3.12 (possibly,
> post -rc1) inclusion.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 17 ++++++++++-------
> arch/arm/boot/dts/r8a73a4.dtsi | 6 +++---
> arch/arm/boot/dts/r8a7740.dtsi | 4 ++--
> arch/arm/boot/dts/r8a7790.dtsi | 8 ++++----
> arch/arm/boot/dts/sh73a0.dtsi | 6 +++---
> drivers/mmc/host/sh_mobile_sdhi.c | 16 ++++++++--------
> 6 files changed, 30 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt index df204e1..6a2a116
> 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -9,12 +9,15 @@ compulsory and any optional properties, common to all
> SD/MMC drivers, as described in mmc.txt, can be used. Additionally the
> following tmio_mmc-specific optional bindings can be used.
>
> +Required properties:
> +- compatible: "renesas,sdhi-shmobile" - a generic sh-mobile SDHI unit
> + "renesas,sdhi-sh7372" - SDHI IP on SH7372 SoC
> + "renesas,sdhi-sh73a0" - SDHI IP on SH73A0 SoC
> + "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> + "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7778" - SDHI IP on R8A7778 SoC
> + "renesas,sdhi-r8a7779" - SDHI IP on R8A7779 SoC
> + "renesas,sdhi-r8a7790" - SDHI IP on R8A7790 SoC
> +
> Optional properties:
> - toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
> -
> -When used with Renesas SDHI hardware, the following compatibility strings
> -configure various model-specific properties:
> -
> -"renesas,sh7372-sdhi": (default) compatible with SH7372
> -"renesas,r8a7740-sdhi": compatible with R8A7740: certain MMC/SD commands
> have to - wait for the interface to become idle.
> diff --git a/arch/arm/boot/dts/r8a73a4.dtsi b/arch/arm/boot/dts/r8a73a4.dtsi
> index b0511b1..cc7211b 100644
> --- a/arch/arm/boot/dts/r8a73a4.dtsi
> +++ b/arch/arm/boot/dts/r8a73a4.dtsi
> @@ -236,7 +236,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -245,7 +245,7 @@
> };
>
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -254,7 +254,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
> index 8fc4a5d..c4992b5 100644
> --- a/arch/arm/boot/dts/r8a7740.dtsi
> +++ b/arch/arm/boot/dts/r8a7740.dtsi
> @@ -155,7 +155,7 @@
> };
>
> sdhi0: sdhi@e6850000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6850000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 117 4
> @@ -167,7 +167,7 @@
> };
>
> sdhi1: sdhi@e6860000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6860000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 121 4
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index 3b879e7..885f9f4 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -152,7 +152,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -161,7 +161,7 @@
> };
>
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -170,7 +170,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> @@ -179,7 +179,7 @@
> };
>
> sdhi3: sdhi@ee160000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee160000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 168 4>;
> diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
> index ba59a58..3955c76 100644
> --- a/arch/arm/boot/dts/sh73a0.dtsi
> +++ b/arch/arm/boot/dts/sh73a0.dtsi
> @@ -196,7 +196,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee100000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 83 4
> @@ -208,7 +208,7 @@
>
> /* SDHI1 and SDHI2 have no CD pins, no need for CD IRQ */
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee120000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 88 4
> @@ -219,7 +219,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee140000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 104 4
> diff --git a/drivers/mmc/host/sh_mobile_sdhi.c
> b/drivers/mmc/host/sh_mobile_sdhi.c index 87ed3fb..f344659 100644
> --- a/drivers/mmc/host/sh_mobile_sdhi.c
> +++ b/drivers/mmc/host/sh_mobile_sdhi.c
> @@ -113,14 +113,14 @@ static const struct sh_mobile_sdhi_ops sdhi_ops = {
> };
>
> static const struct of_device_id sh_mobile_sdhi_of_match[] = {
> - { .compatible = "renesas,shmobile-sdhi" },
> - { .compatible = "renesas,sh7372-sdhi" },
> - { .compatible = "renesas,sh73a0-sdhi", .data = &sh_mobile_sdhi_of_cfg[0],
> }, - { .compatible = "renesas,r8a73a4-sdhi", .data =
> &sh_mobile_sdhi_of_cfg[0], }, - { .compatible = "renesas,r8a7740-sdhi",
> .data = &sh_mobile_sdhi_of_cfg[0], }, - { .compatible =
> "renesas,r8a7778-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, - {
> .compatible = "renesas,r8a7779-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], },
> - { .compatible = "renesas,r8a7790-sdhi", .data =
> &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-shmobile" },
> + { .compatible = "renesas,sdhi-sh7372" },
> + { .compatible = "renesas,sdhi-sh73a0", .data = &sh_mobile_sdhi_of_cfg[0],
> }, + { .compatible = "renesas,sdhi-r8a73a4", .data =
> &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-r8a7740",
> .data = &sh_mobile_sdhi_of_cfg[0], }, + { .compatible =
> "renesas,sdhi-r8a7778", .data = &sh_mobile_sdhi_of_cfg[0], }, + {
> .compatible = "renesas,sdhi-r8a7779", .data = &sh_mobile_sdhi_of_cfg[0], },
> + { .compatible = "renesas,sdhi-r8a7790", .data =
> &sh_mobile_sdhi_of_cfg[0], }, {},
> };
> MODULE_DEVICE_TABLE(of, sh_mobile_sdhi_of_match);
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] ARM: shmobile: update SDHI DT compatibility string to the <unit>-<soc> format
Date: Thu, 29 Aug 2013 23:47:21 +0000 [thread overview]
Message-ID: <1621893.Re68yLczyK@avalon> (raw)
In-Reply-To: <Pine.LNX.4.64.1308291650120.3786@axis700.grange>
Hi Guennadi,
Thank you for the patch.
On Thursday 29 August 2013 17:14:49 Guennadi Liakhovetski wrote:
> Currently DT compatibility strings of both types can be found in the kernel
> sources: <unit>-<soc> and <soc>-<unit>, whereas a unique format should be
> followed and the former one is preferred. This patch converts the SDHI
> MMC driver and its users to the common standard. This is safe for now, since
> ATM no real products are using this driver with DT.
>
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>
> Ok, the "no real products" is an assumption, that Laurent and Simon kind
> of agreed upon, so, I'm using it here to avoid having to carry old
> compatibility strings with us around. If anyone has other information -
> please shout. Otherwise, I think, it would be the easiest for Chris to ack
> this and for Simon to pull this via the ARM tree for late 3.12 (possibly,
> post -rc1) inclusion.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 17 ++++++++++-------
> arch/arm/boot/dts/r8a73a4.dtsi | 6 +++---
> arch/arm/boot/dts/r8a7740.dtsi | 4 ++--
> arch/arm/boot/dts/r8a7790.dtsi | 8 ++++----
> arch/arm/boot/dts/sh73a0.dtsi | 6 +++---
> drivers/mmc/host/sh_mobile_sdhi.c | 16 ++++++++--------
> 6 files changed, 30 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt index df204e1..6a2a116
> 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -9,12 +9,15 @@ compulsory and any optional properties, common to all
> SD/MMC drivers, as described in mmc.txt, can be used. Additionally the
> following tmio_mmc-specific optional bindings can be used.
>
> +Required properties:
> +- compatible: "renesas,sdhi-shmobile" - a generic sh-mobile SDHI unit
> + "renesas,sdhi-sh7372" - SDHI IP on SH7372 SoC
> + "renesas,sdhi-sh73a0" - SDHI IP on SH73A0 SoC
> + "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> + "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7778" - SDHI IP on R8A7778 SoC
> + "renesas,sdhi-r8a7779" - SDHI IP on R8A7779 SoC
> + "renesas,sdhi-r8a7790" - SDHI IP on R8A7790 SoC
> +
> Optional properties:
> - toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
> -
> -When used with Renesas SDHI hardware, the following compatibility strings
> -configure various model-specific properties:
> -
> -"renesas,sh7372-sdhi": (default) compatible with SH7372
> -"renesas,r8a7740-sdhi": compatible with R8A7740: certain MMC/SD commands
> have to - wait for the interface to become idle.
> diff --git a/arch/arm/boot/dts/r8a73a4.dtsi b/arch/arm/boot/dts/r8a73a4.dtsi
> index b0511b1..cc7211b 100644
> --- a/arch/arm/boot/dts/r8a73a4.dtsi
> +++ b/arch/arm/boot/dts/r8a73a4.dtsi
> @@ -236,7 +236,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -245,7 +245,7 @@
> };
>
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -254,7 +254,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
> index 8fc4a5d..c4992b5 100644
> --- a/arch/arm/boot/dts/r8a7740.dtsi
> +++ b/arch/arm/boot/dts/r8a7740.dtsi
> @@ -155,7 +155,7 @@
> };
>
> sdhi0: sdhi@e6850000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6850000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 117 4
> @@ -167,7 +167,7 @@
> };
>
> sdhi1: sdhi@e6860000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6860000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 121 4
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index 3b879e7..885f9f4 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -152,7 +152,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -161,7 +161,7 @@
> };
>
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -170,7 +170,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> @@ -179,7 +179,7 @@
> };
>
> sdhi3: sdhi@ee160000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee160000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 168 4>;
> diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
> index ba59a58..3955c76 100644
> --- a/arch/arm/boot/dts/sh73a0.dtsi
> +++ b/arch/arm/boot/dts/sh73a0.dtsi
> @@ -196,7 +196,7 @@
> };
>
> sdhi0: sdhi@ee100000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee100000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 83 4
> @@ -208,7 +208,7 @@
>
> /* SDHI1 and SDHI2 have no CD pins, no need for CD IRQ */
> sdhi1: sdhi@ee120000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee120000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 88 4
> @@ -219,7 +219,7 @@
> };
>
> sdhi2: sdhi@ee140000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee140000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 104 4
> diff --git a/drivers/mmc/host/sh_mobile_sdhi.c
> b/drivers/mmc/host/sh_mobile_sdhi.c index 87ed3fb..f344659 100644
> --- a/drivers/mmc/host/sh_mobile_sdhi.c
> +++ b/drivers/mmc/host/sh_mobile_sdhi.c
> @@ -113,14 +113,14 @@ static const struct sh_mobile_sdhi_ops sdhi_ops = {
> };
>
> static const struct of_device_id sh_mobile_sdhi_of_match[] = {
> - { .compatible = "renesas,shmobile-sdhi" },
> - { .compatible = "renesas,sh7372-sdhi" },
> - { .compatible = "renesas,sh73a0-sdhi", .data = &sh_mobile_sdhi_of_cfg[0],
> }, - { .compatible = "renesas,r8a73a4-sdhi", .data > &sh_mobile_sdhi_of_cfg[0], }, - { .compatible = "renesas,r8a7740-sdhi",
> .data = &sh_mobile_sdhi_of_cfg[0], }, - { .compatible > "renesas,r8a7778-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, - {
> .compatible = "renesas,r8a7779-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], },
> - { .compatible = "renesas,r8a7790-sdhi", .data > &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-shmobile" },
> + { .compatible = "renesas,sdhi-sh7372" },
> + { .compatible = "renesas,sdhi-sh73a0", .data = &sh_mobile_sdhi_of_cfg[0],
> }, + { .compatible = "renesas,sdhi-r8a73a4", .data > &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-r8a7740",
> .data = &sh_mobile_sdhi_of_cfg[0], }, + { .compatible > "renesas,sdhi-r8a7778", .data = &sh_mobile_sdhi_of_cfg[0], }, + {
> .compatible = "renesas,sdhi-r8a7779", .data = &sh_mobile_sdhi_of_cfg[0], },
> + { .compatible = "renesas,sdhi-r8a7790", .data > &sh_mobile_sdhi_of_cfg[0], }, {},
> };
> MODULE_DEVICE_TABLE(of, sh_mobile_sdhi_of_match);
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: laurent.pinchart@ideasonboard.com (Laurent Pinchart)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: shmobile: update SDHI DT compatibility string to the <unit>-<soc> format
Date: Fri, 30 Aug 2013 01:47:21 +0200 [thread overview]
Message-ID: <1621893.Re68yLczyK@avalon> (raw)
In-Reply-To: <Pine.LNX.4.64.1308291650120.3786@axis700.grange>
Hi Guennadi,
Thank you for the patch.
On Thursday 29 August 2013 17:14:49 Guennadi Liakhovetski wrote:
> Currently DT compatibility strings of both types can be found in the kernel
> sources: <unit>-<soc> and <soc>-<unit>, whereas a unique format should be
> followed and the former one is preferred. This patch converts the SDHI
> MMC driver and its users to the common standard. This is safe for now, since
> ATM no real products are using this driver with DT.
>
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>
> Ok, the "no real products" is an assumption, that Laurent and Simon kind
> of agreed upon, so, I'm using it here to avoid having to carry old
> compatibility strings with us around. If anyone has other information -
> please shout. Otherwise, I think, it would be the easiest for Chris to ack
> this and for Simon to pull this via the ARM tree for late 3.12 (possibly,
> post -rc1) inclusion.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 17 ++++++++++-------
> arch/arm/boot/dts/r8a73a4.dtsi | 6 +++---
> arch/arm/boot/dts/r8a7740.dtsi | 4 ++--
> arch/arm/boot/dts/r8a7790.dtsi | 8 ++++----
> arch/arm/boot/dts/sh73a0.dtsi | 6 +++---
> drivers/mmc/host/sh_mobile_sdhi.c | 16 ++++++++--------
> 6 files changed, 30 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt index df204e1..6a2a116
> 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -9,12 +9,15 @@ compulsory and any optional properties, common to all
> SD/MMC drivers, as described in mmc.txt, can be used. Additionally the
> following tmio_mmc-specific optional bindings can be used.
>
> +Required properties:
> +- compatible: "renesas,sdhi-shmobile" - a generic sh-mobile SDHI unit
> + "renesas,sdhi-sh7372" - SDHI IP on SH7372 SoC
> + "renesas,sdhi-sh73a0" - SDHI IP on SH73A0 SoC
> + "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> + "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7778" - SDHI IP on R8A7778 SoC
> + "renesas,sdhi-r8a7779" - SDHI IP on R8A7779 SoC
> + "renesas,sdhi-r8a7790" - SDHI IP on R8A7790 SoC
> +
> Optional properties:
> - toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
> -
> -When used with Renesas SDHI hardware, the following compatibility strings
> -configure various model-specific properties:
> -
> -"renesas,sh7372-sdhi": (default) compatible with SH7372
> -"renesas,r8a7740-sdhi": compatible with R8A7740: certain MMC/SD commands
> have to - wait for the interface to become idle.
> diff --git a/arch/arm/boot/dts/r8a73a4.dtsi b/arch/arm/boot/dts/r8a73a4.dtsi
> index b0511b1..cc7211b 100644
> --- a/arch/arm/boot/dts/r8a73a4.dtsi
> +++ b/arch/arm/boot/dts/r8a73a4.dtsi
> @@ -236,7 +236,7 @@
> };
>
> sdhi0: sdhi at ee100000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -245,7 +245,7 @@
> };
>
> sdhi1: sdhi at ee120000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -254,7 +254,7 @@
> };
>
> sdhi2: sdhi at ee140000 {
> - compatible = "renesas,r8a73a4-sdhi";
> + compatible = "renesas,sdhi-r8a73a4";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
> index 8fc4a5d..c4992b5 100644
> --- a/arch/arm/boot/dts/r8a7740.dtsi
> +++ b/arch/arm/boot/dts/r8a7740.dtsi
> @@ -155,7 +155,7 @@
> };
>
> sdhi0: sdhi at e6850000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6850000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 117 4
> @@ -167,7 +167,7 @@
> };
>
> sdhi1: sdhi at e6860000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xe6860000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 121 4
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index 3b879e7..885f9f4 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -152,7 +152,7 @@
> };
>
> sdhi0: sdhi at ee100000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee100000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 165 4>;
> @@ -161,7 +161,7 @@
> };
>
> sdhi1: sdhi at ee120000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee120000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 166 4>;
> @@ -170,7 +170,7 @@
> };
>
> sdhi2: sdhi at ee140000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee140000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 167 4>;
> @@ -179,7 +179,7 @@
> };
>
> sdhi3: sdhi at ee160000 {
> - compatible = "renesas,r8a7790-sdhi";
> + compatible = "renesas,sdhi-r8a7790";
> reg = <0 0xee160000 0 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 168 4>;
> diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
> index ba59a58..3955c76 100644
> --- a/arch/arm/boot/dts/sh73a0.dtsi
> +++ b/arch/arm/boot/dts/sh73a0.dtsi
> @@ -196,7 +196,7 @@
> };
>
> sdhi0: sdhi at ee100000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee100000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 83 4
> @@ -208,7 +208,7 @@
>
> /* SDHI1 and SDHI2 have no CD pins, no need for CD IRQ */
> sdhi1: sdhi at ee120000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee120000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 88 4
> @@ -219,7 +219,7 @@
> };
>
> sdhi2: sdhi at ee140000 {
> - compatible = "renesas,r8a7740-sdhi";
> + compatible = "renesas,sdhi-r8a7740";
> reg = <0xee140000 0x100>;
> interrupt-parent = <&gic>;
> interrupts = <0 104 4
> diff --git a/drivers/mmc/host/sh_mobile_sdhi.c
> b/drivers/mmc/host/sh_mobile_sdhi.c index 87ed3fb..f344659 100644
> --- a/drivers/mmc/host/sh_mobile_sdhi.c
> +++ b/drivers/mmc/host/sh_mobile_sdhi.c
> @@ -113,14 +113,14 @@ static const struct sh_mobile_sdhi_ops sdhi_ops = {
> };
>
> static const struct of_device_id sh_mobile_sdhi_of_match[] = {
> - { .compatible = "renesas,shmobile-sdhi" },
> - { .compatible = "renesas,sh7372-sdhi" },
> - { .compatible = "renesas,sh73a0-sdhi", .data = &sh_mobile_sdhi_of_cfg[0],
> }, - { .compatible = "renesas,r8a73a4-sdhi", .data =
> &sh_mobile_sdhi_of_cfg[0], }, - { .compatible = "renesas,r8a7740-sdhi",
> .data = &sh_mobile_sdhi_of_cfg[0], }, - { .compatible =
> "renesas,r8a7778-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, - {
> .compatible = "renesas,r8a7779-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], },
> - { .compatible = "renesas,r8a7790-sdhi", .data =
> &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-shmobile" },
> + { .compatible = "renesas,sdhi-sh7372" },
> + { .compatible = "renesas,sdhi-sh73a0", .data = &sh_mobile_sdhi_of_cfg[0],
> }, + { .compatible = "renesas,sdhi-r8a73a4", .data =
> &sh_mobile_sdhi_of_cfg[0], }, + { .compatible = "renesas,sdhi-r8a7740",
> .data = &sh_mobile_sdhi_of_cfg[0], }, + { .compatible =
> "renesas,sdhi-r8a7778", .data = &sh_mobile_sdhi_of_cfg[0], }, + {
> .compatible = "renesas,sdhi-r8a7779", .data = &sh_mobile_sdhi_of_cfg[0], },
> + { .compatible = "renesas,sdhi-r8a7790", .data =
> &sh_mobile_sdhi_of_cfg[0], }, {},
> };
> MODULE_DEVICE_TABLE(of, sh_mobile_sdhi_of_match);
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2013-08-29 23:47 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-29 15:14 [PATCH] ARM: shmobile: update SDHI DT compatibility string to the <unit>-<soc> format Guennadi Liakhovetski
2013-08-29 15:14 ` Guennadi Liakhovetski
2013-08-29 15:14 ` Guennadi Liakhovetski
2013-08-29 23:47 ` Laurent Pinchart [this message]
2013-08-29 23:47 ` Laurent Pinchart
2013-08-29 23:47 ` Laurent Pinchart
2013-09-05 6:01 ` Simon Horman
2013-09-05 6:01 ` Simon Horman
2013-09-05 6:01 ` Simon Horman
2013-09-05 12:29 ` Chris Ball
2013-09-05 12:29 ` Chris Ball
2013-09-05 12:29 ` Chris Ball
[not found] ` <86ob87tpsi.fsf-zkTaxWjgBN6ccd+RBA7C9g@public.gmane.org>
2013-09-21 17:14 ` Simon Horman
2013-09-21 17:14 ` Simon Horman
2013-09-21 17:14 ` Simon Horman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1621893.Re68yLczyK@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=arnd@arndb.de \
--cc=cjb@laptop.org \
--cc=devicetree@vger.kernel.org \
--cc=g.liakhovetski@gmx.de \
--cc=horms@verge.net.au \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=olof@lixom.net \
--cc=swarren@wwwdotorg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.