[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC fixes for v4.6.


Allow serial to work once again on the Porter board (Revision B) which
does not have the oscillator in question mounted.


The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:

  Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.6

for you to fetch changes up to e885767418bff0c591cc1e45babdd25d91b4a795:

  Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins" (2016-04-08 15:56:28 +0900)

----------------------------------------------------------------
Renesas ARM Based SoC Fixes for v4.6

* Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

----------------------------------------------------------------
Sjoerd Simons (1):
      Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

 arch/arm/boot/dts/r8a7791-porter.dts | 13 -------------
 1 file changed, 13 deletions(-)

[PATCH] Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

[Simon Horman]

From: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

This reverts commit 19417bd9c511 ("ARM: dts: porter: Enable SCIF_CLK
frequency and pins") as according to
http://elinux.org/File:R-CarM2-KOELSCH_PORTER-B_PORTER_C_Comparison.pdf
the external oscillator for SCIF_CLK is not mounted on the porter boards.

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7791-porter.dts | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7791-porter.dts b/arch/arm/boot/dts/r8a7791-porter.dts
index 6c08314427d6..329a9dcc88e9 100644
--- a/arch/arm/boot/dts/r8a7791-porter.dts
+++ b/arch/arm/boot/dts/r8a7791-porter.dts
@@ -143,19 +143,11 @@
 };
 
 &pfc {
-	pinctrl-0 = <&scif_clk_pins>;
-	pinctrl-names = "default";
-
 	scif0_pins: serial0 {
 		renesas,groups = "scif0_data_d";
 		renesas,function = "scif0";
 	};
 
-	scif_clk_pins: scif_clk {
-		renesas,groups = "scif_clk";
-		renesas,function = "scif_clk";
-	};
-
 	ether_pins: ether {
 		renesas,groups = "eth_link", "eth_mdio", "eth_rmii";
 		renesas,function = "eth";
@@ -229,11 +221,6 @@
 	status = "okay";
 };
 
-&scif_clk {
-	clock-frequency = <14745600>;
-	status = "okay";
-};
-
 &ether {
 	pinctrl-0 = <&ether_pins &phy1_pins>;
 	pinctrl-names = "default";
-- 
2.1.4

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Geert Uytterhoeven]

Hi Simon,

On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
<horms+renesas@verge.net.au> wrote:
> Hi Olof, Hi Kevin, Hi Arnd,
>
> Please consider these Renesas ARM based SoC fixes for v4.6.
>
>
> Allow serial to work once again on the Porter board (Revision B) which
> does not have the oscillator in question mounted.

Have you tested this on Porter?

I believe there's another change to be made for this revert to actually work,
cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced optional clocks"
(https://lkml.org/lkml/2016/4/6/350)

> The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
>
>   Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.6
>
> for you to fetch changes up to e885767418bff0c591cc1e45babdd25d91b4a795:
>
>   Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins" (2016-04-08 15:56:28 +0900)
>
> ----------------------------------------------------------------
> Renesas ARM Based SoC Fixes for v4.6
>
> * Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"
>
> ----------------------------------------------------------------
> Sjoerd Simons (1):
>       Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

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

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > Hi Olof, Hi Kevin, Hi Arnd,
> >
> > Please consider these Renesas ARM based SoC fixes for v4.6.
> >
> >
> > Allow serial to work once again on the Porter board (Revision B) which
> > does not have the oscillator in question mounted.
> 
> Have you tested this on Porter?

I don't have working access to a porter at this time.

> I believe there's another change to be made for this revert to actually work,
> cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced optional clocks"
> (https://lkml.org/lkml/2016/4/6/350)

Ok, understood.

Could someone verify that?

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

Hi Geert, Hi Sjoerd,

On Tue, Apr 12, 2016 at 08:02:54AM +0900, Simon Horman wrote:
> On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
> > Hi Simon,
> > 
> > On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
> > <horms+renesas@verge.net.au> wrote:
> > > Hi Olof, Hi Kevin, Hi Arnd,
> > >
> > > Please consider these Renesas ARM based SoC fixes for v4.6.
> > >
> > >
> > > Allow serial to work once again on the Porter board (Revision B) which
> > > does not have the oscillator in question mounted.
> > 
> > Have you tested this on Porter?
> 
> I don't have working access to a porter at this time.
> 
> > I believe there's another change to be made for this revert to actually work,
> > cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced optional clocks"
> > (https://lkml.org/lkml/2016/4/6/350)
> 
> Ok, understood.
> 
> Could someone verify that?

I have added "ARM: dts: r8a7791: Don't disable referenced optional clocks"
to the fixes-for-v4.6 branch and plan to push it in a fresh renesas-next
tag a little later today.

I would appreciate any review and/or testing on porter of that branch.

Thanks!

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Geert Uytterhoeven]

Hi Simon,

On Mon, Apr 18, 2016 at 6:12 AM, Simon Horman <horms@verge.net.au> wrote:
> On Tue, Apr 12, 2016 at 08:02:54AM +0900, Simon Horman wrote:
>> On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
>> > On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
>> > <horms+renesas@verge.net.au> wrote:
>> > > Hi Olof, Hi Kevin, Hi Arnd,
>> > >
>> > > Please consider these Renesas ARM based SoC fixes for v4.6.
>> > >
>> > >
>> > > Allow serial to work once again on the Porter board (Revision B) which
>> > > does not have the oscillator in question mounted.
>> >
>> > Have you tested this on Porter?
>>
>> I don't have working access to a porter at this time.
>>
>> > I believe there's another change to be made for this revert to actually work,
>> > cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced optional clocks"
>> > (https://lkml.org/lkml/2016/4/6/350)
>>
>> Ok, understood.
>>
>> Could someone verify that?
>
> I have added "ARM: dts: r8a7791: Don't disable referenced optional clocks"
> to the fixes-for-v4.6 branch and plan to push it in a fresh renesas-next
> tag a little later today.

Thanks!

I would queue

    ARM: dts: r8a7791: Don't disable referenced optional clocks

before

    Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

to preserve bisectability. So please reorder these two commits.

> I would appreciate any review and/or testing on porter of that branch.

I've provided my Reviewed-By in the original thread.

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

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Sjoerd Simons]

Hey Simon,
On Mon, 2016-04-18 at 14:12 +1000, Simon Horman wrote:
> Hi Geert, Hi Sjoerd,
> 
> On Tue, Apr 12, 2016 at 08:02:54AM +0900, Simon Horman wrote:
> > 
> > On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
> > > 
> > > Hi Simon,
> > > 
> > > On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
> > > <horms+renesas@verge.net.au> wrote:
> > > > 
> > > > Hi Olof, Hi Kevin, Hi Arnd,
> > > > 
> > > > Please consider these Renesas ARM based SoC fixes for v4.6.
> > > > 
> > > > 
> > > > Allow serial to work once again on the Porter board (Revision
> > > > B) which
> > > > does not have the oscillator in question mounted.
> > > Have you tested this on Porter?
> > I don't have working access to a porter at this time.
> > 
> > > 
> > > I believe there's another change to be made for this revert to
> > > actually work,
> > > cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced
> > > optional clocks"
> > > (https://lkml.org/lkml/2016/4/6/350)
> > Ok, understood.
> > 
> > Could someone verify that?
> I have added "ARM: dts: r8a7791: Don't disable referenced optional
> clocks"
> to the fixes-for-v4.6 branch and plan to push it in a fresh renesas-
> next
> tag a little later today.
> 
> I would appreciate any review and/or testing on porter of that
branch.

I've tested this branch (top commit?3d7277614d29) on a Porter revision
B board and can confirm the serial console works as expected.

Feel free to add:
? Tested-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>


-- 
Sjoerd Simons
Collabora Ltd.

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

On Tue, Apr 19, 2016 at 09:23:10AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Mon, Apr 18, 2016 at 6:12 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Tue, Apr 12, 2016 at 08:02:54AM +0900, Simon Horman wrote:
> >> On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
> >> > On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
> >> > <horms+renesas@verge.net.au> wrote:
> >> > > Hi Olof, Hi Kevin, Hi Arnd,
> >> > >
> >> > > Please consider these Renesas ARM based SoC fixes for v4.6.
> >> > >
> >> > >
> >> > > Allow serial to work once again on the Porter board (Revision B) which
> >> > > does not have the oscillator in question mounted.
> >> >
> >> > Have you tested this on Porter?
> >>
> >> I don't have working access to a porter at this time.
> >>
> >> > I believe there's another change to be made for this revert to actually work,
> >> > cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced optional clocks"
> >> > (https://lkml.org/lkml/2016/4/6/350)
> >>
> >> Ok, understood.
> >>
> >> Could someone verify that?
> >
> > I have added "ARM: dts: r8a7791: Don't disable referenced optional clocks"
> > to the fixes-for-v4.6 branch and plan to push it in a fresh renesas-next
> > tag a little later today.
> 
> Thanks!
> 
> I would queue
> 
>     ARM: dts: r8a7791: Don't disable referenced optional clocks
> 
> before
> 
>     Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"
> 
> to preserve bisectability. So please reorder these two commits.

Thanks, I have reordered the commits as you suggest.

> > I would appreciate any review and/or testing on porter of that branch.
> 
> I've provided my Reviewed-By in the original thread.

Thanks, got it.

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

On Tue, Apr 19, 2016 at 11:12:03AM +0200, Sjoerd Simons wrote:
> Hey Simon,
> On Mon, 2016-04-18 at 14:12 +1000, Simon Horman wrote:
> > Hi Geert, Hi Sjoerd,
> > 
> > On Tue, Apr 12, 2016 at 08:02:54AM +0900, Simon Horman wrote:
> > > 
> > > On Mon, Apr 11, 2016 at 09:11:19AM +0200, Geert Uytterhoeven wrote:
> > > > 
> > > > Hi Simon,
> > > > 
> > > > On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
> > > > <horms+renesas@verge.net.au> wrote:
> > > > > 
> > > > > Hi Olof, Hi Kevin, Hi Arnd,
> > > > > 
> > > > > Please consider these Renesas ARM based SoC fixes for v4.6.
> > > > > 
> > > > > 
> > > > > Allow serial to work once again on the Porter board (Revision
> > > > > B) which
> > > > > does not have the oscillator in question mounted.
> > > > Have you tested this on Porter?
> > > I don't have working access to a porter at this time.
> > > 
> > > > 
> > > > I believe there's another change to be made for this revert to
> > > > actually work,
> > > > cfr. "[PATCH] ARM: dts: r8a7791: Don't disable referenced
> > > > optional clocks"
> > > > (https://lkml.org/lkml/2016/4/6/350)
> > > Ok, understood.
> > > 
> > > Could someone verify that?
> > I have added "ARM: dts: r8a7791: Don't disable referenced optional
> > clocks"
> > to the fixes-for-v4.6 branch and plan to push it in a fresh renesas-
> > next
> > tag a little later today.
> > 
> > I would appreciate any review and/or testing on porter of that
> branch.
> 
> I've tested this branch (top commit?3d7277614d29) on a Porter revision
> B board and can confirm the serial console works as expected.
> 
> Feel free to add:
> ? Tested-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Thanks, that is really helpful!

I think I will leave out the extra tag as the patches
in question are already signed off by you.

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Simon Horman]

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC fixes for v4.6.


The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:

  Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.6

for you to fetch changes up to c531fb27e9699eaee478ee5686a3cca5dee73602:

  ARM: shmobile: timer: Fix preset_lpj leading to too short delays (2016-04-20 08:52:55 +1000)

----------------------------------------------------------------
Renesas ARM Based SoC Fixes for v4.6

* Correct preset_lpj calculation which may lead to too short delays
* Correct handling of optional clocks on r8a7791 to restore
  access to the serial port the porter board

----------------------------------------------------------------
Geert Uytterhoeven (1):
      ARM: shmobile: timer: Fix preset_lpj leading to too short delays

Sjoerd Simons (2):
      ARM: dts: r8a7791: Don't disable referenced optional clocks
      Revert "ARM: dts: porter: Enable SCIF_CLK frequency and pins"

 arch/arm/boot/dts/r8a7791-koelsch.dts |  1 +
 arch/arm/boot/dts/r8a7791-porter.dts  | 14 +-------------
 arch/arm/boot/dts/r8a7791.dtsi        |  5 +----
 arch/arm/mach-shmobile/timer.c        | 28 +++++++++++-----------------
 4 files changed, 14 insertions(+), 34 deletions(-)

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Kevin Hilman]

Simon Horman <horms+renesas@verge.net.au> writes:

> Hi Olof, Hi Kevin, Hi Arnd,
>
> Please consider these Renesas ARM based SoC fixes for v4.6.
>
>
> The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
>
>   Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.6
>
> for you to fetch changes up to c531fb27e9699eaee478ee5686a3cca5dee73602:
>
>   ARM: shmobile: timer: Fix preset_lpj leading to too short delays (2016-04-20 08:52:55 +1000)
>
> ----------------------------------------------------------------
> Renesas ARM Based SoC Fixes for v4.6
>
> * Correct preset_lpj calculation which may lead to too short delays
> * Correct handling of optional clocks on r8a7791 to restore
>   access to the serial port the porter board
>
> ----------------------------------------------------------------

Applied to fixes,

Kevin

[GIT PULL] Renesas ARM Based SoC Fixes for v4.6

[Kevin Hilman]

Hi Simon,

Kevin Hilman <khilman@baylibre.com> writes:

> Simon Horman <horms+renesas@verge.net.au> writes:
>
>> Hi Olof, Hi Kevin, Hi Arnd,
>>
>> Please consider these Renesas ARM based SoC fixes for v4.6.
>>
>>
>> The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
>>
>>   Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
>>
>> are available in the git repository at:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-fixes-for-v4.6
>>
>> for you to fetch changes up to c531fb27e9699eaee478ee5686a3cca5dee73602:
>>
>>   ARM: shmobile: timer: Fix preset_lpj leading to too short delays (2016-04-20 08:52:55 +1000)
>>
>> ----------------------------------------------------------------
>> Renesas ARM Based SoC Fixes for v4.6
>>
>> * Correct preset_lpj calculation which may lead to too short delays
>> * Correct handling of optional clocks on r8a7791 to restore
>>   access to the serial port the porter board
>>
>> ----------------------------------------------------------------
>
> Applied to fixes,
>

Note that when merging arm-soc/fixes back into our for-next branch, your
fixes conflict with some other stuff you have queued up for v4.7.

Please double-check my conflict resolutions in for-next.

Thanks,

Kevin