public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Anatolij Gustschin <agust@denx.de>
To: u-boot@lists.denx.de
Subject: [PATCH] power-domain: fix hang in endless loop on i.MX8
Date: Sun, 16 Feb 2020 20:24:38 +0100	[thread overview]
Message-ID: <20200216202438.2d6e30d1@crub> (raw)
In-Reply-To: <CAPnjgZ3Jui=VCyzXY_zziB+b4VLybLAuCay3cXLbc+36z1ys3w@mail.gmail.com>

Hi Simon, Peng,

On Sun, 16 Feb 2020 12:02:55 -0700
Simon Glass sjg at chromium.org wrote:
...
> >  drivers/power/domain/power-domain-uclass.c | 12 ++++++++++++
> >  1 file changed, 12 insertions(+)  
> 
> Can we have a sandbox test for this case?

I didn't check if there are more platforms with such parent & child
power domain devices in U-Boot. If there are some, then it makes
sense to add a test. But for this i.MX8 case we will probably switch
to another power domain bindings and driver like in Linux (we have
to sync i.MX8 device trees with current Linux implementation which
is different and it doesn't have such parent/child PD devices).

The current i.MX8 PD U-Boot implementation seems wrong anyway, here
the parent power domain devices for dma, connectivity or lsio domains
actually do not control power switching, so there is no need to have
such parent&child PD devices on i.MX8.

@Peng: what do you think?

--
Anatolij

  reply	other threads:[~2020-02-16 19:24 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-15 11:34 [PATCH] power-domain: fix hang in endless loop on i.MX8 Anatolij Gustschin
2020-02-16 19:02 ` Simon Glass
2020-02-16 19:24   ` Anatolij Gustschin [this message]
2020-02-17  2:44     ` Peng Fan
2020-02-17  3:27 ` Lokesh Vutla
2020-02-17  9:07 ` Oliver Graute
2020-02-17  9:12   ` Neil Armstrong

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=20200216202438.2d6e30d1@crub \
    --to=agust@denx.de \
    --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