diff options
author | Uros Majstorovic <majstor@majstor.org> | 2018-03-22 05:26:21 +0100 |
---|---|---|
committer | Uros Majstorovic <majstor@majstor.org> | 2018-03-22 05:26:21 +0100 |
commit | 1aaad9e44871ee421548a14dee25f710b77eba20 (patch) | |
tree | 06c8c9ed9be7b377e3d0f07f4fd373d8411c3321 /code | |
parent | 7410152de8c37361b3935a4edf531c8a86bb9453 (diff) |
spi exchange handler simplified; net test pass with asm spi handlers
Diffstat (limited to 'code')
-rw-r--r-- | code/fe310/eos/net.c | 50 | ||||
-rw-r--r-- | code/fe310/eos/spi_def.h | 3 | ||||
-rw-r--r-- | code/fe310/eos/trap_entry.S | 87 |
3 files changed, 66 insertions, 74 deletions
diff --git a/code/fe310/eos/net.c b/code/fe310/eos/net.c index c0b5252..8213262 100644 --- a/code/fe310/eos/net.c +++ b/code/fe310/eos/net.c @@ -56,7 +56,6 @@ static void spi_xchg_reset(void) { // before starting a transaction, set SPI peripheral to desired mode SPI1_REG(SPI_REG_CSMODE) = SPI_CSMODE_HOLD; - while (SPI1_REG(SPI_REG_TXFIFO) & SPI_TXFIFO_FULL); SPI1_REG(SPI_REG_TXFIFO) = 0; SPI1_REG(SPI_REG_RXCTRL) = SPI_RXWM(0); @@ -80,10 +79,7 @@ static void spi_xchg_start(unsigned char cmd, unsigned char *buffer, uint16_t le // before starting a transaction, set SPI peripheral to desired mode SPI1_REG(SPI_REG_CSMODE) = SPI_CSMODE_HOLD; - while (SPI1_REG(SPI_REG_TXFIFO) & SPI_TXFIFO_FULL); SPI1_REG(SPI_REG_TXFIFO) = ((cmd << 3) | (len >> 8)) & 0xFF; - - while (SPI1_REG(SPI_REG_TXFIFO) & SPI_TXFIFO_FULL); SPI1_REG(SPI_REG_TXFIFO) = len & 0xFF; SPI1_REG(SPI_REG_RXCTRL) = SPI_RXWM(1); @@ -108,20 +104,23 @@ static int spi_xchg_next(unsigned char *_buffer) { return 1; } -static void spi_xchg_handler(void) { +static void spi_handler_xchg(void) { volatile uint32_t r1, r2; int i; if (_eos_spi_state_flags & SPI_FLAG_RST) { - while ((r1 = SPI1_REG(SPI_REG_RXFIFO)) & SPI_RXFIFO_EMPTY); + _eos_spi_state_flags &= ~SPI_FLAG_RST; + + r1 = SPI1_REG(SPI_REG_RXFIFO); SPI1_REG(SPI_REG_CSMODE) = SPI_CSMODE_AUTO; SPI1_REG(SPI_REG_IE) = 0x0; - _eos_spi_state_flags &= ~SPI_FLAG_RST; return; } else if (_eos_spi_state_flags & SPI_FLAG_INIT) { - while ((r1 = SPI1_REG(SPI_REG_RXFIFO)) & SPI_RXFIFO_EMPTY); - while ((r2 = SPI1_REG(SPI_REG_RXFIFO)) & SPI_RXFIFO_EMPTY); + _eos_spi_state_flags &= ~SPI_FLAG_INIT; + + r1 = SPI1_REG(SPI_REG_RXFIFO); + r2 = SPI1_REG(SPI_REG_RXFIFO); if (_eos_spi_state_cmd & EOS_NET_CMD_FLAG_ONEW) { r1 = 0; @@ -140,22 +139,19 @@ static void spi_xchg_handler(void) { _eos_spi_state_len = ((_eos_spi_state_len + 2)/4 + 1) * 4 - 2; } - SPI1_REG(SPI_REG_TXCTRL) = SPI_TXWM(SPI_SIZE_TXWM); - SPI1_REG(SPI_REG_RXCTRL) = SPI_RXWM(SPI_SIZE_RXWM); - SPI1_REG(SPI_REG_IE) = SPI_IP_TXWM | SPI_IP_RXWM; - _eos_spi_state_flags &= ~SPI_FLAG_INIT; + SPI1_REG(SPI_REG_TXCTRL) = SPI_TXWM(SPI_SIZE_WM); + SPI1_REG(SPI_REG_IE) = SPI_IP_TXWM; + return; } - if (SPI1_REG(SPI_REG_IP) & SPI_IP_TXWM) { - uint16_t sz_chunk = MIN(_eos_spi_state_len - _eos_spi_state_idx_tx, SPI_SIZE_CHUNK); - for (i=0; i<sz_chunk; i++) { - volatile uint32_t x = SPI1_REG(SPI_REG_TXFIFO); - if (x & SPI_TXFIFO_FULL) break; - SPI1_REG(SPI_REG_TXFIFO) = _eos_spi_state_buf[_eos_spi_state_idx_tx+i]; - } - _eos_spi_state_idx_tx += i; + uint16_t sz_chunk = MIN(_eos_spi_state_len - _eos_spi_state_idx_tx, SPI_SIZE_CHUNK); + for (i=0; i<sz_chunk; i++) { + volatile uint32_t x = SPI1_REG(SPI_REG_TXFIFO); + if (x & SPI_TXFIFO_FULL) break; + SPI1_REG(SPI_REG_TXFIFO) = _eos_spi_state_buf[_eos_spi_state_idx_tx+i]; } + _eos_spi_state_idx_tx += i; for (i=0; i<_eos_spi_state_idx_tx - _eos_spi_state_idx_rx; i++) { volatile uint32_t x = SPI1_REG(SPI_REG_RXFIFO); @@ -177,12 +173,12 @@ static void spi_xchg_handler(void) { spi_bufq_push(_eos_spi_state_buf); } } else if (_eos_spi_state_idx_tx == _eos_spi_state_len) { - SPI1_REG(SPI_REG_RXCTRL) = SPI_RXWM(MIN(_eos_spi_state_len - _eos_spi_state_idx_rx - 1, SPI_SIZE_RXWM)); + SPI1_REG(SPI_REG_RXCTRL) = SPI_RXWM(MIN(_eos_spi_state_len - _eos_spi_state_idx_rx - 1, SPI_SIZE_CHUNK - 1)); SPI1_REG(SPI_REG_IE) = SPI_IP_RXWM; } } -static void spi_cts_hanler(void) { +static void spi_handler_cts(void) { GPIO_REG(GPIO_RISE_IP) = (0x1 << SPI_PIN_CTS); _eos_spi_state_flags |= SPI_FLAG_CTS; @@ -194,7 +190,7 @@ static void spi_cts_hanler(void) { } } -static void spi_rts_hanler(void) { +static void spi_handler_rts(void) { uint32_t rts_offset = (0x1 << SPI_PIN_RTS); if (GPIO_REG(GPIO_RISE_IP) & rts_offset) { GPIO_REG(GPIO_RISE_IP) = rts_offset; @@ -238,20 +234,20 @@ void eos_net_init(void) { eos_msgq_init(&_eos_spi_send_q, spi_sndq_array, SPI_SIZE_BUFQ); GPIO_REG(GPIO_IOF_SEL) &= ~SPI_IOF_MASK; GPIO_REG(GPIO_IOF_EN) |= SPI_IOF_MASK; - eos_intr_set(INT_SPI1_BASE, 5, spi_xchg_handler); + eos_intr_set(INT_SPI1_BASE, 5, spi_handler_xchg); GPIO_REG(GPIO_OUTPUT_EN) &= ~(0x1 << SPI_PIN_CTS); GPIO_REG(GPIO_PULLUP_EN) |= (0x1 << SPI_PIN_CTS); GPIO_REG(GPIO_INPUT_EN) |= (0x1 << SPI_PIN_CTS); GPIO_REG(GPIO_RISE_IE) |= (0x1 << SPI_PIN_CTS); - eos_intr_set(INT_GPIO_BASE + SPI_PIN_CTS, 4, spi_cts_hanler); + eos_intr_set(INT_GPIO_BASE + SPI_PIN_CTS, 4, spi_handler_cts); GPIO_REG(GPIO_OUTPUT_EN) &= ~(0x1 << SPI_PIN_RTS); GPIO_REG(GPIO_PULLUP_EN) |= (0x1 << SPI_PIN_RTS); GPIO_REG(GPIO_INPUT_EN) |= (0x1 << SPI_PIN_RTS); GPIO_REG(GPIO_RISE_IE) |= (0x1 << SPI_PIN_RTS); GPIO_REG(GPIO_FALL_IE) |= (0x1 << SPI_PIN_RTS); - eos_intr_set(INT_GPIO_BASE + SPI_PIN_RTS, 4, spi_rts_hanler); + eos_intr_set(INT_GPIO_BASE + SPI_PIN_RTS, 4, spi_handler_rts); for (i=0; i<EOS_NET_MAX_CMD; i++) { evt_handler[i] = eos_evtq_bad_handler; diff --git a/code/fe310/eos/spi_def.h b/code/fe310/eos/spi_def.h index 48bcfab..4c558fb 100644 --- a/code/fe310/eos/spi_def.h +++ b/code/fe310/eos/spi_def.h @@ -7,8 +7,7 @@ #define SPI_MODE3 0x03 #define SPI_SIZE_CHUNK 4 -#define SPI_SIZE_TXWM 2 -#define SPI_SIZE_RXWM 3 +#define SPI_SIZE_WM 4 #define SPI_PIN_RTS 0 // pin 8 #define SPI_PIN_CTS 23 // pin 7 diff --git a/code/fe310/eos/trap_entry.S b/code/fe310/eos/trap_entry.S index 25412d8..687493d 100644 --- a/code/fe310/eos/trap_entry.S +++ b/code/fe310/eos/trap_entry.S @@ -17,6 +17,13 @@ #define GPIO_CTRL_ADDR 0x10012000 #include "sifive/devices/gpio.h" +#define SPI1_CTRL_ADDR 0x10024000 +#include "sifive/devices/spi.h" + +#include "evt_def.h" +#include "msgq_def.h" +#include "i2s_def.h" + #define INT_PWM0_BASE 40 #define INT_PWM1_BASE 44 #define INT_PWM2_BASE 48 @@ -24,9 +31,13 @@ #define I2S_PWM_CTRL_ADDR_CK PWM0_CTRL_ADDR #define I2S_PWM_CTRL_ADDR_WS PWM2_CTRL_ADDR -#include "i2s_def.h" -#include "evt_def.h" -#include "msgq_def.h" +#include "net_def.h" +#include "spi_def.h" + +#define INT_SPI1_BASE 6 +#define INT_GPIO_BASE 8 + +#define IOF_SPI1_SS2 9 .section .data.entry .align 2 @@ -76,6 +87,12 @@ eos_trap_entry: beq x9, x18, i2s_handler_ws li x18, I2S_IRQ_CI_ID beq x9, x18, i2s_handler_ci + li x18, INT_SPI1_BASE + beq x9, x18, spi_handler_xchg + li x18, INT_GPIO_BASE + SPI_PIN_CTS + beq x9, x18, spi_handler_cts + li x18, INT_GPIO_BASE + SPI_PIN_RTS + beq x9, x18, spi_handler_rts j handler i2s_handler_sd: @@ -376,13 +393,6 @@ i2s_handler_ci_exit: # exit j trap_exit_data -#define SPI1_CTRL_ADDR 0x10024000 -#define IOF_SPI1_SS2 9 - -#include "sifive/devices/spi.h" -#include "net_def.h" -#include "spi_def.h" - # x9 - cmd, x18 - buffer, x19 - len, x20 - &flags, x21 - flags spi_xchg_start: ori x21, x21, SPI_FLAG_INIT @@ -461,11 +471,9 @@ spi_handler_xchg: andi x20, x20, ~SPI_FLAG_INIT sb x20, 0(x18) - li x8, SPI_SIZE_TXWM + li x8, SPI_SIZE_WM sw x8, SPI_REG_TXCTRL(x19) - li x8, SPI_SIZE_RXWM - sw x8, SPI_REG_RXCTRL(x19) - li x8, (SPI_IP_TXWM | SPI_IP_RXWM) + li x8, SPI_IP_TXWM sw x8, SPI_REG_IE(x19) lw x8, SPI_REG_RXFIFO(x19) lw x9, SPI_REG_RXFIFO(x19) @@ -477,7 +485,6 @@ spi_handler_xchg: beqz x23, 2f mv x8, x0 mv x9, x0 - 2: srli x23, x8, 3 sb x23, 0(x18) @@ -490,12 +497,12 @@ spi_handler_xchg: lw x21, 0(x18) bgeu x21, x22, 3f mv x21, x22 + 3: li x8, 6 bgeu x21, x8, 4f mv x21, x8 j 5f - 4: addi x8, x21, 2 li x9, 4 @@ -521,10 +528,6 @@ spi_handler_xchg: la x18, _eos_spi_state_buf lw x23, 0(x18) - lw x8, SPI_REG_IP(x19) - andi x8, x8, SPI_IP_TXWM - beqz x8, 9f - sub x9, x20, x21 li x8, SPI_SIZE_CHUNK bltu x8, x9, 7f @@ -545,10 +548,6 @@ spi_handler_xchg: j 8b 9: - # lw x8, SPI_REG_IP(x19) - # andi x8, x8, SPI_IP_RXWM - # beqz x8, 11f - sub x8, x21, x22 addi x8, x8, -1 add x9, x23, x22 @@ -597,16 +596,17 @@ spi_handler_xchg: lbu x9, 0(x19) beqz x9, 14f 13: - bnez x23, 14f la x19, _eos_spi_state_next_buf - sw x23, 0(x19) + lw x9, 0(x19) + bnez x9, 14f andi x8, x8, ~SPI_FLAG_ONEW sb x8, 0(x18) + sw x23, 0(x19) j spi_handler_xchg_exit 14: - # push spi bufq + # push to spi buf queue la x19, _eos_spi_buf_q lbu x8, SPI_BUFQ_OFF_IDXW(x19) @@ -624,7 +624,7 @@ spi_handler_xchg: bne x21, x20, spi_handler_xchg_exit sub x8, x20, x22 addi x8, x8, -1 - li x9, SPI_SIZE_RXWM + li x9, SPI_SIZE_CHUNK - 1 bltu x8, x9, 16f mv x8, x9 16: @@ -639,10 +639,10 @@ spi_handler_xchg_exit: LOAD x23, 3*REGBYTES(sp) addi sp, sp, 4*REGBYTES - # XXX should complete! - # li x18, I2S_IRQ_SD_ID - # li x19, PLIC_CLAIM - # sw x18, 0(x19) + # complete + li x18, INT_SPI1_BASE + li x19, PLIC_CLAIM + sw x18, 0(x19) # exit j trap_exit_data @@ -685,9 +685,7 @@ spi_handler_cts: lbu x9, MSGQ_ITEM_OFF_CMD(x8) lw x18, MSGQ_ITEM_OFF_BUF(x8) lhu x19, MSGQ_ITEM_OFF_SIZE(x8) - beqz x9, 1f beqz x18, 1f - beqz x19, 1f jal x8, spi_xchg_start j spi_handler_cts_exit @@ -726,10 +724,10 @@ spi_handler_cts_exit: LOAD x23, 3*REGBYTES(sp) addi sp, sp, 4*REGBYTES - # XXX should complete! - # li x18, I2S_IRQ_SD_ID - # li x19, PLIC_CLAIM - # sw x18, 0(x19) + # complete + li x18, INT_GPIO_BASE + SPI_PIN_CTS + li x19, PLIC_CLAIM + sw x18, 0(x19) # exit j trap_exit_data @@ -774,20 +772,19 @@ spi_handler_rts: beqz x9, spi_handler_rts_exit sw x8, GPIO_FALL_IP(x18) - lbu x8, 0(x18) + lbu x8, 0(x19) andi x8, x8, ~SPI_FLAG_RTS sb x8, 0(x19) spi_handler_rts_exit: - # XXX should complete! - # li x18, I2S_IRQ_SD_ID - # li x19, PLIC_CLAIM - # sw x18, 0(x19) + # complete + li x18, INT_GPIO_BASE + SPI_PIN_RTS + li x19, PLIC_CLAIM + sw x18, 0(x19) # exit j trap_exit_data - - + trap_exit_data: # Remain in M-mode after mret li x18, MSTATUS_MPP |