OpenSBI Archive on lore.kernel.org
 help / color / mirror / Atom feed
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



  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