Re: [PATCH] spi: make sure all transfer has bits_per_word set

From: Grant Likely
Date: Thu Dec 06 2012 - 09:15:06 EST


On Mon, 12 Nov 2012 11:03:56 +0100, Jonas Gorski <jonas.gorski@xxxxxxxxx> wrote:
> Hi,
>
> On 9 November 2012 10:06, Laxman Dewangan <ldewangan@xxxxxxxxxx> wrote:
> > When spi client does the spi transfer and does not sets
> > the bits_per_word for each transfer then set it as default
> > of spi device in spi core before calling low level transfer.
>
> I like that (not that it counts ... )!
>
> > Removing the similar code from spi-tegra20-slink driver as
> > it is not required.
>
> Not sure if this should be part of *this* patch.
> Also spi-tegra20-slink isn't the only one fixing up the bits_per_word
> for transfers, so it would be nice if you could remove it from the
> other drivers, too.
>
> In a future patch, maybe even do the same with speed_hz?

That sounds reasonable. Could you craft one up and see how it looks?

Thanks,
g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/