On Wed, Jan 9, 2019 at 11:37 PM Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote:
>
> On Tue, Jan 08, 2019 at 11:11:59PM -0800, Andrey Smirnov wrote:
> > Port of a Linux commit 9e01e2d56db23485a75864b6aeee8e443f024ddb
> >
> > If a regulator requests a deferred probe, the power domain gets
> > initialized twice. This leads to a list double add (without
> > list debugging the kernel hangs due to the double add later):
> >
> > WARNING: CPU: 0 PID: 19 at lib/list_debug.c:31 __list_add_valid+0xbc/0xc4
> > list_add double add: new=c1229754, prev=c12383b4, next=c1229754.
> >
> > Initialize the power domain after we get the regulator. Also do
> > not print an error in case the regulator defers probing.
> >
> > Cc: Fabio Estevam <fabio.estevam@xxxxxxx>
> > Cc: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> > Cc: linux-kernel@xxxxxxxxxxxxxxx
> > Fixes: 03aa12629fc4 ("soc: imx: Add GPCv2 power gating driver")
> > Signed-off-by: Stefan Agner <stefan@xxxxxxxx>
> > Acked-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> > Tested-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> > Signed-off-by: Shawn Guo <shawnguo@xxxxxxxxxx>
> >
> > Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> > ---
> > drivers/soc/imx/gpcv2.c | 15 ++++++++-------
> > 1 file changed, 8 insertions(+), 7 deletions(-)
>
> This file has been added in the last patch. Please squash this patch and
> the next one into it.
>
I purposefully kept this patch separate, since the bugs weren't deal
breakers (PCI still worked despite my bugs) and it would be easier to
see that the fixes that came later were back-ported to BB. I guess I
can squash this in v2.
Thanks,
Andrey Smirnov
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
