361c4ac54f
This is a defensive change to make the transceiver code easier to reason about, and to avoid the possibility of races such as that seen in #1042. Previously, set_transceiver_mode() was called in the vendor request handler for the SET_TRANSCEIVER_MODE request, as well in the callback for a USB configuration change. Both these calls are made from the USB0 ISR, so could interrupt the rx_mode(), tx_mode() and sweep_mode() functions at any point. It was hard to tell if this was safe. Instead, set_transceiver_mode() has been removed, and its work is split into three parts: - request_transceiver_mode(), which is safe to call from ISR context. All this function does is update the requested mode and increment a sequence number. This builds on work already done in PR #1029, but the interface has been simplified to use a shared volatile structure. - transceiver_startup(), which transitions the transceiver from an idle state to the configuration required for a specific mode, including setting up the RF path, configuring the M0, adjusting LEDs and UI etc. - transceiver_shutdown(), which transitions the transceiver back to an idle state. The *_mode() functions that implement the transceiver modes now call transceiver_startup() before starting work, and transceiver_shutdown() before returning, and all this happens in the main thread of execution. As such, it is now guaranteed that all the steps involved happen in a consistent order, with the transceiver starting from an idle state, and being returned to an idle state before control returns to the main loop. For consistency of interface, an off_mode() function has been added to implement the behaviour of the OFF transceiver mode. Since the transceiver is already guaranteed to be in an idle state when this is called, the only work required is to set the UI mode and wait for a new mode request.
75 lines
2.8 KiB
C
75 lines
2.8 KiB
C
/*
|
|
* Copyright 2012 Jared Boone
|
|
* Copyright 2013 Benjamin Vernoux
|
|
*
|
|
* 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 __USB_API_TRANSCEIVER_H__
|
|
#define __USB_API_TRANSCEIVER_H__
|
|
|
|
#include <hackrf_core.h>
|
|
#include <usb_type.h>
|
|
#include <usb_request.h>
|
|
|
|
typedef struct {
|
|
transceiver_mode_t mode;
|
|
uint32_t seq;
|
|
} transceiver_request_t;
|
|
|
|
extern volatile transceiver_request_t transceiver_request;
|
|
|
|
void set_hw_sync_mode(const hw_sync_mode_t new_hw_sync_mode);
|
|
usb_request_status_t usb_vendor_request_set_transceiver_mode(
|
|
usb_endpoint_t* const endpoint,
|
|
const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_baseband_filter_bandwidth(
|
|
usb_endpoint_t* const endpoint,
|
|
const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_freq(
|
|
usb_endpoint_t* const endpoint,
|
|
const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_sample_rate_frac(
|
|
usb_endpoint_t* const endpoint,
|
|
const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_amp_enable(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_lna_gain(
|
|
usb_endpoint_t* const endpoint,
|
|
const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_vga_gain(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_txvga_gain(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_antenna_enable(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_freq_explicit(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
usb_request_status_t usb_vendor_request_set_hw_sync_mode(
|
|
usb_endpoint_t* const endpoint, const usb_transfer_stage_t stage);
|
|
|
|
void request_transceiver_mode(transceiver_mode_t mode);
|
|
void transceiver_startup(transceiver_mode_t mode);
|
|
void transceiver_shutdown(void);
|
|
void start_streaming_on_hw_sync();
|
|
void rx_mode(uint32_t seq);
|
|
void tx_mode(uint32_t seq);
|
|
void off_mode(uint32_t seq);
|
|
|
|
#endif/*__USB_API_TRANSCEIVER_H__*/
|