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

2016-04-11 Thread Geert Uytterhoeven
Hi Simon,

On Mon, Apr 11, 2016 at 3:52 AM, Simon Horman
 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 -- ge...@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


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

2016-04-11 Thread 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
>  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?


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

2016-04-17 Thread 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
> >  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!


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

2016-04-19 Thread Geert Uytterhoeven
Hi Simon,

On Mon, Apr 18, 2016 at 6:12 AM, Simon Horman  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
>> >  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 -- ge...@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


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

2016-04-19 Thread 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
> > >  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 Ltd.


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

2016-04-19 Thread 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  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
> >> >  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.


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

2016-04-19 Thread 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
> > > >  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 

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.


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

2016-04-22 Thread Kevin Hilman
Simon Horman  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


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

2016-04-22 Thread Kevin Hilman
Hi Simon,

Kevin Hilman  writes:

> Simon Horman  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