On 08/20/2012 10:33 AM, Roland Stigge wrote: > On 08/20/2012 04:22 PM, Rob Herring wrote: >>> .../devicetree/bindings/spi/spi_pl022.txt | 15 +++ >>> drivers/spi/spi-pl022.c | 114 ++++++++++++++++---- >>> 2 files changed, 110 insertions(+), 19 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/spi/spi_pl022.txt b/Documentation/devicetree/bindings/spi/spi_pl022.txt >>> index 306ec3f..b089ec7 100644 >>> --- a/Documentation/devicetree/bindings/spi/spi_pl022.txt >>> +++ b/Documentation/devicetree/bindings/spi/spi_pl022.txt >>> @@ -6,7 +6,22 @@ Required properties: >>> - interrupts : Should contain SPI controller interrupt >>> >>> Optional properties: >>> +- pl022,num-chipselects : total number of chipselects >> >> We have this for other spi controllers too. Define a generic property. > > Currently, we have: > > num-cs > ti,spi-num-cs > fsl,spi-num-chipselects > fsl,espi-num-chipselects > (pl022,num-chipselects) > > Would "num-cs" be the right one to use? Well, it is either that or invent something new, so I think num-cs is fine. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html