Em Wed, 7 Mar 2018 06:47:14 -0300 Fabio Estevam <festevam@xxxxxxxxx> escreveu: > Hi Mauro, > > On Wed, Mar 7, 2018 at 6:14 AM, Mauro Carvalho Chehab > <mchehab@xxxxxxxxxxxxxxxx> wrote: > > Not sure why, but, on ia64, with Linaro's gcc 7.3 compiler, > > using #ifdef (CONFIG_I2C) is not OK. > > Looking at the kbuild report the failure happens when CONFIG_I2C=m. > > IS_ENABLED() macro takes care of both built-in and module as it will expand to: > > #if defined(CONFIG_I2C) || defined(CONFIG_I2C_MODULE) Ah, true! Yeah, I forgot that for tri-state vars, it should test for _MODULE variant. > > and that's the reason why IS_ENABLE() fixes the build problem. Thanks for reminding me. Regards, Mauro
