[PATCH 0/4] r8a779[01] I2C clocks fixes

[PATCH 0/4] r8a779[01] I2C clocks fixes

[Laurent Pinchart]

Hello,

The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in the
legacy C code and in DT. This patch set fixes that.

Laurent Pinchart (4):
  ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
  ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
  ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
  ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT

 arch/arm/boot/dts/r8a7790.dtsi         |  2 +-
 arch/arm/boot/dts/r8a7791.dtsi         |  6 +++---
 arch/arm/mach-shmobile/clock-r8a7790.c |  8 ++++----
 arch/arm/mach-shmobile/clock-r8a7791.c | 12 ++++++------
 4 files changed, 14 insertions(+), 14 deletions(-)

-- 
Regards,

Laurent Pinchart

Re: [PATCH 0/4] r8a779[01] I2C clocks fixes

[Geert Uytterhoeven]

Hi Laurent,

On Tue, Apr 1, 2014 at 1:02 PM, Laurent Pinchart
<laurent.pinchart+renesas@ideasonboard.com> wrote:
> The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in the
> legacy C code and in DT. This patch set fixes that.

Thanks!

> Laurent Pinchart (4):
>   ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
>   ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
>   ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
>   ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT

Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>

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

Re: [PATCH 0/4] r8a779[01] I2C clocks fixes

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 624 bytes --]

On Tue, Apr 01, 2014 at 01:02:14PM +0200, Laurent Pinchart wrote:
> Hello,
> 
> The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in the
> legacy C code and in DT. This patch set fixes that.
> 
> Laurent Pinchart (4):
>   ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
>   ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
>   ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
>   ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT

Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>

CC on I2C patches would be appreciated.

Thanks!


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 0/4] r8a779[01] I2C clocks fixes

[Laurent Pinchart]

[-- Attachment #1: Type: text/plain, Size: 799 bytes --]

Hi Wolfram,

On Wednesday 02 April 2014 13:42:20 Wolfram Sang wrote:
> On Tue, Apr 01, 2014 at 01:02:14PM +0200, Laurent Pinchart wrote:
> > Hello,
> > 
> > The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in
> > the legacy C code and in DT. This patch set fixes that.
> > 
> > Laurent Pinchart (4):
> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT
> 
> Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>

Thank you.

> CC on I2C patches would be appreciated.

Sure, sorry for having forgotten.

-- 
Regards,

Laurent Pinchart

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

Re: [PATCH 0/4] r8a779[01] I2C clocks fixes

[Magnus Damm]

Hi Simon,

On Wed, Apr 2, 2014 at 8:51 PM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> Hi Wolfram,
>
> On Wednesday 02 April 2014 13:42:20 Wolfram Sang wrote:
>> On Tue, Apr 01, 2014 at 01:02:14PM +0200, Laurent Pinchart wrote:
>> > Hello,
>> >
>> > The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in
>> > the legacy C code and in DT. This patch set fixes that.
>> >
>> > Laurent Pinchart (4):
>> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
>> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
>> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
>> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT
>>
>> Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>
>
> Thank you.
>
>> CC on I2C patches would be appreciated.
>
> Sure, sorry for having forgotten.

I'm a bit unclear about who is supposed to merge these patches, but
based on path it looks to me like Simon is supposed to do that.

So Simon, can you please pick up this series and include these?

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>

Thanks,

/ magnus

Re: [PATCH 0/4] r8a779[01] I2C clocks fixes

[Simon Horman]

On Thu, Apr 10, 2014 at 03:21:02PM +0900, Magnus Damm wrote:
> Hi Simon,
> 
> On Wed, Apr 2, 2014 at 8:51 PM, Laurent Pinchart
> <laurent.pinchart@ideasonboard.com> wrote:
> > Hi Wolfram,
> >
> > On Wednesday 02 April 2014 13:42:20 Wolfram Sang wrote:
> >> On Tue, Apr 01, 2014 at 01:02:14PM +0200, Laurent Pinchart wrote:
> >> > Hello,
> >> >
> >> > The I2C clocks select an incorrect parent on r8a7790 and r8a7791, both in
> >> > the legacy C code and in DT. This patch set fixes that.
> >> >
> >> > Laurent Pinchart (4):
> >> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in legacy code
> >> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in legacy code
> >> >   ARM: shmobile: r8a7790: Fix the I2C clocks parents in DT
> >> >   ARM: shmobile: r8a7791: Fix the I2C clocks parents in DT
> >>
> >> Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>
> >
> > Thank you.
> >
> >> CC on I2C patches would be appreciated.
> >
> > Sure, sorry for having forgotten.
> 
> I'm a bit unclear about who is supposed to merge these patches, but
> based on path it looks to me like Simon is supposed to do that.

Yes, it seems so.

> So Simon, can you please pick up this series and include these?

Sure, will do.

> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Reviewed-by: Wolfram Sang <wsa@sang-engineering.com>
> 
> Thanks,
> 
> / magnus
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>