Re: [PATCH 1/2] dt-bindings: iio: pressure: honeywell,mprls0025pa

Petre Rodan
2023-12-20T17:25:32Z

[-- Attachment #1: Type: text/plain, Size: 933 bytes --]

hi Jonathan,

On Wed, Dec 20, 2023 at 03:16:45PM +0000, Jonathan Cameron wrote:
> On Tue, 19 Dec 2023 15:02:20 +0200
> Petre Rodan <petre.rodan@subdimension.ro> wrote:
> > honeywell,pmin-pascal:
> > description:
> > Minimum pressure value the sensor can measure in pascal.
> > + To be specified only if honeywell,pressure-triplet is set to "NA".
> That just added a backwards compatibility break. It would be fine
> if there was a default: NA for honeywell,pressure-triplet or a check that either
> one or the other was supplied (which I'd prefer). Thus old bindings will work
> and new ones also supported.

ok, I see your reasoning. but in this second scenario that you prefer how can we
propery define the 'required:' block? an equivalent to

required:
- compatible
- reg
- (honeywell,pmin-pascal && honeywell,pmax-pascal) || honeywell,pressure-triplet
- honeywell,transfer-function

thanks,
peter

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]