On Wed, Jan 9, 2019 at 11:37 PM Sascha Hauer <[email protected]> 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 <[email protected]> > > Cc: Andrey Smirnov <[email protected]> > > Cc: [email protected] > > Cc: [email protected] > > Fixes: 03aa12629fc4 ("soc: imx: Add GPCv2 power gating driver") > > Signed-off-by: Stefan Agner <[email protected]> > > Acked-by: Andrey Smirnov <[email protected]> > > Tested-by: Andrey Smirnov <[email protected]> > > Signed-off-by: Shawn Guo <[email protected]> > > > > Signed-off-by: Andrey Smirnov <[email protected]> > > --- > > 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 [email protected] http://lists.infradead.org/mailman/listinfo/barebox
