diff mbox series

MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer

Message ID 6c1f0131-fb02-f30a-86ed-63ce3788901b@omp.ru
State New
Headers show
Series MAINTAINERS: add myself as Renesas R-Car SATA driver reviewer | expand

Commit Message

Sergey Shtylyov Feb. 3, 2022, 5:51 p.m. UTC
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 <s.shtylyov@omp.ru>

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

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

Comments

Geert Uytterhoeven Feb. 3, 2022, 6:13 p.m. UTC | #1
CC linux-reneas-soc

Hi Sergey,

On Thu, Feb 3, 2022 at 6:51 PM Sergey Shtylyov <s.shtylyov@omp.ru> 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 <s.shtylyov@omp.ru>

Thanks for the offer!

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

> --- 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 <s.shtylyov@omp.ru>
> +L:     linux-ide@vger.kernel.org
> +L:     linux-renesas-soc@vger.kernel.org
> +F:     Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F:     drivers/ata/sata_rcar.c
> +
>  RENESAS R-CAR THERMAL DRIVERS
>  M:     Niklas Söderlund <niklas.soderlund@ragnatech.se>
>  L:     linux-renesas-soc@vger.kernel.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
Sergey Shtylyov Feb. 3, 2022, 6:24 p.m. UTC | #2
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 <s.shtylyov@omp.ru>
> 
> Thanks for the offer!
> 
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

   TY!

[...]

MBR, Sergey
Geert Uytterhoeven Feb. 3, 2022, 6:27 p.m. UTC | #3
Hi Sergey,

On Thu, Feb 3, 2022 at 7:24 PM Sergey Shtylyov <s.shtylyov@omp.ru> wrote:
> 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-)

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

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
Sergey Shtylyov Feb. 3, 2022, 6:34 p.m. UTC | #4
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
Joe Perches Feb. 3, 2022, 6:54 p.m. UTC | #5
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 <s.shtylyov@omp.ru>
> +L:	linux-ide@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +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
Sergey Shtylyov Feb. 3, 2022, 7:17 p.m. UTC | #6
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 <s.shtylyov@omp.ru>
>> +L:	linux-ide@vger.kernel.org
>> +L:	linux-renesas-soc@vger.kernel.org
>> +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
diff mbox series

Patch

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 <s.shtylyov@omp.ru>
+L:	linux-ide@vger.kernel.org
+L:	linux-renesas-soc@vger.kernel.org
+F:	Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
+F:	drivers/ata/sata_rcar.c
+
 RENESAS R-CAR THERMAL DRIVERS
 M:	Niklas Söderlund <niklas.soderlund@ragnatech.se>
 L:	linux-renesas-soc@vger.kernel.org