2023-05-17 21:50:18

by Ira Weiny

[permalink] [raw]
Subject: [PATCH 1/3] MAINTAINERS: Add additional reviewers for CXL

To get better coverage add Dave Jiang and Jonathan Cameron as reviewers.

Cc: Dave Jiang <[email protected]>
Cc: Jonathan Cameron <[email protected]>
Signed-off-by: Ira Weiny <[email protected]>
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 90abe83c02f3..6f8568ec2731 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -5169,6 +5169,8 @@ M: Vishal Verma <[email protected]>
M: Ira Weiny <[email protected]>
M: Ben Widawsky <[email protected]>
M: Dan Williams <[email protected]>
+R: Dave Jiang <[email protected]>
+R: Jonathan Cameron <[email protected]>
L: [email protected]
S: Maintained
F: drivers/cxl/

--
2.40.0



2023-05-17 22:15:05

by Dave Jiang

[permalink] [raw]
Subject: Re: [PATCH 1/3] MAINTAINERS: Add additional reviewers for CXL



On 5/17/23 2:28 PM, Ira Weiny wrote:
> To get better coverage add Dave Jiang and Jonathan Cameron as reviewers.
>
> Cc: Dave Jiang <[email protected]>
> Cc: Jonathan Cameron <[email protected]>
> Signed-off-by: Ira Weiny <[email protected]>

Acked-by: Dave Jiang <[email protected]>

> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 90abe83c02f3..6f8568ec2731 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5169,6 +5169,8 @@ M: Vishal Verma <[email protected]>
> M: Ira Weiny <[email protected]>
> M: Ben Widawsky <[email protected]>
> M: Dan Williams <[email protected]>
> +R: Dave Jiang <[email protected]>
> +R: Jonathan Cameron <[email protected]>
> L: [email protected]
> S: Maintained
> F: drivers/cxl/
>

2023-05-18 09:49:58

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 1/3] MAINTAINERS: Add additional reviewers for CXL

On Wed, 17 May 2023 14:28:10 -0700
Ira Weiny <[email protected]> wrote:

> To get better coverage add Dave Jiang and Jonathan Cameron as reviewers.
>
> Cc: Dave Jiang <[email protected]>
> Cc: Jonathan Cameron <[email protected]>
> Signed-off-by: Ira Weiny <[email protected]>

Acked-by: Jonathan Cameron <[email protected]>

One comment inline though on capitalization.

Thanks,

Jonathan

> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 90abe83c02f3..6f8568ec2731 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5169,6 +5169,8 @@ M: Vishal Verma <[email protected]>
> M: Ira Weiny <[email protected]>
> M: Ben Widawsky <[email protected]>
> M: Dan Williams <[email protected]>
> +R: Dave Jiang <[email protected]>
> +R: Jonathan Cameron <[email protected]>

For consistency go with <[email protected]>
which is what I'm using elsewhere in maintainers and
matches other lower case entries here.



> L: [email protected]
> S: Maintained
> F: drivers/cxl/
>


2023-05-18 14:52:24

by Ira Weiny

[permalink] [raw]
Subject: Re: [PATCH 1/3] MAINTAINERS: Add additional reviewers for CXL

Jonathan Cameron wrote:
> On Wed, 17 May 2023 14:28:10 -0700
> Ira Weiny <[email protected]> wrote:
>
> > To get better coverage add Dave Jiang and Jonathan Cameron as reviewers.
> >
> > Cc: Dave Jiang <[email protected]>
> > Cc: Jonathan Cameron <[email protected]>
> > Signed-off-by: Ira Weiny <[email protected]>
>
> Acked-by: Jonathan Cameron <[email protected]>
>
> One comment inline though on capitalization.
>
> Thanks,
>
> Jonathan
>
> > ---
> > MAINTAINERS | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 90abe83c02f3..6f8568ec2731 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -5169,6 +5169,8 @@ M: Vishal Verma <[email protected]>
> > M: Ira Weiny <[email protected]>
> > M: Ben Widawsky <[email protected]>
> > M: Dan Williams <[email protected]>
> > +R: Dave Jiang <[email protected]>
> > +R: Jonathan Cameron <[email protected]>
>
> For consistency go with <[email protected]>
> which is what I'm using elsewhere in maintainers and
> matches other lower case entries here.

NP on this change. However, FYI b4 picks up your email capitalized a lot.
For example when you ack/review as you did above.

Ira

>
>
>
> > L: [email protected]
> > S: Maintained
> > F: drivers/cxl/
> >
>