Re: [PATCH] Revert "thermal: rockchip: fix up the tsadc pinctrl setting error"

From: Eduardo Valentin
Date: Thu May 23 2019 - 22:23:44 EST


On Wed, May 22, 2019 at 08:34:34AM -0700, Doug Anderson wrote:
> Hi,
>
> On Wed, May 22, 2019 at 7:12 AM Heiko Stuebner <heiko@xxxxxxxxx> wrote:
> >
> > This reverts commit 28694e009e512451ead5519dd801f9869acb1f60.
> >
> > The commit causes multiple issues in that:
> > - the added call to ->control does potentially run unclocked
> > causing a hang of the machine
> > - the added pinctrl-states are undocumented in the binding
> > - the added pinctrl-states are not backwards compatible, breaking
> > old devicetrees.
> >
> > Fixes: 28694e009e51 ("thermal: rockchip: fix up the tsadc pinctrl setting error")
> > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx>
> > ---
> > drivers/thermal/rockchip_thermal.c | 36 +++---------------------------
> > 1 file changed, 3 insertions(+), 33 deletions(-)
>
> In case it helps with the urgency, there are lots of people who have
> all independently needed to identify which commit stopped their boards
> from booting mainline or broke temperature reading. I'm aware of at
> least these reports:
>
> Reported-by: kernelci.org bot <bot@xxxxxxxxxxxx>
> Reported-by: Enric Balletbo Serra <eballetbo@xxxxxxxxx>
> Reported-by: Vicente Bergas <vicencb@xxxxxxxxx>
> Reported-by: Jack Mitchell <ml@xxxxxxxxxxx>
> Reported-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
>
> I can confirm that my board boots again after this revert, thus:
>
> Tested-by: Douglas Anderson <dianders@xxxxxxxxxxxx>

I see. I can collect this revert if that is what unblocks you guys.

Elaine, do you mind checking why these pinctrl changes are causing
boot issues?