2019-10-21 20:54:58

by Bobby Jones

[permalink] [raw]
Subject: [PATCH] ARM: dts: imx: ventana: add fxos8700 on gateworks boards

Add fxos8700 iio imu entries for Gateworks ventana SBCs.

Signed-off-by: Robert Jones <[email protected]>
---
arch/arm/boot/dts/imx6qdl-gw52xx.dtsi | 5 +++++
arch/arm/boot/dts/imx6qdl-gw53xx.dtsi | 5 +++++
arch/arm/boot/dts/imx6qdl-gw54xx.dtsi | 5 +++++
3 files changed, 15 insertions(+)

diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
index 1a9a9d9..2d7d01e 100644
--- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
@@ -313,6 +313,11 @@
interrupts = <12 2>;
wakeup-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>;
};
+
+ fxos8700@1e {
+ compatible = "nxp,fxos8700";
+ reg = <0x1e>;
+ };
};

&ldb {
diff --git a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
index 54b2bea..bf1a2c6 100644
--- a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
@@ -304,6 +304,11 @@
interrupts = <11 2>;
wakeup-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
};
+
+ fxos8700@1e {
+ compatible = "nxp,fxos8700";
+ reg = <0x1e>;
+ };
};

&ldb {
diff --git a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
index 1b6c133..d9e09a9 100644
--- a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
@@ -361,6 +361,11 @@
interrupts = <12 2>;
wakeup-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>;
};
+
+ fxos8700@1e {
+ compatible = "nxp,fxos8700";
+ reg = <0x1e>;
+ };
};

&ldb {
--
2.9.2


2019-11-07 19:11:59

by Bobby Jones

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: imx: ventana: add fxos8700 on gateworks boards

Hello Shawn,

I just wanted to follow up with this and see if you had a chance to
look at this. I submitted this after responding to Marco on my initial
submission but haven't heard anything since and didn't want it to fall
through the cracks. It may be worth mentioning that both the bindings
for the fxos8700 and lsm9ds1 have been accepted by iio.

In addition to this submission I have the following that I'd like to
check in on as well:

[PATCH] ARM: dts: imx: imx6qdl-gw553x.dtsi: add lsm9ds1 iio imu/magn support
[PATCH] ARM: dts: imx: Add GW5907
[PATCH] ARM: dts: imx: Add GW5912
[PATCH] ARM: dts: imx: Add GW5913
[PATCH] ARM: dts: imx: Add GW5910

Please let me know if there's anything I can do. Thanks!

Regards,
Bobby




On Mon, Oct 21, 2019 at 1:54 PM Robert Jones <[email protected]> wrote:
>
> Add fxos8700 iio imu entries for Gateworks ventana SBCs.
>
> Signed-off-by: Robert Jones <[email protected]>
> ---
> arch/arm/boot/dts/imx6qdl-gw52xx.dtsi | 5 +++++
> arch/arm/boot/dts/imx6qdl-gw53xx.dtsi | 5 +++++
> arch/arm/boot/dts/imx6qdl-gw54xx.dtsi | 5 +++++
> 3 files changed, 15 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> index 1a9a9d9..2d7d01e 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw52xx.dtsi
> @@ -313,6 +313,11 @@
> interrupts = <12 2>;
> wakeup-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>;
> };
> +
> + fxos8700@1e {
> + compatible = "nxp,fxos8700";
> + reg = <0x1e>;
> + };
> };
>
> &ldb {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> index 54b2bea..bf1a2c6 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw53xx.dtsi
> @@ -304,6 +304,11 @@
> interrupts = <11 2>;
> wakeup-gpios = <&gpio1 11 GPIO_ACTIVE_LOW>;
> };
> +
> + fxos8700@1e {
> + compatible = "nxp,fxos8700";
> + reg = <0x1e>;
> + };
> };
>
> &ldb {
> diff --git a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> index 1b6c133..d9e09a9 100644
> --- a/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-gw54xx.dtsi
> @@ -361,6 +361,11 @@
> interrupts = <12 2>;
> wakeup-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>;
> };
> +
> + fxos8700@1e {
> + compatible = "nxp,fxos8700";
> + reg = <0x1e>;
> + };
> };
>
> &ldb {
> --
> 2.9.2
>

2019-12-04 11:35:12

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: imx: ventana: add fxos8700 on gateworks boards

On Thu, Nov 07, 2019 at 11:10:19AM -0800, Bobby Jones wrote:
> Hello Shawn,
>
> I just wanted to follow up with this and see if you had a chance to
> look at this. I submitted this after responding to Marco on my initial
> submission but haven't heard anything since and didn't want it to fall
> through the cracks. It may be worth mentioning that both the bindings
> for the fxos8700 and lsm9ds1 have been accepted by iio.
>
> In addition to this submission I have the following that I'd like to
> check in on as well:
>
> [PATCH] ARM: dts: imx: imx6qdl-gw553x.dtsi: add lsm9ds1 iio imu/magn support
> [PATCH] ARM: dts: imx: Add GW5907
> [PATCH] ARM: dts: imx: Add GW5912
> [PATCH] ARM: dts: imx: Add GW5913
> [PATCH] ARM: dts: imx: Add GW5910
>
> Please let me know if there's anything I can do. Thanks!

Hmm, did you copy me on those patches? I cannot find them in my
mailbox.

Shawn

2019-12-05 20:47:16

by Bobby Jones

[permalink] [raw]
Subject: Re: [PATCH] ARM: dts: imx: ventana: add fxos8700 on gateworks boards

On Wed, Dec 4, 2019 at 3:33 AM Shawn Guo <[email protected]> wrote:
>
> On Thu, Nov 07, 2019 at 11:10:19AM -0800, Bobby Jones wrote:
> > Hello Shawn,
> >
> > I just wanted to follow up with this and see if you had a chance to
> > look at this. I submitted this after responding to Marco on my initial
> > submission but haven't heard anything since and didn't want it to fall
> > through the cracks. It may be worth mentioning that both the bindings
> > for the fxos8700 and lsm9ds1 have been accepted by iio.
> >
> > In addition to this submission I have the following that I'd like to
> > check in on as well:
> >
> > [PATCH] ARM: dts: imx: imx6qdl-gw553x.dtsi: add lsm9ds1 iio imu/magn support
> > [PATCH] ARM: dts: imx: Add GW5907
> > [PATCH] ARM: dts: imx: Add GW5912
> > [PATCH] ARM: dts: imx: Add GW5913
> > [PATCH] ARM: dts: imx: Add GW5910
> >
> > Please let me know if there's anything I can do. Thanks!
>
> Hmm, did you copy me on those patches? I cannot find them in my
> mailbox.

I just double checked and they all have you in the recipient list.
However I just had my coworker check who's also subscribed to
linux-arm-kernel and he's only seeing this thread and the "add lsm9ds1
iio imu/magn support" thread. Maybe something funny with our mail
server.

Interestingly enough they appear on the marc.info archive for Oct 3rd:
https://marc.info/?a=147430901100002&r=1&w=2

I'm not sure what happened but I'll send them out again now.

Thanks,
Bobby