[PATCH 0/3] Update email addresses and add missing MAINTAINERS entry

[PATCH 0/3] Update email addresses and add missing MAINTAINERS entry

[Jean-Baptiste Maneyrol via B4 Relay]

Migrate invensense email addresses to TDK domain. Add missing entry
for iio inv_mpu6050 driver.

Signed-off-by: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
---
Jean-Baptiste Maneyrol (3):
      MAINTAINERS: iio: migrate invensense email address to tdk domain
      dt-bindings: iio: imu: migrate InvenSense email to TDK group domain
      MAINAINERS: iio: imu: add entry for InvenSense MPU-6050 driver

 .../devicetree/bindings/iio/imu/invensense,icm42600.yaml       |  2 +-
 .../devicetree/bindings/iio/imu/invensense,mpu6050.yaml        |  2 +-
 MAINTAINERS                                                    | 10 +++++++++-
 3 files changed, 11 insertions(+), 3 deletions(-)
---
base-commit: 550aaa170cd9176655382364308d2ff54623b30b
change-id: 20240923-invn-maintainers-email-update-ac2b3c74e4f1

Best regards,
-- 
Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>

[PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Jean-Baptiste Maneyrol via B4 Relay]

From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>

InvenSense is part of TDK group. Update email address to use the
TDK domain.
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index bcdf43f37660403f3adb11884f1c58f64bc32840..b6b252b991a514e225df309485c708c2af547f57 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11898,7 +11898,7 @@ F:	Documentation/devicetree/bindings/media/i2c/isil,isl79987.yaml
 F:	drivers/media/i2c/isl7998x.c
 
 INVENSENSE ICM-426xx IMU DRIVER
-M:	Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>
+M:	Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
 L:	linux-iio@vger.kernel.org
 S:	Maintained
 W:	https://invensense.tdk.com/

-- 
2.34.1

[PATCH 2/3] dt-bindings: iio: imu: migrate InvenSense email to TDK group domain

[Jean-Baptiste Maneyrol via B4 Relay]

From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>

Migrate maintainer email to TDK domain.
---
 Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml | 2 +-
 Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
index 3769f8e8e98cee3db61cc0138f485c488d55a63e..7e4492bbd0278a336587dc5ac04da7153453da29 100644
--- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
 title: InvenSense ICM-426xx Inertial Measurement Unit
 
 maintainers:
-  - Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>
+  - Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
 
 description: |
   6-axis MotionTracking device that combines a 3-axis gyroscope and a 3-axis
diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml
index a8d30ef015faaca02a11af7def6e06057bde3909..f91954870a44c60cadc76b5bc3917ee6f23e30a2 100644
--- a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
 title: InvenSense MPU-6050 Six-Axis (Gyro + Accelerometer) MEMS MotionTracking Device
 
 maintainers:
-  - Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>
+  - Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
 
 description: |
   These devices support both I2C and SPI bus interfaces.

-- 
2.34.1

[PATCH 3/3] MAINAINERS: iio: imu: add entry for InvenSense MPU-6050 driver

[Jean-Baptiste Maneyrol via B4 Relay]

From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>

Add entry for inv_mpu6050 iio driver supporting InvenSense MPU-6xxx
and ICM-206xxx devices.
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index b6b252b991a514e225df309485c708c2af547f57..fba699e27612997ed1465ae080632dbeb0639be5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11913,6 +11913,14 @@ S:	Maintained
 F:	Documentation/devicetree/bindings/iio/gyroscope/invensense,mpu3050.yaml
 F:	drivers/iio/gyro/mpu3050*
 
+INVENSENSE MPU-6050 IMU DRIVER
+M:	Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
+L:	linux-iio@vger.kernel.org
+S:	Maintained
+W:	https://invensense.tdk.com/
+F:	Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml
+F:	drivers/iio/imu/inv_mpu6050/
+
 IOC3 ETHERNET DRIVER
 M:	Ralf Baechle <ralf@linux-mips.org>
 L:	linux-mips@vger.kernel.org

-- 
2.34.1

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Krzysztof Kozlowski]

On 03/10/2024 09:37, Jean-Baptiste Maneyrol via B4 Relay wrote:
> From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> 
> InvenSense is part of TDK group. Update email address to use the
> TDK domain.

Please run scripts/checkpatch.pl and fix reported warnings. Then please
run `scripts/checkpatch.pl --strict` and (probably) fix more warnings.
Some warnings can be ignored, especially from --strict run, but the code
here looks like it needs a fix. Feel free to get in touch if the warning
is not clear.

Best regards,
Krzysztof

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Jean-Baptiste Maneyrol]

Hello Krzysztof,

this is strange because I run b4 prep --check and was expecting it to run checkpatch on the patches.

I am having trailing whitespaces errors, but not on the part I changed. It looks like these "spaces" that aren't even displayed correctly in an editor are introduced by git.

Do you have any idea on this kind of issues?

Thanks,
JB

________________________________________
From: Krzysztof Kozlowski <krzk@kernel.org>
Sent: Thursday, October 3, 2024 09:45
To: Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>; Jonathan Cameron <jic23@kernel.org>; Lars-Peter Clausen <lars@metafoo.de>; Rob Herring <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>
Cc: linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; devicetree@vger.kernel.org <devicetree@vger.kernel.org>
Subject: Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain
 
This Message Is From an External Sender
This message came from outside your organization.
 
On 03/10/2024 09:37, Jean-Baptiste Maneyrol via B4 Relay wrote:
> From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> 
> InvenSense is part of TDK group. Update email address to use the
> TDK domain.

Please run scripts/checkpatch.pl and fix reported warnings. Then please
run `scripts/checkpatch.pl --strict` and (probably) fix more warnings.
Some warnings can be ignored, especially from --strict run, but the code
here looks like it needs a fix. Feel free to get in touch if the warning
is not clear.

Best regards,
Krzysztof

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Krzysztof Kozlowski]

On 03/10/2024 12:03, Jean-Baptiste Maneyrol wrote:
> Hello Krzysztof,
> 
> this is strange because I run b4 prep --check and was expecting it to run checkpatch on the patches.
> 
> I am having trailing whitespaces errors, but not on the part I changed. It looks like these "spaces" that aren't even displayed correctly in an editor are introduced by git.
> 
> Do you have any idea on this kind of issues?

Please do not top-post.

Which version of b4? Paste the entire b4 prep command output.

Best regards,
Krzysztof

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Krzysztof Kozlowski]

On 03/10/2024 12:03, Jean-Baptiste Maneyrol wrote:
> Hello Krzysztof,
> 
> this is strange because I run b4 prep --check and was expecting it to run checkpatch on the patches.
> 
> I am having trailing whitespaces errors, but not on the part I changed. It looks like these "spaces" that aren't even displayed correctly in an editor are introduced by git.
> 
> Do you have any idea on this kind of issues?
> 
> Thanks,
> JB

Not much details above, but anyway, I think that's `b4 prep --check` and
I reproduced it on 0.14.2.

Konstantin,

`b4 prep --check` does not operate on patches, thus for example SoB
checks are not working.

Reproduce:
1. Download this patchset, create patches (git format-patch).
2. scripts/checkpatch.pl 0* - will report correctly errors.
3. b4 prep --check will say everything is fine. Expected: same errors
about SoB.

P.S. I would bugspray you, but not sure how does it work :)

Best regards,
Krzysztof

> 
> ________________________________________
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Thursday, October 3, 2024 09:45
> To: Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>; Jonathan Cameron <jic23@kernel.org>; Lars-Peter Clausen <lars@metafoo.de>; Rob Herring <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>
> Cc: linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; devicetree@vger.kernel.org <devicetree@vger.kernel.org>
> Subject: Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain
>  
> This Message Is From an External Sender
> This message came from outside your organization.
>  
> On 03/10/2024 09:37, Jean-Baptiste Maneyrol via B4 Relay wrote:
>> From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
>>
>> InvenSense is part of TDK group. Update email address to use the
>> TDK domain.
> 
> Please run scripts/checkpatch.pl and fix reported warnings. Then please
> run `scripts/checkpatch.pl --strict` and (probably) fix more warnings.
> Some warnings can be ignored, especially from --strict run, but the code
> here looks like it needs a fix. Feel free to get in touch if the warning
> is not clear.
> 
> Best regards,
> Krzysztof
> 

Best regards,
Krzysztof

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Jean-Baptiste Maneyrol]

Hello,

I apologize first for doing a top response, but I'm stuck with an email tool that doesn't allows me to have better response than that. And sadly, I don't think there is something I can do about it.

In fact, when trying checkpatch --strict I don't have any warning on my side. I was using in fact checkpatch -f which is for testing source file and not patch. I tried upgrading git to latest version, I don't see any change. b4 is version 0.14.2.


./scripts/checkpatch.pl --strict test/*.patch
--------------------------------------------------------------------
test/0001-MAINTAINERS-iio-migrate-invensense-email-address-to-.patch
--------------------------------------------------------------------
total: 0 errors, 0 warnings, 0 checks, 8 lines checked

test/0001-MAINTAINERS-iio-migrate-invensense-email-address-to-.patch has no obvious style problems and is ready for submission.
--------------------------------------------------------------------
test/0002-dt-bindings-iio-imu-migrate-InvenSense-email-to-TDK-.patch
--------------------------------------------------------------------
total: 0 errors, 0 warnings, 0 checks, 16 lines checked

test/0002-dt-bindings-iio-imu-migrate-InvenSense-email-to-TDK-.patch has no obvious style problems and is ready for submission.
--------------------------------------------------------------------
test/0003-MAINTAINERS-iio-imu-add-entry-for-InvenSense-MPU-605.patch
--------------------------------------------------------------------
total: 0 errors, 0 warnings, 0 checks, 14 lines checked

test/0003-MAINTAINERS-iio-imu-add-entry-for-InvenSense-MPU-605.patch has no obvious style problems and is ready for submission.


What are the errors you are seeing on your side for these patches?

Thanks,
JB

________________________________________
From: Krzysztof Kozlowski <krzk@kernel.org>
Sent: Thursday, October 3, 2024 09:45
To: Jean-Baptiste Maneyrol <Jean-Baptiste.Maneyrol@tdk.com>; Jonathan Cameron <jic23@kernel.org>; Lars-Peter Clausen <lars@metafoo.de>; Rob Herring <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>
Cc: linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; linux-iio@vger.kernel.org <linux-iio@vger.kernel.org>; devicetree@vger.kernel.org <devicetree@vger.kernel.org>
Subject: Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain
 
This Message Is From an External Sender
This message came from outside your organization.
 
On 03/10/2024 09:37, Jean-Baptiste Maneyrol via B4 Relay wrote:
> From: Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>
> 
> InvenSense is part of TDK group. Update email address to use the
> TDK domain.

Please run scripts/checkpatch.pl and fix reported warnings. Then please
run `scripts/checkpatch.pl --strict` and (probably) fix more warnings.
Some warnings can be ignored, especially from --strict run, but the code
here looks like it needs a fix. Feel free to get in touch if the warning
is not clear.

Best regards,
Krzysztof

Re: [PATCH 1/3] MAINTAINERS: iio: migrate invensense email address to tdk domain

[Krzysztof Kozlowski]

On 03/10/2024 14:49, Jean-Baptiste Maneyrol wrote:
> Hello,
> 
> I apologize first for doing a top response, but I'm stuck with an email tool that doesn't allows me to have better response than that. And sadly, I don't think there is something I can do about it.
> 
> In fact, when trying checkpatch --strict I don't have any warning on my side. I was using in fact checkpatch -f which is for testing source file and not patch. I tried upgrading git to latest version, I don't see any change. b4 is version 0.14.2.
> 
> 
> ./scripts/checkpatch.pl --strict test/*.patch
> --------------------------------------------------------------------
> test/0001-MAINTAINERS-iio-migrate-invensense-email-address-to-.patch
> --------------------------------------------------------------------
> total: 0 errors, 0 warnings, 0 checks, 8 lines checked
> 
> test/0001-MAINTAINERS-iio-migrate-invensense-email-address-to-.patch has no obvious style problems and is ready for submission.
> --------------------------------------------------------------------
> test/0002-dt-bindings-iio-imu-migrate-InvenSense-email-to-TDK-.patch
> --------------------------------------------------------------------
> total: 0 errors, 0 warnings, 0 checks, 16 lines checked
> 
> test/0002-dt-bindings-iio-imu-migrate-InvenSense-email-to-TDK-.patch has no obvious style problems and is ready for submission.
> --------------------------------------------------------------------
> test/0003-MAINTAINERS-iio-imu-add-entry-for-InvenSense-MPU-605.patch
> --------------------------------------------------------------------
> total: 0 errors, 0 warnings, 0 checks, 14 lines checked
> 
> test/0003-MAINTAINERS-iio-imu-add-entry-for-InvenSense-MPU-605.patch has no obvious style problems and is ready for submission.
> 
> 
> What are the errors you are seeing on your side for these patches?

Big red warning:

ERROR: Missing Signed-off-by: line by nominal patch author
'Jean-Baptiste Maneyrol <jean-baptiste.maneyrol@tdk.com>'


Best regards,
Krzysztof