Thread (19 messages) 19 messages, 2 authors, 2014-02-27

[PATCH v3 1/9] Input: pmic8xxx-pwrkey - Migrate to regmap APIs

From: Josh Cartwright <hidden>
Date: 2014-02-27 00:02:39
Also in: linux-arm-msm, lkml

On Wed, Feb 26, 2014 at 11:05:54AM -0800, Stephen Boyd wrote:
quoted hunk ↗ jump to hunk
Use the regmap APIs for this driver instead of custom pm8xxx
APIs. This breaks this driver's dependency on the pm8xxx APIs and
allows us to easily port it to other bus protocols in the future.

Signed-off-by: Stephen Boyd <redacted>
---
 drivers/input/misc/pmic8xxx-pwrkey.c | 4 ++++
 1 file changed, 4 insertions(+)
diff --git a/drivers/input/misc/pmic8xxx-pwrkey.c b/drivers/input/misc/pmic8xxx-pwrkey.c
index 0e1a05f95858..cf6125dc3b51 100644
--- a/drivers/input/misc/pmic8xxx-pwrkey.c
+++ b/drivers/input/misc/pmic8xxx-pwrkey.c
@@ -120,6 +120,10 @@ static int pmic8xxx_pwrkey_probe(struct platform_device *pdev)
 		return -ENOMEM;
 	}
 
+	regmap = dev_get_regmap(pdev->dev.parent, NULL);
+	if (!regmap)
+		return -ENODEV;
+
This looks really weird.  A previous version of patch was included in
v3.14-rc1 (1e63bd9cc4), which already included a very similar hunk...

Maybe the fact that the hunk isn't identical is why a rebase didn't drop
it (and instead, now if this gets merged, we'll have unnecessary
duplication).

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help