pinctrl: devicetree: Fix dt_to_map_one_config handling of hogs
authorRichard Fitzgerald <rf@opensource.cirrus.com>
Wed, 28 Feb 2018 15:53:06 +0000 (15:53 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 30 May 2018 05:52:37 +0000 (07:52 +0200)
commit72678f7a2922a719610e076eaf9f97ddc5481d7a
tree8ae3ce67d69685f6d2dac1a3ce09ff7e24f99830
parent39c655c5edfe629fdef92efaecd85a05b958f9a2
pinctrl: devicetree: Fix dt_to_map_one_config handling of hogs

[ Upstream commit b89405b6102fcc3746f43697b826028caa94c823 ]

When dt_to_map_one_config() is called with a pinctrl_dev passed
in, it should only be using this if the node being looked up
is a hog. The code was always using the passed pinctrl_dev
without checking whether the dt node referred to it.

A pin controller can have pinctrl-n dependencies on other pin
controllers in these cases:

- the pin controller hardware is external, for example I2C, so
  needs other pin controller(s) to be setup to communicate with
  the hardware device.

- it is a child of a composite MFD so its of_node is shared with
  the parent MFD and other children of that MFD. Any part of that
  MFD could have dependencies on other pin controllers.

Because of this, dt_to_map_one_config() can't assume that if it
has a pinctrl_dev passed in then the node it looks up must be
a hog. It could be a reference to some other pin controller.

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/pinctrl/devicetree.c