> -----Original Message----- > From: Simon Horman [mailto:horms@xxxxxxxxxxxx] > Sent: 18 August 2017 09:06 > To: Biju Das <biju.das@xxxxxxxxxxxxxx> > Cc: Russell King <linux@xxxxxxxxxxxxxxx>; Magnus Damm > <magnus.damm@xxxxxxxxx>; Fabrizio Castro > <fabrizio.castro@xxxxxxxxxxxxxx>; Chris Paterson > <Chris.Paterson2@xxxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx; linux- > arm-kernel@xxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH ] ARM: multi_v7_defconfig: Enable BQ32000 RTC driver > > On Thu, Aug 17, 2017 at 02:09:05PM +0100, Biju Das wrote: > > The iWave RZ/G1M Q7 SOM supports RTC (TI BQ32000). > > To increase hardware support enable the driver in the > > multi_v7_defconfig multiplatform configuration. > > > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > --- > > This patch is tested against linux-next tag next-20170817 as well as > > renesas-dev tag renesas-devel-20170815-v4.13-rc5 . > > > > arch/arm/configs/multi_v7_defconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/arm/configs/multi_v7_defconfig > > b/arch/arm/configs/multi_v7_defconfig > > index 18e1f80..02dc373 100644 > > --- a/arch/arm/configs/multi_v7_defconfig > > +++ b/arch/arm/configs/multi_v7_defconfig > > @@ -766,6 +766,7 @@ CONFIG_RTC_DRV_MAX8997=m > > CONFIG_RTC_DRV_MAX77686=y CONFIG_RTC_DRV_RK808=m > > CONFIG_RTC_DRV_RS5C372=m > > +CONFIG_RTC_DRV_BQ32K=y > > My assumption is that multi_v7_defconfig produces a very large kernel image > and that we should try to avoid making it larger by using modules where > possible. Is it practical to enable this option as a module (m) rather than a built- > in (y)? I have tested this as module and it works. OK I will send v2. > > CONFIG_RTC_DRV_PALMAS=y > > CONFIG_RTC_DRV_ST_LPC=y > > CONFIG_RTC_DRV_TWL4030=y > > -- > > 1.9.1 > > Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.