* [PATCH] power: supply: cpcap-charger: add OMAP_USB2 dependency
@ 2017-07-21 16:14 Arnd Bergmann
2017-07-24 18:51 ` Sebastian Reichel
0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2017-07-21 16:14 UTC (permalink / raw)
To: Sebastian Reichel; +Cc: Arnd Bergmann, Tony Lindgren, linux-kernel
When CONFIG_OMAP_USB2 is set to 'm' and the charger driver is built-in,
we get this link failure:
drivers/power/supply/cpcap-charger.o: In function `cpcap_charger_probe':
cpcap-charger.c:(.text+0x48c): undefined reference to `omap_usb2_set_comparator'
drivers/power/supply/cpcap-charger.o: In function `cpcap_charger_remove':
cpcap-charger.c:(.text+0x774): undefined reference to `omap_usb2_set_comparator'
This adds a dependency to prevent that problem, while still allowing
compile-testing with the OMAP_USB2 driver completely disabled.
Fixes: 0c9888e3c192 ("power: supply: cpcap-charger: Add minimal CPCAP PMIC battery charger")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/power/supply/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
index 969f5005669c..765f9ac0a4e7 100644
--- a/drivers/power/supply/Kconfig
+++ b/drivers/power/supply/Kconfig
@@ -365,6 +365,7 @@ config BATTERY_RX51
config CHARGER_CPCAP
tristate "CPCAP PMIC Charger Driver"
depends on MFD_CPCAP && IIO
+ depends on OMAP_USB2 || (!OMAP_USB2 && COMPILE_TEST)
default MFD_CPCAP
help
Say Y to enable support for CPCAP PMIC charger driver for Motorola
--
2.9.0
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] power: supply: cpcap-charger: add OMAP_USB2 dependency
2017-07-21 16:14 [PATCH] power: supply: cpcap-charger: add OMAP_USB2 dependency Arnd Bergmann
@ 2017-07-24 18:51 ` Sebastian Reichel
0 siblings, 0 replies; 2+ messages in thread
From: Sebastian Reichel @ 2017-07-24 18:51 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: Tony Lindgren, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 864 bytes --]
Hi,
On Fri, Jul 21, 2017 at 06:14:37PM +0200, Arnd Bergmann wrote:
> When CONFIG_OMAP_USB2 is set to 'm' and the charger driver is built-in,
> we get this link failure:
>
> drivers/power/supply/cpcap-charger.o: In function `cpcap_charger_probe':
> cpcap-charger.c:(.text+0x48c): undefined reference to `omap_usb2_set_comparator'
> drivers/power/supply/cpcap-charger.o: In function `cpcap_charger_remove':
> cpcap-charger.c:(.text+0x774): undefined reference to `omap_usb2_set_comparator'
>
> This adds a dependency to prevent that problem, while still allowing
> compile-testing with the OMAP_USB2 driver completely disabled.
>
> Fixes: 0c9888e3c192 ("power: supply: cpcap-charger: Add minimal CPCAP PMIC battery charger")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
Thanks, applied to power-supply's fixes branch.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-07-24 18:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-21 16:14 [PATCH] power: supply: cpcap-charger: add OMAP_USB2 dependency Arnd Bergmann
2017-07-24 18:51 ` Sebastian Reichel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®