[PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[Laurent Pinchart]

Hello,

The VSP1 clocks were incorrectly named in the r8a7790 and r8a7791 .dtsi. This
patch set fixes the problem.

Laurent Pinchart (2):
  ARM: shmobile: r8a7790: Rename VSP1_(SY|RT) clocks to VSP1_(S|R)
  ARM: shmobile: r8a7791: Rename VSP1_SY clocks to VSP1_S

 arch/arm/boot/dts/r8a7790.dtsi            | 2 +-
 arch/arm/boot/dts/r8a7791.dtsi            | 2 +-
 include/dt-bindings/clock/r8a7790-clock.h | 4 ++--
 include/dt-bindings/clock/r8a7791-clock.h | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

-- 
Regards,

Laurent Pinchart

Re: [PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[Magnus Damm]

Hi Laurent,

[CC Simon]

On Wed, Apr 2, 2014 at 11:31 PM, Laurent Pinchart
<laurent.pinchart+renesas@ideasonboard.com> wrote:
> Hello,
>
> The VSP1 clocks were incorrectly named in the r8a7790 and r8a7791 .dtsi. This
> patch set fixes the problem.
>
> Laurent Pinchart (2):
>   ARM: shmobile: r8a7790: Rename VSP1_(SY|RT) clocks to VSP1_(S|R)
>   ARM: shmobile: r8a7791: Rename VSP1_SY clocks to VSP1_S

Thanks for fixing this. These patches look fine to me.

Acked-by: Magnus Damm <damm@opensource.se>

Simon, can you please pick up?

Cheers,

/ magnus

Re: [PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[Simon Horman]

On Thu, Apr 10, 2014 at 06:09:20PM +0900, Magnus Damm wrote:
> Hi Laurent,
> 
> [CC Simon]

Thanks, it helps my work flow immensely to be CCed on patches
that I should handle one way or another.

> 
> On Wed, Apr 2, 2014 at 11:31 PM, Laurent Pinchart
> <laurent.pinchart+renesas@ideasonboard.com> wrote:
> > Hello,
> >
> > The VSP1 clocks were incorrectly named in the r8a7790 and r8a7791 .dtsi. This
> > patch set fixes the problem.
> >
> > Laurent Pinchart (2):
> >   ARM: shmobile: r8a7790: Rename VSP1_(SY|RT) clocks to VSP1_(S|R)
> >   ARM: shmobile: r8a7791: Rename VSP1_SY clocks to VSP1_S
> 
> Thanks for fixing this. These patches look fine to me.
> 
> Acked-by: Magnus Damm <damm@opensource.se>
> 
> Simon, can you please pick up?

Done

Re: [PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[Laurent Pinchart]

Hi Simon,

On Friday 11 April 2014 07:59:43 Simon Horman wrote:
> On Thu, Apr 10, 2014 at 06:09:20PM +0900, Magnus Damm wrote:
> > Hi Laurent,
> > 
> > [CC Simon]
> 
> Thanks, it helps my work flow immensely to be CCed on patches
> that I should handle one way or another.

Every maintainer has different preferences, and for some reason I thought you 
relied on patchwork more than on e-mails to track patches. I'll make sure to 
CC you in the future.

> > On Wed, Apr 2, 2014 at 11:31 PM, Laurent Pinchart wrote:
> > > Hello,
> > > 
> > > The VSP1 clocks were incorrectly named in the r8a7790 and r8a7791 .dtsi.
> > > This patch set fixes the problem.
> > > 
> > > Laurent Pinchart (2):
> > >   ARM: shmobile: r8a7790: Rename VSP1_(SY|RT) clocks to VSP1_(S|R)
> > >   ARM: shmobile: r8a7791: Rename VSP1_SY clocks to VSP1_S
> > 
> > Thanks for fixing this. These patches look fine to me.
> > 
> > Acked-by: Magnus Damm <damm@opensource.se>
> > 
> > Simon, can you please pick up?
> 
> Done

Thank you.

-- 
Regards,

Laurent Pinchart

Re: [PATCH 0/2] Fix VSP1 clock names in r8a779[01] .dtsi

[Simon Horman]

On Fri, Apr 11, 2014 at 01:32:31AM +0200, Laurent Pinchart wrote:
> Hi Simon,
> 
> On Friday 11 April 2014 07:59:43 Simon Horman wrote:
> > On Thu, Apr 10, 2014 at 06:09:20PM +0900, Magnus Damm wrote:
> > > Hi Laurent,
> > > 
> > > [CC Simon]
> > 
> > Thanks, it helps my work flow immensely to be CCed on patches
> > that I should handle one way or another.
> 
> Every maintainer has different preferences, and for some reason I thought you 
> relied on patchwork more than on e-mails to track patches. I'll make sure to 
> CC you in the future.

Thanks.

I do use patchwork but usually as a secondary reference.
I usually pull patches straight out of email.

> 
> > > On Wed, Apr 2, 2014 at 11:31 PM, Laurent Pinchart wrote:
> > > > Hello,
> > > > 
> > > > The VSP1 clocks were incorrectly named in the r8a7790 and r8a7791 .dtsi.
> > > > This patch set fixes the problem.
> > > > 
> > > > Laurent Pinchart (2):
> > > >   ARM: shmobile: r8a7790: Rename VSP1_(SY|RT) clocks to VSP1_(S|R)
> > > >   ARM: shmobile: r8a7791: Rename VSP1_SY clocks to VSP1_S
> > > 
> > > Thanks for fixing this. These patches look fine to me.
> > > 
> > > Acked-by: Magnus Damm <damm@opensource.se>
> > > 
> > > Simon, can you please pick up?
> > 
> > Done
> 
> Thank you.
> 
> -- 
> Regards,
> 
> Laurent Pinchart
>