Re: [PATCH v2 1/3] regulator: fixed: add possibility to enable by clock

[Guenter Roeck]

On 10/7/19 6:57 AM, Philippe Schenker wrote:
> On Mon, 2019-10-07 at 06:29 -0700, Guenter Roeck wrote:
> > On Tue, Sep 10, 2019 at 06:21:15AM +0000, Philippe Schenker wrote:
> > > This commit adds the possibility to choose the compatible
> > > "regulator-fixed-clock" in devicetree.
> > >
> > > This is a special regulator-fixed that has to have a clock, from
> > > which
> > > the regulator gets switched on and off.
> > >
> > > Signed-off-by: Philippe Schenker <philippe.schenker@xxxxxxxxxxx>
> >
> > This patch causes a crash in one of my qemu tests (kzm machine with
> > imx_v6_v7_defconfig). Reverting this patch fixes the problem.
> > Crash backtrace and bisect log attached below.
> >
> > Guenter
>
> Axel Lin sent a patch to fix this NULL pointer issue with my patch
> (Thanks by the way :)). I guess it will help for your issue. Could you
> please test if this patch solves your issue?

Yes, it does. I replied with Tested-by: on the fixup patch.

Thanks,
Guenter