summaryrefslogtreecommitdiffstats
path: root/drivers/pinctrl/pinctrl-sx150x.c
diff options
context:
space:
mode:
authorPeter Rosin <peda@axentia.se>2016-12-02 11:51:16 +0100
committerLinus Walleij <linus.walleij@linaro.org>2016-12-07 15:27:03 +0100
commit283dc0be75b2fd870ae32a080a568f67c9ac9a4e (patch)
treedf96b1edc92abf67ec9db85b246b3fd0b5f914ed /drivers/pinctrl/pinctrl-sx150x.c
parentc9d26f1aab007a7083f509b4be263bc904ddb0d3 (diff)
downloadblackbird-op-linux-283dc0be75b2fd870ae32a080a568f67c9ac9a4e.tar.gz
blackbird-op-linux-283dc0be75b2fd870ae32a080a568f67c9ac9a4e.zip
pinctrl: sx150x: handle missing 'advanced' reg in sx1504 and sx1505
This fixes a problem where sx150x_regmap_reg_width() returns 8 for the data register (reg 0) for sx1504 where it should return 4, and return a correct 8 for sx1505 but for the wrong reason (both chips lack the 'advanced' register). This is not a real problem, since nothing depends on the function returning 4 or 8, and certainly not if it is returning 8 for the wrong reason. But fix this to avoid nasty surprises down the line. Signed-off-by: Peter Rosin <peda@axentia.se> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinctrl-sx150x.c')
-rw-r--r--drivers/pinctrl/pinctrl-sx150x.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pinctrl/pinctrl-sx150x.c b/drivers/pinctrl/pinctrl-sx150x.c
index 37b8737e61a9..bdb7ea3d9911 100644
--- a/drivers/pinctrl/pinctrl-sx150x.c
+++ b/drivers/pinctrl/pinctrl-sx150x.c
@@ -967,6 +967,7 @@ static int sx150x_regmap_reg_width(struct sx150x_pinctrl *pctl,
reg == data->pri.x123.reg_advanced)
||
(data->model == SX150X_456 &&
+ data->pri.x456.reg_advanced &&
reg == data->pri.x456.reg_advanced)) {
return 8;
} else {
OpenPOWER on IntegriCloud