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

[Arnd Bergmann]

On Wed, Jul 12, 2017 at 1:16 PM, James Hogan <james.hogan@xxxxxxxxxx> wrote:
> On Tue, Jul 11, 2017 at 06:31:17PM -0700, Palmer Dabbelt wrote:
>> From: Jonathan NeuschÃfer <j.neuschaefer@xxxxxxx>
>>
>> RISC-V needs a MAINTAINERS entry. Let's add one.
>>
>> Signed-off-by: Jonathan NeuschÃfer <j.neuschaefer@xxxxxxx>
>> Signed-off-by: Palmer Dabbelt <palmer@xxxxxxxxxxx>
>> ---
>>  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@xxxxxxxxx>
>>  S:   Maintained
>>  F:   drivers/memstick/host/r592.*
>>
>> +RISC-V ARCHITECTURE
>> +M:   Palmer Dabbelt <palmer@xxxxxxxxxx>
>> +M:   Albert Ou <albert@xxxxxxxxxx>
>> +L:   patches@xxxxxxxxxxxxxxxx
>> +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