From: Anup Patel <Anup.Patel@wdc.com>
To: opensbi@lists.infradead.org
Subject: [PATCH 0/4] reboot SiFive Unmatched via PMIC
Date: Fri, 24 Sep 2021 16:02:07 +0000 [thread overview]
Message-ID: <AE4ED4C2-7921-4CF2-A813-FDF25778F855@wdc.com> (raw)
In-Reply-To: <BCDA27E8-DCA4-4621-8E7C-D0DD8632C710@jrtc27.com>
?On 24/09/21, 9:24 PM, "Jessica Clarke" <jrtc27@jrtc27.com> wrote:
On 24 Sep 2021, at 12:33, Nikita Shubin <nikita.shubin@maquefel.me> wrote:
>
> From: Nikita Shubin <n.shubin@yadro.com>
>
> This series introduce rebooting via i2c PMIC, currently on
> SiFive Unmatched board.
>
> teseted via Linux with reset extension and direct ecall from
> u-boot.
>
> With this it becomes possible to reboot/shutdown the board
> if Linux or u-boot is accompanied with OpenSBI reset extension.
>
> "gpio-poweroff" is required to be removed from u-boot dts file, as
> it overrides fdt_reset_da9063.
I disagree with this. The GPIO has been provided by SiFive specifically
for powering off and so should be used. The DA9063 is only used for
reset because there is no reset GPIO, but it?s a bit of a hacky way of
resetting. IMO the DA9063 driver should be the lowest priority and only
used for poweroff or reset when there is no alternative.
We need to improve sbi_system.h APIs so that platform/drivers can
register callback for particular reset types. This will help reset types
provided different reset drivers and we can use GPIO (for poweroff)
+ DA9063 (for reset) on SiFive Unmatched.
Regards,
Anup
Jess
next prev parent reply other threads:[~2021-09-24 16:02 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-24 11:33 [PATCH 0/4] reboot SiFive Unmatched via PMIC Nikita Shubin
2021-09-24 11:33 ` [PATCH 1/4] lib: utils/i2c: Add generic I2C configuration library Nikita Shubin
2021-09-25 4:19 ` Xiang W
2021-09-28 11:07 ` Nikita Shubin
2021-09-27 15:41 ` Alexandre ghiti
2021-09-28 10:51 ` Nikita Shubin
2021-09-28 11:43 ` Alexandre Ghiti
2021-09-24 11:33 ` [PATCH 2/4] lib: utils/gpio: Add simple FDT based I2C framework Nikita Shubin
2021-09-25 4:39 ` Xiang W
2021-09-27 15:42 ` Alexandre ghiti
2021-09-24 11:33 ` [PATCH 3/4] lib: utils/gpio: Add minimal SiFive I2C driver Nikita Shubin
2021-09-27 15:42 ` Alexandre ghiti
2021-09-24 11:33 ` [PATCH 4/4] lib: utils/reset: Add generic da9063 reset driver Nikita Shubin
2021-09-27 15:42 ` Alexandre ghiti
2021-09-28 10:56 ` Nikita Shubin
2021-09-24 15:54 ` [PATCH 0/4] reboot SiFive Unmatched via PMIC Jessica Clarke
2021-09-24 16:02 ` Anup Patel [this message]
2021-09-24 16:06 ` Nikita Shubin
2021-09-24 16:09 ` Anup Patel
2021-09-24 16:20 ` Nikita Shubin
2021-09-24 16:36 ` Nikita Shubin
2021-09-25 1:10 ` Bin Meng
2021-09-25 3:56 ` Anup Patel
2021-09-25 4:45 ` Bin Meng
2021-09-25 6:34 ` Anup Patel
2021-09-25 8:11 ` Heinrich Schuchardt
2021-09-28 10:32 ` Nikita Shubin
2021-09-29 12:46 ` Anup Patel
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=AE4ED4C2-7921-4CF2-A813-FDF25778F855@wdc.com \
--to=anup.patel@wdc.com \
--cc=opensbi@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