From: Simon Horman <horms@verge.net.au>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
Chris Ball <cjb@laptop.org>
Cc: linux-sh@vger.kernel.org, Olof Johansson <olof@lixom.net>,
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
Subject: Re: [PATCH] ARM: shmobile: update SDHI DT compatibility string to the <unit>-<soc> format
Date: Thu, 5 Sep 2013 15:01:03 +0900 [thread overview]
Message-ID: <20130905060102.GH14960@verge.net.au> (raw)
In-Reply-To: <1621893.Re68yLczyK@avalon>
On Fri, Aug 30, 2013 at 01:47:21AM +0200, Laurent Pinchart wrote:
> 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.
Thanks, I believe that is a good plan.
Chris, could we get an Ack?
> >
> > 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
>
next prev parent reply other threads:[~2013-09-05 6:01 UTC|newest]
Thread overview: 5+ 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 23:47 ` Laurent Pinchart
2013-09-05 6:01 ` Simon Horman [this message]
2013-09-05 12:29 ` Chris Ball
[not found] ` <86ob87tpsi.fsf-zkTaxWjgBN6ccd+RBA7C9g@public.gmane.org>
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=20130905060102.GH14960@verge.net.au \
--to=horms@verge.net.au \
--cc=arnd@arndb.de \
--cc=cjb@laptop.org \
--cc=devicetree@vger.kernel.org \
--cc=g.liakhovetski@gmx.de \
--cc=laurent.pinchart@ideasonboard.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).