2018-10-09 14:20:39

by Bhardwaj, Rajneesh

[permalink] [raw]
Subject: [Patch v2] MAINTAINERS: intel_telemetry: Update maintainers info

Add myself and David as the new maintainers for Intel Telemetry driver
since Souvik no longer works with Intel. David and I work on power
management and debug related drivers which are pretty close to this one
in terms of functionalities so volunteering for the task.

Signed-off-by: Box, David E <[email protected]>
Signed-off-by: Rajneesh Bhardwaj <[email protected]>
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 22065048d89d..f33ebd1b8034 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7530,7 +7530,8 @@ F: arch/x86/crypto/sha*-mb/
F: crypto/mcryptd.c

INTEL TELEMETRY DRIVER
-M: Souvik Kumar Chakravarty <[email protected]>
+M: Rajneesh Bhardwaj <[email protected]>
+M: Box, David E <[email protected]>
L: [email protected]
S: Maintained
F: arch/x86/include/asm/intel_telemetry.h
--
2.17.1



2018-10-09 17:47:43

by Joe Perches

[permalink] [raw]
Subject: Re: [Patch v2] MAINTAINERS: intel_telemetry: Update maintainers info

On Tue, 2018-10-09 at 19:48 +0530, Rajneesh Bhardwaj wrote:
> Add myself and David as the new maintainers for Intel Telemetry driver
> since Souvik no longer works with Intel. David and I work on power
> management and debug related drivers which are pretty close to this one
> in terms of functionalities so volunteering for the task.
>
> Signed-off-by: Box, David E <[email protected]>
> Signed-off-by: Rajneesh Bhardwaj <[email protected]>
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 22065048d89d..f33ebd1b8034 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7530,7 +7530,8 @@ F: arch/x86/crypto/sha*-mb/
> F: crypto/mcryptd.c
>
> INTEL TELEMETRY DRIVER
> -M: Souvik Kumar Chakravarty <[email protected]>
> +M: Rajneesh Bhardwaj <[email protected]>
> +M: Box, David E <[email protected]>

It is better to quote your name when it uses
non-alphabetic characters like periods and commas.

Especially the comma because that is commonly used
by email clients as a multiple address separator so
it makes copy/paste use error-prone.

It's also likely better to use a more typical
"First <Initial> Last" name style.

> L: [email protected]
> S: Maintained
> F: arch/x86/include/asm/intel_telemetry.hch mor


2018-10-12 16:31:09

by Box, David E

[permalink] [raw]
Subject: [PATCH V3] MAINTAINERS: intel_telemetry: Update maintainers info

Remove Souvik who has left this role. Add Rajneesh and David who work
jointly on telemetry updates for new platforms.

Signed-off-by: David E. Box <[email protected]>
Signed-off-by: Rajneesh Bhardwaj <[email protected]>
---

V3: Change signoff format

MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 544cac829cf4..cbe1595db4a5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7355,7 +7355,8 @@ F: arch/x86/crypto/sha*-mb
F: crypto/mcryptd.c

INTEL TELEMETRY DRIVER
-M: Souvik Kumar Chakravarty <[email protected]>
+M: Rajneesh Bhardwaj <[email protected]>
+M: "David E. Box" <[email protected]>
L: [email protected]
S: Maintained
F: arch/x86/include/asm/intel_telemetry.h
--
2.14.4


2018-10-12 16:47:03

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH V3] MAINTAINERS: intel_telemetry: Update maintainers info

On Fri, 2018-10-12 at 09:28 -0700, David E. Box wrote:
> Remove Souvik who has left this role. Add Rajneesh and David who work
> jointly on telemetry updates for new platforms.
>
> Signed-off-by: David E. Box <[email protected]>
> Signed-off-by: Rajneesh Bhardwaj <[email protected]>
[]
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -7355,7 +7355,8 @@ F: arch/x86/crypto/sha*-mb
> F: crypto/mcryptd.c
>
> INTEL TELEMETRY DRIVER
> -M: Souvik Kumar Chakravarty <[email protected]>
> +M: Rajneesh Bhardwaj <[email protected]>
> +M: "David E. Box" <[email protected]>
> L: [email protected]
> S: Maintained

S: Status, one of the following:
Supported: Someone is actually paid to look after this.
Maintained: Someone actually looks after it.

Perhaps this is driver status should be marked

"S: Supported"

> F: arch/x86/include/asm/intel_telemetry.h


2018-10-16 15:52:42

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH V3] MAINTAINERS: intel_telemetry: Update maintainers info

On Fri, Oct 12, 2018 at 7:46 PM Joe Perches <[email protected]> wrote:
>
> On Fri, 2018-10-12 at 09:28 -0700, David E. Box wrote:
> > Remove Souvik who has left this role. Add Rajneesh and David who work
> > jointly on telemetry updates for new platforms.
> >
> > Signed-off-by: David E. Box <[email protected]>
> > Signed-off-by: Rajneesh Bhardwaj <[email protected]>
> []
> > diff --git a/MAINTAINERS b/MAINTAINERS
> []
> > @@ -7355,7 +7355,8 @@ F: arch/x86/crypto/sha*-mb
> > F: crypto/mcryptd.c
> >
> > INTEL TELEMETRY DRIVER
> > -M: Souvik Kumar Chakravarty <[email protected]>
> > +M: Rajneesh Bhardwaj <[email protected]>
> > +M: "David E. Box" <[email protected]>
> > L: [email protected]
> > S: Maintained
>
> S: Status, one of the following:
> Supported: Someone is actually paid to look after this.
> Maintained: Someone actually looks after it.
>
> Perhaps this is driver status should be marked
>
> "S: Supported"
>
> > F: arch/x86/include/asm/intel_telemetry.h

Thanks, Joe.
I may fix it when applying.

--
With Best Regards,
Andy Shevchenko

2018-10-19 14:03:03

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH V3] MAINTAINERS: intel_telemetry: Update maintainers info

On Tue, Oct 16, 2018 at 6:51 PM Andy Shevchenko
<[email protected]> wrote:
>
> On Fri, Oct 12, 2018 at 7:46 PM Joe Perches <[email protected]> wrote:
> >
> > On Fri, 2018-10-12 at 09:28 -0700, David E. Box wrote:
> > > Remove Souvik who has left this role. Add Rajneesh and David who work
> > > jointly on telemetry updates for new platforms.
> > >
> > > Signed-off-by: David E. Box <[email protected]>
> > > Signed-off-by: Rajneesh Bhardwaj <[email protected]>
> > []
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > []
> > > @@ -7355,7 +7355,8 @@ F: arch/x86/crypto/sha*-mb
> > > F: crypto/mcryptd.c
> > >
> > > INTEL TELEMETRY DRIVER
> > > -M: Souvik Kumar Chakravarty <[email protected]>
> > > +M: Rajneesh Bhardwaj <[email protected]>
> > > +M: "David E. Box" <[email protected]>
> > > L: [email protected]
> > > S: Maintained
> >
> > S: Status, one of the following:
> > Supported: Someone is actually paid to look after this.
> > Maintained: Someone actually looks after it.
> >
> > Perhaps this is driver status should be marked
> >
> > "S: Supported"
> >
> > > F: arch/x86/include/asm/intel_telemetry.h
>
> Thanks, Joe.
> I may fix it when applying.

Hmm... Since it's not related to the change done in the patch I would
do rather a separate one. And moreover hear an opinion from Darren. I
don't remember if we have any rule of thumb for the drivers to be only
"supported" and not "maintained".

--
With Best Regards,
Andy Shevchenko

2018-10-19 14:04:24

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH V3] MAINTAINERS: intel_telemetry: Update maintainers info

On Fri, Oct 12, 2018 at 7:29 PM David E. Box <[email protected]> wrote:
>
> Remove Souvik who has left this role. Add Rajneesh and David who work
> jointly on telemetry updates for new platforms.
>

Pushed to my reviewing and testing queue, thanks!

> Signed-off-by: David E. Box <[email protected]>
> Signed-off-by: Rajneesh Bhardwaj <[email protected]>
> ---
>
> V3: Change signoff format
>
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 544cac829cf4..cbe1595db4a5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7355,7 +7355,8 @@ F: arch/x86/crypto/sha*-mb
> F: crypto/mcryptd.c
>
> INTEL TELEMETRY DRIVER
> -M: Souvik Kumar Chakravarty <[email protected]>
> +M: Rajneesh Bhardwaj <[email protected]>
> +M: "David E. Box" <[email protected]>
> L: [email protected]
> S: Maintained
> F: arch/x86/include/asm/intel_telemetry.h
> --
> 2.14.4
>


--
With Best Regards,
Andy Shevchenko