* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 11:39 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-14 11:39 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland
Cc: Simon Horman, Magnus Damm, Chris Paterson, Wolfram Sang,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-mmc-u79uwXL29TY76Z2rM5mHXA, Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
---
This patch is compiled and tested againt linux next tag
next-20170811. This patch depend on the driver change
[PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
https://patchwork.kernel.org/patch/9898611/
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..0d507ec 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -34,8 +36,8 @@ Required properties:
"core" and "cd". If the controller only has 1 clock, naming is not
required.
Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
+ 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778, R8A7779,
+ R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
2: R7S72100
Optional properties:
--
1.9.1
--
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
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 11:39 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-14 11:39 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland
Cc: Simon Horman, Magnus Damm, Chris Paterson, Wolfram Sang,
devicetree, linux-renesas-soc, linux-arm-kernel, linux-mmc,
Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
This patch is compiled and tested againt linux next tag
next-20170811. This patch depend on the driver change
[PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
https://patchwork.kernel.org/patch/9898611/
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..0d507ec 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -34,8 +36,8 @@ Required properties:
"core" and "cd". If the controller only has 1 clock, naming is not
required.
Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
+ 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778, R8A7779,
+ R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
2: R7S72100
Optional properties:
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 11:39 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-14 11:39 UTC (permalink / raw)
To: linux-arm-kernel
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
This patch is compiled and tested againt linux next tag
next-20170811. This patch depend on the driver change
[PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
https://patchwork.kernel.org/patch/9898611/
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..0d507ec 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -34,8 +36,8 @@ Required properties:
"core" and "cd". If the controller only has 1 clock, naming is not
required.
Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
+ 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778, R8A7779,
+ R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
2: R7S72100
Optional properties:
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 11:39 ` Biju Das
@ 2017-08-17 20:48 ` Rob Herring
-1 siblings, 0 replies; 43+ messages in thread
From: Rob Herring @ 2017-08-17 20:48 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Mark Rutland, Simon Horman, Magnus Damm,
Chris Paterson, Wolfram Sang, devicetree, linux-renesas-soc,
linux-arm-kernel, linux-mmc
On Mon, Aug 14, 2017 at 12:39:26PM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> This patch is compiled and tested againt linux next tag
> next-20170811. This patch depend on the driver change
> [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
> https://patchwork.kernel.org/patch/9898611/
>
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-17 20:48 ` Rob Herring
0 siblings, 0 replies; 43+ messages in thread
From: Rob Herring @ 2017-08-17 20:48 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 14, 2017 at 12:39:26PM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> This patch is compiled and tested againt linux next tag
> next-20170811. This patch depend on the driver change
> [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
> https://patchwork.kernel.org/patch/9898611/
>
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 11:39 ` Biju Das
@ 2017-08-21 8:25 ` Wolfram Sang
-1 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 8:25 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Simon Horman,
Magnus Damm, Chris Paterson, Wolfram Sang, devicetree,
linux-renesas-soc, linux-arm-kernel, linux-mmc
[-- Attachment #1: Type: text/plain, Size: 491 bytes --]
> Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> + 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778, R8A7779,
> + R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> 2: R7S72100
Hmm, maybe we should convert this to only listing SoCs with two clocks?
That will be easier to maintaine since the majority seems to have one.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 8:25 ` Wolfram Sang
0 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 8:25 UTC (permalink / raw)
To: linux-arm-kernel
> Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> + 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778, R8A7779,
> + R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> 2: R7S72100
Hmm, maybe we should convert this to only listing SoCs with two clocks?
That will be easier to maintaine since the majority seems to have one.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170821/024ba7bd/attachment.sig>
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 8:25 ` Wolfram Sang
@ 2017-08-21 9:47 ` Biju Das
-1 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 9:47 UTC (permalink / raw)
To: Wolfram Sang
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Simon Horman,
Magnus Damm, Chris Paterson, Wolfram Sang, devicetree,
linux-renesas-soc, linux-arm-kernel, linux-mmc
> -----Original Message-----
> From: Wolfram Sang [mailto:wsa@the-dreams.de]
> Sent: 21 August 2017 09:25
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>; Rob Herring <robh+dt@kernel.org>;
> Mark Rutland <mark.rutland@arm.com>; Simon Horman
> <horms@verge.net.au>; Magnus Damm <magnus.damm@gmail.com>; Chris
> Paterson <Chris.Paterson2@renesas.com>; Wolfram Sang
> <wsa+renesas@sang-engineering.com>; devicetree@vger.kernel.org; linux-
> renesas-soc@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> mmc@vger.kernel.org
> Subject: Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
>
>
> > Below is the number clocks for each supported SoC:
> > - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> > - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> > + 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778,
> R8A7779,
> > + R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795,
> R8A7796
> > 2: R7S72100
>
> Hmm, maybe we should convert this to only listing SoCs with two clocks?
> That will be easier to maintaine since the majority seems to have one.
Ok. I will send v2 with the suggested changes.
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 9:47 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 9:47 UTC (permalink / raw)
To: linux-arm-kernel
> -----Original Message-----
> From: Wolfram Sang [mailto:wsa at the-dreams.de]
> Sent: 21 August 2017 09:25
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>; Rob Herring <robh+dt@kernel.org>;
> Mark Rutland <mark.rutland@arm.com>; Simon Horman
> <horms@verge.net.au>; Magnus Damm <magnus.damm@gmail.com>; Chris
> Paterson <Chris.Paterson2@renesas.com>; Wolfram Sang
> <wsa+renesas@sang-engineering.com>; devicetree at vger.kernel.org; linux-
> renesas-soc at vger.kernel.org; linux-arm-kernel at lists.infradead.org; linux-
> mmc at vger.kernel.org
> Subject: Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
>
>
> > Below is the number clocks for each supported SoC:
> > - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> > - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> > + 1: SH73A0, R8A73A4, R8A7740, R8A7743, R8A7745, R8A7778,
> R8A7779,
> > + R8A7790, R8A7791, R8A7792, R8A7793, R8A7794, R8A7795,
> R8A7796
> > 2: R7S72100
>
> Hmm, maybe we should convert this to only listing SoCs with two clocks?
> That will be easier to maintaine since the majority seems to have one.
Ok. I will send v2 with the suggested changes.
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 11:39 ` Biju Das
(?)
@ 2017-08-21 10:38 ` Biju Das
-1 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 10:38 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang
Cc: Simon Horman, Magnus Damm, Chris Paterson,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-mmc-u79uwXL29TY76Z2rM5mHXA, Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
---
v1->v2
-Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..1e3106e 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,9 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Most supported SoCs only have 1 clock. Devices which have more
+ than 1 clock are listed below:
+ 2: R7S72100"
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
--
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
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 10:38 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 10:38 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang
Cc: Simon Horman, Magnus Damm, Chris Paterson, devicetree,
linux-renesas-soc, linux-arm-kernel, linux-mmc, Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
v1->v2
-Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..1e3106e 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,9 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Most supported SoCs only have 1 clock. Devices which have more
+ than 1 clock are listed below:
+ 2: R7S72100"
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 10:38 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 10:38 UTC (permalink / raw)
To: linux-arm-kernel
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
v1->v2
-Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..1e3106e 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,9 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Most supported SoCs only have 1 clock. Devices which have more
+ than 1 clock are listed below:
+ 2: R7S72100"
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 10:38 ` Biju Das
(?)
@ 2017-08-21 11:30 ` Geert Uytterhoeven
-1 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-21 11:30 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang,
Simon Horman, Magnus Damm, Chris Paterson,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Linux MMC List
Hi Biju,
On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
> ---
> v1->v2
> -Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> index 4fd8b7a..1e3106e 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -15,6 +15,8 @@ Required properties:
> "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
> @@ -33,10 +35,9 @@ Required properties:
> If 2 clocks are specified by the hardware, you must name them as
> "core" and "cd". If the controller only has 1 clock, naming is not
> required.
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Most supported SoCs only have 1 clock. Devices which have more
That is actually mentioned in the paragraph above:
- clocks: Most controllers only have 1 clock source per channel. However, on
some variations of this controller, the internal card detection
logic that exists in this controller is sectioned off to be run by a
separate second clock source to allow the main core clock to be turned
off to save power.
so I suggest just integrating the (short) list in that paragraph.
> + than 1 clock are listed below:
> + 2: R7S72100"
Bogus trailing double quote.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
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
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 11:30 ` Geert Uytterhoeven
0 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-21 11:30 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang,
Simon Horman, Magnus Damm, Chris Paterson, devicetree,
Linux-Renesas, linux-arm-kernel, Linux MMC List
Hi Biju,
On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> v1->v2
> -Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> index 4fd8b7a..1e3106e 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -15,6 +15,8 @@ Required properties:
> "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
> @@ -33,10 +35,9 @@ Required properties:
> If 2 clocks are specified by the hardware, you must name them as
> "core" and "cd". If the controller only has 1 clock, naming is not
> required.
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Most supported SoCs only have 1 clock. Devices which have more
That is actually mentioned in the paragraph above:
- clocks: Most controllers only have 1 clock source per channel. However, on
some variations of this controller, the internal card detection
logic that exists in this controller is sectioned off to be run by a
separate second clock source to allow the main core clock to be turned
off to save power.
so I suggest just integrating the (short) list in that paragraph.
> + than 1 clock are listed below:
> + 2: R7S72100"
Bogus trailing double quote.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 11:30 ` Geert Uytterhoeven
0 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-21 11:30 UTC (permalink / raw)
To: linux-arm-kernel
Hi Biju,
On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> v1->v2
> -Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> index 4fd8b7a..1e3106e 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -15,6 +15,8 @@ Required properties:
> "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
> @@ -33,10 +35,9 @@ Required properties:
> If 2 clocks are specified by the hardware, you must name them as
> "core" and "cd". If the controller only has 1 clock, naming is not
> required.
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Most supported SoCs only have 1 clock. Devices which have more
That is actually mentioned in the paragraph above:
- clocks: Most controllers only have 1 clock source per channel. However, on
some variations of this controller, the internal card detection
logic that exists in this controller is sectioned off to be run by a
separate second clock source to allow the main core clock to be turned
off to save power.
so I suggest just integrating the (short) list in that paragraph.
> + than 1 clock are listed below:
> + 2: R7S72100"
Bogus trailing double quote.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 10:38 ` Biju Das
@ 2017-08-21 11:33 ` Wolfram Sang
-1 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 11:33 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Simon Horman,
Magnus Damm, Chris Paterson, devicetree, linux-renesas-soc,
linux-arm-kernel, linux-mmc
[-- Attachment #1: Type: text/plain, Size: 458 bytes --]
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Most supported SoCs only have 1 clock. Devices which have more
> + than 1 clock are listed below:
> + 2: R7S72100"
I like this, but stray " at the end of the line?
With that fixed:
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 11:33 ` Wolfram Sang
0 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 11:33 UTC (permalink / raw)
To: linux-arm-kernel
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Most supported SoCs only have 1 clock. Devices which have more
> + than 1 clock are listed below:
> + 2: R7S72100"
I like this, but stray " at the end of the line?
With that fixed:
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170821/c86ac8bf/attachment.sig>
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 11:30 ` Geert Uytterhoeven
(?)
@ 2017-08-21 12:13 ` Biju Das
-1 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:13 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang,
Simon Horman, Magnus Damm, Chris Paterson,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-Renesas,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Linux MMC List
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 4328 bytes --]
> -----Original Message-----
> From: devicetree-owner@vger.kernel.org [mailto:devicetree-
> owner@vger.kernel.org] On Behalf Of Geert Uytterhoeven
> Sent: 21 August 2017 12:30
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>; Rob Herring <robh+dt@kernel.org>;
> Mark Rutland <mark.rutland@arm.com>; Wolfram Sang <wsa@the-
> dreams.de>; Simon Horman <horms@verge.net.au>; Magnus Damm
> <magnus.damm@gmail.com>; Chris Paterson
> <Chris.Paterson2@renesas.com>; devicetree@vger.kernel.org; Linux-Renesas
> <linux-renesas-soc@vger.kernel.org>; linux-arm-kernel@lists.infradead.org;
> Linux MMC List <linux-mmc@vger.kernel.org>
> Subject: Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
>
> Hi Biju,
>
> On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das@bp.renesas.com>
> wrote:
> > Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > v1->v2
> > -Modified the listing SoC's description related to SoC
> > with 1 Clock and 2 Clocks
> >
> > This patch is compiled and tested against linux next tag
> > next-20170817.
> >
> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> > 1 file changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > index 4fd8b7a..1e3106e 100644
> > --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > @@ -15,6 +15,8 @@ Required properties:
> > "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> > "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> > "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> > + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> > + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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 @@
> > -33,10 +35,9 @@ Required properties:
> > If 2 clocks are specified by the hardware, you must name them as
> > "core" and "cd". If the controller only has 1 clock, naming is not
> > required.
> > - Below is the number clocks for each supported SoC:
> > - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> > - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> > - 2: R7S72100
> > + Most supported SoCs only have 1 clock. Devices which have
> > + more
>
> That is actually mentioned in the paragraph above:
>
> - clocks: Most controllers only have 1 clock source per channel. However, on
> some variations of this controller, the internal card detection
> logic that exists in this controller is sectioned off to be run by a
> separate second clock source to allow the main core clock to be turned
> off to save power.
>
> so I suggest just integrating the (short) list in that paragraph.
>
> > + than 1 clock are listed below:
> > + 2: R7S72100"
>
> Bogus trailing double quote.
Thanks. I will send v3 with above changes.
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in the body
> of a message to majordomo@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
N§²æìr¸yúèØb²X¬¶Ç§vØ^)Þº{.nÇ+·zøzÚÞz)í
æèw*\x1fjg¬±¨\x1e¶Ý¢j.ïÛ°\½½MúgjÌæa×\x02' ©Þ¢¸\f¢·¦j:+v¨wèjØm¶ÿ¾\a«êçzZ+ùÝ¢j"ú!¶i
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 12:13 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:13 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang,
Simon Horman, Magnus Damm, Chris Paterson, devicetree,
Linux-Renesas, linux-arm-kernel, Linux MMC List
> -----Original Message-----
> From: devicetree-owner@vger.kernel.org [mailto:devicetree-
> owner@vger.kernel.org] On Behalf Of Geert Uytterhoeven
> Sent: 21 August 2017 12:30
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>; Rob Herring <robh+dt@kernel.org>;
> Mark Rutland <mark.rutland@arm.com>; Wolfram Sang <wsa@the-
> dreams.de>; Simon Horman <horms@verge.net.au>; Magnus Damm
> <magnus.damm@gmail.com>; Chris Paterson
> <Chris.Paterson2@renesas.com>; devicetree@vger.kernel.org; Linux-Renesas
> <linux-renesas-soc@vger.kernel.org>; linux-arm-kernel@lists.infradead.org;
> Linux MMC List <linux-mmc@vger.kernel.org>
> Subject: Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
>
> Hi Biju,
>
> On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das@bp.renesas.com>
> wrote:
> > Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > v1->v2
> > -Modified the listing SoC's description related to SoC
> > with 1 Clock and 2 Clocks
> >
> > This patch is compiled and tested against linux next tag
> > next-20170817.
> >
> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> > 1 file changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > index 4fd8b7a..1e3106e 100644
> > --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > @@ -15,6 +15,8 @@ Required properties:
> > "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> > "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> > "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> > + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> > + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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 @@
> > -33,10 +35,9 @@ Required properties:
> > If 2 clocks are specified by the hardware, you must name them as
> > "core" and "cd". If the controller only has 1 clock, naming is not
> > required.
> > - Below is the number clocks for each supported SoC:
> > - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> > - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> > - 2: R7S72100
> > + Most supported SoCs only have 1 clock. Devices which have
> > + more
>
> That is actually mentioned in the paragraph above:
>
> - clocks: Most controllers only have 1 clock source per channel. However, on
> some variations of this controller, the internal card detection
> logic that exists in this controller is sectioned off to be run by a
> separate second clock source to allow the main core clock to be turned
> off to save power.
>
> so I suggest just integrating the (short) list in that paragraph.
>
> > + than 1 clock are listed below:
> > + 2: R7S72100"
>
> Bogus trailing double quote.
Thanks. I will send v3 with above changes.
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in the body
> of a message to majordomo@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 12:13 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:13 UTC (permalink / raw)
To: linux-arm-kernel
> -----Original Message-----
> From: devicetree-owner at vger.kernel.org [mailto:devicetree-
> owner at vger.kernel.org] On Behalf Of Geert Uytterhoeven
> Sent: 21 August 2017 12:30
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>; Rob Herring <robh+dt@kernel.org>;
> Mark Rutland <mark.rutland@arm.com>; Wolfram Sang <wsa@the-
> dreams.de>; Simon Horman <horms@verge.net.au>; Magnus Damm
> <magnus.damm@gmail.com>; Chris Paterson
> <Chris.Paterson2@renesas.com>; devicetree at vger.kernel.org; Linux-Renesas
> <linux-renesas-soc@vger.kernel.org>; linux-arm-kernel at lists.infradead.org;
> Linux MMC List <linux-mmc@vger.kernel.org>
> Subject: Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
>
> Hi Biju,
>
> On Mon, Aug 21, 2017 at 12:38 PM, Biju Das <biju.das@bp.renesas.com>
> wrote:
> > Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > v1->v2
> > -Modified the listing SoC's description related to SoC
> > with 1 Clock and 2 Clocks
> >
> > This patch is compiled and tested against linux next tag
> > next-20170817.
> >
> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> > 1 file changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > index 4fd8b7a..1e3106e 100644
> > --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> > @@ -15,6 +15,8 @@ Required properties:
> > "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> > "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> > "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> > + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> > + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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 @@
> > -33,10 +35,9 @@ Required properties:
> > If 2 clocks are specified by the hardware, you must name them as
> > "core" and "cd". If the controller only has 1 clock, naming is not
> > required.
> > - Below is the number clocks for each supported SoC:
> > - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> > - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> > - 2: R7S72100
> > + Most supported SoCs only have 1 clock. Devices which have
> > + more
>
> That is actually mentioned in the paragraph above:
>
> - clocks: Most controllers only have 1 clock source per channel. However, on
> some variations of this controller, the internal card detection
> logic that exists in this controller is sectioned off to be run by a
> separate second clock source to allow the main core clock to be turned
> off to save power.
>
> so I suggest just integrating the (short) list in that paragraph.
>
> > + than 1 clock are listed below:
> > + 2: R7S72100"
>
> Bogus trailing double quote.
Thanks. I will send v3 with above changes.
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in the body
> of a message to majordomo at vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 11:30 ` Geert Uytterhoeven
(?)
@ 2017-08-21 12:25 ` Biju Das
-1 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:25 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang
Cc: Simon Horman, Magnus Damm, Chris Paterson,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-mmc-u79uwXL29TY76Z2rM5mHXA, Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
---
v1->v2
- Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
v2->v3
- Fixed the trailing double quote.
- Updated listing SoC's description.
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..54ef642 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,8 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Devices which have more than 1 clock are listed below:
+ 2: R7S72100
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
--
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
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 12:25 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:25 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Mark Rutland, Wolfram Sang
Cc: Simon Horman, Magnus Damm, Chris Paterson, devicetree,
linux-renesas-soc, linux-arm-kernel, linux-mmc, Biju Das
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
v1->v2
- Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
v2->v3
- Fixed the trailing double quote.
- Updated listing SoC's description.
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..54ef642 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,8 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Devices which have more than 1 clock are listed below:
+ 2: R7S72100
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 12:25 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-21 12:25 UTC (permalink / raw)
To: linux-arm-kernel
Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
v1->v2
- Modified the listing SoC's description related to SoC
with 1 Clock and 2 Clocks
v2->v3
- Fixed the trailing double quote.
- Updated listing SoC's description.
This patch is compiled and tested against linux next tag
next-20170817.
Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
index 4fd8b7a..54ef642 100644
--- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
+++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
@@ -15,6 +15,8 @@ Required properties:
"renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
"renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
"renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
+ "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
+ "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
@@ -33,10 +35,8 @@ Required properties:
If 2 clocks are specified by the hardware, you must name them as
"core" and "cd". If the controller only has 1 clock, naming is not
required.
- Below is the number clocks for each supported SoC:
- 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
- R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
- 2: R7S72100
+ Devices which have more than 1 clock are listed below:
+ 2: R7S72100
Optional properties:
- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* Re: [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 12:25 ` Biju Das
@ 2017-08-21 12:43 ` Wolfram Sang
-1 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 12:43 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Rob Herring, Mark Rutland, Simon Horman,
Magnus Damm, Chris Paterson, devicetree, linux-renesas-soc,
linux-arm-kernel, linux-mmc
[-- Attachment #1: Type: text/plain, Size: 293 bytes --]
On Mon, Aug 21, 2017 at 01:25:11PM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 12:43 ` Wolfram Sang
0 siblings, 0 replies; 43+ messages in thread
From: Wolfram Sang @ 2017-08-21 12:43 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 21, 2017 at 01:25:11PM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170821/ce9d0c85/attachment.sig>
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 10:38 ` Biju Das
@ 2017-08-21 23:34 ` Rob Herring
-1 siblings, 0 replies; 43+ messages in thread
From: Rob Herring @ 2017-08-21 23:34 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Mark Rutland, Wolfram Sang, Simon Horman,
Magnus Damm, Chris Paterson, devicetree, linux-renesas-soc,
linux-arm-kernel, linux-mmc
On Mon, Aug 21, 2017 at 11:38:17AM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> v1->v2
> -Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v2] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-21 23:34 ` Rob Herring
0 siblings, 0 replies; 43+ messages in thread
From: Rob Herring @ 2017-08-21 23:34 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 21, 2017 at 11:38:17AM +0100, Biju Das wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> v1->v2
> -Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 12:43 ` Wolfram Sang
@ 2017-08-22 8:00 ` Simon Horman
-1 siblings, 0 replies; 43+ messages in thread
From: Simon Horman @ 2017-08-22 8:00 UTC (permalink / raw)
To: Wolfram Sang
Cc: Biju Das, Ulf Hansson, Rob Herring, Mark Rutland, Magnus Damm,
Chris Paterson, devicetree, linux-renesas-soc, linux-arm-kernel,
linux-mmc
On Mon, Aug 21, 2017 at 02:43:05PM +0200, Wolfram Sang wrote:
> On Mon, Aug 21, 2017 at 01:25:11PM +0100, Biju Das wrote:
> > Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> > is identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
>
> Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Its probably time for some fallback compatibility strings but
I think that should be done as follow-up (likely by me) in a considered
way. So this patch is fine in my books.
Acked-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-22 8:00 ` Simon Horman
0 siblings, 0 replies; 43+ messages in thread
From: Simon Horman @ 2017-08-22 8:00 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 21, 2017 at 02:43:05PM +0200, Wolfram Sang wrote:
> On Mon, Aug 21, 2017 at 01:25:11PM +0100, Biju Das wrote:
> > Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> > is identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
>
> Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Its probably time for some fallback compatibility strings but
I think that should be done as follow-up (likely by me) in a considered
way. So this patch is fine in my books.
Acked-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-21 12:25 ` Biju Das
@ 2017-08-22 11:13 ` Ulf Hansson
-1 siblings, 0 replies; 43+ messages in thread
From: Ulf Hansson @ 2017-08-22 11:13 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Mark Rutland, Wolfram Sang, Simon Horman,
Magnus Damm, Chris Paterson, devicetree, Linux-Renesas,
linux-arm-kernel, linux-mmc
On 21 August 2017 at 14:25, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Thanks, applied for next!
Kind regards
Uffe
> ---
> v1->v2
> - Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
> v2->v3
> - Fixed the trailing double quote.
> - Updated listing SoC's description.
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> index 4fd8b7a..54ef642 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -15,6 +15,8 @@ Required properties:
> "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
> @@ -33,10 +35,8 @@ Required properties:
> If 2 clocks are specified by the hardware, you must name them as
> "core" and "cd". If the controller only has 1 clock, naming is not
> required.
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Devices which have more than 1 clock are listed below:
> + 2: R7S72100
>
> Optional properties:
> - toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH v3] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-22 11:13 ` Ulf Hansson
0 siblings, 0 replies; 43+ messages in thread
From: Ulf Hansson @ 2017-08-22 11:13 UTC (permalink / raw)
To: linux-arm-kernel
On 21 August 2017 at 14:25, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC. Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Thanks, applied for next!
Kind regards
Uffe
> ---
> v1->v2
> - Modified the listing SoC's description related to SoC
> with 1 Clock and 2 Clocks
> v2->v3
> - Fixed the trailing double quote.
> - Updated listing SoC's description.
>
> This patch is compiled and tested against linux next tag
> next-20170817.
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> index 4fd8b7a..54ef642 100644
> --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt
> @@ -15,6 +15,8 @@ Required properties:
> "renesas,sdhi-r7s72100" - SDHI IP on R7S72100 SoC
> "renesas,sdhi-r8a73a4" - SDHI IP on R8A73A4 SoC
> "renesas,sdhi-r8a7740" - SDHI IP on R8A7740 SoC
> + "renesas,sdhi-r8a7743" - SDHI IP on R8A7743 SoC
> + "renesas,sdhi-r8a7745" - SDHI IP on R8A7745 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
> @@ -33,10 +35,8 @@ Required properties:
> If 2 clocks are specified by the hardware, you must name them as
> "core" and "cd". If the controller only has 1 clock, naming is not
> required.
> - Below is the number clocks for each supported SoC:
> - 1: SH73A0, R8A73A4, R8A7740, R8A7778, R8A7779, R8A7790
> - R8A7791, R8A7792, R8A7793, R8A7794, R8A7795, R8A7796
> - 2: R7S72100
> + Devices which have more than 1 clock are listed below:
> + 2: R7S72100
>
> Optional properties:
> - toshiba,mmc-wrprotect-disable: write-protect detection is unavailable
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 15:20 ` Geert Uytterhoeven
@ 2017-08-14 15:23 ` Chris Paterson
-1 siblings, 0 replies; 43+ messages in thread
From: Chris Paterson @ 2017-08-14 15:23 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Biju Das, Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, devicetree, Linux-Renesas,
linux-arm-kernel, Linux MMC List
Hello Geert,
> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]
> On Behalf Of Geert Uytterhoeven
>
> Hi Chris,
>
> On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson
> <Chris.Paterson2@renesas.com> wrote:
> >> From: geert.uytterhoeven@gmail.com
> >> [mailto:geert.uytterhoeven@gmail.com]
> >> On Behalf Of Geert Uytterhoeven
> >> Sent: 14 August 2017 15:22
> >>
> >> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com>
> wrote:
> >> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> >> > identical to the R-Car Gen2 family.
> >> >
> >> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> >> > ---
> >> > This patch is compiled and tested against mmc/next.
> >> >
> >> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> >> > 1 file changed, 2 insertions(+)
> >> >
> >> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > index 718cb8a..f709f7f 100644
> >> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > @@ -95,6 +95,8 @@
> >> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible,
> },
> >> > { .compatible = "renesas,sdhi-r8a7778", .data =
> >> &of_rcar_gen1_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7779", .data =
> >> > &of_rcar_gen1_compatible, },
> >> > + { .compatible = "renesas,sdhi-r8a7743", .data =
> >> &of_rcar_gen2_compatible, },
> >> > + { .compatible = "renesas,sdhi-r8a7745", .data =
> >> > + &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7790", .data =
> >> &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7791", .data =
> >> &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7792", .data =
> >> > &of_rcar_gen2_compatible, },
> >>
> >> I guess we want generic R-Car Gen2/3 compatible values here, too?
> >
> > Yes, but I assume that should be a separate patch submitted instead of this
> one?
>
> Indeed. That would avoid having to add the SoC-specific compatible values to
> the driver for all RZ/G1 variants.
Okay. I'll send a new patch.
Kind regards, Chris
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 15:23 ` Chris Paterson
0 siblings, 0 replies; 43+ messages in thread
From: Chris Paterson @ 2017-08-14 15:23 UTC (permalink / raw)
To: linux-arm-kernel
Hello Geert,
> From: geert.uytterhoeven at gmail.com [mailto:geert.uytterhoeven at gmail.com]
> On Behalf Of Geert Uytterhoeven
>
> Hi Chris,
>
> On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson
> <Chris.Paterson2@renesas.com> wrote:
> >> From: geert.uytterhoeven at gmail.com
> >> [mailto:geert.uytterhoeven at gmail.com]
> >> On Behalf Of Geert Uytterhoeven
> >> Sent: 14 August 2017 15:22
> >>
> >> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com>
> wrote:
> >> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> >> > identical to the R-Car Gen2 family.
> >> >
> >> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> >> > ---
> >> > This patch is compiled and tested against mmc/next.
> >> >
> >> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> >> > 1 file changed, 2 insertions(+)
> >> >
> >> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > index 718cb8a..f709f7f 100644
> >> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> >> > @@ -95,6 +95,8 @@
> >> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible,
> },
> >> > { .compatible = "renesas,sdhi-r8a7778", .data =
> >> &of_rcar_gen1_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7779", .data =
> >> > &of_rcar_gen1_compatible, },
> >> > + { .compatible = "renesas,sdhi-r8a7743", .data =
> >> &of_rcar_gen2_compatible, },
> >> > + { .compatible = "renesas,sdhi-r8a7745", .data =
> >> > + &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7790", .data =
> >> &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7791", .data =
> >> &of_rcar_gen2_compatible, },
> >> > { .compatible = "renesas,sdhi-r8a7792", .data =
> >> > &of_rcar_gen2_compatible, },
> >>
> >> I guess we want generic R-Car Gen2/3 compatible values here, too?
> >
> > Yes, but I assume that should be a separate patch submitted instead of this
> one?
>
> Indeed. That would avoid having to add the SoC-specific compatible values to
> the driver for all RZ/G1 variants.
Okay. I'll send a new patch.
Kind regards, Chris
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 15:15 ` Chris Paterson
(?)
@ 2017-08-14 15:20 ` Geert Uytterhoeven
-1 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-14 15:20 UTC (permalink / raw)
To: Chris Paterson
Cc: Biju Das, Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, devicetree-u79uwXL29TY76Z2rM5mHXA,
Linux-Renesas, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Linux MMC List
Hi Chris,
On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson
<Chris.Paterson2-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> wrote:
>> From: geert.uytterhoeven-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org [mailto:geert.uytterhoeven-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org]
>> On Behalf Of Geert Uytterhoeven
>> Sent: 14 August 2017 15:22
>>
>> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> wrote:
>> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
>> > identical to the R-Car Gen2 family.
>> >
>> > Signed-off-by: Biju Das <biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
>> > ---
>> > This patch is compiled and tested against mmc/next.
>> >
>> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
>> > 1 file changed, 2 insertions(+)
>> >
>> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > index 718cb8a..f709f7f 100644
>> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > @@ -95,6 +95,8 @@
>> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7778", .data =
>> &of_rcar_gen1_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7779", .data =
>> > &of_rcar_gen1_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7743", .data =
>> &of_rcar_gen2_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7745", .data =
>> > + &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7790", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7791", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7792", .data =
>> > &of_rcar_gen2_compatible, },
>>
>> I guess we want generic R-Car Gen2/3 compatible values here, too?
>
> Yes, but I assume that should be a separate patch submitted instead of this one?
Indeed. That would avoid having to add the SoC-specific compatible values
to the driver for all RZ/G1 variants.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
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
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 15:20 ` Geert Uytterhoeven
0 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-14 15:20 UTC (permalink / raw)
To: Chris Paterson
Cc: Biju Das, Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, devicetree, Linux-Renesas,
linux-arm-kernel, Linux MMC List
Hi Chris,
On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson
<Chris.Paterson2@renesas.com> wrote:
>> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]
>> On Behalf Of Geert Uytterhoeven
>> Sent: 14 August 2017 15:22
>>
>> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
>> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
>> > identical to the R-Car Gen2 family.
>> >
>> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
>> > ---
>> > This patch is compiled and tested against mmc/next.
>> >
>> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
>> > 1 file changed, 2 insertions(+)
>> >
>> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > index 718cb8a..f709f7f 100644
>> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > @@ -95,6 +95,8 @@
>> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7778", .data =
>> &of_rcar_gen1_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7779", .data =
>> > &of_rcar_gen1_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7743", .data =
>> &of_rcar_gen2_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7745", .data =
>> > + &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7790", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7791", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7792", .data =
>> > &of_rcar_gen2_compatible, },
>>
>> I guess we want generic R-Car Gen2/3 compatible values here, too?
>
> Yes, but I assume that should be a separate patch submitted instead of this one?
Indeed. That would avoid having to add the SoC-specific compatible values
to the driver for all RZ/G1 variants.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 15:20 ` Geert Uytterhoeven
0 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-14 15:20 UTC (permalink / raw)
To: linux-arm-kernel
Hi Chris,
On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson
<Chris.Paterson2@renesas.com> wrote:
>> From: geert.uytterhoeven at gmail.com [mailto:geert.uytterhoeven at gmail.com]
>> On Behalf Of Geert Uytterhoeven
>> Sent: 14 August 2017 15:22
>>
>> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
>> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
>> > identical to the R-Car Gen2 family.
>> >
>> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
>> > ---
>> > This patch is compiled and tested against mmc/next.
>> >
>> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
>> > 1 file changed, 2 insertions(+)
>> >
>> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > index 718cb8a..f709f7f 100644
>> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
>> > @@ -95,6 +95,8 @@
>> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7778", .data =
>> &of_rcar_gen1_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7779", .data =
>> > &of_rcar_gen1_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7743", .data =
>> &of_rcar_gen2_compatible, },
>> > + { .compatible = "renesas,sdhi-r8a7745", .data =
>> > + &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7790", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7791", .data =
>> &of_rcar_gen2_compatible, },
>> > { .compatible = "renesas,sdhi-r8a7792", .data =
>> > &of_rcar_gen2_compatible, },
>>
>> I guess we want generic R-Car Gen2/3 compatible values here, too?
>
> Yes, but I assume that should be a separate patch submitted instead of this one?
Indeed. That would avoid having to add the SoC-specific compatible values
to the driver for all RZ/G1 variants.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 14:22 ` Geert Uytterhoeven
(?)
@ 2017-08-14 15:15 ` Chris Paterson
-1 siblings, 0 replies; 43+ messages in thread
From: Chris Paterson @ 2017-08-14 15:15 UTC (permalink / raw)
To: Geert Uytterhoeven, Biju Das
Cc: Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, devicetree-u79uwXL29TY76Z2rM5mHXA,
Linux-Renesas, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Linux MMC List
> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]
> On Behalf Of Geert Uytterhoeven
> Sent: 14 August 2017 15:22
>
> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > This patch is compiled and tested against mmc/next.
> >
> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > index 718cb8a..f709f7f 100644
> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > @@ -95,6 +95,8 @@
> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
> > { .compatible = "renesas,sdhi-r8a7778", .data =
> &of_rcar_gen1_compatible, },
> > { .compatible = "renesas,sdhi-r8a7779", .data =
> > &of_rcar_gen1_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7743", .data =
> &of_rcar_gen2_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7745", .data =
> > + &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7790", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7791", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7792", .data =
> > &of_rcar_gen2_compatible, },
>
> I guess we want generic R-Car Gen2/3 compatible values here, too?
Yes, but I assume that should be a separate patch submitted instead of this one?
Kind regards, Chris
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* RE: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 15:15 ` Chris Paterson
0 siblings, 0 replies; 43+ messages in thread
From: Chris Paterson @ 2017-08-14 15:15 UTC (permalink / raw)
To: Geert Uytterhoeven, Biju Das
Cc: Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, devicetree, Linux-Renesas,
linux-arm-kernel, Linux MMC List
> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]
> On Behalf Of Geert Uytterhoeven
> Sent: 14 August 2017 15:22
>
> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > This patch is compiled and tested against mmc/next.
> >
> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > index 718cb8a..f709f7f 100644
> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > @@ -95,6 +95,8 @@
> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
> > { .compatible = "renesas,sdhi-r8a7778", .data =
> &of_rcar_gen1_compatible, },
> > { .compatible = "renesas,sdhi-r8a7779", .data =
> > &of_rcar_gen1_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7743", .data =
> &of_rcar_gen2_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7745", .data =
> > + &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7790", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7791", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7792", .data =
> > &of_rcar_gen2_compatible, },
>
> I guess we want generic R-Car Gen2/3 compatible values here, too?
Yes, but I assume that should be a separate patch submitted instead of this one?
Kind regards, Chris
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 15:15 ` Chris Paterson
0 siblings, 0 replies; 43+ messages in thread
From: Chris Paterson @ 2017-08-14 15:15 UTC (permalink / raw)
To: linux-arm-kernel
> From: geert.uytterhoeven at gmail.com [mailto:geert.uytterhoeven at gmail.com]
> On Behalf Of Geert Uytterhoeven
> Sent: 14 August 2017 15:22
>
> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is
> > identical to the R-Car Gen2 family.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > ---
> > This patch is compiled and tested against mmc/next.
> >
> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > index 718cb8a..f709f7f 100644
> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > @@ -95,6 +95,8 @@
> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
> > { .compatible = "renesas,sdhi-r8a7778", .data =
> &of_rcar_gen1_compatible, },
> > { .compatible = "renesas,sdhi-r8a7779", .data =
> > &of_rcar_gen1_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7743", .data =
> &of_rcar_gen2_compatible, },
> > + { .compatible = "renesas,sdhi-r8a7745", .data =
> > + &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7790", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7791", .data =
> &of_rcar_gen2_compatible, },
> > { .compatible = "renesas,sdhi-r8a7792", .data =
> > &of_rcar_gen2_compatible, },
>
> I guess we want generic R-Car Gen2/3 compatible values here, too?
Yes, but I assume that should be a separate patch submitted instead of this one?
Kind regards, Chris
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
2017-08-14 11:19 ` Biju Das
@ 2017-08-14 14:22 ` Geert Uytterhoeven
-1 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-14 14:22 UTC (permalink / raw)
To: Biju Das
Cc: Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland,
Simon Horman, Magnus Damm, Chris Paterson, devicetree,
Linux-Renesas, linux-arm-kernel, Linux MMC List
On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> This patch is compiled and tested against mmc/next.
>
> drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> index 718cb8a..f709f7f 100644
> --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> @@ -95,6 +95,8 @@
> { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
> { .compatible = "renesas,sdhi-r8a7778", .data = &of_rcar_gen1_compatible, },
> { .compatible = "renesas,sdhi-r8a7779", .data = &of_rcar_gen1_compatible, },
> + { .compatible = "renesas,sdhi-r8a7743", .data = &of_rcar_gen2_compatible, },
> + { .compatible = "renesas,sdhi-r8a7745", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7790", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7791", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7792", .data = &of_rcar_gen2_compatible, },
I guess we want generic R-Car Gen2/3 compatible values here, too?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 14:22 ` Geert Uytterhoeven
0 siblings, 0 replies; 43+ messages in thread
From: Geert Uytterhoeven @ 2017-08-14 14:22 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Aug 14, 2017 at 1:19 PM, Biju Das <biju.das@bp.renesas.com> wrote:
> Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI
> is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> ---
> This patch is compiled and tested against mmc/next.
>
> drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> index 718cb8a..f709f7f 100644
> --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> @@ -95,6 +95,8 @@
> { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
> { .compatible = "renesas,sdhi-r8a7778", .data = &of_rcar_gen1_compatible, },
> { .compatible = "renesas,sdhi-r8a7779", .data = &of_rcar_gen1_compatible, },
> + { .compatible = "renesas,sdhi-r8a7743", .data = &of_rcar_gen2_compatible, },
> + { .compatible = "renesas,sdhi-r8a7745", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7790", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7791", .data = &of_rcar_gen2_compatible, },
> { .compatible = "renesas,sdhi-r8a7792", .data = &of_rcar_gen2_compatible, },
I guess we want generic R-Car Gen2/3 compatible values here, too?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 11:19 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-14 11:19 UTC (permalink / raw)
To: Ulf Hansson, Wolfram Sang, Rob Herring, Mark Rutland
Cc: Simon Horman, Magnus Damm, Chris Paterson, devicetree,
linux-renesas-soc, linux-arm-kernel, linux-mmc, Biju Das
Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
This patch is compiled and tested against mmc/next.
drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
index 718cb8a..f709f7f 100644
--- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
+++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
@@ -95,6 +95,8 @@
{ .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
{ .compatible = "renesas,sdhi-r8a7778", .data = &of_rcar_gen1_compatible, },
{ .compatible = "renesas,sdhi-r8a7779", .data = &of_rcar_gen1_compatible, },
+ { .compatible = "renesas,sdhi-r8a7743", .data = &of_rcar_gen2_compatible, },
+ { .compatible = "renesas,sdhi-r8a7745", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7790", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7791", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7792", .data = &of_rcar_gen2_compatible, },
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
* [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support
@ 2017-08-14 11:19 ` Biju Das
0 siblings, 0 replies; 43+ messages in thread
From: Biju Das @ 2017-08-14 11:19 UTC (permalink / raw)
To: linux-arm-kernel
Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI
is identical to the R-Car Gen2 family.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
---
This patch is compiled and tested against mmc/next.
drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
index 718cb8a..f709f7f 100644
--- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c
+++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c
@@ -95,6 +95,8 @@
{ .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, },
{ .compatible = "renesas,sdhi-r8a7778", .data = &of_rcar_gen1_compatible, },
{ .compatible = "renesas,sdhi-r8a7779", .data = &of_rcar_gen1_compatible, },
+ { .compatible = "renesas,sdhi-r8a7743", .data = &of_rcar_gen2_compatible, },
+ { .compatible = "renesas,sdhi-r8a7745", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7790", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7791", .data = &of_rcar_gen2_compatible, },
{ .compatible = "renesas,sdhi-r8a7792", .data = &of_rcar_gen2_compatible, },
--
1.9.1
^ permalink raw reply related [flat|nested] 43+ messages in thread
end of thread, other threads:[~2017-08-22 11:13 UTC | newest]
Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-14 11:39 [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support Biju Das
2017-08-14 11:39 ` Biju Das
2017-08-14 11:39 ` Biju Das
2017-08-17 20:48 ` Rob Herring
2017-08-17 20:48 ` Rob Herring
2017-08-21 8:25 ` Wolfram Sang
2017-08-21 8:25 ` Wolfram Sang
2017-08-21 9:47 ` Biju Das
2017-08-21 9:47 ` Biju Das
[not found] ` <1502710766-44361-1-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
2017-08-21 10:38 ` [PATCH v2] " Biju Das
2017-08-21 10:38 ` Biju Das
2017-08-21 10:38 ` Biju Das
[not found] ` <1503311897-18807-1-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org>
2017-08-21 11:30 ` Geert Uytterhoeven
2017-08-21 11:30 ` Geert Uytterhoeven
2017-08-21 11:30 ` Geert Uytterhoeven
[not found] ` <CAMuHMdWUxxp75u1VSLY3WnObmhSMDvP3skr9Uf3rV4Bf-ixy_w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-08-21 12:13 ` Biju Das
2017-08-21 12:13 ` Biju Das
2017-08-21 12:13 ` Biju Das
2017-08-21 12:25 ` [PATCH v3] " Biju Das
2017-08-21 12:25 ` Biju Das
2017-08-21 12:25 ` Biju Das
2017-08-21 12:43 ` Wolfram Sang
2017-08-21 12:43 ` Wolfram Sang
2017-08-22 8:00 ` Simon Horman
2017-08-22 8:00 ` Simon Horman
2017-08-22 11:13 ` Ulf Hansson
2017-08-22 11:13 ` Ulf Hansson
2017-08-21 11:33 ` [PATCH v2] " Wolfram Sang
2017-08-21 11:33 ` Wolfram Sang
2017-08-21 23:34 ` Rob Herring
2017-08-21 23:34 ` Rob Herring
-- strict thread matches above, loose matches on Subject: below --
2017-08-14 11:19 [PATCH ] " Biju Das
2017-08-14 11:19 ` Biju Das
2017-08-14 14:22 ` Geert Uytterhoeven
2017-08-14 14:22 ` Geert Uytterhoeven
[not found] ` <CAMuHMdVDqgbwKzxFoxajLLFHZC80NT5vrDLnpb9sFj+DL-1zDw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-08-14 15:15 ` Chris Paterson
2017-08-14 15:15 ` Chris Paterson
2017-08-14 15:15 ` Chris Paterson
[not found] ` <TY1PR06MB070269842CD923E4DE4FE6EDB78C0-/PRLmSCtZ14u2TXDOttQZW0DtJ1/0DrXvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2017-08-14 15:20 ` Geert Uytterhoeven
2017-08-14 15:20 ` Geert Uytterhoeven
2017-08-14 15:20 ` Geert Uytterhoeven
2017-08-14 15:23 ` Chris Paterson
2017-08-14 15:23 ` Chris Paterson
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.