Skip to content
This repository was archived by the owner on Apr 6, 2022. It is now read-only.

Conversation

@yboutreux
Copy link

Add CONFIG_SHIFT to Kconfig exclusion.
This definition is from LL from STM32MP1X Cube,
in ext/hal/st/stm32cube/stm32mp1xx/drivers/include/stm32mp1xx_ll_rcc.h.
This solve false positive in zephyrproject-rtos/zephyr#15426

Signed-off-by: Yaël BOUTREUX [email protected]

Add CONFIG_SHIFT to Kconfig exclusion.
This definition is from LL from STM32MP1X Cube,
in ext/hal/st/stm32cube/stm32mp1xx/drivers/include/stm32mp1xx_ll_rcc.h.
This solve false positive in zephyrproject-rtos/zephyr#15426

Signed-off-by: Yaël BOUTREUX <[email protected]>
@yboutreux yboutreux requested a review from nashif as a code owner April 12, 2019 14:24
@galak galak requested a review from ulfalizer April 12, 2019 15:08
Copy link
Collaborator

@ulfalizer ulfalizer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another option might be to call it CFG_SHIFT or the like, unless CONFIG_SHIFT is a standard name from some data sheet or the like.

"CONFIG_REG1",
"CONFIG_REG2",
"CONFIG_SEL",
"CONFIG_SHIFT", #From STM32MP1 Cube (ext/hal/), file stm32mp1xx_ll_rcc.h
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Please put a space after # (and /*, and //) in the future. It's a much more common style.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will remember this for next time, thank you !

@ulfalizer
Copy link
Collaborator

CC @nashif

@yboutreux
Copy link
Author

Another option might be to call it CFG_SHIFT or the like, unless CONFIG_SHIFT is a standard name from some data sheet or the like.

You are right, the variable will be renamed to CFG_SHIFT, thank you !

@yboutreux yboutreux closed this Apr 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants