Fourth Round of Renesas ARM Based SoC Clock Updates for v3.17

GIT PULL New Simon Horman 5 participants 9 messages Jul 31, 2014 – Aug 22, 2014

Subsystems (4)

Arm (32-bit) Arm sub-architectures SuperH Arm/RISC-V/Renesas architecture

lore

Share & tools

lore

Simon Horman

author

2014-07-31 00:33 UTC #0 lore raw

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these fourth round of Renesas ARM based SoC clock updates for v3.17.

I am sending this even though v3.16-rc6 has long since been released as this pull-request only includes a single one line fix for a problem introduced in patches you have already queued up for v3.16.

At it turns out the problem should not manifest at run time as it is an error in an clock that is not used without DT modifications. As such I am happy for you to ask me to defer this to v3.18 if you like.

This pull request is based on the previous round of such requests, tagged as renesas-clock3-for-v3.17, which you have already merged.

The following changes since commit a0f7e7496d56ac2da7c684e2035815318c17973a:

ARM: shmobile: sh73a0: add CMT1 clock support for DT (2014-07-15 13:34:17 +0900)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-clock4-for-v3.17

for you to fetch changes up to 2c3758b01aa6f11382d2b018614b5d48ecf30350:

ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name (2014-07-16 23:09:18 +0900)

----------------------------------------------------------------
Fourth Round of Renesas ARM Based SoC Clock Updates for v3.17

* Remove spurious 0x from SCIFB clock name

----------------------------------------------------------------
Simon Horman (1):

      ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Simon Horman

author

2014-07-31 00:33 UTC #1 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.

This change should not have any run-time affect at this time as the clock in question is used by a SCIF device that is not enabled by default.

Reported-by Sergei Shtylyov <[email protected]>

Signed-off-by Simon Horman <[email protected]>

---
 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

arch/arm/mach-shmobile/clock-sh73a0.c +1 -1

index 0d77f65..b0993a5 100644
@@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
 	CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
 	CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
 	CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
-	CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
+	CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
 	CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
 	CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
 	CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */

Signature · 1 line

2.0.1

Olof Johansson

2014-07-31 00:35 UTC #2 lore raw

In reply to Simon Horman

Quoted 9 lines · Simon Horman

Hi Olof, Hi Kevin, Hi Arnd,
Please consider these fourth round of Renesas ARM based SoC clock updates for v3.17.
I am sending this even though v3.16-rc6 has long since been released as this pull-request only includes a single one line fix for a problem introduced in patches you have already queued up for v3.16.

Quoted 7 lines

At it turns out the problem should not manifest at run time as it is an error in an clock that is not used without DT modifications. As such I am happy for you to ask me to defer this to v3.18 if you like.
This pull request is based on the previous round of such requests, tagged as renesas-clock3-for-v3.17, which you have already merged.

It is less work to just apply a patch when there's a single one like this. Please just send it and ask us to apply it on top of the previous branch.

-Olof

Simon Horman

2014-07-31 00:42 UTC #3 lore raw

Quoted 15 lines · Olof Johansson

Quoted 13 lines · Simon Horman
Hi Olof, Hi Kevin, Hi Arnd,
Please consider these fourth round of Renesas ARM based SoC clock updates for v3.17.
I am sending this even though v3.16-rc6 has long since been released as this pull-request only includes a single one line fix for a problem introduced in patches you have already queued up for v3.16.
At it turns out the problem should not manifest at run time as it is an error in an clock that is not used without DT modifications. As such I am happy for you to ask me to defer this to v3.18 if you like.

Quoted 7 lines

Quoted 3 lines
This pull request is based on the previous round of such requests, tagged as renesas-clock3-for-v3.17, which you have already merged.
It is less work to just apply a patch when there's a single one like this. Please just send it and ask us to apply it on top of the previous branch.

Thanks, I wasn't aware of that.

The patch was sent as a reply to this email, though you were not CCed on the patch email. I will repost it with you CCed.

Simon Horman

author

2014-07-31 00:42 UTC #4 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

In reply to Simon Horman

A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.

This change should not have any run-time affect at this time as the clock in question is used by a SCIF device that is not enabled by default.

Reported-by Sergei Shtylyov <[email protected]>

Signed-off-by Simon Horman <[email protected]>

---
 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Hi Olof,

this is a repost of the patch you asked for so you can apply it directly rather than as pulling a pull request.

arch/arm/mach-shmobile/clock-sh73a0.c +1 -1

index 0d77f65..b0993a5 100644
@@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
 	CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
 	CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
 	CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
-	CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
+	CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
 	CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
 	CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
 	CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */

Signature · 1 line

2.0.1

Andreas Färber

2014-07-31 04:07 UTC #5 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

Am 31.07.2014 02:42, schrieb Simon Horman:

Quoted 4 lines

A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.
This change should not have any run-time affect at this time as

"effect", in case Olof wants to fix it up.

Quoted 5 lines

the clock in question is used by a SCIF device that is not enabled by default.
Reported-by Sergei Shtylyov <[email protected]>
Signed-off-by Simon Horman <[email protected]>

Cheers,
Andreas

Signature · 2 lines

SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imend?rffer; HRB 16746 AG N?rnberg

Geert Uytterhoeven

2014-08-18 08:32 UTC #6 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

In reply to Simon Horman

Hi Simon, Olof,

Quoted 9 lines · Simon Horman

A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.
This change should not have any run-time affect at this time as the clock in question is used by a SCIF device that is not enabled by default.
Reported-by Sergei Shtylyov <[email protected]>
Signed-off-by Simon Horman <[email protected]>

Quoted 8 lines

---
 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Hi Olof,
this is a repost of the patch you asked for so you can apply it directly rather than as pulling a pull request.

It seems Olof missed this one.
As a consequence, we now have the bad version in v3.17-rc1, and in renesas-devel-20140818-v3.17-rc1.

Quoted 16 lines

arch/arm/mach-shmobile/clock-sh73a0.c +1 -2

index 0d77f65..b0993a5 100644
@@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
        CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
-       CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
+       CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
        CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */
--

2.0.1

Quoted 4 lines

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

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

Simon Horman

2014-08-18 23:56 UTC #7 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

Hi Geert, thanks for noting that.

I will add this back to the devel branch in the renesas tree.

Olof, could you pick up the patch below for v3.17-rcX?

Quoted 58 lines · Geert Uytterhoeven

Hi Simon, Olof,
Quoted 9 lines · Simon Horman
A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.
This change should not have any run-time affect at this time as the clock in question is used by a SCIF device that is not enabled by default.
Reported-by Sergei Shtylyov <[email protected]>
Signed-off-by Simon Horman <[email protected]>
Quoted 8 lines
---
 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Hi Olof,
this is a repost of the patch you asked for so you can apply it directly rather than as pulling a pull request.
It seems Olof missed this one.
As a consequence, we now have the bad version in v3.17-rc1, and in renesas-devel-20140818-v3.17-rc1.
Quoted 16 lines
arch/arm/mach-shmobile/clock-sh73a0.c +1 -2
index 0d77f65..b0993a5 100644
@@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
        CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
-       CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
+       CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
        CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */
--
2.0.1
Quoted 4 lines
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
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

Simon Horman

2014-08-22 03:53 UTC #8 lore raw

PATCH 1/1 ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

Quoted 5 lines · Simon Horman

Hi Geert, thanks for noting that.
I will add this back to the devel branch in the renesas tree.
Olof, could you pick up the patch below for v3.17-rcX?

Olof, I have two more clock fixes for v3.17. I'll send a pull request for them and the one below once things have sat in next for a bit.

Quoted 65 lines · Geert Uytterhoeven

Quoted 49 lines
Hi Simon, Olof,
Quoted 9 lines · Simon Horman
A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1 clock support for DT"). This patch removes it.
This change should not have any run-time affect at this time as the clock in question is used by a SCIF device that is not enabled by default.
Reported-by Sergei Shtylyov <[email protected]>
Signed-off-by Simon Horman <[email protected]>
Quoted 8 lines
---
 arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Hi Olof,
this is a repost of the patch you asked for so you can apply it directly rather than as pulling a pull request.
It seems Olof missed this one.
As a consequence, we now have the bad version in v3.17-rc1, and in renesas-devel-20140818-v3.17-rc1.
Quoted 16 lines
arch/arm/mach-shmobile/clock-sh73a0.c +1 -2
index 0d77f65..b0993a5 100644
@@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
        CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
        CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
-       CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
+       CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
        CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
        CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */
--
2.0.1
Quoted 4 lines
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Gr{oetje,eeting}s,
Quoted 9 lines
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 linux-sh" in the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html