From 0c69d0a7f7c78b306cbf22de58d9b63eba856350 Mon Sep 17 00:00:00 2001 From: Jared Boone Date: Tue, 4 Nov 2014 12:35:38 -0800 Subject: [PATCH 05/68] RFFC5071: Separate hardware-specific code. Conflicts: firmware/CMakeLists.txt firmware/hackrf_usb/Makefile firmware/mixertx/Makefile firmware/sgpio/CMakeLists.txt firmware/simpletx/Makefile firmware/spiflash/CMakeLists.txt firmware/startup/Makefile firmware/startup_systick/Makefile firmware/startup_systick_perfo/Makefile --- firmware/common/rffc5071.c | 200 +---------------------------------- firmware/common/rffc5071_drv.c | 229 +++++++++++++++++++++++++++++++++++++++++ firmware/common/rffc5071_drv.h | 31 ++++++ firmware/hackrf-common.cmake | 1 + 4 files changed, 264 insertions(+), 197 deletions(-) create mode 100644 firmware/common/rffc5071_drv.c create mode 100644 firmware/common/rffc5071_drv.h --- a/firmware/common/rffc5071.c +++ b/firmware/common/rffc5071.c @@ -34,6 +34,7 @@ #include #include #include "rffc5071.h" +#include "rffc5071_drv.h" #include "rffc5071_regs.def" // private register def macros #if (defined DEBUG) @@ -41,9 +42,6 @@ #define LOG printf #else #define LOG(x,...) -#include -#include -#include #include "hackrf_core.h" #endif @@ -105,25 +103,8 @@ { rffc5071_init(); LOG("# rffc5071_setup\n"); -#if !defined TEST - /* Configure GPIO pins. */ - scu_pinmux(SCU_MIXER_ENX, SCU_GPIO_FAST); - scu_pinmux(SCU_MIXER_SCLK, SCU_GPIO_FAST | SCU_CONF_FUNCTION4); - scu_pinmux(SCU_MIXER_SDATA, SCU_GPIO_FAST); - scu_pinmux(SCU_MIXER_RESETX, SCU_GPIO_FAST); - - /* Set GPIO pins as outputs. */ - GPIO_DIR(PORT_MIXER_ENX) |= PIN_MIXER_ENX; - GPIO_DIR(PORT_MIXER_SCLK) |= PIN_MIXER_SCLK; - GPIO_DIR(PORT_MIXER_SDATA) |= PIN_MIXER_SDATA; - GPIO_DIR(PORT_MIXER_RESETX) |= PIN_MIXER_RESETX; - - /* set to known state */ - gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); /* active low */ - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - gpio_set(PORT_MIXER_RESETX, PIN_MIXER_RESETX); /* active low */ -#endif + + rffc5071_pin_config(); /* initial setup */ /* put zeros in freq contol registers */ @@ -152,181 +133,6 @@ rffc5071_regs_commit(); } -void serial_delay(void) -{ - uint32_t i; - - for (i = 0; i < 2; i++) - __asm__("nop"); -} - -/* SPI register read. - * - * Send 9 bits: - * first bit is ignored, - * second bit is one for read operation, - * next 7 bits are register address. - * Then receive 16 bits (register value). - */ -uint16_t rffc5071_spi_read(uint8_t r) { - - int bits = 9; - int msb = 1 << (bits -1); - uint32_t data = 0x80 | (r & 0x7f); - -#if DEBUG - LOG("reg%d = 0\n", r); - return 0; -#else - /* make sure everything is starting in the correct state */ - gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - - /* - * The device requires two clocks while ENX is high before a serial - * transaction. This is not clearly documented. - */ - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - /* start transaction by bringing ENX low */ - gpio_clear(PORT_MIXER_ENX, PIN_MIXER_ENX); - - while (bits--) { - if (data & msb) - gpio_set(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - else - gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - data <<= 1; - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - } - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - bits = 16; - data = 0; - /* set SDATA line as input */ - GPIO_DIR(PORT_MIXER_SDATA) &= ~PIN_MIXER_SDATA; - - while (bits--) { - data <<= 1; - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - if (MIXER_SDATA_STATE) - data |= 1; - } - /* set SDATA line as output */ - GPIO_DIR(PORT_MIXER_SDATA) |= PIN_MIXER_SDATA; - - serial_delay(); - gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); - - /* - * The device requires a clock while ENX is high after a serial - * transaction. This is not clearly documented. - */ - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - return data; -#endif /* DEBUG */ -} - -/* SPI register write - * - * Send 25 bits: - * first bit is ignored, - * second bit is zero for write operation, - * next 7 bits are register address, - * next 16 bits are register value. - */ -void rffc5071_spi_write(uint8_t r, uint16_t v) { - -#if DEBUG - LOG("0x%04x -> reg%d\n", v, r); -#else - - int bits = 25; - int msb = 1 << (bits -1); - uint32_t data = ((r & 0x7f) << 16) | v; - - /* make sure everything is starting in the correct state */ - gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - - /* - * The device requires two clocks while ENX is high before a serial - * transaction. This is not clearly documented. - */ - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - /* start transaction by bringing ENX low */ - gpio_clear(PORT_MIXER_ENX, PIN_MIXER_ENX); - - while (bits--) { - if (data & msb) - gpio_set(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - else - gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); - data <<= 1; - - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - } - - gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); - - /* - * The device requires a clock while ENX is high after a serial - * transaction. This is not clearly documented. - */ - serial_delay(); - gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); - - serial_delay(); - gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); -#endif -} - uint16_t rffc5071_reg_read(uint8_t r) { /* Readback register is not cached. */ --- /dev/null +++ b/firmware/common/rffc5071_drv.c @@ -0,0 +1,229 @@ +/* + * Copyright 2012 Michael Ossmann + * Copyright 2014 Jared Boone + * + * This file is part of HackRF. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2, or (at your option) + * any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; see the file COPYING. If not, write to + * the Free Software Foundation, Inc., 51 Franklin Street, + * Boston, MA 02110-1301, USA. + */ + +#if (defined DEBUG) +#include +#define LOG printf +#else +#define LOG(x,...) +#include +#include +#include +#include "hackrf_core.h" +#endif + +void rffc5071_pin_config() { +#if !defined TEST + /* Configure GPIO pins. */ + scu_pinmux(SCU_MIXER_ENX, SCU_GPIO_FAST); + scu_pinmux(SCU_MIXER_SCLK, SCU_GPIO_FAST | SCU_CONF_FUNCTION4); + scu_pinmux(SCU_MIXER_SDATA, SCU_GPIO_FAST); + scu_pinmux(SCU_MIXER_RESETX, SCU_GPIO_FAST); + + /* Set GPIO pins as outputs. */ + GPIO_DIR(PORT_MIXER_ENX) |= PIN_MIXER_ENX; + GPIO_DIR(PORT_MIXER_SCLK) |= PIN_MIXER_SCLK; + GPIO_DIR(PORT_MIXER_SDATA) |= PIN_MIXER_SDATA; + GPIO_DIR(PORT_MIXER_RESETX) |= PIN_MIXER_RESETX; + + /* set to known state */ + gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); /* active low */ + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + gpio_set(PORT_MIXER_RESETX, PIN_MIXER_RESETX); /* active low */ +#endif +} + +static void serial_delay(void) +{ + uint32_t i; + + for (i = 0; i < 2; i++) + __asm__("nop"); +} + +/* SPI register read. + * + * Send 9 bits: + * first bit is ignored, + * second bit is one for read operation, + * next 7 bits are register address. + * Then receive 16 bits (register value). + */ +uint16_t rffc5071_spi_read(uint8_t r) { + + int bits = 9; + int msb = 1 << (bits -1); + uint32_t data = 0x80 | (r & 0x7f); + +#if DEBUG + LOG("reg%d = 0\n", r); + return 0; +#else + /* make sure everything is starting in the correct state */ + gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + + /* + * The device requires two clocks while ENX is high before a serial + * transaction. This is not clearly documented. + */ + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + /* start transaction by bringing ENX low */ + gpio_clear(PORT_MIXER_ENX, PIN_MIXER_ENX); + + while (bits--) { + if (data & msb) + gpio_set(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + else + gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + data <<= 1; + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + } + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + bits = 16; + data = 0; + /* set SDATA line as input */ + GPIO_DIR(PORT_MIXER_SDATA) &= ~PIN_MIXER_SDATA; + + while (bits--) { + data <<= 1; + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + if (MIXER_SDATA_STATE) + data |= 1; + } + /* set SDATA line as output */ + GPIO_DIR(PORT_MIXER_SDATA) |= PIN_MIXER_SDATA; + + serial_delay(); + gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); + + /* + * The device requires a clock while ENX is high after a serial + * transaction. This is not clearly documented. + */ + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + return data; +#endif /* DEBUG */ +} + +/* SPI register write + * + * Send 25 bits: + * first bit is ignored, + * second bit is zero for write operation, + * next 7 bits are register address, + * next 16 bits are register value. + */ +void rffc5071_spi_write(uint8_t r, uint16_t v) { + +#if DEBUG + LOG("0x%04x -> reg%d\n", v, r); +#else + + int bits = 25; + int msb = 1 << (bits -1); + uint32_t data = ((r & 0x7f) << 16) | v; + + /* make sure everything is starting in the correct state */ + gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + + /* + * The device requires two clocks while ENX is high before a serial + * transaction. This is not clearly documented. + */ + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + /* start transaction by bringing ENX low */ + gpio_clear(PORT_MIXER_ENX, PIN_MIXER_ENX); + + while (bits--) { + if (data & msb) + gpio_set(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + else + gpio_clear(PORT_MIXER_SDATA, PIN_MIXER_SDATA); + data <<= 1; + + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + } + + gpio_set(PORT_MIXER_ENX, PIN_MIXER_ENX); + + /* + * The device requires a clock while ENX is high after a serial + * transaction. This is not clearly documented. + */ + serial_delay(); + gpio_set(PORT_MIXER_SCLK, PIN_MIXER_SCLK); + + serial_delay(); + gpio_clear(PORT_MIXER_SCLK, PIN_MIXER_SCLK); +#endif +} --- /dev/null +++ b/firmware/common/rffc5071_drv.h @@ -0,0 +1,31 @@ +/* + * Copyright 2012 Michael Ossmann + * Copyright 2014 Jared Boone + * + * This file is part of HackRF. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2, or (at your option) + * any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; see the file COPYING. If not, write to + * the Free Software Foundation, Inc., 51 Franklin Street, + * Boston, MA 02110-1301, USA. + */ + +#ifndef __RFFC5071_DRV_H +#define __RFFC5071_DRV_H + +void rffc5071_pin_config(); + +uint16_t rffc5071_spi_read(uint8_t r); +void rffc5071_spi_write(uint8_t r, uint16_t v); + +#endif // __RFFC5071_DRV_H --- a/firmware/hackrf-common.cmake +++ b/firmware/hackrf-common.cmake @@ -139,6 +139,7 @@ ${PATH_HACKRF_FIRMWARE_COMMON}/max2837_drv.c ${PATH_HACKRF_FIRMWARE_COMMON}/max5864.c ${PATH_HACKRF_FIRMWARE_COMMON}/rffc5071.c + ${PATH_HACKRF_FIRMWARE_COMMON}/rffc5071_drv.c m0_bin.s )