On Fri, Oct 12, 2012 at 5:55 PM, Daniel Mack <zonque@xxxxxxxxx> wrote: > This allows DT driven boards to allocate and configure the pinmux once > the driver is probed. > > Signed-off-by: Daniel Mack <zonque@xxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Cc: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> (...) > + /* request pin mux */ > + pinctrl = devm_pinctrl_get_select_default(dev); > + if (IS_ERR(pinctrl)) > + dev_warn(dev, "pins are not configured from the driver\n"); I think dev_warn() is rather nasty to throw in here, dev_info() is OK. However I suspect this driver could actually handle default, idle and sleep states, especially after the runtime PM patches discussed elsewhere, but that can be patched later. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
