MAINTAINERS: Add the nios2 interrupt controller to the nios2 section

[PATCH] MAINTAINERS: Add the nios2 interrupt controller to the nios2 section

Posted by Thomas Huth 7 months, 1 week ago

These files obviously belong to the nios2 target, so they should
be listed in the nios2 section in the MAINTAINERS file.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 6eecd617e0..45eb230465 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -296,7 +296,9 @@ R: Marek Vasut <marex@denx.de>
 S: Orphan
 F: target/nios2/
 F: hw/nios2/
+F: hw/intc/nios2_vic.c
 F: disas/nios2.c
+F: include/hw/intc/nios2_vic.h
 F: configs/devices/nios2-softmmu/default.mak
 F: tests/docker/dockerfiles/debian-nios2-cross.d/build-toolchain.sh
 F: tests/tcg/nios2/
-- 
2.41.0

Re: [PATCH] MAINTAINERS: Add the nios2 interrupt controller to the nios2 section

Posted by Michael Tokarev 7 months ago

13.10.2023 09:54, Thomas Huth wrote:
> These files obviously belong to the nios2 target, so they should
> be listed in the nios2 section in the MAINTAINERS file.

Queued to trivial-patches, thanks!

/mjt

Re: [PATCH] MAINTAINERS: Add the nios2 interrupt controller to the nios2 section

Posted by Philippe Mathieu-Daudé 7 months, 1 week ago

On 13/10/23 08:54, Thomas Huth wrote:
> These files obviously belong to the nios2 target, so they should
> be listed in the nios2 section in the MAINTAINERS file.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6eecd617e0..45eb230465 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -296,7 +296,9 @@ R: Marek Vasut <marex@denx.de>
>   S: Orphan
>   F: target/nios2/
>   F: hw/nios2/
> +F: hw/intc/nios2_vic.c
>   F: disas/nios2.c
> +F: include/hw/intc/nios2_vic.h
>   F: configs/devices/nios2-softmmu/default.mak
>   F: tests/docker/dockerfiles/debian-nios2-cross.d/build-toolchain.sh
>   F: tests/tcg/nios2/

Could we have a NiosII hardware section instead?

Re: [PATCH] MAINTAINERS: Add the nios2 interrupt controller to the nios2 section

Posted by Thomas Huth 7 months, 1 week ago

On 13/10/2023 10.51, Philippe Mathieu-Daudé wrote:
> On 13/10/23 08:54, Thomas Huth wrote:
>> These files obviously belong to the nios2 target, so they should
>> be listed in the nios2 section in the MAINTAINERS file.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>   MAINTAINERS | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 6eecd617e0..45eb230465 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -296,7 +296,9 @@ R: Marek Vasut <marex@denx.de>
>>   S: Orphan
>>   F: target/nios2/
>>   F: hw/nios2/
>> +F: hw/intc/nios2_vic.c
>>   F: disas/nios2.c
>> +F: include/hw/intc/nios2_vic.h
>>   F: configs/devices/nios2-softmmu/default.mak
>>   F: tests/docker/dockerfiles/debian-nios2-cross.d/build-toolchain.sh
>>   F: tests/tcg/nios2/
> 
> Could we have a NiosII hardware section instead?

Is it worth the effort? ... the code is in "Orphan" state anyway...

  Thomas