On Fri, 6 Mar 2020 09:50:37 +0100, Geert Uytterhoeven wrote: > Currently, the DT bindings for an SPI controller specify that > "#address-cells" must be fixed to one. However, that applies to an SPI > controller in master mode only. When running in SPI slave mode, > "#address-cells" should not be specified. > > Fix this making "#address-cells" mutually-exclusive with "spi-slave". > > Fixes: 0a1b929356830257 ("spi: Add YAML schemas for the generic SPI options") > Reported-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- > v3: > - Make #address-cells mutually-exclusive with spi-slave, > > v2: > - Use "enum: [0, 1]" instead of min/max limit, > - use "- spi-slave" instead of "[ spi-slave ]". > --- > Documentation/devicetree/bindings/spi/spi-controller.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>