public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports
@ 2013-06-25 11:45 Barry Song
  2013-06-25 15:40 ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Barry Song @ 2013-06-25 11:45 UTC (permalink / raw)
  To: linux-arm-kernel

From: Qipan Li <Qipan.Li@csr.com>

Missing this will cause some user cases fail when they want to change spi
transfer mode.

Signed-off-by: Qipan Li <Qipan.Li@csr.com>
Signed-off-by: Barry Song <Baohua.Song@csr.com>
---
 rebased to for-next branch of broonie/spi.git

 drivers/spi/spi-sirf.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/spi/spi-sirf.c b/drivers/spi/spi-sirf.c
index fc20bcf..9608716 100644
--- a/drivers/spi/spi-sirf.c
+++ b/drivers/spi/spi-sirf.c
@@ -538,6 +538,7 @@ static int spi_sirfsoc_probe(struct platform_device *pdev)
 	sspi->bitbang.txrx_bufs = spi_sirfsoc_transfer;
 	sspi->bitbang.master->setup = spi_sirfsoc_setup;
 	master->bus_num = pdev->id;
+	master->mode_bits = SPI_CPOL | SPI_CPHA | SPI_LSB_FIRST | SPI_CS_HIGH;
 	master->bits_per_word_mask = SPI_BPW_MASK(8) | SPI_BPW_MASK(12) |
 					SPI_BPW_MASK(16) | SPI_BPW_MASK(32);
 	sspi->bitbang.master->dev.of_node = pdev->dev.of_node;
-- 
1.8.2.3



Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom
More information can be found at www.csr.com. Follow CSR on Twitter at http://twitter.com/CSR_PLC and read our blog at www.csr.com/blog

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports
  2013-06-25 11:45 [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports Barry Song
@ 2013-06-25 15:40 ` Mark Brown
  2013-06-25 16:17   ` Barry Song
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Brown @ 2013-06-25 15:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Jun 25, 2013 at 07:45:29PM +0800, Barry Song wrote:
> From: Qipan Li <Qipan.Li@csr.com>
> 
> Missing this will cause some user cases fail when they want to change spi
> transfer mode.

This doesn't seem to apply against the current SPI code or mainline -
can you please check and resend?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130625/a3e10f12/attachment.sig>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports
  2013-06-25 15:40 ` Mark Brown
@ 2013-06-25 16:17   ` Barry Song
  2013-06-26  8:08     ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Barry Song @ 2013-06-25 16:17 UTC (permalink / raw)
  To: linux-arm-kernel

2013/6/25 Mark Brown <broonie@kernel.org>:
> On Tue, Jun 25, 2013 at 07:45:29PM +0800, Barry Song wrote:
>> From: Qipan Li <Qipan.Li@csr.com>
>>
>> Missing this will cause some user cases fail when they want to change spi
>> transfer mode.
>
> This doesn't seem to apply against the current SPI code or mainline -
> can you please check and resend?

this was againest your for-next branch of spi as i guessed it will be
in the coming version. you want one againest mainline?

-barry

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports
  2013-06-25 16:17   ` Barry Song
@ 2013-06-26  8:08     ` Mark Brown
  0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2013-06-26  8:08 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jun 26, 2013 at 12:17:24AM +0800, Barry Song wrote:

> this was againest your for-next branch of spi as i guessed it will be
> in the coming version. you want one againest mainline?

I'm looking for things against topic/sirf.  Just checked again and it's
due to a dependency on topic/bpw.  I've merged that in and applied,
thanks.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130626/4e17af80/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-06-26  8:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-25 11:45 [PATCH] spi: sirf: add missed spi mode_bits that SiRFSoC hardware supports Barry Song
2013-06-25 15:40 ` Mark Brown
2013-06-25 16:17   ` Barry Song
2013-06-26  8:08     ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox