From: hkallweit1@gmail.com (Heiner Kallweit)
To: linus-amlogic@lists.infradead.org
Subject: Faults on Odroid-C2 during shutdown and fix proposal
Date: Sat, 16 Dec 2017 00:19:56 +0100 [thread overview]
Message-ID: <c28e0ecf-9bf3-f6e8-b8c8-91b14bf797f6@gmail.com> (raw)
Neil,
during shutdown I experience faults as described in the commit message of
339cd0ea0822 "soc: amlogic: meson-gx-pwrc-vpu: fix power-off when powered by bootloader"
My best guess:
I operate the board headless, therefore the domain is never powered on, leaving the
clocks disabled. The shutdown function then tries to disable the already disabled
clocks.
The following change fixes it for me, however there may be scenarios I miss,
therefore I'd like to hear your opinion.
static void meson_gx_pwrc_vpu_shutdown(struct platform_device *pdev)
{
- meson_gx_pwrc_vpu_power_off(&vpu_hdmi_pd.genpd);
+ bool powered_off;
+
+ powered_off = meson_gx_pwrc_vpu_get_power(&vpu_hdmi_pd);
+ if (!powered_off)
+ meson_gx_pwrc_vpu_power_off(&vpu_hdmi_pd.genpd);
}
And something completely different:
I get a misleading error message "failed to get reset lines", caused by the fact
that devm_reset_control_array_get() returns -EPROBE_DEFER. So I think we should
print the error message only if the error is not caused by a probe deferral.
Kind regards,
Heiner
next reply other threads:[~2017-12-15 23:19 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-15 23:19 Heiner Kallweit [this message]
2017-12-19 10:28 ` 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=c28e0ecf-9bf3-f6e8-b8c8-91b14bf797f6@gmail.com \
--to=hkallweit1@gmail.com \
--cc=linus-amlogic@lists.infradead.org \
/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
all inboxes | Powered by JetHome®