2022-05-05 22:16:02

by Conor Dooley

[permalink] [raw]
Subject: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

Hardware random, PCI and clock drivers for the PolarFire SoC have been
upstreamed but are not covered by the MAINTAINERS entry, so add them.
Daire is the author of the clock & PCI drivers, so add him as a
maintainer in place of Lewis.

Signed-off-by: Conor Dooley <[email protected]>
---
MAINTAINERS | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index fd768d43e048..d7602658b0a5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -16939,12 +16939,15 @@ N: riscv
K: riscv

RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
-M: Lewis Hanly <[email protected]>
M: Conor Dooley <[email protected]>
+M: Daire McNamara <[email protected]>
L: [email protected]
S: Supported
F: arch/riscv/boot/dts/microchip/
+F: drivers/char/hw_random/mpfs-rng.c
+F: drivers/clk/microchip/clk-mpfs.c
F: drivers/mailbox/mailbox-mpfs.c
+F: drivers/pci/controller/pcie-microchip-host.c
F: drivers/soc/microchip/
F: include/soc/microchip/mpfs.h

--
2.35.2



2022-05-23 11:43:14

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On 05/05/2022 11:55, Conor Dooley wrote:
> Hardware random, PCI and clock drivers for the PolarFire SoC have been
> upstreamed but are not covered by the MAINTAINERS entry, so add them.
> Daire is the author of the clock & PCI drivers, so add him as a
> maintainer in place of Lewis.
>
> Signed-off-by: Conor Dooley <[email protected]>

Hey Palmer,
I know youre busy etc but just a reminder :)
Thanks,
Conor.

> ---
> MAINTAINERS | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index fd768d43e048..d7602658b0a5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -16939,12 +16939,15 @@ N: riscv
> K: riscv
>
> RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
> -M: Lewis Hanly <[email protected]>
> M: Conor Dooley <[email protected]>
> +M: Daire McNamara <[email protected]>
> L: [email protected]
> S: Supported
> F: arch/riscv/boot/dts/microchip/
> +F: drivers/char/hw_random/mpfs-rng.c
> +F: drivers/clk/microchip/clk-mpfs.c
> F: drivers/mailbox/mailbox-mpfs.c
> +F: drivers/pci/controller/pcie-microchip-host.c
> F: drivers/soc/microchip/
> F: include/soc/microchip/mpfs.h
>

2022-05-23 20:43:35

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
> On 05/05/2022 11:55, Conor Dooley wrote:
>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
>> Daire is the author of the clock & PCI drivers, so add him as a
>> maintainer in place of Lewis.
>>
>> Signed-off-by: Conor Dooley <[email protected]>
>
> Hey Palmer,
> I know youre busy etc but just a reminder :)

Sorry, I didn't realize this was aimed at the RISC-V tree. I'm fine
taking it, but it seems like these should have gone in along with the
drivers.

Arnd: maybe this is really an SOC tree sort of thing? No big deal
either way on my end, just let me know.

> Thanks,
> Conor.
>
>> ---
>> MAINTAINERS | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index fd768d43e048..d7602658b0a5 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -16939,12 +16939,15 @@ N: riscv
>> K: riscv
>>
>> RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
>> -M: Lewis Hanly <[email protected]>
>> M: Conor Dooley <[email protected]>
>> +M: Daire McNamara <[email protected]>
>> L: [email protected]
>> S: Supported
>> F: arch/riscv/boot/dts/microchip/
>> +F: drivers/char/hw_random/mpfs-rng.c
>> +F: drivers/clk/microchip/clk-mpfs.c
>> F: drivers/mailbox/mailbox-mpfs.c
>> +F: drivers/pci/controller/pcie-microchip-host.c
>> F: drivers/soc/microchip/
>> F: include/soc/microchip/mpfs.h
>>
>

2022-05-23 20:44:06

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On 23/05/2022 20:52, Palmer Dabbelt wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
>> On 05/05/2022 11:55, Conor Dooley wrote:
>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
>>> Daire is the author of the clock & PCI drivers, so add him as a
>>> maintainer in place of Lewis.
>>>
>>> Signed-off-by: Conor Dooley <[email protected]>
>>
>> Hey Palmer,
>> I know youre busy etc but just a reminder :)
>
> Sorry, I didn't realize this was aimed at the RISC-V tree.  I'm fine
> taking it, but it seems like these should have gone in along with the
> drivers.

Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
the rng bundled this way b/c I didn't want to end up a conflict.
Obv. there's not a rush so I can always split it back out if needs be.

>
> Arnd: maybe this is really an SOC tree sort of thing?  No big deal
> either way on my end, just let me know.
>
>> Thanks,
>> Conor.
>>
>>> ---
>>>   MAINTAINERS | 5 ++++-
>>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index fd768d43e048..d7602658b0a5 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -16939,12 +16939,15 @@ N: riscv
>>>   K: riscv
>>>
>>>   RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
>>> -M:  Lewis Hanly <[email protected]>
>>>   M: Conor Dooley <[email protected]>
>>> +M:  Daire McNamara <[email protected]>
>>>   L: [email protected]
>>>   S: Supported
>>>   F: arch/riscv/boot/dts/microchip/
>>> +F:  drivers/char/hw_random/mpfs-rng.c
>>> +F:  drivers/clk/microchip/clk-mpfs.c
>>>   F: drivers/mailbox/mailbox-mpfs.c
>>> +F:  drivers/pci/controller/pcie-microchip-host.c
>>>   F: drivers/soc/microchip/
>>>   F: include/soc/microchip/mpfs.h
>>>
>>

2022-06-02 01:55:57

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On Mon, 23 May 2022 13:00:01 PDT (-0700), [email protected] wrote:
> On 23/05/2022 20:52, Palmer Dabbelt wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
>>> On 05/05/2022 11:55, Conor Dooley wrote:
>>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
>>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
>>>> Daire is the author of the clock & PCI drivers, so add him as a
>>>> maintainer in place of Lewis.
>>>>
>>>> Signed-off-by: Conor Dooley <[email protected]>
>>>
>>> Hey Palmer,
>>> I know youre busy etc but just a reminder :)
>>
>> Sorry, I didn't realize this was aimed at the RISC-V tree.  I'm fine
>> taking it, but it seems like these should have gone in along with the
>> drivers.
>
> Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
> the rng bundled this way b/c I didn't want to end up a conflict.
> Obv. there's not a rush so I can always split it back out if needs be.

I'm adding a bunch of subsystem maintainers just to check again. I
don't have any problem with it, just not really a RISC-V thing and don't
wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
for now.

Sorry if I'm being overly pedantic about this one...

>
>>
>> Arnd: maybe this is really an SOC tree sort of thing?  No big deal
>> either way on my end, just let me know.
>>
>>> Thanks,
>>> Conor.
>>>
>>>> ---
>>>>   MAINTAINERS | 5 ++++-
>>>>   1 file changed, 4 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index fd768d43e048..d7602658b0a5 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -16939,12 +16939,15 @@ N: riscv
>>>>   K: riscv
>>>>
>>>>   RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
>>>> -M:  Lewis Hanly <[email protected]>
>>>>   M: Conor Dooley <[email protected]>
>>>> +M:  Daire McNamara <[email protected]>
>>>>   L: [email protected]
>>>>   S: Supported
>>>>   F: arch/riscv/boot/dts/microchip/
>>>> +F:  drivers/char/hw_random/mpfs-rng.c
>>>> +F:  drivers/clk/microchip/clk-mpfs.c
>>>>   F: drivers/mailbox/mailbox-mpfs.c
>>>> +F:  drivers/pci/controller/pcie-microchip-host.c
>>>>   F: drivers/soc/microchip/
>>>>   F: include/soc/microchip/mpfs.h
>>>>
>>>
>

2022-06-02 06:19:20

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On 02/06/2022 02:55, Palmer Dabbelt wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> On Mon, 23 May 2022 13:00:01 PDT (-0700), [email protected] wrote:
>> On 23/05/2022 20:52, Palmer Dabbelt wrote:
>>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>>
>>> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
>>>> On 05/05/2022 11:55, Conor Dooley wrote:
>>>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
>>>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
>>>>> Daire is the author of the clock & PCI drivers, so add him as a
>>>>> maintainer in place of Lewis.
>>>>>
>>>>> Signed-off-by: Conor Dooley <[email protected]>
>>>>
>>>> Hey Palmer,
>>>> I know youre busy etc but just a reminder :)
>>>
>>> Sorry, I didn't realize this was aimed at the RISC-V tree. I'm fine
>>> taking it, but it seems like these should have gone in along with the
>>> drivers.
>>
>> Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
>> the rng bundled this way b/c I didn't want to end up a conflict.
>> Obv. there's not a rush so I can always split it back out if needs be.
>
> I'm adding a bunch of subsystem maintainers just to check again. I
> don't have any problem with it, just not really a RISC-V thing and don't
> wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
> for now.
>
> Sorry if I'm being overly pedantic about this one...

I don't mind. Maybe this should go via Andrew next cycle or w/e?
There's obviously no hurry etc

>
>>
>>>
>>> Arnd: maybe this is really an SOC tree sort of thing? No big deal
>>> either way on my end, just let me know.
>>>
>>>> Thanks,
>>>> Conor.
>>>>
>>>>> ---
>>>>> MAINTAINERS | 5 ++++-
>>>>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>> index fd768d43e048..d7602658b0a5 100644
>>>>> --- a/MAINTAINERS
>>>>> +++ b/MAINTAINERS
>>>>> @@ -16939,12 +16939,15 @@ N: riscv
>>>>> K: riscv
>>>>>
>>>>> RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
>>>>> -M: Lewis Hanly <[email protected]>
>>>>> M: Conor Dooley <[email protected]>
>>>>> +M: Daire McNamara <[email protected]>
>>>>> L: [email protected]
>>>>> S: Supported
>>>>> F: arch/riscv/boot/dts/microchip/
>>>>> +F: drivers/char/hw_random/mpfs-rng.c
>>>>> +F: drivers/clk/microchip/clk-mpfs.c
>>>>> F: drivers/mailbox/mailbox-mpfs.c
>>>>> +F: drivers/pci/controller/pcie-microchip-host.c
>>>>> F: drivers/soc/microchip/
>>>>> F: include/soc/microchip/mpfs.h
>>>>>
>>>>
>>
>
>
> _______________________________________________
> linux-riscv mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-riscv

2022-06-03 05:49:30

by Bjorn Helgaas

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On Thu, Jun 02, 2022 at 04:39:08AM +0000, [email protected] wrote:
> On 02/06/2022 02:55, Palmer Dabbelt wrote:

> > I'm adding a bunch of subsystem maintainers just to check again. I
> > don't have any problem with it, just not really a RISC-V thing and don't
> > wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
> > for now.
> >
> > Sorry if I'm being overly pedantic about this one...
>
> I don't mind. Maybe this should go via Andrew next cycle or w/e?
> There's obviously no hurry etc

My turn to be overly pedantic :) IMHO there's no benefit in delaying
MAINTAINERS updates. There's zero risk, and delaying only means
people will miss out on bug reports and other things they should learn
about.

Bjorn

2022-06-03 16:22:11

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On Thu, 02 Jun 2022 09:31:52 PDT (-0700), [email protected] wrote:
> On Thu, Jun 02, 2022 at 04:39:08AM +0000, [email protected] wrote:
>> On 02/06/2022 02:55, Palmer Dabbelt wrote:
>
>> > I'm adding a bunch of subsystem maintainers just to check again. I
>> > don't have any problem with it, just not really a RISC-V thing and don't
>> > wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
>> > for now.
>> >
>> > Sorry if I'm being overly pedantic about this one...
>>
>> I don't mind. Maybe this should go via Andrew next cycle or w/e?
>> There's obviously no hurry etc
>
> My turn to be overly pedantic :) IMHO there's no benefit in delaying
> MAINTAINERS updates. There's zero risk, and delaying only means
> people will miss out on bug reports and other things they should learn
> about.

If by "delay" you mean wait until a merge window, then I definately
agree -- that's just more cofusing for folks to have defacto
maintainership outside of the tree, might as well get these in. It's
not like a MAINTAINERS update is going to introduce a regression or
anything.

I'm just delaying because I just want to wait to make sure folks from
the subsystems are OK with the updates, as these aren't really anything
to do with RISC-V so it's not really my decision to make.

2022-06-03 17:46:56

by Bjorn Helgaas

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

On Wed, Jun 01, 2022 at 06:55:40PM -0700, Palmer Dabbelt wrote:
> On Mon, 23 May 2022 13:00:01 PDT (-0700), [email protected] wrote:
> > On 23/05/2022 20:52, Palmer Dabbelt wrote:
> >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >>
> >> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
> >>> On 05/05/2022 11:55, Conor Dooley wrote:
> >>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
> >>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
> >>>> Daire is the author of the clock & PCI drivers, so add him as a
> >>>> maintainer in place of Lewis.
> >>>>
> >>>> Signed-off-by: Conor Dooley <[email protected]>
> >>>
> >>> Hey Palmer,
> >>> I know youre busy etc but just a reminder :)
> >>
> >> Sorry, I didn't realize this was aimed at the RISC-V tree.? I'm fine
> >> taking it, but it seems like these should have gone in along with the
> >> drivers.
> >
> > Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
> > the rng bundled this way b/c I didn't want to end up a conflict.
> > Obv. there's not a rush so I can always split it back out if needs be.
>
> I'm adding a bunch of subsystem maintainers just to check again. I
> don't have any problem with it, just not really a RISC-V thing and don't
> wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
> for now.

Fine with me, if you want it:

Acked-by: Bjorn Helgaas <[email protected]>

> >> Arnd: maybe this is really an SOC tree sort of thing?? No big deal
> >> either way on my end, just let me know.
> >>
> >>> Thanks,
> >>> Conor.
> >>>
> >>>> ---
> >>>> ? MAINTAINERS | 5 ++++-
> >>>> ? 1 file changed, 4 insertions(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>>> index fd768d43e048..d7602658b0a5 100644
> >>>> --- a/MAINTAINERS
> >>>> +++ b/MAINTAINERS
> >>>> @@ -16939,12 +16939,15 @@ N: riscv
> >>>> ? K: riscv
> >>>>
> >>>> ? RISC-V/MICROCHIP POLARFIRE SOC SUPPORT
> >>>> -M:? Lewis Hanly <[email protected]>
> >>>> ? M: Conor Dooley <[email protected]>
> >>>> +M:? Daire McNamara <[email protected]>
> >>>> ? L: [email protected]
> >>>> ? S: Supported
> >>>> ? F: arch/riscv/boot/dts/microchip/
> >>>> +F:? drivers/char/hw_random/mpfs-rng.c
> >>>> +F:? drivers/clk/microchip/clk-mpfs.c
> >>>> ? F: drivers/mailbox/mailbox-mpfs.c
> >>>> +F:? drivers/pci/controller/pcie-microchip-host.c
> >>>> ? F: drivers/soc/microchip/
> >>>> ? F: include/soc/microchip/mpfs.h
> >>>>
> >>>
> >

2022-06-09 23:06:46

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers



On 09/06/2022 23:43, Stephen Boyd wrote:
> Quoting Palmer Dabbelt (2022-06-01 18:55:40)
>> On Mon, 23 May 2022 13:00:01 PDT (-0700), [email protected] wrote:
>>> On 23/05/2022 20:52, Palmer Dabbelt wrote:
>>>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>>>
>>>> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
>>>>> On 05/05/2022 11:55, Conor Dooley wrote:
>>>>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
>>>>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
>>>>>> Daire is the author of the clock & PCI drivers, so add him as a
>>>>>> maintainer in place of Lewis.
>>>>>>
>>>>>> Signed-off-by: Conor Dooley <[email protected]>
>>>>>
>>>>> Hey Palmer,
>>>>> I know youre busy etc but just a reminder :)
>>>>
>>>> Sorry, I didn't realize this was aimed at the RISC-V tree.�� I'm fine
>>>> taking it, but it seems like these should have gone in along with the
>>>> drivers.
>>>
>>> Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
>>> the rng bundled this way b/c I didn't want to end up a conflict.
>>> Obv. there's not a rush so I can always split it back out if needs be.
>>
>> I'm adding a bunch of subsystem maintainers just to check again. I
>> don't have any problem with it, just not really a RISC-V thing and don't
>> wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
>> for now.
>>
>
> Acked-by: Stephen Boyd <[email protected]>
>

+CC Herbert for the last one.

This is likely to generate a conflict in -next though b/c
my spi driver that was applied for 5.20 also touches this
entry. It really pisses me off that I allowed this to
happen & become a burden..

What am I "meant" to do here? Do the merges myself & provide
the conflict resolution? Or for something as obvious as this
is it enough to notify that there will be one?

Sorry,
Conor.

2022-06-09 23:49:41

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH v4 1/1] MAINTAINERS: add polarfire rng, pci and clock drivers

Quoting Palmer Dabbelt (2022-06-01 18:55:40)
> On Mon, 23 May 2022 13:00:01 PDT (-0700), [email protected] wrote:
> > On 23/05/2022 20:52, Palmer Dabbelt wrote:
> >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >>
> >> On Mon, 23 May 2022 04:42:53 PDT (-0700), [email protected] wrote:
> >>> On 05/05/2022 11:55, Conor Dooley wrote:
> >>>> Hardware random, PCI and clock drivers for the PolarFire SoC have been
> >>>> upstreamed but are not covered by the MAINTAINERS entry, so add them.
> >>>> Daire is the author of the clock & PCI drivers, so add him as a
> >>>> maintainer in place of Lewis.
> >>>>
> >>>> Signed-off-by: Conor Dooley <[email protected]>
> >>>
> >>> Hey Palmer,
> >>> I know youre busy etc but just a reminder :)
> >>
> >> Sorry, I didn't realize this was aimed at the RISC-V tree.�� I'm fine
> >> taking it, but it seems like these should have gone in along with the
> >> drivers.
> >
> > Yeah, sorry. In hindsight it should've but that ship has sailed. I sent
> > the rng bundled this way b/c I didn't want to end up a conflict.
> > Obv. there's not a rush so I can always split it back out if needs be.
>
> I'm adding a bunch of subsystem maintainers just to check again. I
> don't have any problem with it, just not really a RISC-V thing and don't
> wan to make a mess. I've stashed it over at palmer/pcsoc-maintainers
> for now.
>

Acked-by: Stephen Boyd <[email protected]>