2023-10-25 09:16:44

by Lukas Bulwahn

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: correct file entry in BOSCH SENSORTEC BMI323 IMU IIO DRIVER

Commit b512c767e7bc ("iio: imu: Add driver for BMI323 IMU") adds the
MAINTAINERS section BOSCH SENSORTEC BMI323 IMU IIO DRIVER and refers to a
non-existing device-tree file.

Probably, this mistake was introduced by copying from the BOSCH SENSORTEC
BMA400 ACCELEROMETER IIO DRIVER section and missing to adjust the file
entry properly. This is however easily caught, as the script
./scripts/get_maintainer.pl --self-test=patterns complains about a broken
reference.

The related commit 77583938740e ("dt-bindings: iio: imu: Add Bosch BMI323")
adds bosch,bmi323.yaml, so refer to that intended file instead.

Signed-off-by: Lukas Bulwahn <[email protected]>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a493d96ffd85..82ba3695f614 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3642,7 +3642,7 @@ BOSCH SENSORTEC BMI323 IMU IIO DRIVER
M: Jagath Jog J <[email protected]>
L: [email protected]
S: Maintained
-F: Documentation/devicetree/bindings/iio/imu/bosch,bma400.yaml
+F: Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml
F: drivers/iio/imu/bmi323/

BPF JIT for ARM
--
2.17.1


2023-10-25 19:50:28

by Jagath Jog J

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: correct file entry in BOSCH SENSORTEC BMI323 IMU IIO DRIVER

On Wed, Oct 25, 2023 at 2:45 PM Lukas Bulwahn <[email protected]> wrote:
>
> Commit b512c767e7bc ("iio: imu: Add driver for BMI323 IMU") adds the
> MAINTAINERS section BOSCH SENSORTEC BMI323 IMU IIO DRIVER and refers to a
> non-existing device-tree file.
>
> Probably, this mistake was introduced by copying from the BOSCH SENSORTEC
> BMA400 ACCELEROMETER IIO DRIVER section and missing to adjust the file
> entry properly. This is however easily caught, as the script
> ./scripts/get_maintainer.pl --self-test=patterns complains about a broken
> reference.
>
> The related commit 77583938740e ("dt-bindings: iio: imu: Add Bosch BMI323")
> adds bosch,bmi323.yaml, so refer to that intended file instead.

My apologies for the silly mistake, and thank you for fixing it.
Reviewed-by: Jagath Jog J <[email protected]>


>
> Signed-off-by: Lukas Bulwahn <[email protected]>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a493d96ffd85..82ba3695f614 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3642,7 +3642,7 @@ BOSCH SENSORTEC BMI323 IMU IIO DRIVER
> M: Jagath Jog J <[email protected]>
> L: [email protected]
> S: Maintained
> -F: Documentation/devicetree/bindings/iio/imu/bosch,bma400.yaml
> +F: Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml
> F: drivers/iio/imu/bmi323/
>
> BPF JIT for ARM
> --
> 2.17.1
>

2023-10-27 13:30:23

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: correct file entry in BOSCH SENSORTEC BMI323 IMU IIO DRIVER

On Thu, 26 Oct 2023 01:19:51 +0530
Jagath Jog J <[email protected]> wrote:

> On Wed, Oct 25, 2023 at 2:45 PM Lukas Bulwahn <[email protected]> wrote:
> >
> > Commit b512c767e7bc ("iio: imu: Add driver for BMI323 IMU") adds the
> > MAINTAINERS section BOSCH SENSORTEC BMI323 IMU IIO DRIVER and refers to a
> > non-existing device-tree file.
> >
> > Probably, this mistake was introduced by copying from the BOSCH SENSORTEC
> > BMA400 ACCELEROMETER IIO DRIVER section and missing to adjust the file
> > entry properly. This is however easily caught, as the script
> > ./scripts/get_maintainer.pl --self-test=patterns complains about a broken
> > reference.
> >
> > The related commit 77583938740e ("dt-bindings: iio: imu: Add Bosch BMI323")
> > adds bosch,bmi323.yaml, so refer to that intended file instead.
>
> My apologies for the silly mistake, and thank you for fixing it.
> Reviewed-by: Jagath Jog J <[email protected]>
>
Applied to the togreg branch of iio.git and pushed out as testing.

Thanks,