2022-02-03 20:00:32

by Sergey Shtylyov

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
I do have the manuals still! :-)

Signed-off-by: Sergey Shtylyov <[email protected]>

---
This patch is against the 'master' branch of Damien Le Moal's 'libata.git'
repo.

MAINTAINERS | 7 +++++++
1 file changed, 7 insertions(+)

Index: libata/MAINTAINERS
===================================================================
--- libata.orig/MAINTAINERS
+++ libata/MAINTAINERS
@@ -16469,6 +16469,13 @@ F: Documentation/devicetree/bindings/i2c
F: drivers/i2c/busses/i2c-rcar.c
F: drivers/i2c/busses/i2c-sh_mobile.c

+RENESAS R-CAR SATA DRIVER
+R: Sergey Shtylyov <[email protected]>
+L: [email protected]
+L: [email protected]
+F: Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
+F: drivers/ata/sata_rcar.c
+
RENESAS R-CAR THERMAL DRIVERS
M: Niklas Söderlund <[email protected]>
L: [email protected]


2022-02-03 21:59:41

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

On Thu, 2022-02-03 at 20:51 +0300, Sergey Shtylyov wrote:
> Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
> the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
> I do have the manuals still! :-)

Thanks.

> --- libata.orig/MAINTAINERS
> +++ libata/MAINTAINERS
> @@ -16469,6 +16469,13 @@ F: Documentation/devicetree/bindings/i2c
> F: drivers/i2c/busses/i2c-rcar.c
> F: drivers/i2c/busses/i2c-sh_mobile.c
>
> +RENESAS R-CAR SATA DRIVER
> +R: Sergey Shtylyov <[email protected]>
> +L: [email protected]
> +L: [email protected]
> +F: Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F: drivers/ata/sata_rcar.c

I believe it's better for every section to have an S: entry


2022-02-04 13:58:55

by Sergey Shtylyov

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

On 2/3/22 9:13 PM, Geert Uytterhoeven wrote:

>> Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
>> the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
>> I do have the manuals still! :-)
>
> I guess some of the manuals are publicly available on the Renesas
> website?

Have they finally published the R-Car manuals? 8-)

>> Signed-off-by: Sergey Shtylyov <[email protected]>
>
> Thanks for the offer!
>
> Acked-by: Geert Uytterhoeven <[email protected]>

TY!

[...]

MBR, Sergey

2022-02-04 20:32:16

by Sergey Shtylyov

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

On 2/3/22 9:27 PM, Geert Uytterhoeven wrote:

[...]
>>>> Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
>>>> the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
>>>> I do have the manuals still! :-)
>>>
>>> I guess some of the manuals are publicly available on the Renesas
>>> website?
>>
>> Have they finally published the R-Car manuals? 8-)
>
> The same SATA block is also used in other SoC families. See e.g.
> "RZ/G Series User's Manual: Hardware" on
> https://www.renesas.com/eu/en/products/microcontrollers-microprocessors/rz-arm-based-high-end-32-64-bit-mpus/rzg1h-ultra-high-performance-microprocessors-quad-core-arm-cortex-a15-and-quad-core-arm-cortex-a7-cpus-3d

Oh, I that know, and have redirected the people in search of the R-Car manuals couple times. ;-)

> Gr{oetje,eeting}s,
>
> Geert

MBR, Sergey

2022-02-07 05:53:41

by Sergey Shtylyov

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

On 2/3/22 9:54 PM, Joe Perches wrote:

>> Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
>> the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
>> I do have the manuals still! :-)
>
> Thanks.

Should have done this while still with Cogent (and working on Renesas stuff)
but the patch traffic appeared low enough and I saw the patches flowing by on
linux-ide anyways (if I wasn't the author, that is :-))...

>> --- libata.orig/MAINTAINERS
>> +++ libata/MAINTAINERS
>> @@ -16469,6 +16469,13 @@ F: Documentation/devicetree/bindings/i2c
>> F: drivers/i2c/busses/i2c-rcar.c
>> F: drivers/i2c/busses/i2c-sh_mobile.c
>>
>> +RENESAS R-CAR SATA DRIVER
>> +R: Sergey Shtylyov <[email protected]>
>> +L: [email protected]
>> +L: [email protected]
>> +F: Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
>> +F: drivers/ata/sata_rcar.c
>
> I believe it's better for every section to have an S: entry

Hm, you're right. I'll recast...

MBR, Sergey

2022-02-07 13:52:04

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

CC linux-reneas-soc

Hi Sergey,

On Thu, Feb 3, 2022 at 6:51 PM Sergey Shtylyov <[email protected]> wrote:
> Add myself as a reviewer for the Renesas R-Car SATA driver -- I don't have
> the hardware anymore (Geert Uytterhoeven does have a lot of hardware!) but
> I do have the manuals still! :-)

I guess some of the manuals are publicly available on the Renesas
website?

> Signed-off-by: Sergey Shtylyov <[email protected]>

Thanks for the offer!

Acked-by: Geert Uytterhoeven <[email protected]>

> --- libata.orig/MAINTAINERS
> +++ libata/MAINTAINERS
> @@ -16469,6 +16469,13 @@ F: Documentation/devicetree/bindings/i2c
> F: drivers/i2c/busses/i2c-rcar.c
> F: drivers/i2c/busses/i2c-sh_mobile.c
>
> +RENESAS R-CAR SATA DRIVER
> +R: Sergey Shtylyov <[email protected]>
> +L: [email protected]
> +L: [email protected]
> +F: Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F: drivers/ata/sata_rcar.c
> +
> RENESAS R-CAR THERMAL DRIVERS
> M: Niklas Söderlund <[email protected]>
> L: [email protected]

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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