diff mbox

[RESEND] input/wm97xx: make missing platform data non-fatal

Message ID 20170119084353.4250-1-manuel.lauss@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Manuel Lauss Jan. 19, 2017, 8:43 a.m. UTC
Commit 6480af4915d6 ("power_supply: wm97xx_battery: use
power_supply_get_drvdata") made wm97xx platform data mandatory,
although it's still optional.

This patch fixes an oops during driver probe on one
of my MIPS boards with a wm9712.  Touchscreen continues
to work fine, and I don't have any battery hooked up
to it either, so I'm not missing any functionality.

Signed-off-by: Manuel Lauss <manuel.lauss@gmail.com>
Cc: linux-input@vger.kernel.org
Cc: patches@opensource.wolfsonmicro.com
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
---
Resend, added Maintainers to Cc.

 drivers/input/touchscreen/wm97xx-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Charles Keepax Jan. 19, 2017, 9:10 a.m. UTC | #1
On Thu, Jan 19, 2017 at 09:43:53AM +0100, Manuel Lauss wrote:
> Commit 6480af4915d6 ("power_supply: wm97xx_battery: use
> power_supply_get_drvdata") made wm97xx platform data mandatory,
> although it's still optional.
> 
> This patch fixes an oops during driver probe on one
> of my MIPS boards with a wm9712.  Touchscreen continues
> to work fine, and I don't have any battery hooked up
> to it either, so I'm not missing any functionality.
> 
> Signed-off-by: Manuel Lauss <manuel.lauss@gmail.com>
> Cc: linux-input@vger.kernel.org
> Cc: patches@opensource.wolfsonmicro.com
> Cc: Robert Jarzmik <robert.jarzmik@free.fr>
> ---

Acked-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>

Thanks,
Charles
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/input/touchscreen/wm97xx-core.c b/drivers/input/touchscreen/wm97xx-core.c
index 83cf11312fd9..c9d1c91e1887 100644
--- a/drivers/input/touchscreen/wm97xx-core.c
+++ b/drivers/input/touchscreen/wm97xx-core.c
@@ -682,7 +682,7 @@  static int wm97xx_probe(struct device *dev)
 	}
 	platform_set_drvdata(wm->battery_dev, wm);
 	wm->battery_dev->dev.parent = dev;
-	wm->battery_dev->dev.platform_data = pdata->batt_pdata;
+	wm->battery_dev->dev.platform_data = pdata ? pdata->batt_pdata : NULL;
 	ret = platform_device_add(wm->battery_dev);
 	if (ret < 0)
 		goto batt_reg_err;