linux-riscv.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Anup Patel <Anup.Patel@wdc.com>
To: Troy Benjegerdes <troy.benjegerdes@sifive.com>,
	Anup Patel <anup@brainfault.org>
Cc: "Björn Töpel" <bjorn.topel@gmail.com>,
	"Palmer Dabbelt" <palmer@sifive.com>,
	"Atish Patra" <Atish.Patra@wdc.com>,
	"Lukas Auer" <lukas.auer@aisec.fraunhofer.de>,
	"Paul Walmsley" <paul.walmsley@sifive.com>,
	"linux-riscv@lists.infradead.org"
	<linux-riscv@lists.infradead.org>,
	"Bin Meng" <bmeng.cn@gmail.com>
Subject: RE: 5.2-rc1 boot on Unleashed
Date: Wed, 29 May 2019 05:00:50 +0000	[thread overview]
Message-ID: <MN2PR04MB60610DD66EC6E93C25D18F5F8D1F0@MN2PR04MB6061.namprd04.prod.outlook.com> (raw)
In-Reply-To: <4572851B-EF46-45A1-A806-663894673EF9@sifive.com>



> -----Original Message-----
> From: linux-riscv <linux-riscv-bounces@lists.infradead.org> On Behalf Of
> Troy Benjegerdes
> Sent: Saturday, May 25, 2019 7:46 PM
> To: Anup Patel <anup@brainfault.org>
> Cc: Palmer Dabbelt <palmer@sifive.com>; Björn Töpel
> <bjorn.topel@gmail.com>; Anup Patel <Anup.Patel@wdc.com>; Atish Patra
> <Atish.Patra@wdc.com>; Lukas Auer <lukas.auer@aisec.fraunhofer.de>;
> Paul Walmsley <paul.walmsley@sifive.com>; linux-riscv@lists.infradead.org;
> Bin Meng <bmeng.cn@gmail.com>
> Subject: Re: 5.2-rc1 boot on Unleashed
> 
> 
> 
> > On May 25, 2019, at 3:06 AM, Anup Patel <anup@brainfault.org> wrote:
> >
> > Hi Troy,
> >
> > On Thu, May 23, 2019 at 3:45 AM Troy Benjegerdes
> > <troy.benjegerdes@sifive.com> wrote:
> >>
> >>
> >>> On May 22, 2019, at 3:43 PM, Paul Walmsley
> <paul.walmsley@sifive.com> wrote:
> >>>
> >>> + Anup, Troy
> >>>
> >>> On Wed, 22 May 2019, Atish Patra wrote:
> >>>
> >>>> On 5/22/19 9:42 AM, Paul Walmsley wrote:
> >>>>> Am checking on this now and hope to have some conclusion on it this
> week.
> >>>>
> >>>> Thanks. Any chance you can take a look at the U-Boot clock driver
> >>>> to patch it as per the DT changes ?
> >>>
> >>> I probably won't have the chance to get to it for a while.  Looks
> >>> like Anup was the one who posted the clock driver to U-Boot - can he
> >>> take a look at it?
> >>>
> >>> We've also asked Troy to look at upstream U-boot related issues, but
> >>> I'm unsure when patches will start flowing upstream.
> >>>
> >>>
> >>> - Paul
> >>
> >> Once I figure out why I’m getting TX timeouts on the macb driver and
> have something working I can start upstreaming the u-boot patches.
> >>
> >> For now, the work-in-progress is at
> >> https://github.com/sifive/u-boot/tree/sandbox
> >
> > Based on your commits in above GitRepo, it seems you are trying entire
> > U-Boot in M-mode.
> >
> > Current boot-flow of S-mode U-Boot is:
> > ZSBL (M-mode) -> FSBL (M-mode) -> OpenSBI/BBL (M-mode) -> U-Boot
> > (S-mode)
> >
> > The U-Boot SPL can perfectly replace SiFive FSBL to have following
> > boot-flow:
> > ZSBL (M-mode) -> U-Boot SPL (M-mode) -> OpenSBI/BBL (M-mode) -> U-
> Boot
> > (S-mode)
> >
> > Can you explain advantages of using full U-Boot M-mode to replace FSBL
> > as compared to U-Boot SPL M-mode replacing FSBL ?
> >
> > Regards,
> > Anup
> 
> The current flow in Freedom-u-sdk (with
> https://github.com/sifive/HiFive_U-Boot) is this:
> ZSBL (M-mode) -> U-Boot (M-mode) -> BBL -> Linux kernel
> 
> The major driver for full U-Boot M-mode is automated regression testing and
> being able to load the SBI interface (BBL or OpenSBI) and linux kernel via
> TFTP.

This is a very specific use-case. The RISC-V boot flow should be compliant with
major architectures (ARM/ARM64 and x86) so that it's very easy (and boring)
for folks (familiar with these major architectures) to try-out RISC-V.

To achieve this, we need to ensure that users are able to boot kernel Image
and Image.gz without embedding it in OpenSBI/BBL. In fact, we also need to
ensure that users can update SBI runtime (OpenSBI/BBL) and bootloader
(i.e. U-Boot) separately.

If we try to address above requirements then we end-up having U-Boot S-mode
(or any S-mode bootloader) as last stage before kernel and SBI runtime will be
before U-Boot S-mode as separate binary.

Due to above, we have come-up with FW_JUMP and FW_DYNAMIC firmwares
in OpenSBI. These firmwares don't embed Linux kernel or Bootloader binary
and can be loaded separately.

> 
> I do agree U-Boot SPL is a very good idea, I have been working on replicating
> the functionality of the old HiFive U-boot first and then once that works I
> think it will be a lot easier to work out the SPL option.

Yes, U-Boot SPI (M-mode) will be the right way to go and it will fit in lot
of use-cases.

Regards,
Anup
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

  reply	other threads:[~2019-05-29  5:01 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-20 23:31 5.2-rc1 boot on Unleashed Atish Patra
2019-05-21  9:40 ` Andreas Schwab
2019-05-21 18:14   ` Atish Patra
2019-05-22  7:31     ` Andreas Schwab
2019-05-22  8:40     ` Andreas Schwab
2019-05-22  9:58     ` Andreas Schwab
2019-05-22 21:13       ` Paul Walmsley
2019-05-21 17:47 ` Kevin Hilman
2019-05-21 18:18   ` Atish Patra
2019-05-23  9:42     ` Andreas Schwab
2019-05-22  4:42 ` Sagar Kadam
2019-05-22  9:51   ` Atish Patra
2019-05-22 16:42 ` Paul Walmsley
2019-05-22 17:38   ` Atish Patra
2019-05-22 20:43     ` Paul Walmsley
2019-05-22 22:14       ` Troy Benjegerdes
2019-05-25  8:06         ` Anup Patel
2019-05-25 14:16           ` Troy Benjegerdes
2019-05-29  5:00             ` Anup Patel [this message]
2019-05-29  8:31               ` Andreas Schwab
2019-05-29 17:38                 ` Troy Benjegerdes
2019-05-29 18:59                   ` Karsten Merker
2019-05-30 17:00                     ` Palmer Dabbelt
2019-06-03 10:49                     ` Andreas Schwab
2019-06-03 14:44                       ` Distro bootcmd: " Troy Benjegerdes
2019-06-03 17:02                         ` [U-Boot] " Tom Rini
2019-06-03 19:17                           ` Palmer Dabbelt
2019-06-03 19:32                             ` Tom Rini
2019-06-03 19:47                           ` Troy Benjegerdes
2019-05-24  0:23       ` Troy Benjegerdes
2019-05-24  3:59         ` Atish Patra
2019-05-24 14:46           ` Troy Benjegerdes
2019-05-24 15:35             ` Auer, Lukas
2019-05-24 16:01               ` Troy Benjegerdes
2019-05-24 17:40                 ` Atish Patra
     [not found]                   ` <CAF5mof3_PSwpsSMnjMALpfvmOPdeAgDLa2p6jp4y+Hmas83mbw@mail.gmail.com>
2019-05-24 21:34                     ` Auer, Lukas
2019-05-25  3:49                       ` Anup Patel
2019-05-25 16:23                         ` Troy Benjegerdes
2019-05-25 18:09             ` Paul Walmsley
2019-05-25 18:40               ` Troy Benjegerdes
2019-05-27 12:21                 ` Andreas Schwab
2019-05-27 16:51                   ` Troy Benjegerdes

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=MN2PR04MB60610DD66EC6E93C25D18F5F8D1F0@MN2PR04MB6061.namprd04.prod.outlook.com \
    --to=anup.patel@wdc.com \
    --cc=Atish.Patra@wdc.com \
    --cc=anup@brainfault.org \
    --cc=bjorn.topel@gmail.com \
    --cc=bmeng.cn@gmail.com \
    --cc=linux-riscv@lists.infradead.org \
    --cc=lukas.auer@aisec.fraunhofer.de \
    --cc=palmer@sifive.com \
    --cc=paul.walmsley@sifive.com \
    --cc=troy.benjegerdes@sifive.com \
    /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;
as well as URLs for NNTP newsgroup(s).