All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Remove bmp280 and add iio-hwmon for DPS310
@ 2020-04-01 19:06 Eddie James
  2020-04-01 19:39 ` Eddie James
  0 siblings, 1 reply; 3+ messages in thread
From: Eddie James @ 2020-04-01 19:06 UTC (permalink / raw)
  To: openbmc

The BMP280 was replaced by the DPS310 on Witherspoon, which needs an
iio-hwmon node to connect to the hwmon subsystem.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
---
 arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
index 044011bb8a04..846a8a2e1a78 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
@@ -88,6 +88,11 @@
 			linux,code = <7>;
 		};
 	};
+
+	iio-hwmon-dps310 {
+		compatible = "iio-hwmon";
+		io-channels = <&dps 0>;
+	};
 };
 
 &gpio0 {
@@ -442,12 +447,6 @@
 &i2c3 {
 	status = "okay";
 
-	bmp: bmp280@77 {
-		compatible = "bosch,bmp280";
-		reg = <0x77>;
-		#io-channel-cells = <1>;
-	};
-
 	max31785@52 {
 		compatible = "maxim,max31785a";
 		reg = <0x52>;
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Remove bmp280 and add iio-hwmon for DPS310
  2020-04-01 19:06 [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Remove bmp280 and add iio-hwmon for DPS310 Eddie James
@ 2020-04-01 19:39 ` Eddie James
  2020-04-01 21:58   ` Joel Stanley
  0 siblings, 1 reply; 3+ messages in thread
From: Eddie James @ 2020-04-01 19:39 UTC (permalink / raw)
  To: openbmc


On 4/1/20 2:06 PM, Eddie James wrote:
> The BMP280 was replaced by the DPS310 on Witherspoon, which needs an
> iio-hwmon node to connect to the hwmon subsystem.
>
> Signed-off-by: Eddie James <eajames@linux.ibm.com>
> ---
>   arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 11 +++++------
>   1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> index 044011bb8a04..846a8a2e1a78 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> @@ -88,6 +88,11 @@
>   			linux,code = <7>;
>   		};
>   	};
> +
> +	iio-hwmon-dps310 {
> +		compatible = "iio-hwmon";
> +		io-channels = <&dps 0>;
> +	};
>   };
>   
>   &gpio0 {
> @@ -442,12 +447,6 @@
>   &i2c3 {
>   	status = "okay";
>   
> -	bmp: bmp280@77 {
> -		compatible = "bosch,bmp280";
> -		reg = <0x77>;
> -		#io-channel-cells = <1>;
> -	};


Nope, oops, the device is physically there so we should keep it in the dts.


> -
>   	max31785@52 {
>   		compatible = "maxim,max31785a";
>   		reg = <0x52>;

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Remove bmp280 and add iio-hwmon for DPS310
  2020-04-01 19:39 ` Eddie James
@ 2020-04-01 21:58   ` Joel Stanley
  0 siblings, 0 replies; 3+ messages in thread
From: Joel Stanley @ 2020-04-01 21:58 UTC (permalink / raw)
  To: Eddie James; +Cc: OpenBMC Maillist

On Wed, 1 Apr 2020 at 19:40, Eddie James <eajames@linux.ibm.com> wrote:
>
>
> On 4/1/20 2:06 PM, Eddie James wrote:
> > The BMP280 was replaced by the DPS310 on Witherspoon, which needs an
> > iio-hwmon node to connect to the hwmon subsystem.
> >
> > Signed-off-by: Eddie James <eajames@linux.ibm.com>
> > ---
> >   arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 11 +++++------
> >   1 file changed, 5 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> > index 044011bb8a04..846a8a2e1a78 100644
> > --- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> > +++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> > @@ -88,6 +88,11 @@
> >                       linux,code = <7>;
> >               };
> >       };
> > +
> > +     iio-hwmon-dps310 {
> > +             compatible = "iio-hwmon";
> > +             io-channels = <&dps 0>;
> > +     };
> >   };
> >
> >   &gpio0 {
> > @@ -442,12 +447,6 @@
> >   &i2c3 {
> >       status = "okay";
> >
> > -     bmp: bmp280@77 {
> > -             compatible = "bosch,bmp280";
> > -             reg = <0x77>;
> > -             #io-channel-cells = <1>;
> > -     };
>
>
> Nope, oops, the device is physically there so we should keep it in the dts.

Ok, I've dropped this one and applied the other.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-04-01 21:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-01 19:06 [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Remove bmp280 and add iio-hwmon for DPS310 Eddie James
2020-04-01 19:39 ` Eddie James
2020-04-01 21:58   ` Joel Stanley

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.