Message ID | 20161027081605.26475-1-judge.packham@gmail.com |
---|---|
State | Accepted |
Commit | ebfa18cb3d8b5e6f57ac22e39b21c4fc1c1d3597 |
Delegated to: | Jagannadha Sutradharudu Teki |
Headers | show |
On Thu, Oct 27, 2016 at 1:46 PM, Chris Packham <judge.packham@gmail.com> wrote: > Set the appropriate bits in the interface config register based > on the SPI_ mode flags. > > Signed-off-by: Chris Packham <judge.packham@gmail.com> Missed updating Stefan Reviewed-by tag? please check for next time. > --- > > Changes in v2: > - Clear bits before updating Applied, thanks!
On 27/10/2016 10:41 PM, "Jagan Teki" <jagan@openedev.com> wrote: > > On Thu, Oct 27, 2016 at 1:46 PM, Chris Packham <judge.packham@gmail.com> wrote: > > Set the appropriate bits in the interface config register based > > on the SPI_ mode flags. > > > > Signed-off-by: Chris Packham <judge.packham@gmail.com> > > Missed updating Stefan Reviewed-by tag? please check for next time. That was semi-intentional. I thought v2 was different enough that I didn't want to assume Stefan's review would carry through. > > > --- > > > > Changes in v2: > > - Clear bits before updating > > Applied, thanks! > > -- > Jagan Teki > Free Software Engineer | www.openedev.com > U-Boot, Linux | Upstream Maintainer > Hyderabad, India.
On 27.10.2016 11:49, Chris Packham wrote: > On 27/10/2016 10:41 PM, "Jagan Teki" <jagan@openedev.com > <mailto:jagan@openedev.com>> wrote: >> >> On Thu, Oct 27, 2016 at 1:46 PM, Chris Packham > <judge.packham@gmail.com <mailto:judge.packham@gmail.com>> wrote: >> > Set the appropriate bits in the interface config register based >> > on the SPI_ mode flags. >> > >> > Signed-off-by: Chris Packham <judge.packham@gmail.com > <mailto:judge.packham@gmail.com>> >> >> Missed updating Stefan Reviewed-by tag? please check for next time. > > That was semi-intentional. I thought v2 was different enough that I > didn't want to assume Stefan's review would carry through. Fair enough. I've overlooked the clearing of the bits spotted by Jagan. Please add my Reviewed-by now as well. Thanks, Stefan
On Thu, Oct 27, 2016 at 3:21 PM, Stefan Roese <sr@denx.de> wrote: > On 27.10.2016 11:49, Chris Packham wrote: >> >> On 27/10/2016 10:41 PM, "Jagan Teki" <jagan@openedev.com >> <mailto:jagan@openedev.com>> wrote: >>> >>> >>> On Thu, Oct 27, 2016 at 1:46 PM, Chris Packham >> >> <judge.packham@gmail.com <mailto:judge.packham@gmail.com>> wrote: >>> >>> > Set the appropriate bits in the interface config register based >>> > on the SPI_ mode flags. >>> > >>> > Signed-off-by: Chris Packham <judge.packham@gmail.com >> >> <mailto:judge.packham@gmail.com>> >>> >>> >>> Missed updating Stefan Reviewed-by tag? please check for next time. >> >> >> That was semi-intentional. I thought v2 was different enough that I >> didn't want to assume Stefan's review would carry through. > > > Fair enough. I've overlooked the clearing of the bits spotted by > Jagan. Please add my Reviewed-by now as well. Yeah, added and applied!
diff --git a/arch/arm/include/asm/arch-mvebu/spi.h b/arch/arm/include/asm/arch-mvebu/spi.h index 78869a253d1f..3545aed17347 100644 --- a/arch/arm/include/asm/arch-mvebu/spi.h +++ b/arch/arm/include/asm/arch-mvebu/spi.h @@ -52,6 +52,10 @@ struct kwspi_registers { #define KWSPI_ADRLEN_3BYTE (2 << 8) #define KWSPI_ADRLEN_4BYTE (3 << 8) #define KWSPI_ADRLEN_MASK (3 << 8) +#define KWSPI_CPOL (1 << 11) +#define KWSPI_CPHA (1 << 12) +#define KWSPI_TXLSBF (1 << 13) +#define KWSPI_RXLSBF (1 << 14) #define KWSPI_IRQUNMASK 1 /* unmask SPI interrupt */ #define KWSPI_IRQMASK 0 /* mask SPI interrupt */ diff --git a/drivers/spi/kirkwood_spi.c b/drivers/spi/kirkwood_spi.c index 6851ba942f51..791f3e8099c8 100644 --- a/drivers/spi/kirkwood_spi.c +++ b/drivers/spi/kirkwood_spi.c @@ -271,6 +271,21 @@ static int mvebu_spi_set_speed(struct udevice *bus, uint hz) static int mvebu_spi_set_mode(struct udevice *bus, uint mode) { + struct mvebu_spi_platdata *plat = dev_get_platdata(bus); + struct kwspi_registers *reg = plat->spireg; + u32 data = readl(®->cfg); + + data &= ~(KWSPI_CPHA | KWSPI_CPOL | KWSPI_RXLSBF | KWSPI_TXLSBF); + + if (mode & SPI_CPHA) + data |= KWSPI_CPHA; + if (mode & SPI_CPOL) + data |= KWSPI_CPOL; + if (mode & SPI_LSB_FIRST) + data |= (KWSPI_RXLSBF | KWSPI_TXLSBF); + + writel(data, ®->cfg); + return 0; }
Set the appropriate bits in the interface config register based on the SPI_ mode flags. Signed-off-by: Chris Packham <judge.packham@gmail.com> --- Changes in v2: - Clear bits before updating arch/arm/include/asm/arch-mvebu/spi.h | 4 ++++ drivers/spi/kirkwood_spi.c | 15 +++++++++++++++ 2 files changed, 19 insertions(+)