Messages in this thread |  | | Date | Sat, 8 Feb 2025 15:07:08 +0800 | | Subject | Re: [PATCH 2/3] dt-bindings: iio: chemical: sensirion,senxx: Add yaml description | | From | Hermes Zhang <> |
| |
Hi,
On 2025/2/7 2:20, Conor Dooley wrote: > diff --git a/Documentation/devicetree/bindings/iio/chemical/sensirion,senxx.yaml b/Documentation/devicetree/bindings/iio/chemical/sensirion,senxx.yaml > new file mode 100644 > index 000000000000..4d998eabe441 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/chemical/sensirion,senxx.yaml > filename matching a compatible please. So sensirion,sen66.yaml? > + https://sensirion.com/media/documents/6791EFA0/62A1F68F/Sensirion_Datasheet_Environmental_Node_SEN5x.pdf > + https://sensirion.com/media/documents/FAFC548D/6731FFFA/Sensirion_Datasheet_SEN6x.pdf > + > +properties: > + compatible: > + enum: > + - sensirion,sen50 > + - sensirion,sen54 > + - sensirion,sen55 > + - sensirion,sen60 > + - sensirion,sen65 > + - sensirion,sen66 > I'd like a note in the commit message as to how all of these devices are > different please. Sure, will fix in v2. >> + >> + reg: >> + maxItems: 1 >> + >> +required: >> + - compatible >> + - reg > No supplies needed for this device? Seems like you would need at least > one, no?
You mean the vdd-supply? The chip require a e.g. 3.3v VDD, but in our HW, we have no gpio/regulator to control it, connect directly by the HW, should I still need to have one vdd-supply here?
Best Regards,
Hermes
|  |