Re: [PATCH v2] dt-bindings: trivial-devices: Document spi-cpha and spi-cpol

[Krzysztof Kozlowski <krzk@xxxxxxxxxx>]

On Fri, Aug 30, 2024 at 02:18:49PM -0300, Fabio Estevam wrote:
> There may be cases where a trivial-device needs to describe
> the SPI clock polarity and phase via spi-cpol and spi-cpha
> properties.
> 
> Document these properties to fix the following dt-schema warnings:
> 
> rv1108-elgin-r1.dtb: display@0: 'spi-cpha', 'spi-cpol' do not match any of the regexes: 'pinctrl-[0-9]+'
> 
> Signed-off-by: Fabio Estevam <festevam@xxxxxxxxx>
> ---
> Changes since v1:
> - Add spi-cpha and spi-cpol to trivial-devices.yaml. (Conor)
> 

No, this does not look correct. Why suddenly all devices get CPHA/CPOL?
This is supposed to be only for devices REALLY needing it (as discussed
with patch moving it out of spi-peripheral-props, did anything change
here?).

>  Documentation/devicetree/bindings/trivial-devices.yaml | 4 ++++
>  1 file changed, 4 insertions(+)

Best regards,
Krzysztof