pinctrl: Initialize pinctrl_dev.node
authorThierry Reding <treding@nvidia.com>
Thu, 12 Jan 2017 16:03:34 +0000 (17:03 +0100)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 13 Jan 2017 15:30:52 +0000 (16:30 +0100)
commit46daed6ebd4bf357f77d231075f1b001f8707b48
tree124a6c0c3084b560dfa003af586a52da99d8a86a
parent950b0d91dc108f54bccca5a2f75bb46f2df63d29
pinctrl: Initialize pinctrl_dev.node

The struct pinctrl_dev's node field is not properly set up, which means
the .prev and .next fields will be NULL. That's not something that the
linked list code can deal with, so extra care must be taken when using
these fields. An example of this is introduced in commit 3429fb3cda34
("pinctrl: Fix panic when pinctrl devices with hogs are unregistered")
where list_del() is made conditional on the pinctrl device being part
of the pinctrl device list. This is to ensure that list_del() won't
crash upon encountering a NULL pointer in .prev and/or .next.

After initializing the list head there's no need to jump through these
extra hoops and list_del() will work unconditionally. This is because
the initialized list head points to itself and therefore the .prev and
.next fields can be properly dereferenced.

Signed-off-by: Thierry Reding <treding@nvidia.com>
Acked-by: Jon Hunter <jonathanh@nvidia.com>
Tested-by: Jon Hunter <jonathanh@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/core.c