ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

stm32: use default in switch case 

As commented during PR review, better use default case.
pull/3665/head
Laurent MEUNIER 2017-01-31 16:35:08 +01:00
parent 5317ea5bb6
commit 273ac81680
9 changed files with 18 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
targets/TARGET_STM/TARGET_STM32F0/pin_device.h
View File

@ -43,15 +43,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32F1/pin_device.h
View File

@ -97,8 +97,7 @@ static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint3
{ {
uint32_t function = LL_GPIO_GetPinMode(gpio, ll_pin); uint32_t function = LL_GPIO_GetPinMode(gpio, ll_pin);
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
if (function == LL_GPIO_MODE_FLOATING) if (function == LL_GPIO_MODE_FLOATING)
LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_INPUT); LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_INPUT);
@ -109,7 +108,7 @@ static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint3
LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_INPUT); LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_INPUT);
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
/* Input+NoPull = Floating for F1 family */ /* Input+NoPull = Floating for F1 family */
if (function == LL_GPIO_MODE_INPUT) if (function == LL_GPIO_MODE_INPUT)
LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_FLOATING); LL_GPIO_SetPinMode(gpio, ll_pin, LL_GPIO_MODE_FLOATING);

5
targets/TARGET_STM/TARGET_STM32F2/pin_device.h
View File

@ -113,15 +113,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32F3/pin_device.h
View File

@ -44,15 +44,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32F4/pin_device.h
View File

@ -112,15 +112,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32F7/pin_device.h
View File

@ -114,15 +114,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32L0/pin_device.h
View File

@ -43,15 +43,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32L1/pin_device.h
View File

@ -43,15 +43,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }

5
targets/TARGET_STM/TARGET_STM32L4/pin_device.h
View File

@ -43,15 +43,14 @@ static inline void stm_pin_DisconnectDebug(PinName pin)
static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config) static inline void stm_pin_PullConfig(GPIO_TypeDef *gpio, uint32_t ll_pin, uint32_t pull_config)
{ {
switch (pull_config) switch (pull_config) {
{
case GPIO_PULLUP: case GPIO_PULLUP:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_UP);
break; break;
case GPIO_PULLDOWN: case GPIO_PULLDOWN:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_DOWN);
break; break;
case GPIO_NOPULL: default:
LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO); LL_GPIO_SetPinPull(gpio, ll_pin, LL_GPIO_PULL_NO);
break; break;
} }