Re: [PATCH 1/3] spi: add xtfpga SPI controller driver

From: Mark Brown
Date: Wed Mar 12 2014 - 09:25:00 EST


On Wed, Mar 12, 2014 at 05:48:16AM +0400, Max Filippov wrote:
> On Wed, Mar 12, 2014 at 5:11 AM, Mark Brown <broonie@xxxxxxxxxx> wrote:

> > So I just looked again - the SPI code isn't in mainline, there must
> > be some out of tree patches here that can't have been tested since the
> > driver was converted to regmap (or the byte swapping you're doing in the
> > controller is buggy for 16 bits per word).

> I'm successfully running this driver with the patches in your ASoC tree
> branch topic/tlv320aic23.

Ah, I missed that during review - the driver shouldn't be forcing 16
bits per word, any swapping that's needed should be being done in the
regmap API. Though based on the discussion of the SPI driver it seems
like this may be compensating for an unwanted byte swap there.

Attachment: signature.asc
Description: Digital signature