diff mbox series

[3/3] phy: mapphone-mdm6600: add gpiolib dependency

Message ID 20190304202406.1716350-3-arnd@arndb.de
State Accepted
Commit 208d3423ee463ab257908456f6bbca4024ab63f7
Headers show
Series [1/3] ASoC: ti: fix davinci_mcasp_probe dependencies | expand

Commit Message

Arnd Bergmann March 4, 2019, 8:23 p.m. UTC
gcc points out that when CONFIG_GPIOLIB is disabled,
gpiod_get_array_value_cansleep() returns 0 but fails to set its output:

drivers/phy/motorola/phy-mapphone-mdm6600.c: In function 'phy_mdm6600_status':
drivers/phy/motorola/phy-mapphone-mdm6600.c:220:24: error: 'values[0]' is used uninitialized in this function [-Werror=uninitialized]

This could be fixed more generally in gpiolib by returning a failure
code, but for this specific case, the easier workaround is to add a
gpiolib dependency.

Fixes: 5d1ebbda0318 ("phy: mapphone-mdm6600: Add USB PHY driver for MDM6600 on Droid 4")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>

---
 drivers/phy/motorola/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.20.0

Comments

Tony Lindgren March 5, 2019, 3:46 p.m. UTC | #1
* Arnd Bergmann <arnd@arndb.de> [190304 20:24]:
> gcc points out that when CONFIG_GPIOLIB is disabled,

> gpiod_get_array_value_cansleep() returns 0 but fails to set its output:

> 

> drivers/phy/motorola/phy-mapphone-mdm6600.c: In function 'phy_mdm6600_status':

> drivers/phy/motorola/phy-mapphone-mdm6600.c:220:24: error: 'values[0]' is used uninitialized in this function [-Werror=uninitialized]

> 

> This could be fixed more generally in gpiolib by returning a failure

> code, but for this specific case, the easier workaround is to add a

> gpiolib dependency.


Thanks for fixing it:

Acked-by: Tony Lindgren <tony@atomide.com>
diff mbox series

Patch

diff --git a/drivers/phy/motorola/Kconfig b/drivers/phy/motorola/Kconfig
index 82651524ffb9..718f8729701d 100644
--- a/drivers/phy/motorola/Kconfig
+++ b/drivers/phy/motorola/Kconfig
@@ -13,7 +13,7 @@  config PHY_CPCAP_USB
 
 config PHY_MAPPHONE_MDM6600
 	tristate "Motorola Mapphone MDM6600 modem USB PHY driver"
-	depends on OF && USB_SUPPORT
+	depends on OF && USB_SUPPORT && GPIOLIB
 	select GENERIC_PHY
 	help
 	  Enable this for MDM6600 USB modem to work on Motorola phones