diff mbox series

[v1,1/1] software node: Update MAINTAINERS data base

Message ID 20211221071409.14361-1-andriy.shevchenko@linux.intel.com (mailing list archive)
State Superseded, archived
Headers show
Series [v1,1/1] software node: Update MAINTAINERS data base | expand

Commit Message

Andy Shevchenko Dec. 21, 2021, 7:14 a.m. UTC
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 <andriy.shevchenko@linux.intel.com>
---

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(-)

Comments

Daniel Scally Dec. 21, 2021, 9:15 a.m. UTC | #1
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 <andriy.shevchenko@linux.intel.com>
> ---
>
> 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 <andriy.shevchenko@linux.intel.com>
>  R:	Heikki Krogerus <heikki.krogerus@linux.intel.com>
> +R:	Daniel Scally <djrscally@gmail.com>
>  L:	linux-acpi@vger.kernel.org
>  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 <song@kernel.org>
Andy Shevchenko Dec. 21, 2021, 9:16 a.m. UTC | #2
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 <andriy.shevchenko@linux.intel.com>
> > ---
> >
> > 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).
Daniel Scally Dec. 21, 2021, 9:23 a.m. UTC | #3
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 <andriy.shevchenko@linux.intel.com>
>>> ---
>>>
>>> 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 <djrscally@gmail.com>
Sakari Ailus Dec. 21, 2021, 11:54 a.m. UTC | #4
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 <andriy.shevchenko@linux.intel.com>
> ---
> 
> 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 <andriy.shevchenko@linux.intel.com>
>  R:	Heikki Krogerus <heikki.krogerus@linux.intel.com>
> +R:	Daniel Scally <djrscally@gmail.com>

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

>  L:	linux-acpi@vger.kernel.org
>  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 <song@kernel.org>
Heikki Krogerus Dec. 21, 2021, 12:22 p.m. UTC | #5
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 <andriy.shevchenko@linux.intel.com>

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 <heikki.krogerus@linux.intel.com>

> ---
> 
> 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 <andriy.shevchenko@linux.intel.com>
>  R:	Heikki Krogerus <heikki.krogerus@linux.intel.com>
> +R:	Daniel Scally <djrscally@gmail.com>
>  L:	linux-acpi@vger.kernel.org
>  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 <song@kernel.org>
> -- 
> 2.34.1

thanks,
Andy Shevchenko Dec. 21, 2021, 1:31 p.m. UTC | #6
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 <djrscally@gmail.com>
> 
> I wouldn't mind being added as reviewer, too. :-)

NP, I have just sent a v2 with your name included.
Andy Shevchenko Jan. 4, 2022, 3:43 p.m. UTC | #7
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 <andriy.shevchenko@linux.intel.com>
> 
> 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 <heikki.krogerus@linux.intel.com>

Thanks!
diff mbox series

Patch

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 <andriy.shevchenko@linux.intel.com>
 R:	Heikki Krogerus <heikki.krogerus@linux.intel.com>
+R:	Daniel Scally <djrscally@gmail.com>
 L:	linux-acpi@vger.kernel.org
 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 <song@kernel.org>