From e80890e1649c230cfc94d169a8161b93ca768f9a Mon Sep 17 00:00:00 2001 From: Oleg Semyonov Date: Fri, 15 Mar 2013 20:42:54 +0200 Subject: [PATCH] pios: STM32F4xx PIOS #ifdef guards --- flight/PiOS/STM32F4xx/pios_adc.c | 4 +++- flight/PiOS/STM32F4xx/pios_bl_helper.c | 8 +++++--- flight/PiOS/STM32F4xx/pios_debug.c | 1 - flight/PiOS/STM32F4xx/pios_delay.c | 5 ++--- flight/PiOS/STM32F4xx/pios_dsm.c | 10 +++++----- flight/PiOS/STM32F4xx/pios_exti.c | 3 +-- flight/PiOS/STM32F4xx/pios_flash_internal.c | 4 ++-- flight/PiOS/STM32F4xx/pios_gpio.c | 5 ++--- flight/PiOS/STM32F4xx/pios_i2c.c | 7 +++---- flight/PiOS/STM32F4xx/pios_iap.c | 4 ++++ flight/PiOS/STM32F4xx/pios_irq.c | 5 ++--- flight/PiOS/STM32F4xx/pios_led.c | 5 ++--- flight/PiOS/STM32F4xx/pios_overo.c | 8 ++++++-- flight/PiOS/STM32F4xx/pios_ppm.c | 8 ++++---- flight/PiOS/STM32F4xx/pios_pwm.c | 8 ++++---- flight/PiOS/STM32F4xx/pios_rtc.c | 7 ++++--- flight/PiOS/STM32F4xx/pios_servo.c | 6 +++++- flight/PiOS/STM32F4xx/pios_spi.c | 4 ++-- flight/PiOS/STM32F4xx/pios_sys.c | 5 ++--- flight/PiOS/STM32F4xx/pios_tim.c | 5 ++++- flight/PiOS/STM32F4xx/pios_usart.c | 5 ++--- flight/PiOS/STM32F4xx/pios_usb.c | 8 ++++---- flight/PiOS/STM32F4xx/pios_usb_cdc.c | 6 ++---- flight/PiOS/STM32F4xx/pios_usb_hid.c | 6 ++---- flight/PiOS/STM32F4xx/pios_wdg.c | 5 +++++ 25 files changed, 77 insertions(+), 65 deletions(-) diff --git a/flight/PiOS/STM32F4xx/pios_adc.c b/flight/PiOS/STM32F4xx/pios_adc.c index a1320a1be..7131e440a 100644 --- a/flight/PiOS/STM32F4xx/pios_adc.c +++ b/flight/PiOS/STM32F4xx/pios_adc.c @@ -43,9 +43,11 @@ */ #include "pios.h" + +#ifdef PIOS_INCLUDE_ADC + #include -#if defined(PIOS_INCLUDE_ADC) #if !defined(PIOS_ADC_MAX_SAMPLES) #define PIOS_ADC_MAX_SAMPLES 0 diff --git a/flight/PiOS/STM32F4xx/pios_bl_helper.c b/flight/PiOS/STM32F4xx/pios_bl_helper.c index 05de4a305..afa59b3e5 100644 --- a/flight/PiOS/STM32F4xx/pios_bl_helper.c +++ b/flight/PiOS/STM32F4xx/pios_bl_helper.c @@ -28,9 +28,10 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_BL_HELPER) + +#ifdef PIOS_INCLUDE_BL_HELPER + #include #include "stm32f4xx_flash.h" #include @@ -192,4 +193,5 @@ void PIOS_BL_HELPER_FLASH_Read_Description(uint8_t * array, uint8_t size) void PIOS_BL_HELPER_CRC_Ini() { } -#endif + +#endif /* PIOS_INCLUDE_BL_HELPER */ diff --git a/flight/PiOS/STM32F4xx/pios_debug.c b/flight/PiOS/STM32F4xx/pios_debug.c index 339b9d427..0e607cf47 100644 --- a/flight/PiOS/STM32F4xx/pios_debug.c +++ b/flight/PiOS/STM32F4xx/pios_debug.c @@ -28,7 +28,6 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" // Global variables diff --git a/flight/PiOS/STM32F4xx/pios_delay.c b/flight/PiOS/STM32F4xx/pios_delay.c index 947998c3f..5faeef2c7 100644 --- a/flight/PiOS/STM32F4xx/pios_delay.c +++ b/flight/PiOS/STM32F4xx/pios_delay.c @@ -30,10 +30,9 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include -#if defined(PIOS_INCLUDE_DELAY) +#ifdef PIOS_INCLUDE_DELAY /* these should be defined by CMSIS, but they aren't */ #define DWT_CTRL (*(volatile uint32_t *)0xe0001000) @@ -167,7 +166,7 @@ uint32_t PIOS_DELAY_DiffuS(uint32_t raw) return diff / us_ticks; } -#endif +#endif /* PIOS_INCLUDE_DELAY */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_dsm.c b/flight/PiOS/STM32F4xx/pios_dsm.c index 7c5348560..4bcc0187f 100644 --- a/flight/PiOS/STM32F4xx/pios_dsm.c +++ b/flight/PiOS/STM32F4xx/pios_dsm.c @@ -28,13 +28,13 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" + +#ifdef PIOS_INCLUDE_DSM + #include "pios_dsm_priv.h" -#if defined(PIOS_INCLUDE_DSM) - -#if !defined(PIOS_INCLUDE_RTC) +#ifndef PIOS_INCLUDE_RTC #error PIOS_INCLUDE_RTC must be used to use DSM #endif @@ -401,7 +401,7 @@ static void PIOS_DSM_Supervisor(uint32_t dsm_id) } } -#endif /* PIOS_INCLUDE_DSM */ +#endif /* PIOS_INCLUDE_DSM */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_exti.c b/flight/PiOS/STM32F4xx/pios_exti.c index 45efb2ae5..3dc1e1158 100644 --- a/flight/PiOS/STM32F4xx/pios_exti.c +++ b/flight/PiOS/STM32F4xx/pios_exti.c @@ -29,10 +29,9 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_EXTI) +#ifdef PIOS_INCLUDE_EXTI /* Map EXTI line to full config */ #define EXTI_MAX_LINES 16 diff --git a/flight/PiOS/STM32F4xx/pios_flash_internal.c b/flight/PiOS/STM32F4xx/pios_flash_internal.c index 534b911fd..fceae8038 100644 --- a/flight/PiOS/STM32F4xx/pios_flash_internal.c +++ b/flight/PiOS/STM32F4xx/pios_flash_internal.c @@ -26,7 +26,7 @@ #include "pios.h" -#if defined(PIOS_INCLUDE_FLASH_INTERNAL) +#ifdef PIOS_INCLUDE_FLASH_INTERNAL #include "stm32f4xx_flash.h" #include "pios_flash_internal_priv.h" @@ -334,4 +334,4 @@ const struct pios_flash_driver pios_internal_flash_driver = { .read_data = PIOS_Flash_Internal_ReadData, }; -#endif /* defined(PIOS_INCLUDE_FLASH_INTERNAL) */ +#endif /* PIOS_INCLUDE_FLASH_INTERNAL */ diff --git a/flight/PiOS/STM32F4xx/pios_gpio.c b/flight/PiOS/STM32F4xx/pios_gpio.c index 88f342a74..18c083d9b 100644 --- a/flight/PiOS/STM32F4xx/pios_gpio.c +++ b/flight/PiOS/STM32F4xx/pios_gpio.c @@ -28,10 +28,9 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_GPIO) +#ifdef PIOS_INCLUDE_GPIO /* Private Function Prototypes */ @@ -92,7 +91,7 @@ void PIOS_GPIO_Toggle(uint8_t Pin) GPIO_ToggleBits(GPIO_PORT[Pin], GPIO_PIN[Pin]); } -#endif +#endif /* PIOS_INCLUDE_GPIO */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_i2c.c b/flight/PiOS/STM32F4xx/pios_i2c.c index bb2a1bfc1..9ceb72337 100644 --- a/flight/PiOS/STM32F4xx/pios_i2c.c +++ b/flight/PiOS/STM32F4xx/pios_i2c.c @@ -33,12 +33,11 @@ * be merged. */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_I2C) +#ifdef PIOS_INCLUDE_I2C -#if defined(PIOS_INCLUDE_FREERTOS) +#ifdef PIOS_INCLUDE_FREERTOS #define USE_FREERTOS #endif @@ -1232,7 +1231,7 @@ void PIOS_I2C_ER_IRQ_Handler(uint32_t i2c_id) } } -#endif +#endif /* PIOS_INCLUDE_I2C */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_iap.c b/flight/PiOS/STM32F4xx/pios_iap.c index ca4ef89ba..e08ccd09a 100644 --- a/flight/PiOS/STM32F4xx/pios_iap.c +++ b/flight/PiOS/STM32F4xx/pios_iap.c @@ -33,6 +33,8 @@ ****************************************************************************************/ #include +#ifdef PIOS_INCLUDE_IAP + /**************************************************************************************** * Private Definitions/Macros ****************************************************************************************/ @@ -140,3 +142,5 @@ void PIOS_IAP_WriteBootCount (uint16_t boot_count) { RTC_WriteBackupRegister ( IAP_BOOTCOUNT, boot_count ); } + +#endif /* PIOS_INCLUDE_IAP */ diff --git a/flight/PiOS/STM32F4xx/pios_irq.c b/flight/PiOS/STM32F4xx/pios_irq.c index b556931bc..d14120829 100644 --- a/flight/PiOS/STM32F4xx/pios_irq.c +++ b/flight/PiOS/STM32F4xx/pios_irq.c @@ -32,10 +32,9 @@ * @todo This should be shared with the F1xx code. */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_IRQ) +#ifdef PIOS_INCLUDE_IRQ /* Private Function Prototypes */ @@ -93,7 +92,7 @@ int32_t PIOS_IRQ_Enable(void) return 0; } -#endif +#endif /* PIOS_INCLUDE_IRQ */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_led.c b/flight/PiOS/STM32F4xx/pios_led.c index 1f7fd6365..7ff5dec42 100644 --- a/flight/PiOS/STM32F4xx/pios_led.c +++ b/flight/PiOS/STM32F4xx/pios_led.c @@ -28,10 +28,9 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_LED) +#ifdef PIOS_INCLUDE_LED #include @@ -166,7 +165,7 @@ void PIOS_LED_Toggle(uint32_t led_id) } } -#endif +#endif /* PIOS_INCLUDE_LED */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_overo.c b/flight/PiOS/STM32F4xx/pios_overo.c index c535d413e..2de75591a 100644 --- a/flight/PiOS/STM32F4xx/pios_overo.c +++ b/flight/PiOS/STM32F4xx/pios_overo.c @@ -31,6 +31,8 @@ #include +#ifdef PIOS_INCLUDE_OVERO + /** * Configures the SPI device to use a double buffered DMA for transferring * data. At the end of each transfer (NSS goes high) it makes sure to reset @@ -38,7 +40,7 @@ * buffer */ -#if defined(PIOS_INCLUDE_SPI) +#ifdef PIOS_INCLUDE_SPI #include @@ -368,7 +370,9 @@ static void PIOS_OVERO_RxStart(uint32_t overo_id, uint16_t rx_bytes_avail) {}; #endif /* PIOS_INCLUDE_FREERTOS */ -#endif +#endif /* PIOS_INCLUDE_SPI */ + +#endif /* PIOS_INCLUDE_OVERO */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_ppm.c b/flight/PiOS/STM32F4xx/pios_ppm.c index e17b342b7..3040287da 100644 --- a/flight/PiOS/STM32F4xx/pios_ppm.c +++ b/flight/PiOS/STM32F4xx/pios_ppm.c @@ -28,11 +28,11 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#include "pios_ppm_priv.h" -#if defined(PIOS_INCLUDE_PPM) +#ifdef PIOS_INCLUDE_PPM + +#include "pios_ppm_priv.h" /* Provide a RCVR driver */ static int32_t PIOS_PPM_Get(uint32_t rcvr_id, uint8_t channel); @@ -349,7 +349,7 @@ static void PIOS_PPM_Supervisor(uint32_t ppm_id) { ppm_dev->Fresh = false; } -#endif +#endif /* PIOS_INCLUDE_PPM */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_pwm.c b/flight/PiOS/STM32F4xx/pios_pwm.c index 5bbec32aa..4f7f6c942 100644 --- a/flight/PiOS/STM32F4xx/pios_pwm.c +++ b/flight/PiOS/STM32F4xx/pios_pwm.c @@ -28,11 +28,11 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#include "pios_pwm_priv.h" -#if defined(PIOS_INCLUDE_PWM) +#ifdef PIOS_INCLUDE_PWM + +#include "pios_pwm_priv.h" /* Provide a RCVR driver */ static int32_t PIOS_PWM_Get(uint32_t rcvr_id, uint8_t channel); @@ -274,7 +274,7 @@ static void PIOS_PWM_tim_edge_cb (uint32_t tim_id, uint32_t context, uint8_t cha } -#endif +#endif /* PIOS_INCLUDE_PWM */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_rtc.c b/flight/PiOS/STM32F4xx/pios_rtc.c index cc2de2ce2..e5b2e64ab 100644 --- a/flight/PiOS/STM32F4xx/pios_rtc.c +++ b/flight/PiOS/STM32F4xx/pios_rtc.c @@ -28,10 +28,10 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_RTC) +#ifdef PIOS_INCLUDE_RTC + #include #ifndef PIOS_RTC_PRESCALER @@ -130,7 +130,8 @@ void PIOS_RTC_irq_handler (void) if (EXTI_GetITStatus(EXTI_Line22) != RESET) EXTI_ClearITPendingBit(EXTI_Line22); } -#endif + +#endif /* PIOS_INCLUDE_RTC */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_servo.c b/flight/PiOS/STM32F4xx/pios_servo.c index 05188d563..5c5d6ef52 100644 --- a/flight/PiOS/STM32F4xx/pios_servo.c +++ b/flight/PiOS/STM32F4xx/pios_servo.c @@ -28,8 +28,10 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" + +#ifdef PIOS_INCLUDE_SERVO + #include "pios_servo_priv.h" #include "pios_tim_priv.h" @@ -153,3 +155,5 @@ void PIOS_Servo_Set(uint8_t servo, uint16_t position) break; } } + +#endif /* PIOS_INCLUDE_SERVO */ diff --git a/flight/PiOS/STM32F4xx/pios_spi.c b/flight/PiOS/STM32F4xx/pios_spi.c index 87a8e0e9e..2bd07284a 100644 --- a/flight/PiOS/STM32F4xx/pios_spi.c +++ b/flight/PiOS/STM32F4xx/pios_spi.c @@ -38,7 +38,7 @@ */ #include -#if defined(PIOS_INCLUDE_SPI) +#ifdef PIOS_INCLUDE_SPI #include @@ -659,7 +659,7 @@ void PIOS_SPI_IRQ_Handler(uint32_t spi_id) } } -#endif +#endif /* PIOS_INCLUDE_SPI */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_sys.c b/flight/PiOS/STM32F4xx/pios_sys.c index 1bb0a090b..a25e01182 100644 --- a/flight/PiOS/STM32F4xx/pios_sys.c +++ b/flight/PiOS/STM32F4xx/pios_sys.c @@ -29,10 +29,9 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_SYS) +#ifdef PIOS_INCLUDE_SYS /* Private Function Prototypes */ void NVIC_Configuration(void); @@ -320,7 +319,7 @@ void assert_failed(uint8_t * file, uint32_t line) } #endif -#endif +#endif /* PIOS_INCLUDE_SYS */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_tim.c b/flight/PiOS/STM32F4xx/pios_tim.c index 129e88a86..4f3828d68 100644 --- a/flight/PiOS/STM32F4xx/pios_tim.c +++ b/flight/PiOS/STM32F4xx/pios_tim.c @@ -31,7 +31,8 @@ #include "pios.h" -#include "pios_tim.h" +#ifdef PIOS_INCLUDE_TIM + #include "pios_tim_priv.h" enum pios_tim_dev_magic { @@ -465,6 +466,8 @@ static void PIOS_TIM8_TRG_COM_TIM14_IRQHandler (void) } } +#endif /* PIOS_INCLUDE_TIM */ + /** * @} * @} diff --git a/flight/PiOS/STM32F4xx/pios_usart.c b/flight/PiOS/STM32F4xx/pios_usart.c index 2fe780e48..8080cbb69 100644 --- a/flight/PiOS/STM32F4xx/pios_usart.c +++ b/flight/PiOS/STM32F4xx/pios_usart.c @@ -32,10 +32,9 @@ * @todo This is virtually identical to the F1xx driver and should be merged. */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_USART) +#ifdef PIOS_INCLUDE_USART #include @@ -350,7 +349,7 @@ static void PIOS_USART_generic_irq_handler(uint32_t usart_id) #endif /* PIOS_INCLUDE_FREERTOS */ } -#endif +#endif /* PIOS_INCLUDE_USART */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_usb.c b/flight/PiOS/STM32F4xx/pios_usb.c index ae74d0e89..50ae0837f 100644 --- a/flight/PiOS/STM32F4xx/pios_usb.c +++ b/flight/PiOS/STM32F4xx/pios_usb.c @@ -28,14 +28,14 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" + +#ifdef PIOS_INCLUDE_USB + #include "usb_core.h" #include "pios_usb_board_data.h" -#include "pios_usb.h" #include "pios_usb_priv.h" -#if defined(PIOS_INCLUDE_USB) /* Rx/Tx status */ static uint8_t transfer_possible = 0; @@ -261,7 +261,7 @@ void USB_OTG_BSP_TimerIRQ (void) } -#endif /* PIOS_INCLUDE_USB */ +#endif /* PIOS_INCLUDE_USB */ /** * @} diff --git a/flight/PiOS/STM32F4xx/pios_usb_cdc.c b/flight/PiOS/STM32F4xx/pios_usb_cdc.c index c26997ccd..30151330e 100644 --- a/flight/PiOS/STM32F4xx/pios_usb_cdc.c +++ b/flight/PiOS/STM32F4xx/pios_usb_cdc.c @@ -29,12 +29,10 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_USB_CDC) +#ifdef PIOS_INCLUDE_USB_CDC -#include "pios_usb.h" #include "pios_usb_cdc_priv.h" #include "pios_usb_board_data.h" /* PIOS_BOARD_*_DATA_LENGTH */ @@ -398,4 +396,4 @@ static void PIOS_USB_CDC_CTRL_EP_IN_Callback(void) SetEPTxValid(usb_cdc_dev->cfg->data_tx_ep); } -#endif /* PIOS_INCLUDE_USB_CDC */ +#endif /* PIOS_INCLUDE_USB_CDC */ diff --git a/flight/PiOS/STM32F4xx/pios_usb_hid.c b/flight/PiOS/STM32F4xx/pios_usb_hid.c index 4fa50960c..88f45978b 100644 --- a/flight/PiOS/STM32F4xx/pios_usb_hid.c +++ b/flight/PiOS/STM32F4xx/pios_usb_hid.c @@ -29,12 +29,10 @@ * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -/* Project Includes */ #include "pios.h" -#if defined(PIOS_INCLUDE_USB_HID) +#ifdef PIOS_INCLUDE_USB_HID -#include "pios_usb.h" #include "pios_usb_hid_priv.h" #include "pios_usb_board_data.h" /* PIOS_BOARD_*_DATA_LENGTH */ #include "pios_usbhook.h" /* PIOS_USBHOOK_* */ @@ -525,4 +523,4 @@ static bool PIOS_USB_HID_EP_OUT_Callback(uint32_t usb_hid_id, uint8_t epnum, uin return rc; } -#endif /* PIOS_INCLUDE_USB_HID */ +#endif /* PIOS_INCLUDE_USB_HID */ diff --git a/flight/PiOS/STM32F4xx/pios_wdg.c b/flight/PiOS/STM32F4xx/pios_wdg.c index 2c718ecfd..f1a5e5d10 100644 --- a/flight/PiOS/STM32F4xx/pios_wdg.c +++ b/flight/PiOS/STM32F4xx/pios_wdg.c @@ -37,6 +37,9 @@ */ #include "pios.h" + +#ifdef PIOS_INCLUDE_WDG + #include "stm32f4xx_iwdg.h" #include "stm32f4xx_dbgmcu.h" #include "stm32f4xx_rtc.h" @@ -177,3 +180,5 @@ void PIOS_WDG_Clear(void) IWDG_ReloadCounter(); #endif } + +#endif /* PIOS_INCLUDE_WDG */