| Message ID | 20200419170810.5738-10-robh@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Modularizing Versatile Express | expand |
On Sun, Apr 19, 2020 at 7:08 PM Rob Herring <robh@kernel.org> wrote: > > Nothing in the VExpress sysregs nor the MFD child drivers use > CONFIG_CLKSRC_MMIO. There's the 24MHz counter, but that's handled by > drivers/clocksource/timer-versatile.c which doesn't use > CONFIG_CLKSRC_MMIO either. So let's just drop CONFIG_CLKSRC_MMIO. > > As the !ARCH_USES_GETTIMEOFFSET dependency was added for > CONFIG_CLKSRC_MMIO, that can be dropped, too. > > Cc: Liviu Dudau <liviu.dudau@arm.com> > Cc: Sudeep Holla <sudeep.holla@arm.com> > Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> > Cc: Linus Walleij <linus.walleij@linaro.org> > Cc: Lee Jones <lee.jones@linaro.org> > Signed-off-by: Rob Herring <robh@kernel.org> Acked-by: Arnd Bergmann <arnd@arndb.de>
On Sun, Apr 19, 2020 at 12:08:02PM -0500, Rob Herring wrote: > Nothing in the VExpress sysregs nor the MFD child drivers use > CONFIG_CLKSRC_MMIO. There's the 24MHz counter, but that's handled by > drivers/clocksource/timer-versatile.c which doesn't use > CONFIG_CLKSRC_MMIO either. So let's just drop CONFIG_CLKSRC_MMIO. > > As the !ARCH_USES_GETTIMEOFFSET dependency was added for > CONFIG_CLKSRC_MMIO, that can be dropped, too. > > Cc: Liviu Dudau <liviu.dudau@arm.com> Acked-by: Liviu Dudau <liviu.dudau@arm.com> > Cc: Sudeep Holla <sudeep.holla@arm.com> > Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> > Cc: Linus Walleij <linus.walleij@linaro.org> > Cc: Lee Jones <lee.jones@linaro.org> > Signed-off-by: Rob Herring <robh@kernel.org> > --- > drivers/mfd/Kconfig | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 0a59249198d3..b1311dea2da1 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -2029,9 +2029,8 @@ endmenu > > config MFD_VEXPRESS_SYSREG > bool "Versatile Express System Registers" > - depends on VEXPRESS_CONFIG && GPIOLIB && !ARCH_USES_GETTIMEOFFSET > + depends on VEXPRESS_CONFIG && GPIOLIB > default y > - select CLKSRC_MMIO > select GPIO_GENERIC_PLATFORM > select MFD_CORE > select MFD_SYSCON > -- > 2.20.1 >
diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig index 0a59249198d3..b1311dea2da1 100644 --- a/drivers/mfd/Kconfig +++ b/drivers/mfd/Kconfig @@ -2029,9 +2029,8 @@ endmenu config MFD_VEXPRESS_SYSREG bool "Versatile Express System Registers" - depends on VEXPRESS_CONFIG && GPIOLIB && !ARCH_USES_GETTIMEOFFSET + depends on VEXPRESS_CONFIG && GPIOLIB default y - select CLKSRC_MMIO select GPIO_GENERIC_PLATFORM select MFD_CORE select MFD_SYSCON
Nothing in the VExpress sysregs nor the MFD child drivers use CONFIG_CLKSRC_MMIO. There's the 24MHz counter, but that's handled by drivers/clocksource/timer-versatile.c which doesn't use CONFIG_CLKSRC_MMIO either. So let's just drop CONFIG_CLKSRC_MMIO. As the !ARCH_USES_GETTIMEOFFSET dependency was added for CONFIG_CLKSRC_MMIO, that can be dropped, too. Cc: Liviu Dudau <liviu.dudau@arm.com> Cc: Sudeep Holla <sudeep.holla@arm.com> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> Cc: Linus Walleij <linus.walleij@linaro.org> Cc: Lee Jones <lee.jones@linaro.org> Signed-off-by: Rob Herring <robh@kernel.org> --- drivers/mfd/Kconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)