All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
@ 2020-03-30 21:14 Eddie James
  2020-03-30 21:36 ` Joel Stanley
  0 siblings, 1 reply; 6+ messages in thread
From: Eddie James @ 2020-03-30 21:14 UTC (permalink / raw)
  To: openbmc; +Cc: joel, msbarth, Eddie James

Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
nodes.

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

diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
index ff49ec76fa7c..374512a83f33 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
@@ -89,6 +89,16 @@
 			linux,code = <7>;
 		};
 	};
+
+	iio-hwmon-dps310 {
+		compatible = "iio-hwmon";
+		io-channels = <&dps 0>;
+	};
+
+	iio-hwmon-bmp280 {
+		compatible = "iio-hwmon";
+		io-channels = <&bmp 1>;
+	};
 };
 
 &fmc {
-- 
2.24.0

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

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
  2020-03-30 21:14 [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices Eddie James
@ 2020-03-30 21:36 ` Joel Stanley
  2020-04-01 18:57   ` Eddie James
  0 siblings, 1 reply; 6+ messages in thread
From: Joel Stanley @ 2020-03-30 21:36 UTC (permalink / raw)
  To: Eddie James; +Cc: OpenBMC Maillist, msbarth

On Mon, 30 Mar 2020 at 21:14, Eddie James <eajames@linux.ibm.com> wrote:
>
> Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
> nodes.

The BMP280 was on pre-production witherspoons. I imagine most of the
ones we're using have dps310. Can we drop the bmp280 support?

Cheers,

Joel

>
> Signed-off-by: Eddie James <eajames@linux.ibm.com>
> ---
>  arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> index ff49ec76fa7c..374512a83f33 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
> @@ -89,6 +89,16 @@
>                         linux,code = <7>;
>                 };
>         };
> +
> +       iio-hwmon-dps310 {
> +               compatible = "iio-hwmon";
> +               io-channels = <&dps 0>;
> +       };
> +
> +       iio-hwmon-bmp280 {
> +               compatible = "iio-hwmon";
> +               io-channels = <&bmp 1>;
> +       };
>  };
>
>  &fmc {
> --
> 2.24.0
>

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

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
  2020-03-30 21:36 ` Joel Stanley
@ 2020-04-01 18:57   ` Eddie James
  2020-04-01 19:38     ` Eddie James
  0 siblings, 1 reply; 6+ messages in thread
From: Eddie James @ 2020-04-01 18:57 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC Maillist


On 3/30/20 4:36 PM, Joel Stanley wrote:
> On Mon, 30 Mar 2020 at 21:14, Eddie James <eajames@linux.ibm.com> wrote:
>> Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
>> nodes.
> The BMP280 was on pre-production witherspoons. I imagine most of the
> ones we're using have dps310. Can we drop the bmp280 support?


Yes, sent a new patch for that, so this one can be abandoned.


Thanks,

Eddie


>
> Cheers,
>
> Joel
>
>> Signed-off-by: Eddie James <eajames@linux.ibm.com>
>> ---
>>   arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>> index ff49ec76fa7c..374512a83f33 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>> @@ -89,6 +89,16 @@
>>                          linux,code = <7>;
>>                  };
>>          };
>> +
>> +       iio-hwmon-dps310 {
>> +               compatible = "iio-hwmon";
>> +               io-channels = <&dps 0>;
>> +       };
>> +
>> +       iio-hwmon-bmp280 {
>> +               compatible = "iio-hwmon";
>> +               io-channels = <&bmp 1>;
>> +       };
>>   };
>>
>>   &fmc {
>> --
>> 2.24.0
>>

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

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
  2020-04-01 18:57   ` Eddie James
@ 2020-04-01 19:38     ` Eddie James
  2020-04-01 21:15       ` Joel Stanley
  0 siblings, 1 reply; 6+ messages in thread
From: Eddie James @ 2020-04-01 19:38 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC Maillist


On 4/1/20 1:57 PM, Eddie James wrote:
>
> On 3/30/20 4:36 PM, Joel Stanley wrote:
>> On Mon, 30 Mar 2020 at 21:14, Eddie James <eajames@linux.ibm.com> wrote:
>>> Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
>>> nodes.
>> The BMP280 was on pre-production witherspoons. I imagine most of the
>> ones we're using have dps310. Can we drop the bmp280 support?
>
>
> Yes, sent a new patch for that, so this one can be abandoned.


Nevermind... This one is better, because the device is physically there 
on all the Tacomas I've checked. It's also present in the regular old 
Witherspoon devicetree.


>
>
> Thanks,
>
> Eddie
>
>
>>
>> Cheers,
>>
>> Joel
>>
>>> Signed-off-by: Eddie James <eajames@linux.ibm.com>
>>> ---
>>>   arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts | 10 ++++++++++
>>>   1 file changed, 10 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts 
>>> b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>>> index ff49ec76fa7c..374512a83f33 100644
>>> --- a/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-tacoma.dts
>>> @@ -89,6 +89,16 @@
>>>                          linux,code = <7>;
>>>                  };
>>>          };
>>> +
>>> +       iio-hwmon-dps310 {
>>> +               compatible = "iio-hwmon";
>>> +               io-channels = <&dps 0>;
>>> +       };
>>> +
>>> +       iio-hwmon-bmp280 {
>>> +               compatible = "iio-hwmon";
>>> +               io-channels = <&bmp 1>;
>>> +       };
>>>   };
>>>
>>>   &fmc {
>>> -- 
>>> 2.24.0
>>>

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

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
  2020-04-01 19:38     ` Eddie James
@ 2020-04-01 21:15       ` Joel Stanley
  2020-04-01 21:16         ` Eddie James
  0 siblings, 1 reply; 6+ messages in thread
From: Joel Stanley @ 2020-04-01 21:15 UTC (permalink / raw)
  To: Eddie James; +Cc: OpenBMC Maillist

On Wed, 1 Apr 2020 at 19:39, Eddie James <eajames@linux.ibm.com> wrote:
>
>
> On 4/1/20 1:57 PM, Eddie James wrote:
> >
> > On 3/30/20 4:36 PM, Joel Stanley wrote:
> >> On Mon, 30 Mar 2020 at 21:14, Eddie James <eajames@linux.ibm.com> wrote:
> >>> Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
> >>> nodes.
> >> The BMP280 was on pre-production witherspoons. I imagine most of the
> >> ones we're using have dps310. Can we drop the bmp280 support?
> >
> >
> > Yes, sent a new patch for that, so this one can be abandoned.
>
>
> Nevermind... This one is better, because the device is physically there
> on all the Tacomas I've checked. It's also present in the regular old
> Witherspoon devicetree.

Sorry, I don't follow. The machines have both the DPS and the BMP?

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

* Re: [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices
  2020-04-01 21:15       ` Joel Stanley
@ 2020-04-01 21:16         ` Eddie James
  0 siblings, 0 replies; 6+ messages in thread
From: Eddie James @ 2020-04-01 21:16 UTC (permalink / raw)
  To: Joel Stanley; +Cc: OpenBMC Maillist


On 4/1/20 4:15 PM, Joel Stanley wrote:
> On Wed, 1 Apr 2020 at 19:39, Eddie James <eajames@linux.ibm.com> wrote:
>>
>> On 4/1/20 1:57 PM, Eddie James wrote:
>>> On 3/30/20 4:36 PM, Joel Stanley wrote:
>>>> On Mon, 30 Mar 2020 at 21:14, Eddie James <eajames@linux.ibm.com> wrote:
>>>>> Connect the BMP280 and DPS310 to the hwmon subsystem with iio-hwmon
>>>>> nodes.
>>>> The BMP280 was on pre-production witherspoons. I imagine most of the
>>>> ones we're using have dps310. Can we drop the bmp280 support?
>>>
>>> Yes, sent a new patch for that, so this one can be abandoned.
>>
>> Nevermind... This one is better, because the device is physically there
>> on all the Tacomas I've checked. It's also present in the regular old
>> Witherspoon devicetree.
> Sorry, I don't follow. The machines have both the DPS and the BMP?


Yep.

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

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

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-03-30 21:14 [PATCH linux dev-5.4] ARM: dts: Aspeed: Tacoma: Add iio-hwmon nodes for IIO devices Eddie James
2020-03-30 21:36 ` Joel Stanley
2020-04-01 18:57   ` Eddie James
2020-04-01 19:38     ` Eddie James
2020-04-01 21:15       ` Joel Stanley
2020-04-01 21:16         ` Eddie James

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.