2021-12-21 07:14:11

by Andy Shevchenko

[permalink] [raw]
Subject: [PATCH v1 1/1] software node: Update MAINTAINERS data base

There are two updates to the MAINTAINERS regarding to software node API:
- add Dan Scally to be designated reviewer
- add rather tightly related device property files to the list
- adjust section name accordingly

Signed-off-by: Andy Shevchenko <[email protected]>
---

Based on the latest Dan's involvement and amount of patches seen recently
I went ahead and added his name to the list. Dan, please tell me if it's
not appropriate.

MAINTAINERS | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8912b2c1260c..ccb4aa744540 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17666,12 +17666,16 @@ F: drivers/firmware/arm_sdei.c
F: include/linux/arm_sdei.h
F: include/uapi/linux/arm_sdei.h

-SOFTWARE NODES
+SOFTWARE NODES AND DEVICE PROPERTIES
R: Andy Shevchenko <[email protected]>
R: Heikki Krogerus <[email protected]>
+R: Daniel Scally <[email protected]>
L: [email protected]
S: Maintained
+F: drivers/base/property.c
F: drivers/base/swnode.c
+F: include/linux/fwnode.h
+F: include/linux/property.h

SOFTWARE RAID (Multiple Disks) SUPPORT
M: Song Liu <[email protected]>
--
2.34.1



2021-12-21 09:15:35

by Daniel Scally

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

Morning

On 21/12/2021 07:14, Andy Shevchenko wrote:
> There are two updates to the MAINTAINERS regarding to software node API:
> - add Dan Scally to be designated reviewer
> - add rather tightly related device property files to the list
> - adjust section name accordingly
>
> Signed-off-by: Andy Shevchenko <[email protected]>
> ---
>
> Based on the latest Dan's involvement and amount of patches seen recently
> I went ahead and added his name to the list. Dan, please tell me if it's
> not appropriate.


This is fine by me - I don't know if you need a tag from me for this but
feel free to add whichever's appropriate

>
> MAINTAINERS | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8912b2c1260c..ccb4aa744540 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17666,12 +17666,16 @@ F: drivers/firmware/arm_sdei.c
> F: include/linux/arm_sdei.h
> F: include/uapi/linux/arm_sdei.h
>
> -SOFTWARE NODES
> +SOFTWARE NODES AND DEVICE PROPERTIES
> R: Andy Shevchenko <[email protected]>
> R: Heikki Krogerus <[email protected]>
> +R: Daniel Scally <[email protected]>
> L: [email protected]
> S: Maintained
> +F: drivers/base/property.c
> F: drivers/base/swnode.c
> +F: include/linux/fwnode.h
> +F: include/linux/property.h
>
> SOFTWARE RAID (Multiple Disks) SUPPORT
> M: Song Liu <[email protected]>

2021-12-21 09:18:31

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

On Tue, Dec 21, 2021 at 09:15:30AM +0000, Daniel Scally wrote:
> Morning
>
> On 21/12/2021 07:14, Andy Shevchenko wrote:
> > There are two updates to the MAINTAINERS regarding to software node API:
> > - add Dan Scally to be designated reviewer
> > - add rather tightly related device property files to the list
> > - adjust section name accordingly
> >
> > Signed-off-by: Andy Shevchenko <[email protected]>
> > ---
> >
> > Based on the latest Dan's involvement and amount of patches seen recently
> > I went ahead and added his name to the list. Dan, please tell me if it's
> > not appropriate.
>
> This is fine by me - I don't know if you need a tag from me for this but
> feel free to add whichever's appropriate

Yep, please give a formal tag (usually Acked-by in this kind of cases).


--
With Best Regards,
Andy Shevchenko



2021-12-21 09:24:01

by Daniel Scally

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base


On 21/12/2021 09:16, Andy Shevchenko wrote:
> On Tue, Dec 21, 2021 at 09:15:30AM +0000, Daniel Scally wrote:
>> Morning
>>
>> On 21/12/2021 07:14, Andy Shevchenko wrote:
>>> There are two updates to the MAINTAINERS regarding to software node API:
>>> - add Dan Scally to be designated reviewer
>>> - add rather tightly related device property files to the list
>>> - adjust section name accordingly
>>>
>>> Signed-off-by: Andy Shevchenko <[email protected]>
>>> ---
>>>
>>> Based on the latest Dan's involvement and amount of patches seen recently
>>> I went ahead and added his name to the list. Dan, please tell me if it's
>>> not appropriate.
>> This is fine by me - I don't know if you need a tag from me for this but
>> feel free to add whichever's appropriate
> Yep, please give a formal tag (usually Acked-by in this kind of cases).
>
Acked-by: Daniel Scally <[email protected]>

2021-12-21 11:55:10

by Sakari Ailus

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

Hi Andy,

On Tue, Dec 21, 2021 at 09:14:09AM +0200, Andy Shevchenko wrote:
> There are two updates to the MAINTAINERS regarding to software node API:
> - add Dan Scally to be designated reviewer
> - add rather tightly related device property files to the list
> - adjust section name accordingly
>
> Signed-off-by: Andy Shevchenko <[email protected]>
> ---
>
> Based on the latest Dan's involvement and amount of patches seen recently
> I went ahead and added his name to the list. Dan, please tell me if it's
> not appropriate.
>
> MAINTAINERS | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8912b2c1260c..ccb4aa744540 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17666,12 +17666,16 @@ F: drivers/firmware/arm_sdei.c
> F: include/linux/arm_sdei.h
> F: include/uapi/linux/arm_sdei.h
>
> -SOFTWARE NODES
> +SOFTWARE NODES AND DEVICE PROPERTIES
> R: Andy Shevchenko <[email protected]>
> R: Heikki Krogerus <[email protected]>
> +R: Daniel Scally <[email protected]>

I wouldn't mind being added as reviewer, too. :-)

> L: [email protected]
> S: Maintained
> +F: drivers/base/property.c
> F: drivers/base/swnode.c
> +F: include/linux/fwnode.h
> +F: include/linux/property.h
>
> SOFTWARE RAID (Multiple Disks) SUPPORT
> M: Song Liu <[email protected]>

--
Regards,

Sakari Ailus

2021-12-21 12:22:36

by Heikki Krogerus

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

On Tue, Dec 21, 2021 at 09:14:09AM +0200, Andy Shevchenko wrote:
> There are two updates to the MAINTAINERS regarding to software node API:
> - add Dan Scally to be designated reviewer
> - add rather tightly related device property files to the list
> - adjust section name accordingly
>
> Signed-off-by: Andy Shevchenko <[email protected]>

Another way would be to have a separate entry for the unified device
property API, and that could then have you (Andy), Sakari and Daniel
as the reviers. I don't think I have much to say about those parts -
I'm only interested in the software nodes. But it's up to you. FWIW:

Acked-by: Heikki Krogerus <[email protected]>

> ---
>
> Based on the latest Dan's involvement and amount of patches seen recently
> I went ahead and added his name to the list. Dan, please tell me if it's
> not appropriate.
>
> MAINTAINERS | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8912b2c1260c..ccb4aa744540 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17666,12 +17666,16 @@ F: drivers/firmware/arm_sdei.c
> F: include/linux/arm_sdei.h
> F: include/uapi/linux/arm_sdei.h
>
> -SOFTWARE NODES
> +SOFTWARE NODES AND DEVICE PROPERTIES
> R: Andy Shevchenko <[email protected]>
> R: Heikki Krogerus <[email protected]>
> +R: Daniel Scally <[email protected]>
> L: [email protected]
> S: Maintained
> +F: drivers/base/property.c
> F: drivers/base/swnode.c
> +F: include/linux/fwnode.h
> +F: include/linux/property.h
>
> SOFTWARE RAID (Multiple Disks) SUPPORT
> M: Song Liu <[email protected]>
> --
> 2.34.1

thanks,

--
heikki

2021-12-21 13:32:55

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

On Tue, Dec 21, 2021 at 01:54:58PM +0200, Sakari Ailus wrote:
> On Tue, Dec 21, 2021 at 09:14:09AM +0200, Andy Shevchenko wrote:

> > +R: Daniel Scally <[email protected]>
>
> I wouldn't mind being added as reviewer, too. :-)

NP, I have just sent a v2 with your name included.

--
With Best Regards,
Andy Shevchenko



2022-01-04 15:44:51

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] software node: Update MAINTAINERS data base

On Tue, Dec 21, 2021 at 02:22:31PM +0200, Heikki Krogerus wrote:
> On Tue, Dec 21, 2021 at 09:14:09AM +0200, Andy Shevchenko wrote:
> > There are two updates to the MAINTAINERS regarding to software node API:
> > - add Dan Scally to be designated reviewer
> > - add rather tightly related device property files to the list
> > - adjust section name accordingly
> >
> > Signed-off-by: Andy Shevchenko <[email protected]>
>
> Another way would be to have a separate entry for the unified device
> property API, and that could then have you (Andy), Sakari and Daniel
> as the reviers. I don't think I have much to say about those parts -
> I'm only interested in the software nodes. But it's up to you. FWIW:

Currently property.h shares information that is highly used by swnode.c.
I believe the current variant describes the state of affairs as is.

> Acked-by: Heikki Krogerus <[email protected]>

Thanks!

--
With Best Regards,
Andy Shevchenko