On Fri, Apr 28, 2023 at 02:15:21PM +0200, Valentin Caron wrote:To add some documentation about cs-gpios and don't allow to use
From: Alain Volmat <alain.volmat@xxxxxxxxxxx>Why? What problem are you trying to solve.
Update the spi-stm32 binding yaml regarding to the SPI slave support.
We try to keep consistency with spi master, but, as you said, it is useless.
Signed-off-by: Alain Volmat <alain.volmat@xxxxxxxxxxx>I don't understand. I though cs-gpios was for master mode. You want to
Signed-off-by: Valentin Caron <valentin.caron@xxxxxxxxxxx>
---
Documentation/devicetree/bindings/spi/st,stm32-spi.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml
index c599eb359d56..1d26fa2658c5 100644
--- a/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml
+++ b/Documentation/devicetree/bindings/spi/st,stm32-spi.yaml
@@ -27,6 +27,7 @@ allOf:
then:
properties:
st,spi-midi-ns: false
+ spi-slave: false
properties:
"#address-cells": true
@@ -62,6 +63,13 @@ properties:
- const: rx
- const: tx
+ cs-gpios:
+ description:
+ In case of spi-slave not defined, cs-gpios behave as defined in
+ spi-controller.yaml.
+ In case of spi-slave defined, if <0>, indicate that SS should be
+ detected via the dedicated HW pin
define 'cs-gpios = <0>;'? How would that be different than just omitting
cs-gpios?
Rob