Fix a few recent (STM32F1 DOGM, SPI) warnings (#14669)
This commit is contained in:
		
				
					committed by
					
						 Scott Lahteine
						Scott Lahteine
					
				
			
			
				
	
			
			
			
						parent
						
							39e1726498
						
					
				
				
					commit
					977cfa4494
				
			| @@ -260,7 +260,11 @@ extern "C" { | |||||||
| void HAL_adc_init(void) { | void HAL_adc_init(void) { | ||||||
|   // configure the ADC |   // configure the ADC | ||||||
|   adc.calibrate(); |   adc.calibrate(); | ||||||
|   adc.setSampleRate(ADC_SMPR_41_5); // ? |   #if F_CPU > 72000000 | ||||||
|  |     adc.setSampleRate(ADC_SMPR_71_5); // 71.5 ADC cycles | ||||||
|  |   #else | ||||||
|  |     adc.setSampleRate(ADC_SMPR_41_5); // 41.5 ADC cycles | ||||||
|  |   #endif | ||||||
|   adc.setPins(adc_pins, ADC_PIN_COUNT); |   adc.setPins(adc_pins, ADC_PIN_COUNT); | ||||||
|   adc.setDMA(HAL_adc_results, (uint16_t)ADC_PIN_COUNT, (uint32_t)(DMA_MINC_MODE | DMA_CIRC_MODE), nullptr); |   adc.setDMA(HAL_adc_results, (uint16_t)ADC_PIN_COUNT, (uint32_t)(DMA_MINC_MODE | DMA_CIRC_MODE), nullptr); | ||||||
|   adc.setScanMode(); |   adc.setScanMode(); | ||||||
|   | |||||||
| @@ -104,7 +104,7 @@ void spiInit(uint8_t spiRate) { | |||||||
|  * @details |  * @details | ||||||
|  */ |  */ | ||||||
| uint8_t spiRec(void) { | uint8_t spiRec(void) { | ||||||
|   uint8_t returnByte = SPI.transfer(0xFF); |   uint8_t returnByte = SPI.transfer(ff); | ||||||
|   return returnByte; |   return returnByte; | ||||||
| } | } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -4,7 +4,6 @@ import sys | |||||||
| #dynamic build flags for generic compile options | #dynamic build flags for generic compile options | ||||||
| if __name__ == "__main__": | if __name__ == "__main__": | ||||||
|   args = " ".join([ "-std=gnu11", |   args = " ".join([ "-std=gnu11", | ||||||
|                     "-std=gnu++11", |  | ||||||
|                     "-Os", |                     "-Os", | ||||||
|                     "-mcpu=cortex-m3", |                     "-mcpu=cortex-m3", | ||||||
|                     "-mthumb", |                     "-mthumb", | ||||||
|   | |||||||
| @@ -25,7 +25,6 @@ | |||||||
|  |  | ||||||
| #ifndef U8G_HAL_LINKS | #ifndef U8G_HAL_LINKS | ||||||
|  |  | ||||||
|  |  | ||||||
|   #ifdef __SAM3X8E__ |   #ifdef __SAM3X8E__ | ||||||
|     uint8_t u8g_com_HAL_DUE_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |     uint8_t u8g_com_HAL_DUE_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|     #define U8G_COM_HAL_SW_SPI_FN  u8g_com_HAL_DUE_sw_spi_fn |     #define U8G_COM_HAL_SW_SPI_FN  u8g_com_HAL_DUE_sw_spi_fn | ||||||
| @@ -33,44 +32,41 @@ | |||||||
|     #define U8G_COM_HAL_HW_SPI_FN u8g_com_HAL_DUE_shared_hw_spi_fn |     #define U8G_COM_HAL_HW_SPI_FN u8g_com_HAL_DUE_shared_hw_spi_fn | ||||||
|     uint8_t u8g_com_HAL_DUE_ST7920_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |     uint8_t u8g_com_HAL_DUE_ST7920_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_HAL_DUE_ST7920_sw_spi_fn |     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_HAL_DUE_ST7920_sw_spi_fn | ||||||
|  |     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_arduino_st7920_hw_spi_fn | ||||||
|   #elif defined(__STM32F1__) |   #elif defined(__STM32F1__) | ||||||
|     uint8_t u8g_com_HAL_STM32F1_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |     uint8_t u8g_com_HAL_STM32F1_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|     #define U8G_COM_HAL_SW_SPI_FN u8g_com_HAL_STM32F1_sw_spi_fn |     #define U8G_COM_HAL_SW_SPI_FN u8g_com_HAL_STM32F1_sw_spi_fn | ||||||
|     uint8_t u8g_com_arduino_hw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     //#define U8G_COM_HAL_HW_SPI_FN u8g_com_arduino_hw_spi_fn |  | ||||||
|     #define U8G_COM_HAL_HW_SPI_FN u8g_com_stm32duino_hw_spi_fn |     #define U8G_COM_HAL_HW_SPI_FN u8g_com_stm32duino_hw_spi_fn | ||||||
|     uint8_t u8g_com_arduino_st7920_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     //#define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn |  | ||||||
|     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_std_sw_spi_fn |     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_std_sw_spi_fn | ||||||
|     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_stm32duino_hw_spi_fn |     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_stm32duino_hw_spi_fn | ||||||
|   #elif defined(ARDUINO_ARCH_STM32) |   #elif defined(ARDUINO_ARCH_STM32) | ||||||
|     uint8_t u8g_com_arduino_std_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     #define U8G_COM_HAL_SW_SPI_FN u8g_com_arduino_std_sw_spi_fn |     #define U8G_COM_HAL_SW_SPI_FN u8g_com_arduino_std_sw_spi_fn | ||||||
|     uint8_t u8g_com_stm32duino_hw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     #define U8G_COM_HAL_HW_SPI_FN u8g_com_stm32duino_hw_spi_fn |     #define U8G_COM_HAL_HW_SPI_FN u8g_com_stm32duino_hw_spi_fn | ||||||
|     uint8_t u8g_com_arduino_st7920_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn |     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn | ||||||
|   #else |     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_stm32duino_hw_spi_fn | ||||||
|  |   #elif defined(__AVR__) | ||||||
|     uint8_t u8g_com_HAL_AVR_sw_sp_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |     uint8_t u8g_com_HAL_AVR_sw_sp_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|     #define U8G_COM_HAL_SW_SPI_FN  u8g_com_HAL_AVR_sw_sp_fn // AVR ? |     #define U8G_COM_HAL_SW_SPI_FN  u8g_com_HAL_AVR_sw_sp_fn | ||||||
|     uint8_t u8g_com_arduino_hw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     #define U8G_COM_HAL_HW_SPI_FN u8g_com_arduino_hw_spi_fn |     #define U8G_COM_HAL_HW_SPI_FN u8g_com_arduino_hw_spi_fn | ||||||
|     uint8_t u8g_com_arduino_st7920_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn |     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn | ||||||
|  |     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_arduino_st7920_hw_spi_fn | ||||||
|  |   #else | ||||||
|  |     #define U8G_COM_HAL_SW_SPI_FN u8g_com_arduino_std_sw_spi_fn | ||||||
|  |     #define U8G_COM_HAL_HW_SPI_FN u8g_com_arduino_hw_spi_fn | ||||||
|  |     #define U8G_COM_ST7920_HAL_SW_SPI u8g_com_arduino_st7920_spi_fn | ||||||
|  |     #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_arduino_st7920_hw_spi_fn | ||||||
|   #endif |   #endif | ||||||
|  |  | ||||||
|   uint8_t u8g_com_arduino_st7920_hw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |  | ||||||
|   #define U8G_COM_ST7920_HAL_HW_SPI u8g_com_arduino_st7920_hw_spi_fn |  | ||||||
|  |  | ||||||
|   uint8_t u8g_com_HAL_LPC1768_ssd_hw_i2c_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |   uint8_t u8g_com_HAL_LPC1768_ssd_hw_i2c_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|   #define U8G_COM_SSD_I2C_HAL u8g_com_arduino_ssd_i2c_fn |   #define U8G_COM_SSD_I2C_HAL u8g_com_arduino_ssd_i2c_fn | ||||||
|  |  | ||||||
|   #if defined(ARDUINO_ARCH_STM32F1) && PIN_EXISTS(FSMC_CS) |   #if PIN_EXISTS(FSMC_CS) | ||||||
|     uint8_t u8g_com_stm32duino_fsmc_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |     uint8_t u8g_com_stm32duino_fsmc_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|     #define U8G_COM_HAL_FSMC_FN u8g_com_stm32duino_fsmc_fn |     #define U8G_COM_HAL_FSMC_FN u8g_com_stm32duino_fsmc_fn | ||||||
|   #else |   #else | ||||||
|     #define U8G_COM_HAL_FSMC_FN u8g_com_null_fn |     #define U8G_COM_HAL_FSMC_FN u8g_com_null_fn | ||||||
|   #endif |   #endif | ||||||
|  |  | ||||||
| #elif TARGET_LPC1768 | #elif TARGET_LPC1768 | ||||||
|   uint8_t u8g_com_HAL_LPC1768_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); |   uint8_t u8g_com_HAL_LPC1768_sw_spi_fn(u8g_t *u8g, uint8_t msg, uint8_t arg_val, void *arg_ptr); | ||||||
|   #define U8G_COM_HAL_SW_SPI_FN u8g_com_HAL_LPC1768_sw_spi_fn |   #define U8G_COM_HAL_SW_SPI_FN u8g_com_HAL_LPC1768_sw_spi_fn | ||||||
|   | |||||||
| @@ -334,7 +334,6 @@ uint8_t u8g_dev_tft_320x240_upscale_from_128x64_fn(u8g_t *u8g, u8g_dev_t *dev, u | |||||||
|   #else |   #else | ||||||
|     uint16_t buffer[256]; // 16-bit RGB 565 pixel line buffer |     uint16_t buffer[256]; // 16-bit RGB 565 pixel line buffer | ||||||
|   #endif |   #endif | ||||||
|   uint16_t i; |  | ||||||
|   switch (msg) { |   switch (msg) { | ||||||
|     case U8G_DEV_MSG_INIT: |     case U8G_DEV_MSG_INIT: | ||||||
|       dev->com_fn(u8g, U8G_COM_MSG_INIT, U8G_SPI_CLK_CYCLE_NONE, &lcd_id); |       dev->com_fn(u8g, U8G_COM_MSG_INIT, U8G_SPI_CLK_CYCLE_NONE, &lcd_id); | ||||||
|   | |||||||
| @@ -1619,8 +1619,11 @@ | |||||||
|   // Middle point of circle |   // Middle point of circle | ||||||
|   #define NOZZLE_CLEAN_CIRCLE_MIDDLE NOZZLE_CLEAN_START_POINT |   #define NOZZLE_CLEAN_CIRCLE_MIDDLE NOZZLE_CLEAN_START_POINT | ||||||
|  |  | ||||||
|   // Moves the nozzle to the initial position |   // Move the nozzle to the initial position after cleaning | ||||||
|   #define NOZZLE_CLEAN_GOBACK |   #define NOZZLE_CLEAN_GOBACK | ||||||
|  |  | ||||||
|  |   // Enable for a purge/clean station that's always at the gantry height (thus no Z move) | ||||||
|  |   //#define NOZZLE_CLEAN_NO_Z | ||||||
| #endif | #endif | ||||||
|  |  | ||||||
| /** | /** | ||||||
| @@ -2123,7 +2126,7 @@ | |||||||
| #define MKS_ROBIN_TFT | #define MKS_ROBIN_TFT | ||||||
|  |  | ||||||
| //============================================================================= | //============================================================================= | ||||||
| //============================= SPI Touch Screens ============================= | //============================  Other Controllers  ============================ | ||||||
| //============================================================================= | //============================================================================= | ||||||
|  |  | ||||||
| #define TOUCH_BUTTONS | #define TOUCH_BUTTONS | ||||||
| @@ -2147,14 +2150,6 @@ | |||||||
|   #endif |   #endif | ||||||
| #endif | #endif | ||||||
|  |  | ||||||
| //============================================================================= |  | ||||||
| //============================  Other Controllers  ============================ |  | ||||||
| //============================================================================= |  | ||||||
|  |  | ||||||
| // |  | ||||||
| // CONTROLLER TYPE: Keypad / Add-on |  | ||||||
| // |  | ||||||
|  |  | ||||||
| // | // | ||||||
| // RepRapWorld REPRAPWORLD_KEYPAD v1.1 | // RepRapWorld REPRAPWORLD_KEYPAD v1.1 | ||||||
| // http://reprapworld.com/?products_details&products_id=202&cPath=1591_1626 | // http://reprapworld.com/?products_details&products_id=202&cPath=1591_1626 | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user