All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 3/3] sunxi: Normalise FEL support
Date: Sat, 7 Feb 2015 11:02:04 -0700	[thread overview]
Message-ID: <CAPnjgZ3PxH+ReJ785YAXTAGVDjv+146tz6zNig+mCzYQyRCEEA@mail.gmail.com> (raw)
In-Reply-To: <54D6526A.7060404@redhat.com>

Hi Hans,

On 7 February 2015 at 10:59, Hans de Goede <hdegoede@redhat.com> wrote:
> Hi,
>
> On 02/07/2015 06:47 PM, Simon Glass wrote:
>>
>> Make sunxi's FEL code fit with the normal U-Boot boot sequence instead of
>> creating its own. There are some #ifdefs required in start.S. Future work
>> will hopefully remove these.
>
>
> About the #ifdefs, I would really like to see us move to having 1 unified
> loader for sunxi, which means getting rid of them. Even though we do use
> a label to return from save_boot_params, save_boot_params could still
> use r0 to return stuff, like you did in your previous patch-set, or we
> could add a global variable to start.S which lives in .data and gets
> initialized with 0, and save_boot_params could optionally save some
> skip flags there.
>
> Note this dropping of #ifdefs can wait till after v2015.04, for now this
> patch-set should get fel mode working as before which is the goal for
> v2015.04.

OK. It's just as easy for me to do this now, but Albert was not keen
on doing this at run-time.

So if you can remove the #ifdefs by calling back into the BROM that
would probably be better. For now I think Albert prefers the #ifdefs.

>> This series is available at u-boot-dm, branch sunxi-working.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>

[snip]

Regards,
Simon

  reply	other threads:[~2015-02-07 18:02 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-07 17:47 [U-Boot] [PATCH v2 1/3] arm: Allow lr to be saved by board code Simon Glass
2015-02-07 17:47 ` [U-Boot] [PATCH v2 2/3] arm: spl: Provide for a board-specific loader Simon Glass
2015-02-08  3:44   ` Siarhei Siamashka
2015-02-07 17:47 ` [U-Boot] [PATCH v2 3/3] sunxi: Normalise FEL support Simon Glass
2015-02-07 17:59   ` Hans de Goede
2015-02-07 18:02     ` Simon Glass [this message]
2015-02-07 21:46       ` Hans de Goede
2015-02-08  3:48   ` Siarhei Siamashka
2015-02-09 22:23     ` Simon Glass
2015-02-11  3:05       ` Siarhei Siamashka
2015-02-11  4:45         ` Simon Glass
2015-02-08  3:43 ` [U-Boot] [PATCH v2 1/3] arm: Allow lr to be saved by board code Siarhei Siamashka

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=CAPnjgZ3PxH+ReJ785YAXTAGVDjv+146tz6zNig+mCzYQyRCEEA@mail.gmail.com \
    --to=sjg@chromium.org \
    --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.