lkml.org 
[lkml]   [2010]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2] ad7877: fix spi word size to 16 bit
    Date
    With no word size given in the users platform data,
    a generic spi host controller driver will assume a default
    word size of eight bit. This causes transmission to be
    performed bytewise, which will fail on little endian
    machines for sure. Failure on big endian depends on usage
    of slave select to mark word boundaries.

    Anyway, ad7877 is specified to work with 16 bit per word,
    so unconditionally set the word size accordingly.
    Flag an error where 16 bit per word is not available.

    Signed-off-by: Oskar Schirmer <os@emlix.com>
    Signed-off-by: Daniel Glöckner <dg@emlix.com>
    Signed-off-by: Oliver Schneidewind <osw@emlix.com>
    ---
    drivers/input/touchscreen/ad7877.c | 7 +++++++
    1 files changed, 7 insertions(+), 0 deletions(-)

    v2: check return value for spi_setup failure

    diff --git a/drivers/input/touchscreen/ad7877.c b/drivers/input/touchscreen/ad7877.c
    index 698d6e7..eebde5b 100644
    --- a/drivers/input/touchscreen/ad7877.c
    +++ b/drivers/input/touchscreen/ad7877.c
    @@ -680,6 +680,13 @@ static int __devinit ad7877_probe(struct spi_device *spi)
    return -EINVAL;
    }

    + spi->bits_per_word = 16;
    + err = spi_setup(spi);
    + if (err) {
    + dev_dbg(&spi->dev, "spi master doesn't support 16 bits/word\n");
    + return err;
    + }
    +
    ts = kzalloc(sizeof(struct ad7877), GFP_KERNEL);
    input_dev = input_allocate_device();
    if (!ts || !input_dev) {
    --
    1.5.3.7
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2010-05-18 10:37    [W:4.117 / U:0.048 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site