From: "Cédric Le Goater" <clg@kaod.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Andrew Jeffery <andrew@aj.id.au>,
qemu-arm@nongnu.org, Joel Stanley <joel@jms.id.au>,
Peter Delevoryas <pdel@fb.com>,
qemu-devel@nongnu.org
Subject: Re: [PATCH 0/4] aspeed/smc: Improve support for the alternate boot function
Date: Mon, 18 Oct 2021 10:54:51 +0200 [thread overview]
Message-ID: <0b06bada-4f7e-1d22-a81f-8c0241a9dcdc@kaod.org> (raw)
In-Reply-To: <20211004154635.394258-1-clg@kaod.org>
On 10/4/21 17:46, Cédric Le Goater wrote:
> Hello,
>
> The Aspeed SoCs have a dual boot function for firmware fail-over
> recovery. The system auto-reboots from the second flash if the main
> flash does not boot successfully within a certain amount of time. This
> function is called alternate boot (ABR) in the FMC controllers.
>
> On the AST2600, the ABR registers controlling the 2nd watchdog timer
> were moved from the watchdog register to the FMC controller. To
> control WDT2 through the FMC model register set, this series creates a
> local address space on top of WDT2 memory region.
>
> To test on the fuji-bmc machine, run :
>
> devmem 0x1e620064
> devmem 0x1e78504C
>
> devmem 0x1e620064 32 0xffffffff
> devmem 0x1e620064
> devmem 0x1e78504C
>
> Thanks
>
> C.
>
>
> Cédric Le Goater (4):
> aspeed/wdt: Add trace events
> aspeed/smc: Dump address offset in trace events
> aspeed/wdt: Add an alias for the MMIO region
> aspeed/smc: Improve support for the alternate boot function
Andrew, Peter D., Joel,
Would you have time to tell me what you think about the last 2 patches ?
It would be a nice extension for the Fuji in QEMU 6.2.
Here are some images for tests.
https://github.com/peterdelevoryas/openbmc/releases/download/fuji-v0.1-alpha/fuji.mtd
This one is recent :
https://github.com/peterdelevoryas/openbmc/releases/download/fuji.mtd.0/fuji.mtd
U-Boot 2019.04 fuji-bd6ee58668 (Sep 13 2021 - 21:29:46 +0000)
[ 0.000000] Linux version 5.10.23-fuji (oe-user@oe-host) (arm-fb-linux-gnueabi-gcc (GCC) 9.3.0, GNU ld (GNU Binutils) 2.34.0.20200220) #1 SMP Thu Sep 9 23:22:29 UTC 2021
Thanks,
C.
>
> include/hw/ssi/aspeed_smc.h | 3 ++
> include/hw/watchdog/wdt_aspeed.h | 1 +
> hw/arm/aspeed_ast2600.c | 2 +
> hw/ssi/aspeed_smc.c | 84 ++++++++++++++++++++++++++++++--
> hw/watchdog/wdt_aspeed.c | 20 ++++++--
> hw/ssi/trace-events | 1 +
> hw/watchdog/trace-events | 4 ++
> 7 files changed, 107 insertions(+), 8 deletions(-)
>
prev parent reply other threads:[~2021-10-18 8:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-04 15:46 [PATCH 0/4] aspeed/smc: Improve support for the alternate boot function Cédric Le Goater
2021-10-04 15:46 ` [PATCH 1/4] aspeed/wdt: Add trace events Cédric Le Goater
2021-10-05 9:16 ` Francisco Iglesias
2021-10-18 8:57 ` Philippe Mathieu-Daudé
2021-10-04 15:46 ` [PATCH 2/4] aspeed/smc: Dump address offset in " Cédric Le Goater
2021-10-05 8:55 ` Francisco Iglesias
2021-10-18 8:58 ` Philippe Mathieu-Daudé
2021-10-04 15:46 ` [PATCH 3/4] aspeed/wdt: Add an alias for the MMIO region Cédric Le Goater
2021-10-18 9:04 ` Philippe Mathieu-Daudé
2021-10-18 13:07 ` Cédric Le Goater
2021-10-04 15:46 ` [PATCH 4/4] aspeed/smc: Improve support for the alternate boot function Cédric Le Goater
2021-10-18 8:54 ` Cédric Le Goater [this message]
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=0b06bada-4f7e-1d22-a81f-8c0241a9dcdc@kaod.org \
--to=clg@kaod.org \
--cc=andrew@aj.id.au \
--cc=joel@jms.id.au \
--cc=pdel@fb.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.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 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.