diff options
author | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-03-07 17:40:23 +0000 |
---|---|---|
committer | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-03-07 17:40:23 +0000 |
commit | 52e4f12c798f8e28445af1ec3809364ba39d9000 (patch) | |
tree | cd81d113f701d250631b47c306492b74de869d92 /nuttx/arch/mips/src/pic32mx | |
parent | 360fc8ba4b531ca44e553be6259c44afc3862175 (diff) | |
download | px4-nuttx-52e4f12c798f8e28445af1ec3809364ba39d9000.tar.gz px4-nuttx-52e4f12c798f8e28445af1ec3809364ba39d9000.tar.bz2 px4-nuttx-52e4f12c798f8e28445af1ec3809364ba39d9000.zip |
Updates to PIC32 SPI driver
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4461 42af7a65-404d-4744-a932-0658087f49c3
Diffstat (limited to 'nuttx/arch/mips/src/pic32mx')
-rw-r--r-- | nuttx/arch/mips/src/pic32mx/pic32mx-internal.h | 138 | ||||
-rw-r--r-- | nuttx/arch/mips/src/pic32mx/pic32mx-spi.c | 190 |
2 files changed, 227 insertions, 101 deletions
diff --git a/nuttx/arch/mips/src/pic32mx/pic32mx-internal.h b/nuttx/arch/mips/src/pic32mx/pic32mx-internal.h index 190fd8a47..6d9373df8 100644 --- a/nuttx/arch/mips/src/pic32mx/pic32mx-internal.h +++ b/nuttx/arch/mips/src/pic32mx/pic32mx-internal.h @@ -47,6 +47,8 @@ #include <stdint.h> #include <stdbool.h> +#include <nuttx/spi.h> + #include "up_internal.h" #include "chip.h" #include "pic32mx-config.h" @@ -396,42 +398,48 @@ EXTERN void pic32mx_dumpgpio(uint32_t pinset, const char *msg); * ************************************************************************************/ -struct spi_dev_s; -enum spi_dev_e; - #ifdef CONFIG_PIC32MX_SPI1 -EXTERN void pic32mx_spi1select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool selected); +EXTERN void pic32mx_spi1select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool selected); EXTERN uint8_t pic32mx_spi1status(FAR struct spi_dev_s *dev, enum spi_dev_e devid); #ifdef CONFIG_SPI_CMDDATA -EXTERN int pic32mx_spi1cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool cmd); +EXTERN int pic32mx_spi1cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool cmd); #endif #endif #ifdef CONFIG_PIC32MX_SPI2 -EXTERN void pic32mx_spi2select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool selected); +EXTERN void pic32mx_spi2select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool selected); EXTERN uint8_t pic32mx_spi2status(FAR struct spi_dev_s *dev, enum spi_dev_e devid); #ifdef CONFIG_SPI_CMDDATA -EXTERN int pic32mx_spi2cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool cmd); +EXTERN int pic32mx_spi2cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool cmd); #endif #endif #ifdef CONFIG_PIC32MX_SPI3 -EXTERN void pic32mx_spi3select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool selected); +EXTERN void pic32mx_spi3select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool selected); EXTERN uint8_t pic32mx_spi3status(FAR struct spi_dev_s *dev, enum spi_dev_e devid); #ifdef CONFIG_SPI_CMDDATA -EXTERN int pic32mx_spi3cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool cmd); +EXTERN int pic32mx_spi3cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool cmd); #endif #endif #ifdef CONFIG_PIC32MX_SPI3 -EXTERN void pic32mx_spi3select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool selected); +EXTERN void pic32mx_spi3select(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool selected); EXTERN uint8_t pic32mx_spi3status(FAR struct spi_dev_s *dev, enum spi_dev_e devid); #ifdef CONFIG_SPI_CMDDATA -EXTERN int pic32mx_spi3cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, bool cmd); +EXTERN int pic32mx_spi3cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, + bool cmd); #endif #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmainitialize * * Description: @@ -440,54 +448,60 @@ EXTERN int pic32mx_spi3cmddata(FAR struct spi_dev_s *dev, enum spi_dev_e devid, * Returned Value: * None * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN void pic32mx_dmainitilaize(void); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmachannel * * Description: - * Allocate a DMA channel. This function sets aside a DMA channel and - * gives the caller exclusive access to the DMA channel. + * Allocate a DMA channel. This function sets aside a DMA channel and gives the + * caller exclusive access to the DMA channel. * * Returned Value: - * One success, this function returns a non-NULL, void* DMA channel - * handle. NULL is returned on any failure. This function can fail only - * if no DMA channel is available. + * One success, this function returns a non-NULL, void* DMA channel handle. NULL + * is returned on any failure. This function can fail only if no DMA channel is + * available. * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN DMA_HANDLE pic32mx_dmachannel(void); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmafree * * Description: - * Release a DMA channel. NOTE: The 'handle' used in this argument must - * NEVER be used again until pic32mx_dmachannel() is called again to re-gain - * a valid handle. + * Release a DMA channel. NOTE: The 'handle' used in this argument must NEVER be + * used again until pic32mx_dmachannel() is called again to re-gain a valid handle. * * Returned Value: * None * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN void pic32mx_dmafree(DMA_HANDLE handle); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmasetup * * Description: * Configure DMA for one transfer. * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN int pic32mx_dmarxsetup(DMA_HANDLE handle, @@ -496,39 +510,45 @@ EXTERN int pic32mx_dmarxsetup(DMA_HANDLE handle, size_t nbytes); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmastart * * Description: * Start the DMA transfer * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN int pic32mx_dmastart(DMA_HANDLE handle, dma_callback_t callback, void *arg); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmastop * * Description: - * Cancel the DMA. After pic32mx_dmastop() is called, the DMA channel is - * reset and pic32mx_dmasetup() must be called before pic32mx_dmastart() can be - * called again + * Cancel the DMA. After pic32mx_dmastop() is called, the DMA channel is reset + * and pic32mx_dmasetup() must be called before pic32mx_dmastart() can be called + * again * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA EXTERN void pic32mx_dmastop(DMA_HANDLE handle); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmasample * * Description: * Sample DMA register contents * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA #ifdef CONFIG_DEBUG_DMA @@ -538,13 +558,15 @@ EXTERN void pic32mx_dmasample(DMA_HANDLE handle, struct pic32mx_dmaregs_s *regs) #endif #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_dmadump * * Description: * Dump previously sampled DMA register contents * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_DMA #ifdef CONFIG_DEBUG_DMA @@ -555,46 +577,52 @@ EXTERN void pic32mx_dmadump(DMA_HANDLE handle, const struct pic32mx_dmaregs_s *r #endif #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_usbpullup * * Description: - * If USB is supported and the board supports a pullup via GPIO (for USB - * software connect and disconnect), then the board software must provide - * stm32_pullup. See include/nuttx/usb/usbdev.h for additional description - * of this method. Alternatively, if no pull-up GPIO the following EXTERN - * can be redefined to be NULL. + * If USB is supported and the board supports a pullup via GPIO (for USB software + * connect and disconnect), then the board software must provide pic32mx_pullup. + * See include/nuttx/usb/usbdev.h for additional description of this method. + * Alternatively, if no pull-up GPIO the following EXTERN can be redefined to be + * NULL. * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_USBDEV struct usbdev_s; EXTERN int pic32mx_usbpullup(FAR struct usbdev_s *dev, bool enable); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_usbsuspend * * Description: - * Board logic must provide the stm32_usbsuspend logic if the USBDEV driver - * is used. This function is called whenever the USB enters or leaves - * suspend mode. This is an opportunity for the board logic to shutdown - * clocks, power, etc. while the USB is suspended. + * Board logic must provide the pic32mx_usbsuspend logic if the USBDEV driver is + * used. This function is called whenever the USB enters or leaves suspend mode. + * This is an opportunity for the board logic to shutdown clocks, power, etc. while + * the USB is suspended. * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_USBDEV EXTERN void pic32mx_usbsuspend(FAR struct usbdev_s *dev, bool resume); #endif -/**************************************************************************** +/************************************************************************************ + * Name: pic32mx_usbattach and pic32mx_usbdetach * * Description: - * The USB stack must be notified when the device is attached or detached - * by calling one of these functions. + * The USB stack must be notified when the device is attached or detached by + * calling one of these functions. * - ****************************************************************************/ + ************************************************************************************/ + #ifdef CONFIG_PIC32MX_USBDEV EXTERN void pic32mx_usbattach(void); diff --git a/nuttx/arch/mips/src/pic32mx/pic32mx-spi.c b/nuttx/arch/mips/src/pic32mx/pic32mx-spi.c index d884511e8..fc81f3d25 100644 --- a/nuttx/arch/mips/src/pic32mx/pic32mx-spi.c +++ b/nuttx/arch/mips/src/pic32mx/pic32mx-spi.c @@ -63,10 +63,15 @@ /**************************************************************************** * Definitions ****************************************************************************/ -/* Enables non-standard debug output from this file */ +/* Enables non-standard debug output from this file. + * + * CONFIG_SPI_DEBUG && CONFIG_DEBUG - Define to enable basic SPI debug + * CONFIG_DEBUG_VERBOSE - Define to enable verbose SPI debug + */ #ifndef CONFIG_DEBUG # undef CONFIG_DEBUG_SPI +# undef CONFIG_DEBUG_VERBOSE #endif #ifdef CONFIG_DEBUG_SPI @@ -77,7 +82,6 @@ # define spivdbg(x...) # endif #else -# undef CONFIG_DEBUG_VERBOSE # define spidbg(x...) # define spivdbg(x...) #endif @@ -401,6 +405,13 @@ static uint32_t spi_setfrequency(FAR struct spi_dev_s *dev, uint32_t frequency) uint32_t actual; uint32_t regval; +#ifndef CONFIG_SPI_OWNBUS + spivdbg("Old frequency: %d actual: %d New frequency: %d\n", + priv->frequency, priv->actual, frequency); +#else + spivdbg("New frequency: %d\n", regval); +#endif + /* Check if the requested frequency is the same as the frequency selection */ #ifndef CONFIG_SPI_OWNBUS @@ -435,8 +446,10 @@ static uint32_t spi_setfrequency(FAR struct spi_dev_s *dev, uint32_t frequency) /* Save the new BRG value */ spi_putreg(priv, PIC32MX_SPI_BRG_OFFSET, regval); + spivdbg("PBCLOCK: %d frequency: %d divisor: %d BRG: %d\n", + BOARD_PBCLOCK, frequency, divisor, regval); - /* Calculate the new actual frequency" + /* Calculate the new actual frequency. * * frequency = BOARD_PBCLOCK / (2 * divisor) */ @@ -450,7 +463,7 @@ static uint32_t spi_setfrequency(FAR struct spi_dev_s *dev, uint32_t frequency) priv->actual = actual; #endif - spidbg("Frequency %d->%d\n", frequency, actual); + spidbg("New frequency: %d Actual: %d\n", frequency, actual); return actual; } @@ -474,16 +487,50 @@ static void spi_setmode(FAR struct spi_dev_s *dev, enum spi_mode_e mode) FAR struct pic32mx_dev_s *priv = (FAR struct pic32mx_dev_s *)dev; uint32_t regval; +#ifndef CONFIG_SPI_OWNBUS + spivdbg("Old mode: %d New mode: %d\n", priv->mode, mode); +#else + spivdbg("New mode: %d\n", mode); +#endif + /* Has the mode changed? */ #ifndef CONFIG_SPI_OWNBUS if (mode != priv->mode) { #endif - /* Yes... Set CR appropriately */ + /* Yes... Set CON register appropriately. + * + * Standard terminology is as follows: + * + * Mode CPOL CPHA + * 0 0 0 + * 1 0 1 + * 2 1 0 + * 3 1 1 + * + * CPOL=0: The inactive value of the clock is zero + * CPOL=1: The inactive value of the clock is one + * CPHA=0: Data is captured on the clock's inactive-to-active edge and + * data is propagated on a active-to-inactive edge. + * CPHA=1: Data is captured on the clock's active-to-inactive edge and + * data is propagated on a active-to-inactive edge. + * + * CON Register mapping: + * CPOL=0 corresponds to CON:CKP=0; CPOL=1 corresponds to CON:CKP=1 + * CPHA=0 corresponds to CON:CKE=1; CPHA=1 corresponds to CON:CKE=1 + * + * In addition, the CON register supports SMP: SPI Data Input Sample + * Phase bit: + * + * 1 = Input data sampled at end of data output time + * 0 = Input data sampled at middle of data output time + * + * Which is hardcoded to 1. + */ regval = spi_getreg(priv, PIC32MX_SPI_CON_OFFSET); - regval &= ~(SPI_CON_CKP|SPI_CON_SMP); + regval &= ~(SPI_CON_CKP|SPI_CON_CKE); switch (mode) { @@ -491,7 +538,7 @@ static void spi_setmode(FAR struct spi_dev_s *dev, enum spi_mode_e mode) break; case SPIDEV_MODE1: /* CPOL=0; CPHA=1 */ - regval |= SPI_CON_SMP; + regval |= SPI_CON_CKE; break; case SPIDEV_MODE2: /* CPOL=1; CPHA=0 */ @@ -499,7 +546,7 @@ static void spi_setmode(FAR struct spi_dev_s *dev, enum spi_mode_e mode) break; case SPIDEV_MODE3: /* CPOL=1; CPHA=1 */ - regval |= (SPI_CON_CKP|SPI_CON_SMP); + regval |= (SPI_CON_CKP|SPI_CON_CKE); break; default: @@ -508,6 +555,7 @@ static void spi_setmode(FAR struct spi_dev_s *dev, enum spi_mode_e mode) } spi_putreg(priv, PIC32MX_SPI_CON_OFFSET, regval); + spivdbg("CON: %08x\n", regval); /* Save the mode so that subsequent re-configuratins will be faster */ @@ -535,9 +583,15 @@ static void spi_setmode(FAR struct spi_dev_s *dev, enum spi_mode_e mode) static void spi_setbits(FAR struct spi_dev_s *dev, int nbits) { FAR struct pic32mx_dev_s *priv = (FAR struct pic32mx_dev_s *)dev; - uint32_t mode; + uint32_t setting; uint32_t regval; +#ifndef CONFIG_SPI_OWNBUS + spivdbg("Old nbits: %d New nbits: %d\n", priv->nbits, nbits); +#else + spivdbg("New nbits: %d\n", nbits); +#endif + /* Has the number of bits changed? */ DEBUGASSERT(priv && nbits > 7 && nbits < 17); @@ -549,15 +603,15 @@ static void spi_setbits(FAR struct spi_dev_s *dev, int nbits) if (nbits == 8) { - mode = SPI_CON_MODE_8BIT; + setting = SPI_CON_MODE_8BIT; } else if (nbits == 16) { - mode = SPI_CON_MODE_8BIT; + setting = SPI_CON_MODE_8BIT; } else if (nbits == 32) { - mode = SPI_CON_MODE_8BIT; + setting = SPI_CON_MODE_8BIT; } else { @@ -567,8 +621,9 @@ static void spi_setbits(FAR struct spi_dev_s *dev, int nbits) regval = spi_getreg(priv, PIC32MX_SPI_CON_OFFSET); regval &= ~SPI_CON_MODE_MASK; - regval |= mode; + regval |= setting; regval = spi_getreg(priv, PIC32MX_SPI_CON_OFFSET); + spivdbg("CON: %08x\n", regval); /* Save the selection so the subsequence re-configurations will be faster */ @@ -598,15 +653,26 @@ static uint16_t spi_send(FAR struct spi_dev_s *dev, uint16_t wd) { FAR struct pic32mx_dev_s *priv = (FAR struct pic32mx_dev_s *)dev; + spivdbg("wd: %04x\n", wd); + /* Write the data to transmitted to the SPI Data Register */ spi_putreg(priv, PIC32MX_SPI_BUF_OFFSET, (uint32_t)wd); - /* Wait for the SPITBE bit in the SPI Status Register to be set to 1. The - * SPITBE bit will be set when the receive buffer is not empty. +#ifdef CONFIG_PIC32MX_SPI_ENHBUF + /* Wait for the SPIRBE bit in the SPI Status Register to be set to 0. In + * enhanced buffer mode, the SPIRBE bit will be cleared in when the + * receive buffer is not empty. + */ + + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBE) != 0); +#else + /* Wait for the SPIRBF bit in the SPI Status Register to be set to 1. In + * normal mode, the SPIRBF bit will be set when receive data is available. */ - while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPITBE) == 0); + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBF) == 0); +#endif /* Return the SPI data */ @@ -639,7 +705,7 @@ static void spi_sndblock(FAR struct spi_dev_s *dev, FAR const void *buffer, size uint32_t regval; uint8_t data; - spidbg("nwords: %d\n", nwords); + spivdbg("nwords: %d\n", nwords); while (nwords) { /* Write the data to transmitted to the SPI Data Register */ @@ -647,11 +713,20 @@ static void spi_sndblock(FAR struct spi_dev_s *dev, FAR const void *buffer, size data = *ptr++; spi_putreg(priv, PIC32MX_SPI_BUF_OFFSET, (uint32_t)data); - /* Wait for the SPITBE bit in the SPI Status Register to be set to 1. - * The SPITBE bit will be set when the receive buffer is not empty. +#ifdef CONFIG_PIC32MX_SPI_ENHBUF + /* Wait for the SPIRBE bit in the SPI Status Register to be set to 0. In + * enhanced buffer mode, the SPIRBE bit will be cleared in when the + * receive buffer is not empty. */ - while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPITBE) == 0); + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBE) != 0); +#else + /* Wait for the SPIRBF bit in the SPI Status Register to be set to 1. In + * normal mode, the SPIRBF bit will be set when receive data is available. + */ + + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBF) == 0); +#endif /* Read from the buffer register to clear the status bit */ @@ -684,7 +759,7 @@ static void spi_recvblock(FAR struct spi_dev_s *dev, FAR void *buffer, size_t nw FAR struct pic32mx_dev_s *priv = (FAR struct pic32mx_dev_s *)dev; FAR uint8_t *ptr = (FAR uint8_t*)buffer; - spidbg("nwords: %d\n", nwords); + spivdbg("nwords: %d\n", nwords); while (nwords) { /* Write some dummy data to the SPI Data Register in order to clock the @@ -693,11 +768,20 @@ static void spi_recvblock(FAR struct spi_dev_s *dev, FAR void *buffer, size_t nw spi_putreg(priv, PIC32MX_SPI_BUF_OFFSET, 0xff); - /* Wait for the SPITBE bit in the SPI Status Register to be set to 1. - * The SPITBE bit will be set when the receive buffer is not empty. +#ifdef CONFIG_PIC32MX_SPI_ENHBUF + /* Wait for the SPIRBE bit in the SPI Status Register to be set to 0. In + * enhanced buffer mode, the SPIRBE bit will be cleared in when the + * receive buffer is not empty. */ - while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPITBE) == 0); + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBE) != 0); +#else + /* Wait for the SPIRBF bit in the SPI Status Register to be set to 1. In + * normal mode, the SPIRBF bit will be set when receive data is available. + */ + + while ((spi_getreg(priv, PIC32MX_SPI_STAT_OFFSET) & SPI_STAT_SPIRBF) == 0); +#endif /* Read the received data from the SPI Data Register */ @@ -730,6 +814,8 @@ FAR struct spi_dev_s *up_spiinitialize(int port) irqstate_t flags; uint32_t regval; + spivdbg("port: %d\n", port); + /* Select the SPI state structure for this port */ #ifdef CONFIG_PIC32MX_SPI1 @@ -782,10 +868,10 @@ FAR struct spi_dev_s *up_spiinitialize(int port) regval = spi_getreg(priv, PIC32MX_SPI_BUF_OFFSET); - /* Set the ENHBUF bit if using Enhanced Buffer mode. */ - #ifdef CONFIG_PIC32MX_SPI_INTERRUPTS - /* Attach the interrupt vector */ + /* Attach the interrupt vector. We do this early to make sure that the + * resource is available. + */ ret = irq_attach(priv->vector, spi_interrupt); if (ret < 0) @@ -793,21 +879,6 @@ FAR struct spi_dev_s *up_spiinitialize(int port) spidbg("Failed to attach vector: %d port: %d\n", priv->vector, port); goto errout; } - - /* Enable SPI interrupts */ - - up_enable_irq(priv->eirq); - up_enable_irq(priv->txirq); - up_enable_irq(priv->rxirq); - - /* Set the interrupt priority */ - - ret = up_prioritize_irq(priv->vector, CONFIG_PIC32MX_SPI_PRIORITY) - if (ret < 0) - { - spidbg("up_prioritize_irq failed: %d\n", ret); - goto errout; - } #endif /* Select a default frequency of approx. 400KHz */ @@ -818,13 +889,20 @@ FAR struct spi_dev_s *up_spiinitialize(int port) spi_putreg(priv, PIC32MX_SPI_STATCLR_OFFSET, SPI_STAT_SPIROV); - /* Initial settings 8 bit + master mode + mode 0*/ + /* Initial settings 8 bit + master mode + mode 0. NOTE that MSSEN + * not set: The slave select pin must be driven manually via the + * board-specific pic32mx_spiNselect() interface. + */ - regval = (SPI_CON_MSTEN | SPI_CON_MODE_8BIT | SPI_CON_ON); -#ifdef CONFIG_PIC32MX_SPI_INTERRUPTS - regval |= (SPI_CON_RTXISEL_HALF | SPI_CON_STXISEL_HALF); + regval = (SPI_CON_MSTEN | SPI_CON_SMP | SPI_CON_MODE_8BIT | SPI_CON_ON); + + /* Set the ENHBUF bit if using Enhanced Buffer mode. */ + +#ifdef CONFIG_PIC32MX_SPI_ENHBUF + regval |= (SPI_CON_ENHBUF | SPI_CON_RTXISEL_HALF | SPI_CON_STXISEL_HALF); #endif spi_putreg(priv, PIC32MX_SPI_CON_OFFSET, regval); + spivdbg("CON: %08x\n", regval); /* Set the initial SPI configuration */ @@ -838,6 +916,26 @@ FAR struct spi_dev_s *up_spiinitialize(int port) #ifndef CONFIG_SPI_OWNBUS sem_init(&priv->exclsem, 0, 1); #endif + +#ifdef CONFIG_PIC32MX_SPI_INTERRUPTS + /* Enable interrupts at the SPI controller */ + + up_enable_irq(priv->eirq); + up_enable_irq(priv->txirq); + up_enable_irq(priv->rxirq); + + /* Set the SPI interrupt priority */ + + ret = up_prioritize_irq(priv->vector, CONFIG_PIC32MX_SPI_PRIORITY) + if (ret < 0) + { + spidbg("up_prioritize_irq failed: %d\n", ret); + goto errout; + } +#endif + + /* Enable interrupts at the interrupt controller */ + irqrestore(flags); return &priv->spidev; |