Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

soc: gd32: Drop PINCTRL from Kconfig.defconfig #80697

Conversation

nandojve
Copy link
Member

This Kconfig has wrongly been added to defconfig files. It is not the right place for it. It has never been the right place for it. Drivers that need it should select the symbol in their Kconfig entries. Drop PINCTL from Kconfig.defconfig and add proper select at Kconfig.gd32.

Fixes #78619

This Kconfig has wrongly been added to defconfig files. It is not the
right place for it. It has never been the right place for it. Drivers
that need it should select the symbol in their Kconfig entries. Drop
PINCTL from Kconfig.defconfig and add proper select at Kconfig.gd32.

Fixes zephyrproject-rtos#78619

Signed-off-by: Gerson Fernando Budke <[email protected]>
@zephyrbot zephyrbot added the area: SPI SPI bus label Oct 31, 2024
@zephyrbot zephyrbot added area: ADC Analog-to-Digital Converter (ADC) platform: GD32 GigaDevice area: UART Universal Asynchronous Receiver-Transmitter area: DAC Digital-to-Analog Converter area: I2C area: PWM Pulse Width Modulation labels Oct 31, 2024
@nandojve nandojve added this to the v4.0.0 milestone Oct 31, 2024
@nandojve nandojve requested a review from nordicjm October 31, 2024 18:11
@mmahadevan108 mmahadevan108 merged commit 52d21d7 into zephyrproject-rtos:main Nov 4, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: DAC Digital-to-Analog Converter area: I2C area: PWM Pulse Width Modulation area: SPI SPI bus area: UART Universal Asynchronous Receiver-Transmitter platform: GD32 GigaDevice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove CONFIG_PINCTRL from all defconfig files
6 participants