Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1685443 > unrolled thread

[PATCH 04/17] MAINTAINERS: Add RISC-V

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH 04/17] MAINTAINERS: Add RISC-V Palmer Dabbelt <palmer@dabbelt.com> - 2017-07-12 03:40 +0200
    Re: [PATCH 04/17] MAINTAINERS: Add RISC-V James Hogan <james.hogan@imgtec.com> - 2017-07-12 13:20 +0200
      Re: [PATCH 04/17] MAINTAINERS: Add RISC-V Arnd Bergmann <arnd@arndb.de> - 2017-07-12 13:40 +0200
        Re: [patches] Re: [PATCH 04/17] MAINTAINERS: Add RISC-V Jonathan Neuschäfer <j.neuschaefer@gmx.net> - 2017-07-12 17:00 +0200

#1685443 — [PATCH 04/17] MAINTAINERS: Add RISC-V

From: Jonathan Neuschäfer <j.neuschaefer@gmx.net>

RISC-V needs a MAINTAINERS entry. Let's add one.

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Signed-off-by: Palmer Dabbelt <palmer@dabbelt.com>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 2af3ab3f7702..3f39f16046dd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11246,6 +11246,14 @@ M:	Maxim Levitsky <maximlevitsky@gmail.com>
 S:	Maintained
 F:	drivers/memstick/host/r592.*
 
+RISC-V ARCHITECTURE
+M:	Palmer Dabbelt <palmer@sifive.com>
+M:	Albert Ou <albert@sifive.com>
+L:	patches@groups.riscv.org
+T:	git https://github.com/riscv/riscv-linux
+S:	Supported
+F:	arch/riscv/
+
 ROCCAT DRIVERS
 M:	Stefan Achatz <erazor_de@users.sourceforge.net>
 W:	http://sourceforge.net/projects/roccat/
-- 
2.13.0

[toc] | [next] | [standalone]

#1685705

[Multipart message — attachments visible in raw view] — view raw

On Tue, Jul 11, 2017 at 06:31:17PM -0700, Palmer Dabbelt wrote:
> From: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> 
> RISC-V needs a MAINTAINERS entry. Let's add one.
> 
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> Signed-off-by: Palmer Dabbelt <palmer@dabbelt.com>
> ---
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2af3ab3f7702..3f39f16046dd 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11246,6 +11246,14 @@ M:	Maxim Levitsky <maximlevitsky@gmail.com>
>  S:	Maintained
>  F:	drivers/memstick/host/r592.*
>  
> +RISC-V ARCHITECTURE
> +M:	Palmer Dabbelt <palmer@sifive.com>
> +M:	Albert Ou <albert@sifive.com>
> +L:	patches@groups.riscv.org
> +T:	git https://github.com/riscv/riscv-linux
> +S:	Supported
> +F:	arch/riscv/

Should this include other tightly riscv specific drivers outside of
arch/riscv/ (or maybe they should have their own MAINTAINERS entries if
they aren't so tightly bound to the architecture or are maintained
separately)?

drivers/clocksource/riscv_timer.c
drivers/irqchip/irq-riscv-intc.c
drivers/irqchip/irq-riscv-plic.c
drivers/tty/hvc/hvc_sbi.c
include/linux/timer_riscv.h

Cheers
James

> +
>  ROCCAT DRIVERS
>  M:	Stefan Achatz <erazor_de@users.sourceforge.net>
>  W:	http://sourceforge.net/projects/roccat/
> -- 
> 2.13.0
> 

[toc] | [prev] | [next] | [standalone]

#1685716

On Wed, Jul 12, 2017 at 1:16 PM, James Hogan <james.hogan@imgtec.com> wrote:
> On Tue, Jul 11, 2017 at 06:31:17PM -0700, Palmer Dabbelt wrote:
>> From: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
>>
>> RISC-V needs a MAINTAINERS entry. Let's add one.
>>
>> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
>> Signed-off-by: Palmer Dabbelt <palmer@dabbelt.com>
>> ---
>>  MAINTAINERS | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 2af3ab3f7702..3f39f16046dd 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -11246,6 +11246,14 @@ M:   Maxim Levitsky <maximlevitsky@gmail.com>
>>  S:   Maintained
>>  F:   drivers/memstick/host/r592.*
>>
>> +RISC-V ARCHITECTURE
>> +M:   Palmer Dabbelt <palmer@sifive.com>
>> +M:   Albert Ou <albert@sifive.com>
>> +L:   patches@groups.riscv.org
>> +T:   git https://github.com/riscv/riscv-linux
>> +S:   Supported
>> +F:   arch/riscv/
>
> Should this include other tightly riscv specific drivers outside of
> arch/riscv/ (or maybe they should have their own MAINTAINERS entries if
> they aren't so tightly bound to the architecture or are maintained
> separately)?
>
> drivers/clocksource/riscv_timer.c
> drivers/irqchip/irq-riscv-intc.c
> drivers/irqchip/irq-riscv-plic.c
> drivers/tty/hvc/hvc_sbi.c
> include/linux/timer_riscv.h
>

I would suggest including a

K: riscv

keyword line that should catch the closely related drivers, and have
separate entries for the rest. Maybe rename the hvc_sbi file
to fall within that keyword match.

        Arnd

[toc] | [prev] | [next] | [standalone]

#1685838 — Re: [patches] Re: [PATCH 04/17] MAINTAINERS: Add RISC-V

[Multipart message — attachments visible in raw view] — view raw

On Wed, Jul 12, 2017 at 01:31:17PM +0200, Arnd Bergmann wrote:
> On Wed, Jul 12, 2017 at 1:16 PM, James Hogan <james.hogan@imgtec.com> wrote:
> > On Tue, Jul 11, 2017 at 06:31:17PM -0700, Palmer Dabbelt wrote:
[...]
> >> +RISC-V ARCHITECTURE
> >> +M:   Palmer Dabbelt <palmer@sifive.com>
> >> +M:   Albert Ou <albert@sifive.com>
> >> +L:   patches@groups.riscv.org
> >> +T:   git https://github.com/riscv/riscv-linux
> >> +S:   Supported
> >> +F:   arch/riscv/
> >
> > Should this include other tightly riscv specific drivers outside of
> > arch/riscv/ (or maybe they should have their own MAINTAINERS entries if
> > they aren't so tightly bound to the architecture or are maintained
> > separately)?
> >
> > drivers/clocksource/riscv_timer.c
> > drivers/irqchip/irq-riscv-intc.c
> > drivers/irqchip/irq-riscv-plic.c
> > drivers/tty/hvc/hvc_sbi.c
> > include/linux/timer_riscv.h
> >
> 
> I would suggest including a
> 
> K: riscv
> 
> keyword line that should catch the closely related drivers, and have
> separate entries for the rest. Maybe rename the hvc_sbi file
> to fall within that keyword match.
> 
>         Arnd

All good suggestions. Palmer, would you mind to take this patch from
here, to reduce the communication overhead? (Besides, you know better
than I what to include, anyway)


Jonathan

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web