MIME-Version: 1.0
In-Reply-To: <20170712111600.GT6973@jhogan-linux.le.imgtec.org>
References: <20170712013130.14792-1-palmer@dabbelt.com> <20170712013130.14792-5-palmer@dabbelt.com>
 <20170712111600.GT6973@jhogan-linux.le.imgtec.org>
From: Arnd Bergmann <arnd@arndb.de>
Date: Wed, 12 Jul 2017 13:31:17 +0200
Message-ID: <CAK8P3a2tC34jZA5_B9TkfhgKnt=b=-w5QHmaAaknSS1kTyrPQQ@mail.gmail.com>
Subject: Re: [PATCH 04/17] MAINTAINERS: Add RISC-V
To: James Hogan <james.hogan@imgtec.com>
Cc: Palmer Dabbelt <palmer@dabbelt.com>, Olof Johansson <olof@lixom.net>,
        Andrew Morton <akpm@linux-foundation.org>, albert@sifive.com,
        Masahiro Yamada <yamada.masahiro@socionext.com>,
        Michal Marek <mmarek@suse.com>, Will Deacon <will.deacon@arm.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Boqun Feng <boqun.feng@gmail.com>, Ingo Molnar <mingo@redhat.com>,
        Daniel Lezcano <daniel.lezcano@linaro.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Jason Cooper <jason@lakedaemon.net>,
        Marc Zyngier <marc.zyngier@arm.com>,
        gregkh <gregkh@linuxfoundation.org>, Jiri Slaby <jslaby@suse.com>,
        David Miller <davem@davemloft.net>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        =?UTF-8?B?RnLDqWTDqXJpYyBXZWlzYmVja2Vy?= <fweisbec@gmail.com>,
        Al Viro <viro@zeniv.linux.org.uk>,
        "Luis R. Rodriguez" <mcgrof@kernel.org>,
        Doug Ledford <dledford@redhat.com>,
        Bart Van Assche <bart.vanassche@sandisk.com>, sstabellini@kernel.org,
        Daniel Vetter <daniel.vetter@ffwll.ch>,
        Michael Ellerman <mpe@ellerman.id.au>,
        Mark Salter <msalter@redhat.com>,
        Nicolas Dichtel <nicolas.dichtel@6wind.com>,
        Paul Gortmaker <paul.gortmaker@windriver.com>,
        Guenter Roeck <linux@roeck-us.net>,
        Heiko Carstens <heiko.carstens@de.ibm.com>,
        Martin Schwidefsky <schwidefsky@de.ibm.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        patches@groups.riscv.org,
        "nathan=20Neusch=C3=A4fer?=" <j.neuschaefer@gmx.net>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 1557
Lines: 49

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