From: Anatolij Gustschin <agust@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 2/6] spi: omap3: add support for more word lengths
Date: Tue, 12 Nov 2013 10:18:11 +0100 [thread overview]
Message-ID: <20131112101811.50cb192f@crub> (raw)
In-Reply-To: <1381933409-8127-3-git-send-email-nikita@compulab.co.il>
On Wed, 16 Oct 2013 17:23:25 +0300
Nikita Kiryanov <nikita@compulab.co.il> wrote:
> Current implementation only supports 8 bit word lengths, even though
> omap3 can handle anything between 4 and 32.
>
> Update the spi interface to support changing the SPI word length,
> and implement it in omap3_spi driver to support the full range of
> possible word lengths.
> This implementation is backwards compatible by defaulting to the old
> behavior of 8 bit word lengths.
> Also, it required a change to the omap3_spi non static I/O functions,
> but since they are not used anywhere else, no collateral changes are required.
>
> Cc: Tom Rini <trini@ti.com>
> Cc: Jagannadha Sutradharudu Teki <jagannadh.teki@gmail.com>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il>
> ---
> Changes in V2:
> - Removed semicolon from #define SPI_DEFAULT_WORDLEN (Gerhard Sittig,
> Igor Grinberg)
> - Moved wordlen to generic spi_slave struct, and added generic
> implementation for spi_set_wordlen which only updates the struct without
> touching the hardware (Igor Grinberg)
> - Update wordlen in hardware just before doing SPI transactions, not
> when changing wordlen (Igor Grinberg)
> - OMAP3 specific check of wordlen value from old implementation of
> spi_set_wordlen moved to xfer. It refines the more general check done
> in the new spi_set_wordlen.
> - Fixed comment style in spi.h following a rebase on top of latest
> U-Boot
>
> drivers/spi/omap3_spi.c | 69 +++++++++++++++++++++++++++++++++++--------------
> drivers/spi/omap3_spi.h | 8 +++---
> drivers/spi/spi.c | 13 ++++++++++
> include/spi.h | 16 ++++++++++++
> 4 files changed, 82 insertions(+), 24 deletions(-)
applied to u-boot-video/master, thanks!
Anatolij
next prev parent reply other threads:[~2013-11-12 9:18 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-16 14:23 [U-Boot] [PATCH V2 0/6] Add support for SPI based DataImage LCD panel Nikita Kiryanov
2013-10-16 14:23 ` [U-Boot] [PATCH V2 1/6] spi: omap3: remove semicolon from #define Nikita Kiryanov
2013-11-12 9:15 ` Anatolij Gustschin
2013-10-16 14:23 ` [U-Boot] [PATCH V2 2/6] spi: omap3: add support for more word lengths Nikita Kiryanov
2013-11-12 9:18 ` Anatolij Gustschin [this message]
2013-10-16 14:23 ` [U-Boot] [PATCH V2 3/6] spi: define SPI_XFER_ONCE Nikita Kiryanov
2013-11-12 9:19 ` Anatolij Gustschin
2013-10-16 14:23 ` [U-Boot] [PATCH V2 4/6] lcd: add DataImage SCF0403x LCD panel support Nikita Kiryanov
2013-10-17 16:22 ` Anatolij Gustschin
2013-11-12 9:20 ` Anatolij Gustschin
2013-10-16 14:23 ` [U-Boot] [PATCH V2 5/6] omap3_dss: define DSS_ONOFF Nikita Kiryanov
2013-10-17 16:23 ` Anatolij Gustschin
2013-11-12 9:22 ` Anatolij Gustschin
2013-10-16 14:23 ` [U-Boot] [PATCH V2 6/6] cm_t35: use scf0403 driver Nikita Kiryanov
2013-11-12 9:23 ` Anatolij Gustschin
2013-11-04 20:49 ` [U-Boot] [PATCH V2 0/6] Add support for SPI based DataImage LCD panel Tom Rini
2013-11-06 9:46 ` Nikita Kiryanov
2013-11-06 10:14 ` Anatolij Gustschin
2013-11-06 12:17 ` Nikita Kiryanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131112101811.50cb192f@crub \
--to=agust@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.