diff options
author | Nick Foster <nick@ettus.com> | 2011-08-09 13:59:32 -0700 |
---|---|---|
committer | Josh Blum <josh@joshknows.com> | 2011-08-09 17:49:43 -0700 |
commit | a2ff19e38282e8ad641b9bbabbb07af315864548 (patch) | |
tree | d4c696f1d5e44857055f9be1af2783646a2f7dc2 /firmware/zpu/usrp2p | |
parent | d44279498462911c7ea698ac5924da6025f2112a (diff) | |
download | uhd-a2ff19e38282e8ad641b9bbabbb07af315864548.tar.gz uhd-a2ff19e38282e8ad641b9bbabbb07af315864548.tar.bz2 uhd-a2ff19e38282e8ad641b9bbabbb07af315864548.zip |
N2XX: fix bug in firmware updater (don't use local variables for static data). this fixes the "n210 bricking on update while streaming" bug.
Diffstat (limited to 'firmware/zpu/usrp2p')
-rw-r--r-- | firmware/zpu/usrp2p/spi_flash.c | 5 | ||||
-rw-r--r-- | firmware/zpu/usrp2p/udp_fw_update.c | 4 |
2 files changed, 6 insertions, 3 deletions
diff --git a/firmware/zpu/usrp2p/spi_flash.c b/firmware/zpu/usrp2p/spi_flash.c index 2033b8035..9406f8042 100644 --- a/firmware/zpu/usrp2p/spi_flash.c +++ b/firmware/zpu/usrp2p/spi_flash.c @@ -50,7 +50,7 @@ spi_flash_wait(void) void spi_flash_erase_sector_start(uint32_t flash_addr) { - //uprintf(UART_DEBUG, "spi_flash_erase_sector_start: addr = 0x%x\n", flash_addr); + //printf("spi_flash_erase_sector_start: addr = 0x%x\n", flash_addr); if(flash_addr > spi_flash_memory_size()) return; @@ -166,6 +166,9 @@ void spi_flash_async_erase_start(spi_flash_async_state_t *s, uint32_t flash_addr, size_t nbytes) { + + //printf("got command to erase %d bytes at 0x%x\n", nbytes, flash_addr); + if ((nbytes == 0) || ((flash_addr + nbytes) > spi_flash_memory_size())){ s->first = s->last = s->current = 0; return; diff --git a/firmware/zpu/usrp2p/udp_fw_update.c b/firmware/zpu/usrp2p/udp_fw_update.c index 68105aab3..793011651 100644 --- a/firmware/zpu/usrp2p/udp_fw_update.c +++ b/firmware/zpu/usrp2p/udp_fw_update.c @@ -35,6 +35,8 @@ uint16_t get_hw_rev(void) { return tmp; } +spi_flash_async_state_t spi_flash_async_state; + //Firmware update packet handler void handle_udp_fw_update_packet(struct socket_address src, struct socket_address dst, unsigned char *payload, int payload_len) { @@ -60,8 +62,6 @@ void handle_udp_fw_update_packet(struct socket_address src, struct socket_addres update_data_in_id = USRP2_FW_UPDATE_ID_WAT; } - spi_flash_async_state_t spi_flash_async_state; - switch(update_data_in_id) { case USRP2_FW_UPDATE_ID_OHAI_LOL: //why hello there you handsome devil update_data_out.id = USRP2_FW_UPDATE_ID_OHAI_OMG; |