2022-10-19 14:09:26

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH 3/3] doc: iio: pressure: ms5611: added max SPI frequency setting to the example

On Wed, Oct 19, 2022 at 3:55 PM Mitja Spes <[email protected]> wrote:
>
> Added max SPI frequency setting to the example. It is now honored by the
> driver.

Is it possible to add a constraint here? So the schema validator will
issue the warning / error if the speed is too high.

--
With Best Regards,
Andy Shevchenko


2022-10-20 06:05:44

by Mitja Špes

[permalink] [raw]
Subject: Re: [PATCH 3/3] doc: iio: pressure: ms5611: added max SPI frequency setting to the example

On Wed, Oct 19, 2022 at 3:23 PM Andy Shevchenko
<[email protected]> wrote:
>
> On Wed, Oct 19, 2022 at 3:55 PM Mitja Spes <[email protected]> wrote:
> >
> > Added max SPI frequency setting to the example. It is now honored by the
> > driver.
>
> Is it possible to add a constraint here? So the schema validator will
> issue the warning / error if the speed is too high.

The constraint is already there:
https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/pressure/meas%2Cms5611.yaml
```
spi-max-frequency:
maximum: 20000000
```

But in any case the patch 2/3 limits the max frequency.

Kind regards,
Mitja Spes