Commit 2b7950c7 authored by Sergey Shtylyov's avatar Sergey Shtylyov Committed by Sebastian Reichel
Browse files

power: supply: mp2629_charger: use platform_get_irq() 



Calling platform_get_irq_optional() doesn't make sense if you then bail out
on any error it returns.  Switch to calling platform_get_irq() instead and
remove dev_err() call as platform_get_irq() already curses loudly on error.

Signed-off-by: default avatarSergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
parent 441d38c6

drivers/power/supply/mp2629_charger.c

+2 −4
Original line number Diff line number Diff line
@@ -580,11 +580,9 @@ static int mp2629_charger_probe(struct platform_device *pdev) 
	charger->dev = dev;

	platform_set_drvdata(pdev, charger);



	irq = platform_get_irq_optional(to_platform_device(dev->parent), 0);

	if (irq < 0) {

		dev_err(dev, "get irq fail: %d\n", irq);

	irq = platform_get_irq(to_platform_device(dev->parent), 0);

	if (irq < 0)

		return irq;

	}



	for (i = 0; i < MP2629_MAX_FIELD; i++) {

		charger->regmap_fields[i] = devm_regmap_field_alloc(dev,