Re: [PATCH] pinctrl: implement pinctrl deferred probing

From: Linus Walleij
Date: Wed Apr 11 2012 - 04:35:35 EST


On Tue, Apr 10, 2012 at 5:36 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:

>> @@ -521,8 +521,11 @@ static int add_setting(struct pinctrl *p, struct pinctrl_map const *map)
>>               dev_err(p->dev, "unknown pinctrl device %s in map entry",
>>                       map->ctrl_dev_name);
>
> Perhaps this dev_err (and the one in the other hunk) should be
> downgraded to warn/info/debug?

I've downgraded to dev_info() for now (see v2), I suspect people may have some
problems with deferred probing ATM so better log something for them. Later we
may set it to dev_dbg() if it looks too noisy...

Thanks,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/