From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH 4/4] devicetree: Add Renesas SH Mobile MSIOF spi controller binding doc Date: Thu, 06 Dec 2012 14:08:49 +0000 Message-ID: <20121206140849.280243E089D@localhost> References: <1352288407-20594-1-git-send-email-hechtb@gmail.com> <1352288407-20594-5-git-send-email-hechtb@gmail.com> Cc: Magnus Damm , linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org To: Bastian Hecht , spi-devel-general@lists.sourceforge.net Return-path: In-Reply-To: <1352288407-20594-5-git-send-email-hechtb@gmail.com> Sender: linux-sh-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org On Wed, 7 Nov 2012 12:40:07 +0100, Bastian Hecht wrote: > Add binding documentation for Renesas' MSIOF SPI controller. > > Signed-off-by: Bastian Hecht > --- > Documentation/devicetree/bindings/spi/sh-msiof.txt | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > create mode 100644 Documentation/devicetree/bindings/spi/sh-msiof.txt > > diff --git a/Documentation/devicetree/bindings/spi/sh-msiof.txt b/Documentation/devicetree/bindings/spi/sh-msiof.txt > new file mode 100644 > index 0000000..b62312e > --- /dev/null > +++ b/Documentation/devicetree/bindings/spi/sh-msiof.txt > @@ -0,0 +1,12 @@ > +Renesas MSIOF spi controller > + > +Required properties: > +- compatible : "renesas,sh-msiof" for SuperH or > + "renesas,sh-mobile-msiof" for SH Mobile series > +- reg : Offset and length of the register set for the device > +- interrupts : interrupt line used by MSIOF > + > +Optional properties: > +- chip_select : Chip select, defaults to 0 This doesn't make a lot of sense to me. What is this property for? Is there more than one CS? And if there is, shouldn't the SPI driver be able to manipulate more than one? > +- tx_fifo_size : Overrides the default tx fifo size given in words > +- rx_fifo_size : Overrides the default rx fifo size given in words Nit: property names should use '-' instead of '_' (by convention) and custom properties should be prefixed with the manufacturer prefix to avoid namespace collisions. ie. "renesas,tx-fifo-size. g.