linux-kernel - Re: [PATCH v7 RESEND 3/7] dt-bindings: iio: Add Infineon DPS310 sensor documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <8ab87732-71e1-4101-9aed-14f68c27fea1@linux.ibm.com>
Date: Mon, 6 Oct 2025 08:44:55 -0500
From: Eddie James <eajames@...ux.ibm.com>
To: Jonathan Cameron <jonathan.cameron@...wei.com>
Cc: linux-hwmon@...r.kernel.org, linux-iio@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-aspeed@...ts.ozlabs.org, robh@...nel.org, krzk+dt@...nel.org,
        conor+dt@...nel.org, andrew@...econstruct.com.au, joel@....id.au,
        linux@...ck-us.net, chanh@...amperecomputing.com, jic23@...nel.org,
        dlechner@...libre.com, nuno.sa@...log.com, andy@...nel.org
Subject: Re: [PATCH v7 RESEND 3/7] dt-bindings: iio: Add Infineon DPS310
 sensor documentation


On 10/3/25 9:01 AM, Jonathan Cameron wrote:
> On Wed,  1 Oct 2025 09:44:37 -0500
> Eddie James <eajames@...ux.ibm.com> wrote:
>
>> The DPS310 is a barometric pressure and temperature sensor with
>> an I2C interface. Remove it from trivial-devices.yaml and add its
>> own documentation.
> Hi Eddie,
>
> Why?  I guess you need the #io-channel-cells which trivial devices
> doesn't allow because you have a consumer driver?


Correct.


>
> Obviously the binding patch shouldn't mention that, but it could call
> out that there can be such consumers.


OK, I can add that.


>
> I'd also expect to see some supplies even if the driver doesn't yet
> explicitly handle them.


You mean in the example section? Sure. I'll send a patch separately 
since the rest is merged


Thanks,

Eddie


>
> Jonathan
>
>> Signed-off-by: Eddie James <eajames@...ux.ibm.com>
>> Reviewed-by: Rob Herring (Arm) <robh@...nel.org>
>> ---
>>   .../iio/pressure/infineon,dps310.yaml         | 44 +++++++++++++++++++
>>   .../devicetree/bindings/trivial-devices.yaml  |  2 -
>>   MAINTAINERS                                   |  1 +
>>   3 files changed, 45 insertions(+), 2 deletions(-)
>>   create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml b/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
>> new file mode 100644
>> index 0000000000000..7c0782e2a821b
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
>> @@ -0,0 +1,44 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/iio/pressure/infineon,dps310.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Infineon DPS310 barometric pressure and temperature sensor
>> +
>> +maintainers:
>> +  - Eddie James <eajames@...ux.ibm.com>
>> +
>> +description:
>> +  The DPS310 is a barometric pressure and temperature sensor with an I2C
>> +  interface.
>> +
>> +properties:
>> +  compatible:
>> +    enum:
>> +      - infineon,dps310
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  "#io-channel-cells":
>> +    const: 0
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +
>> +additionalProperties: false
>> +
>> +examples:
>> +  - |
>> +    i2c {
>> +        #address-cells = <1>;
>> +        #size-cells = <0>;
>> +
>> +        pressure-sensor@76 {
>> +          compatible = "infineon,dps310";
>> +          reg = <0x76>;
>> +          #io-channel-cells = <0>;
>> +        };
>> +    };
>> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
>> index 7609acaa752d5..a72b7fabc7034 100644
>> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
>> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
>> @@ -127,8 +127,6 @@ properties:
>>             - ibm,cffps2
>>               # IBM On-Chip Controller hwmon device
>>             - ibm,p8-occ-hwmon
>> -            # Infineon barometric pressure and temperature sensor
>> -          - infineon,dps310
>>               # Infineon IR36021 digital POL buck controller
>>             - infineon,ir36021
>>               # Infineon IRPS5401 Voltage Regulator (PMIC)
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 0c8281ea4cc64..92b9854a0e07d 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -12191,6 +12191,7 @@ INFINEON DPS310 Driver
>>   M:	Eddie James <eajames@...ux.ibm.com>
>>   L:	linux-iio@...r.kernel.org
>>   S:	Maintained
>> +F:	Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
>>   F:	drivers/iio/pressure/dps310.c
>>   
>>   INFINEON PEB2466 ASoC CODEC
>