All of lore.kernel.org
 help / color / mirror / Atom feed
From: Urja Rannikko <urjaman@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/3] sysreset: switch to using SYSRESET_POWER_OFF for poweroff
Date: Thu, 16 May 2019 11:37:17 +0000	[thread overview]
Message-ID: <CAPCnQJ=t+gF5jo1TwSjLGiLNChnZRE6CBRwxJSo5=jG3FpMpSg@mail.gmail.com> (raw)
In-Reply-To: <64df8c3cf46b4941b17d7cd60ad831b9@SFHDAG6NODE3.st.com>

Hi,

On Thu, May 16, 2019 at 7:44 AM Patrick DELAUNAY
<patrick.delaunay@st.com> wrote:
>
> Hi Urja,
>
> > -     if (type != SYSRESET_POWER)
> > +     if (type != SYSRESET_POWER_OFF)
> >               return -EPROTONOSUPPORT;
>
> In fact in the next part of the code, we are supporting
> only SYSRESET_POWER (reset with PMIC1 switch OFF and restart)
> and not SYSRESET_POWER_OFF....
>
> because Power Cycle if RREQ_EN=1
>
> I think you need to remove the update on this file for your patch and
> I will modified this function is to support both mode....
Okay I will drop this part, but just to confirm... this means that
currently you have a configuration where using the poweroff command
causes a power-cycle? Neat.

-- 
Urja Rannikko

  reply	other threads:[~2019-05-16 11:37 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-13 12:17 [U-Boot] [PATCH 1/3] sysreset: switch to using SYSRESET_POWER_OFF for poweroff Urja Rannikko
2019-05-13 12:17 ` [U-Boot] [PATCH 2/3] sysreset: move stm32mp sysreset poweroff implementation to sysreset uclass Urja Rannikko
2019-05-14 16:07   ` Patrice CHOTARD
2019-05-16 11:41     ` Urja Rannikko
2019-05-13 12:17 ` [U-Boot] [PATCH 3/3] rk8xx: add a sysreset driver for poweroff Urja Rannikko
2019-05-18 16:08   ` Simon Glass
2019-05-16  7:44 ` [U-Boot] [PATCH 1/3] sysreset: switch to using SYSRESET_POWER_OFF " Patrick DELAUNAY
2019-05-16 11:37   ` Urja Rannikko [this message]
2019-05-20 13:38     ` Patrick DELAUNAY
2019-07-26  7:37       ` Kever Yang

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='CAPCnQJ=t+gF5jo1TwSjLGiLNChnZRE6CBRwxJSo5=jG3FpMpSg@mail.gmail.com' \
    --to=urjaman@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.