Merge branch 'master' of git://git.denx.de/u-boot-spi
- mpc8xxx spi driver fixes (Mario) - mpc8xxx spi dm conversion (Mario, Jagan) - SPI DM Migration update (Jagan)
This commit is contained in:
commit
07798764c2
|
@ -134,7 +134,6 @@ CONFIG_CMD_ENV_FLAGS=y
|
||||||
CONFIG_CMD_I2C=y
|
CONFIG_CMD_I2C=y
|
||||||
CONFIG_CMD_NAND=y
|
CONFIG_CMD_NAND=y
|
||||||
CONFIG_CMD_NAND_TRIMFFS=y
|
CONFIG_CMD_NAND_TRIMFFS=y
|
||||||
CONFIG_CMD_SPI=y
|
|
||||||
# CONFIG_CMD_SETEXPR is not set
|
# CONFIG_CMD_SETEXPR is not set
|
||||||
CONFIG_CMD_DHCP=y
|
CONFIG_CMD_DHCP=y
|
||||||
CONFIG_CMD_MII=y
|
CONFIG_CMD_MII=y
|
||||||
|
@ -160,6 +159,4 @@ CONFIG_MII=y
|
||||||
CONFIG_TSEC_ENET=y
|
CONFIG_TSEC_ENET=y
|
||||||
# CONFIG_PCI is not set
|
# CONFIG_PCI is not set
|
||||||
CONFIG_SYS_NS16550=y
|
CONFIG_SYS_NS16550=y
|
||||||
CONFIG_SPI=y
|
|
||||||
CONFIG_MPC8XXX_SPI=y
|
|
||||||
CONFIG_OF_LIBFDT=y
|
CONFIG_OF_LIBFDT=y
|
||||||
|
|
|
@ -59,10 +59,7 @@ No dm conversion yet:
|
||||||
drivers/spi/cf_spi.c
|
drivers/spi/cf_spi.c
|
||||||
drivers/spi/fsl_espi.c
|
drivers/spi/fsl_espi.c
|
||||||
drivers/spi/lpc32xx_ssp.c
|
drivers/spi/lpc32xx_ssp.c
|
||||||
drivers/spi/mpc8xx_spi.c
|
|
||||||
drivers/spi/mpc8xxx_spi.c
|
|
||||||
drivers/spi/mxs_spi.c
|
drivers/spi/mxs_spi.c
|
||||||
drivers/spi/sh_qspi.c
|
|
||||||
drivers/spi/sh_spi.c
|
drivers/spi/sh_spi.c
|
||||||
drivers/spi/soft_spi_legacy.c
|
drivers/spi/soft_spi_legacy.c
|
||||||
|
|
||||||
|
@ -70,13 +67,12 @@ No dm conversion yet:
|
||||||
Deadline: 2019.04
|
Deadline: 2019.04
|
||||||
|
|
||||||
Partially converted:
|
Partially converted:
|
||||||
drivers/spi/atcspi200_spi.c
|
|
||||||
drivers/spi/davinci_spi.c
|
drivers/spi/davinci_spi.c
|
||||||
drivers/spi/fsl_dspi.c
|
drivers/spi/fsl_dspi.c
|
||||||
drivers/spi/fsl_qspi.c
|
|
||||||
drivers/spi/kirkwood_spi.c
|
drivers/spi/kirkwood_spi.c
|
||||||
drivers/spi/mxc_spi.c
|
drivers/spi/mxc_spi.c
|
||||||
drivers/spi/omap3_spi.c
|
drivers/spi/omap3_spi.c
|
||||||
|
drivers/spi/sh_qspi.c
|
||||||
|
|
||||||
Status: In progress
|
Status: In progress
|
||||||
Deadline: 2019.07
|
Deadline: 2019.07
|
||||||
|
|
|
@ -137,6 +137,11 @@ config MPC8XX_SPI
|
||||||
help
|
help
|
||||||
Enable support for SPI on MPC8XX
|
Enable support for SPI on MPC8XX
|
||||||
|
|
||||||
|
config MPC8XXX_SPI
|
||||||
|
bool "MPC8XXX SPI Driver"
|
||||||
|
help
|
||||||
|
Enable support for SPI on the MPC8XXX PowerPC SoCs.
|
||||||
|
|
||||||
config MT7621_SPI
|
config MT7621_SPI
|
||||||
bool "MediaTek MT7621 SPI driver"
|
bool "MediaTek MT7621 SPI driver"
|
||||||
depends on SOC_MT7628
|
depends on SOC_MT7628
|
||||||
|
@ -370,11 +375,6 @@ config LPC32XX_SSP
|
||||||
help
|
help
|
||||||
Enable support for SPI on LPC32xx
|
Enable support for SPI on LPC32xx
|
||||||
|
|
||||||
config MPC8XXX_SPI
|
|
||||||
bool "MPC8XXX SPI Driver"
|
|
||||||
help
|
|
||||||
Enable support for SPI on the MPC8XXX PowerPC SoCs.
|
|
||||||
|
|
||||||
config MXC_SPI
|
config MXC_SPI
|
||||||
bool "MXC SPI Driver"
|
bool "MXC SPI Driver"
|
||||||
help
|
help
|
||||||
|
|
|
@ -5,161 +5,258 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#include <common.h>
|
#include <common.h>
|
||||||
|
#include <dm.h>
|
||||||
|
#include <errno.h>
|
||||||
#include <malloc.h>
|
#include <malloc.h>
|
||||||
#include <spi.h>
|
#include <spi.h>
|
||||||
#include <asm/mpc8xxx_spi.h>
|
#include <asm/mpc8xxx_spi.h>
|
||||||
|
#include <asm-generic/gpio.h>
|
||||||
|
|
||||||
#define SPI_EV_NE (0x80000000 >> 22) /* Receiver Not Empty */
|
enum {
|
||||||
#define SPI_EV_NF (0x80000000 >> 23) /* Transmitter Not Full */
|
SPI_EV_NE = BIT(31 - 22), /* Receiver Not Empty */
|
||||||
|
SPI_EV_NF = BIT(31 - 23), /* Transmitter Not Full */
|
||||||
|
};
|
||||||
|
|
||||||
#define SPI_MODE_LOOP (0x80000000 >> 1) /* Loopback mode */
|
enum {
|
||||||
#define SPI_MODE_REV (0x80000000 >> 5) /* Reverse mode - MSB first */
|
SPI_MODE_LOOP = BIT(31 - 1), /* Loopback mode */
|
||||||
#define SPI_MODE_MS (0x80000000 >> 6) /* Always master */
|
SPI_MODE_CI = BIT(31 - 2), /* Clock invert */
|
||||||
#define SPI_MODE_EN (0x80000000 >> 7) /* Enable interface */
|
SPI_MODE_CP = BIT(31 - 3), /* Clock phase */
|
||||||
|
SPI_MODE_DIV16 = BIT(31 - 4), /* Divide clock source by 16 */
|
||||||
|
SPI_MODE_REV = BIT(31 - 5), /* Reverse mode - MSB first */
|
||||||
|
SPI_MODE_MS = BIT(31 - 6), /* Always master */
|
||||||
|
SPI_MODE_EN = BIT(31 - 7), /* Enable interface */
|
||||||
|
|
||||||
|
SPI_MODE_LEN_MASK = 0xf00000,
|
||||||
|
SPI_MODE_PM_MASK = 0xf0000,
|
||||||
|
|
||||||
|
SPI_COM_LST = BIT(31 - 9),
|
||||||
|
};
|
||||||
|
|
||||||
|
struct mpc8xxx_priv {
|
||||||
|
spi8xxx_t *spi;
|
||||||
|
struct gpio_desc gpios[16];
|
||||||
|
int max_cs;
|
||||||
|
};
|
||||||
|
|
||||||
|
static inline u32 to_prescale_mod(u32 val)
|
||||||
|
{
|
||||||
|
return (min(val, (u32)15) << 16);
|
||||||
|
}
|
||||||
|
|
||||||
|
static void set_char_len(spi8xxx_t *spi, u32 val)
|
||||||
|
{
|
||||||
|
clrsetbits_be32(&spi->mode, SPI_MODE_LEN_MASK, (val << 20));
|
||||||
|
}
|
||||||
|
|
||||||
#define SPI_TIMEOUT 1000
|
#define SPI_TIMEOUT 1000
|
||||||
|
|
||||||
struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
|
static int __spi_set_speed(spi8xxx_t *spi, uint speed)
|
||||||
unsigned int max_hz, unsigned int mode)
|
|
||||||
{
|
{
|
||||||
struct spi_slave *slave;
|
/* TODO(mario.six@gdsys.cc): This only ever sets one fixed speed */
|
||||||
|
|
||||||
if (!spi_cs_is_valid(bus, cs))
|
/* Use SYSCLK / 8 (16.67MHz typ.) */
|
||||||
return NULL;
|
clrsetbits_be32(&spi->mode, SPI_MODE_PM_MASK, to_prescale_mod(1));
|
||||||
|
|
||||||
slave = spi_alloc_slave_base(bus, cs);
|
return 0;
|
||||||
if (!slave)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
/*
|
|
||||||
* TODO: Some of the code in spi_init() should probably move
|
|
||||||
* here, or into spi_claim_bus() below.
|
|
||||||
*/
|
|
||||||
|
|
||||||
return slave;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
void spi_free_slave(struct spi_slave *slave)
|
static int mpc8xxx_spi_ofdata_to_platdata(struct udevice *dev)
|
||||||
{
|
{
|
||||||
free(slave);
|
struct mpc8xxx_priv *priv = dev_get_priv(dev);
|
||||||
|
int ret;
|
||||||
|
|
||||||
|
priv->spi = (spi8xxx_t *)dev_read_addr(dev);
|
||||||
|
|
||||||
|
/* TODO(mario.six@gdsys.cc): Read clock and save the value */
|
||||||
|
|
||||||
|
ret = gpio_request_list_by_name(dev, "gpios", priv->gpios,
|
||||||
|
ARRAY_SIZE(priv->gpios), GPIOD_IS_OUT | GPIOD_ACTIVE_LOW);
|
||||||
|
if (ret < 0)
|
||||||
|
return -EINVAL;
|
||||||
|
|
||||||
|
priv->max_cs = ret;
|
||||||
|
|
||||||
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
void spi_init(void)
|
static int mpc8xxx_spi_probe(struct udevice *dev)
|
||||||
{
|
{
|
||||||
volatile spi8xxx_t *spi = &((immap_t *) (CONFIG_SYS_IMMR))->spi;
|
struct mpc8xxx_priv *priv = dev_get_priv(dev);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* SPI pins on the MPC83xx are not muxed, so all we do is initialize
|
* SPI pins on the MPC83xx are not muxed, so all we do is initialize
|
||||||
* some registers
|
* some registers
|
||||||
*/
|
*/
|
||||||
spi->mode = SPI_MODE_REV | SPI_MODE_MS | SPI_MODE_EN;
|
out_be32(&priv->spi->mode, SPI_MODE_REV | SPI_MODE_MS | SPI_MODE_EN);
|
||||||
spi->mode = (spi->mode & 0xfff0ffff) | BIT(16); /* Use SYSCLK / 8
|
|
||||||
(16.67MHz typ.) */
|
__spi_set_speed(priv->spi, 16666667);
|
||||||
spi->event = 0xffffffff; /* Clear all SPI events */
|
|
||||||
spi->mask = 0x00000000; /* Mask all SPI interrupts */
|
/* Clear all SPI events */
|
||||||
spi->com = 0; /* LST bit doesn't do anything, so disregard */
|
setbits_be32(&priv->spi->event, 0xffffffff);
|
||||||
}
|
/* Mask all SPI interrupts */
|
||||||
|
clrbits_be32(&priv->spi->mask, 0xffffffff);
|
||||||
|
/* LST bit doesn't do anything, so disregard */
|
||||||
|
out_be32(&priv->spi->com, 0);
|
||||||
|
|
||||||
int spi_claim_bus(struct spi_slave *slave)
|
|
||||||
{
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
void spi_release_bus(struct spi_slave *slave)
|
static void mpc8xxx_spi_cs_activate(struct udevice *dev)
|
||||||
{
|
{
|
||||||
|
struct mpc8xxx_priv *priv = dev_get_priv(dev->parent);
|
||||||
|
struct dm_spi_slave_platdata *platdata = dev_get_parent_platdata(dev);
|
||||||
|
|
||||||
|
dm_gpio_set_dir_flags(&priv->gpios[platdata->cs], GPIOD_IS_OUT);
|
||||||
|
dm_gpio_set_value(&priv->gpios[platdata->cs], 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
|
static void mpc8xxx_spi_cs_deactivate(struct udevice *dev)
|
||||||
void *din, unsigned long flags)
|
|
||||||
{
|
{
|
||||||
volatile spi8xxx_t *spi = &((immap_t *) (CONFIG_SYS_IMMR))->spi;
|
struct mpc8xxx_priv *priv = dev_get_priv(dev->parent);
|
||||||
unsigned int tmpdout, tmpdin, event;
|
struct dm_spi_slave_platdata *platdata = dev_get_parent_platdata(dev);
|
||||||
int numBlks = DIV_ROUND_UP(bitlen, 32);
|
|
||||||
int tm, isRead = 0;
|
|
||||||
unsigned char charSize = 32;
|
|
||||||
|
|
||||||
debug("spi_xfer: slave %u:%u dout %08X din %08X bitlen %u\n",
|
dm_gpio_set_dir_flags(&priv->gpios[platdata->cs], GPIOD_IS_OUT);
|
||||||
slave->bus, slave->cs, *(uint *) dout, *(uint *) din, bitlen);
|
dm_gpio_set_value(&priv->gpios[platdata->cs], 1);
|
||||||
|
}
|
||||||
|
|
||||||
|
static int mpc8xxx_spi_xfer(struct udevice *dev, uint bitlen,
|
||||||
|
const void *dout, void *din, ulong flags)
|
||||||
|
{
|
||||||
|
struct udevice *bus = dev->parent;
|
||||||
|
struct mpc8xxx_priv *priv = dev_get_priv(bus);
|
||||||
|
spi8xxx_t *spi = priv->spi;
|
||||||
|
struct dm_spi_slave_platdata *platdata = dev_get_parent_platdata(dev);
|
||||||
|
u32 tmpdin = 0;
|
||||||
|
int num_blks = DIV_ROUND_UP(bitlen, 32);
|
||||||
|
|
||||||
|
debug("%s: slave %s:%u dout %08X din %08X bitlen %u\n", __func__,
|
||||||
|
bus->name, platdata->cs, *(uint *)dout, *(uint *)din, bitlen);
|
||||||
|
|
||||||
if (flags & SPI_XFER_BEGIN)
|
if (flags & SPI_XFER_BEGIN)
|
||||||
spi_cs_activate(slave);
|
mpc8xxx_spi_cs_activate(dev);
|
||||||
|
|
||||||
spi->event = 0xffffffff; /* Clear all SPI events */
|
/* Clear all SPI events */
|
||||||
|
setbits_be32(&spi->event, 0xffffffff);
|
||||||
|
|
||||||
/* handle data in 32-bit chunks */
|
/* Handle data in 32-bit chunks */
|
||||||
while (numBlks--) {
|
while (num_blks--) {
|
||||||
tmpdout = 0;
|
u32 tmpdout = 0;
|
||||||
charSize = (bitlen >= 32 ? 32 : bitlen);
|
uchar xfer_bitlen = (bitlen >= 32 ? 32 : bitlen);
|
||||||
|
ulong start;
|
||||||
|
|
||||||
|
clrbits_be32(&spi->mode, SPI_MODE_EN);
|
||||||
|
|
||||||
|
/* Set up length for this transfer */
|
||||||
|
|
||||||
|
if (bitlen <= 4) /* 4 bits or less */
|
||||||
|
set_char_len(spi, 3);
|
||||||
|
else if (bitlen <= 16) /* at most 16 bits */
|
||||||
|
set_char_len(spi, bitlen - 1);
|
||||||
|
else /* more than 16 bits -> full 32 bit transfer */
|
||||||
|
set_char_len(spi, 0);
|
||||||
|
|
||||||
|
setbits_be32(&spi->mode, SPI_MODE_EN);
|
||||||
|
|
||||||
/* Shift data so it's msb-justified */
|
/* Shift data so it's msb-justified */
|
||||||
tmpdout = *(u32 *) dout >> (32 - charSize);
|
tmpdout = *(u32 *)dout >> (32 - xfer_bitlen);
|
||||||
|
|
||||||
/* The LEN field of the SPMODE register is set as follows:
|
if (bitlen > 32) {
|
||||||
*
|
|
||||||
* Bit length setting
|
|
||||||
* len <= 4 3
|
|
||||||
* 4 < len <= 16 len - 1
|
|
||||||
* len > 16 0
|
|
||||||
*/
|
|
||||||
|
|
||||||
spi->mode &= ~SPI_MODE_EN;
|
|
||||||
|
|
||||||
if (bitlen <= 16) {
|
|
||||||
if (bitlen <= 4)
|
|
||||||
spi->mode = (spi->mode & 0xff0fffff) |
|
|
||||||
(3 << 20);
|
|
||||||
else
|
|
||||||
spi->mode = (spi->mode & 0xff0fffff) |
|
|
||||||
((bitlen - 1) << 20);
|
|
||||||
} else {
|
|
||||||
spi->mode = (spi->mode & 0xff0fffff);
|
|
||||||
/* Set up the next iteration if sending > 32 bits */
|
/* Set up the next iteration if sending > 32 bits */
|
||||||
bitlen -= 32;
|
bitlen -= 32;
|
||||||
dout += 4;
|
dout += 4;
|
||||||
}
|
}
|
||||||
|
|
||||||
spi->mode |= SPI_MODE_EN;
|
/* Write the data out */
|
||||||
|
out_be32(&spi->tx, tmpdout);
|
||||||
|
|
||||||
spi->tx = tmpdout; /* Write the data out */
|
debug("*** %s: ... %08x written\n", __func__, tmpdout);
|
||||||
debug("*** spi_xfer: ... %08x written\n", tmpdout);
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Wait for SPI transmit to get out
|
* Wait for SPI transmit to get out
|
||||||
* or time out (1 second = 1000 ms)
|
* or time out (1 second = 1000 ms)
|
||||||
* The NE event must be read and cleared first
|
* The NE event must be read and cleared first
|
||||||
*/
|
*/
|
||||||
for (tm = 0, isRead = 0; tm < SPI_TIMEOUT; ++tm) {
|
start = get_timer(0);
|
||||||
event = spi->event;
|
do {
|
||||||
if (event & SPI_EV_NE) {
|
u32 event = in_be32(&spi->event);
|
||||||
tmpdin = spi->rx;
|
bool have_ne = event & SPI_EV_NE;
|
||||||
spi->event |= SPI_EV_NE;
|
bool have_nf = event & SPI_EV_NF;
|
||||||
isRead = 1;
|
|
||||||
|
|
||||||
*(u32 *) din = (tmpdin << (32 - charSize));
|
if (!have_ne)
|
||||||
if (charSize == 32) {
|
continue;
|
||||||
/* Advance output buffer by 32 bits */
|
|
||||||
din += 4;
|
tmpdin = in_be32(&spi->rx);
|
||||||
}
|
setbits_be32(&spi->event, SPI_EV_NE);
|
||||||
|
|
||||||
|
*(u32 *)din = (tmpdin << (32 - xfer_bitlen));
|
||||||
|
if (xfer_bitlen == 32) {
|
||||||
|
/* Advance output buffer by 32 bits */
|
||||||
|
din += 4;
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Only bail when we've had both NE and NF events.
|
* Only bail when we've had both NE and NF events.
|
||||||
* This will cause timeouts on RO devices, so maybe
|
* This will cause timeouts on RO devices, so maybe
|
||||||
* in the future put an arbitrary delay after writing
|
* in the future put an arbitrary delay after writing
|
||||||
* the device. Arbitrary delays suck, though...
|
* the device. Arbitrary delays suck, though...
|
||||||
*/
|
*/
|
||||||
if (isRead && (event & SPI_EV_NF))
|
if (have_nf)
|
||||||
break;
|
break;
|
||||||
}
|
|
||||||
if (tm >= SPI_TIMEOUT)
|
|
||||||
puts("*** spi_xfer: Time out during SPI transfer");
|
|
||||||
|
|
||||||
debug("*** spi_xfer: transfer ended. Value=%08x\n", tmpdin);
|
mdelay(1);
|
||||||
|
} while (get_timer(start) < SPI_TIMEOUT);
|
||||||
|
|
||||||
|
if (get_timer(start) >= SPI_TIMEOUT) {
|
||||||
|
debug("*** %s: Time out during SPI transfer\n",
|
||||||
|
__func__);
|
||||||
|
return -ETIMEDOUT;
|
||||||
|
}
|
||||||
|
|
||||||
|
debug("*** %s: transfer ended. Value=%08x\n", __func__, tmpdin);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (flags & SPI_XFER_END)
|
if (flags & SPI_XFER_END)
|
||||||
spi_cs_deactivate(slave);
|
mpc8xxx_spi_cs_deactivate(dev);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int mpc8xxx_spi_set_speed(struct udevice *dev, uint speed)
|
||||||
|
{
|
||||||
|
struct mpc8xxx_priv *priv = dev_get_priv(dev);
|
||||||
|
|
||||||
|
return __spi_set_speed(priv->spi, speed);
|
||||||
|
}
|
||||||
|
|
||||||
|
static int mpc8xxx_spi_set_mode(struct udevice *dev, uint mode)
|
||||||
|
{
|
||||||
|
/* TODO(mario.six@gdsys.cc): Using SPI_CPHA (for clock phase) and
|
||||||
|
* SPI_CPOL (for clock polarity) should work
|
||||||
|
*/
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
static const struct dm_spi_ops mpc8xxx_spi_ops = {
|
||||||
|
.xfer = mpc8xxx_spi_xfer,
|
||||||
|
.set_speed = mpc8xxx_spi_set_speed,
|
||||||
|
.set_mode = mpc8xxx_spi_set_mode,
|
||||||
|
/*
|
||||||
|
* cs_info is not needed, since we require all chip selects to be
|
||||||
|
* in the device tree explicitly
|
||||||
|
*/
|
||||||
|
};
|
||||||
|
|
||||||
|
static const struct udevice_id mpc8xxx_spi_ids[] = {
|
||||||
|
{ .compatible = "fsl,spi" },
|
||||||
|
{ }
|
||||||
|
};
|
||||||
|
|
||||||
|
U_BOOT_DRIVER(mpc8xxx_spi) = {
|
||||||
|
.name = "mpc8xxx_spi",
|
||||||
|
.id = UCLASS_SPI,
|
||||||
|
.of_match = mpc8xxx_spi_ids,
|
||||||
|
.ops = &mpc8xxx_spi_ops,
|
||||||
|
.ofdata_to_platdata = mpc8xxx_spi_ofdata_to_platdata,
|
||||||
|
.probe = mpc8xxx_spi_probe,
|
||||||
|
.priv_auto_alloc_size = sizeof(struct mpc8xxx_priv),
|
||||||
|
};
|
||||||
|
|
Loading…
Reference in New Issue