From: Oliver Graute <oliver.graute@gmail.com>
To: u-boot@lists.denx.de
Subject: [RFC 1/2] hack to boot with 2020.01
Date: Wed, 5 Feb 2020 09:48:02 +0100 [thread overview]
Message-ID: <20200205084802.GB9493@optiplex> (raw)
In-Reply-To: <20200204170711.GL13379@bill-the-cat>
On 04/02/20, Tom Rini wrote:
> On Mon, Feb 03, 2020 at 01:59:14PM +0000, Oliver Graute wrote:
> > As proposed here:
> >
> > https://lists.denx.de/pipermail/u-boot/2020-January/396749.html
> >
> > Both of my imx8qm boards (Advantech and Congatec) aren't booting
> > 2020.01 without this change. Whats the proper way to fix this on my side?
> >
> > ---
> > drivers/core/device.c | 7 ++-----
> > 1 file changed, 2 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/core/device.c b/drivers/core/device.c
> > index 4e037083a6..8358051d60 100644
> > --- a/drivers/core/device.c
> > +++ b/drivers/core/device.c
> > @@ -395,11 +395,8 @@ int device_probe(struct udevice *dev)
> >
> > if (CONFIG_IS_ENABLED(POWER_DOMAIN) && dev->parent &&
> > (device_get_uclass_id(dev) != UCLASS_POWER_DOMAIN) &&
> > - !(drv->flags & DM_FLAG_DEFAULT_PD_CTRL_OFF)) {
> > - ret = dev_power_domain_on(dev);
> > - if (ret)
> > - goto fail;
> > - }
> > + !(drv->flags & DM_FLAG_DEFAULT_PD_CTRL_OFF))
> > + dev_power_domain_on(dev);
> >
> > ret = uclass_pre_probe_device(dev);
> > if (ret)
>
> Adding Lokesh and quoting him from
> http://patchwork.ozlabs.org/patch/1211325/
>
> "Can you check by not returning on failure here? If yes then check the
> power-domain/driver that is failing. If any driver doesn't expect core
> to enable power-domain then enable DM_FLAG_DEFAULT_PD_CTRL_OFF in the
> respective driver."
I tried, if I'am not returning on failure here. U-Boot boots well and
any logging shows me a return value 0 for dev_power_domain_on(). As soon
as I add the if statement with the goto fail U-Boot is stuck and I don't
see any msg on the uart neither return value for dev_power_domain_on.
So currently I can't tell which power-domain driver cause the issue.
Best Regards,
Oliver
next prev parent reply other threads:[~2020-02-05 8:48 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-03 13:59 [PATCH v2] imx: support for conga-QMX8 board Oliver Graute
2020-02-03 13:59 ` [RFC 1/2] hack to boot with 2020.01 Oliver Graute
2020-02-03 17:15 ` Simon Glass
2020-02-04 17:07 ` Tom Rini
2020-02-05 8:48 ` Oliver Graute [this message]
2020-02-03 13:59 ` [RFC 2/2] Revert "dm: core: device: switch off power domain after device removal" Oliver Graute
2020-02-13 17:12 ` Neil Armstrong
2020-02-14 4:20 ` Lokesh Vutla
2020-02-14 9:18 ` Oliver Graute
2020-02-14 9:30 ` Lokesh Vutla
2020-02-14 10:30 ` Oliver Graute
2020-02-14 10:01 ` Anatolij Gustschin
2020-02-14 10:14 ` Anatolij Gustschin
2020-02-14 10:22 ` Lokesh Vutla
2020-02-14 10:50 ` Oliver Graute
2020-02-17 8:55 ` Neil Armstrong
2020-02-17 13:17 ` Anatolij Gustschin
2020-02-17 13:05 ` Anatolij Gustschin
2020-02-17 13:37 ` Lokesh Vutla
2020-05-04 7:40 ` [PATCH v2] imx: support for conga-QMX8 board Stefano Babic
2020-05-04 14:25 ` Oliver Graute
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200205084802.GB9493@optiplex \
--to=oliver.graute@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox