From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Thu, 19 Nov 2015 16:23:31 +0100 Subject: [PATCH] scpi: hide get_scpi_ops in module from built-in code In-Reply-To: <9hhoaespm8p.fsf@e105922-lin.cambridge.arm.com> References: <4740279.aWlP1tqJLa@wuerfel> <9hhoaespm8p.fsf@e105922-lin.cambridge.arm.com> Message-ID: <3802372.zWzDBWFCTp@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 17 November 2015 17:30:14 Punit Agrawal wrote: > Arnd Bergmann writes: > > > The scpi_clock driver can be built-in when CONFIG_COMPILE_TEST > > is set even when ARM_SCPI_PROTOCOL is a loadable module, and > > that results in a link error: > > > > drivers/built-in.o: In function `scpi_clocks_probe': > > (.text+0x14453c): undefined reference to `get_scpi_ops' > > > > Using #if IS_REACHABLE() around the get_scpi_ops() declaration > > makes it build successfully in this case for compile-testing, > > but the effect is the same as when ARM_SCPI_PROTOCOL is > > disabled, as the code will not be used. > > > > Signed-off-by: Arnd Bergmann > > Makes sense. Thanks, Arnd! > > Acked-by: Punit Agrawal > I've added it to arm-soc fixes now. Arnd