Re: [PATCH 1/2] pinctrl: sh-pfc: r8a7794: add SSI pin groups

[Sergei Shtylyov]

Hello.

On 2/18/2016 10:52 AM, Geert Uytterhoeven wrote:


--- linux-pinctrl.orig/drivers/pinctrl/sh-pfc/pfc-r8a7794.c
+++ linux-pinctrl/drivers/pinctrl/sh-pfc/pfc-r8a7794.c
@@ -1,9 +1,9 @@
   /*
* r8a7794 processor support - PFC hardware block.
*
- * Copyright (C) 2014 Renesas Electronics Corporation
+ * Copyright (C) 2014-2015 Renesas Electronics Corporation
* Copyright (C) 2015 Renesas Solutions Corp.
- * Copyright (C) 2015 Cogent  Embedded, Inc., 
+ * Copyright (C) 2015-2016 Cogent  Embedded, Inc.,



Geert, could you turn the double space into single?


Sure, done.


   Thank you! Overlooked it. :-/


Gr{oetje,eeting}s,

 Geert


MBR, Sergei

Re: [PATCH 1/2] pinctrl: sh-pfc: r8a7794: add SSI pin groups

[Geert Uytterhoeven]

Hi Sergei,

On Wed, Feb 17, 2016 at 11:08 PM, Sergei Shtylyov
 wrote:
>> --- linux-pinctrl.orig/drivers/pinctrl/sh-pfc/pfc-r8a7794.c
>> +++ linux-pinctrl/drivers/pinctrl/sh-pfc/pfc-r8a7794.c
>> @@ -1,9 +1,9 @@
>>   /*
>>* r8a7794 processor support - PFC hardware block.
>>*
>> - * Copyright (C) 2014 Renesas Electronics Corporation
>> + * Copyright (C) 2014-2015 Renesas Electronics Corporation
>>* Copyright (C) 2015 Renesas Solutions Corp.
>> - * Copyright (C) 2015 Cogent  Embedded, Inc., 
>> + * Copyright (C) 2015-2016 Cogent  Embedded, Inc.,
>> 
>
>
>Geert, could you turn the double space into single?

Sure, done.

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: [PATCH 1/2] pinctrl: sh-pfc: r8a7794: add SSI pin groups

[Linus Walleij]

On Wed, Feb 10, 2016 at 11:38 PM, Sergei Shtylyov
 wrote:

> From: Ryo Kataoka 
>
> Add the SSI pin groups to the R8A7794 PFC driver.
>
> [Sergei: fixed inconsistent alternate pin group naming, split SSI5/6 pin
> groups into data/control ones, moved SSI7 data B group to its proper place,
> fixed  pin names in  the comments to *_pins[], extended Cogent Embedded's
> copyright, added the changelog, renamed the patch.]
>
> Signed-off-by: Ryo Kataoka 
> Signed-off-by: Sergei Shtylyov 

These look fine to me, Acked-by for both.

Geert, will you & Laurent review and queue this please.

Yours,
Linus Walleij