All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.