* [PATCH RESEND] input/wm97xx: make missing platform data non-fatal
@ 2017-01-19 8:43 Manuel Lauss
2017-01-19 9:10 ` Charles Keepax
0 siblings, 1 reply; 2+ messages in thread
From: Manuel Lauss @ 2017-01-19 8:43 UTC (permalink / raw)
To: linux-input; +Cc: Manuel Lauss, patches, Robert Jarzmik
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(-)
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;
--
2.11.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH RESEND] input/wm97xx: make missing platform data non-fatal
2017-01-19 8:43 [PATCH RESEND] input/wm97xx: make missing platform data non-fatal Manuel Lauss
@ 2017-01-19 9:10 ` Charles Keepax
0 siblings, 0 replies; 2+ messages in thread
From: Charles Keepax @ 2017-01-19 9:10 UTC (permalink / raw)
To: Manuel Lauss; +Cc: linux-input, patches, Robert Jarzmik
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-01-19 9:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-19 8:43 [PATCH RESEND] input/wm97xx: make missing platform data non-fatal Manuel Lauss
2017-01-19 9:10 ` Charles Keepax
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.