Thread (34 messages) 34 messages, 7 authors, 2013-08-24
STALE4671d

[PATCH V1 3/5] mtd: m25p80: add the quad-read support

From: yuhang wang <hidden>
Date: 2013-08-23 13:20:53

Hi,

2013/8/23 Brian Norris [off-list ref]:
On 08/23/2013 04:46 AM, Brian Norris wrote:
quoted
(Now that I've been pointed to the support merged into the SPI tree...)

Aren't the following new DT properties (for the SPI slave) sufficient?

spi-rx-nbits
spi-tx-nbits
...
quoted
They're already in the following commit:

commit f477b7fb13df2b843997559ff34e87d054ba6538
Author: wangyuhang [off-list ref]
Date:   Sun Aug 11 18:15:17 2013 +0800

     spi: DUAL and QUAD support

     fix the previous patch some mistake below:
     1. DT in slave node, use "spi-tx-nbits = <1/2/4>" in place of using
        "spi-tx-dual, spi-tx-quad" directly, same to rx. So correct the
        previous way to get the property in @of_register_spi_devices().
     2. Change the value of transfer bit macro(SPI_NBITS_SINGLE,
SPI_NBITS_DUAL
        SPI_NBITS_QUAD) to 0x01, 0x02 and 0x04 to match the actual wires.
     3. Add the following check
        (1)keep the tx_nbits and rx_nbits in spi_transfer is not beyond
the
           single, dual and quad.
        (2)keep tx_nbits and rx_nbits are contained by @spi_device->mode
           example: if @spi_device->mode = DUAL, then tx/rx_nbits can not
be set
                    to QUAD(SPI_NBITS_QUAD)
        (3)if "@spi_device->mode & SPI_3WIRE", then tx/rx_nbits should be
in
           single(SPI_NBITS_SINGLE)

     Signed-off-by: wangyuhang [off-list ref]
     Signed-off-by: Mark Brown [off-list ref]

Speaking of which, the new device-tree properties are not documented in
Documentation/devicetree/bindings/spi/spi-bus.txt yet.

Brian
Sorry, because I don't have the entire environment in my own PC, so I
will update the document patch as soon as possible when I go back to
company.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help