On Fri, Jan 30, 2015 at 11:27 AM, Stanimir Varbanov <svarbanov@xxxxxxxxxx> wrote: > This makes the pinctrl driver to use the generic pinconf > interface. Mainly it gives us a way to use debugfs to dump > group configurations. > > Signed-off-by: Stanimir Varbanov <svarbanov@xxxxxxxxxx> Nice! Looking for Björn's review though. > +++ b/drivers/pinctrl/qcom/pinctrl-msm.c > @@ -197,7 +197,6 @@ static int msm_config_reg(struct msm_pinctrl *pctrl, > *mask = 1; > break; > default: > - dev_err(pctrl->dev, "Invalid config param %04x\n", param); > return -ENOTSUPP; > } > > @@ -262,8 +261,6 @@ static int msm_config_group_get(struct pinctrl_dev *pctldev, > arg = !!(val & BIT(g->in_bit)); > break; > default: > - dev_err(pctrl->dev, "Unsupported config parameter: %x\n", > - param); > return -EINVAL; Please change this one to -ENOTSUPP as well in this patch. Apart from that it looks good to me. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
