| Message ID | 20210920190350.3860821-5-willmcvicker@google.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: Kconfig: Update ARCH_EXYNOS select configs | expand |
On 20/09/2021 19:03:49+0000, Will McVicker wrote: > Switches the default config logic of HAVE_S3C_RTC to use "default y if > (ARCH_EXYNOS && RTC_CLASS)" versus having ARCH_EXYNOS directly select > it. This provides vendors flexibility to disable the config if desired > or modularize it in the presence of a generic kernel. > > Verified this change doesn't impact the .config. > > Signed-off-by: Will McVicker <willmcvicker@google.com> Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com> > --- > arch/arm64/Kconfig.platforms | 1 - > drivers/rtc/Kconfig | 1 + > 2 files changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index a884e5da8b0f..f9f829aab511 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -91,7 +91,6 @@ config ARCH_BRCMSTB > > config ARCH_EXYNOS > bool "ARMv8 based Samsung Exynos SoC family" > - select HAVE_S3C_RTC if RTC_CLASS > select PINCTRL > select PM_GENERIC_DOMAINS if PM > help > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig > index e1bc5214494e..40afdb37d2a5 100644 > --- a/drivers/rtc/Kconfig > +++ b/drivers/rtc/Kconfig > @@ -1406,6 +1406,7 @@ config RTC_DRV_OMAP > > config HAVE_S3C_RTC > bool > + default y if (ARCH_EXYNOS && RTC_CLASS) > help > This will include RTC support for Samsung SoCs. If > you want to include RTC support for any machine, kindly > -- > 2.33.0.464.g1972c5931b-goog >
On 20/09/2021 21:03, Will McVicker wrote: > Switches the default config logic of HAVE_S3C_RTC to use "default y if > (ARCH_EXYNOS && RTC_CLASS)" versus having ARCH_EXYNOS directly select > it. This provides vendors flexibility to disable the config if desired > or modularize it in the presence of a generic kernel. > > Verified this change doesn't impact the .config. > > Signed-off-by: Will McVicker <willmcvicker@google.com> > --- > arch/arm64/Kconfig.platforms | 1 - > drivers/rtc/Kconfig | 1 + > 2 files changed, 1 insertion(+), 1 deletion(-) > It took me some effort to find this patch because you did not send it to neither me, nor Samsung SoC list. Since you touch arm64 code, this should go either via SoC tree or with SoC ack, therefore you should simply use get_maintainers.pl on entire patchset, not on some pieces. > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index a884e5da8b0f..f9f829aab511 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -91,7 +91,6 @@ config ARCH_BRCMSTB > > config ARCH_EXYNOS > bool "ARMv8 based Samsung Exynos SoC family" > - select HAVE_S3C_RTC if RTC_CLASS Just remove HAVE_S3C_RTC entirely like in commit: 7dd3cae90d856e97e93bc1c32904e5aed7210f7b > select PINCTRL > select PM_GENERIC_DOMAINS if PM > help > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig > index e1bc5214494e..40afdb37d2a5 100644 > --- a/drivers/rtc/Kconfig > +++ b/drivers/rtc/Kconfig > @@ -1406,6 +1406,7 @@ config RTC_DRV_OMAP > > config HAVE_S3C_RTC > bool > + default y if (ARCH_EXYNOS && RTC_CLASS) > help > This will include RTC support for Samsung SoCs. If > you want to include RTC support for any machine, kindly > Best regards, Krzysztof
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms index a884e5da8b0f..f9f829aab511 100644 --- a/arch/arm64/Kconfig.platforms +++ b/arch/arm64/Kconfig.platforms @@ -91,7 +91,6 @@ config ARCH_BRCMSTB config ARCH_EXYNOS bool "ARMv8 based Samsung Exynos SoC family" - select HAVE_S3C_RTC if RTC_CLASS select PINCTRL select PM_GENERIC_DOMAINS if PM help diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index e1bc5214494e..40afdb37d2a5 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -1406,6 +1406,7 @@ config RTC_DRV_OMAP config HAVE_S3C_RTC bool + default y if (ARCH_EXYNOS && RTC_CLASS) help This will include RTC support for Samsung SoCs. If you want to include RTC support for any machine, kindly
Switches the default config logic of HAVE_S3C_RTC to use "default y if (ARCH_EXYNOS && RTC_CLASS)" versus having ARCH_EXYNOS directly select it. This provides vendors flexibility to disable the config if desired or modularize it in the presence of a generic kernel. Verified this change doesn't impact the .config. Signed-off-by: Will McVicker <willmcvicker@google.com> --- arch/arm64/Kconfig.platforms | 1 - drivers/rtc/Kconfig | 1 + 2 files changed, 1 insertion(+), 1 deletion(-)