public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/5] RFC: sunxi: WIP FEL support
Date: Thu, 05 Feb 2015 09:40:04 +0100	[thread overview]
Message-ID: <54D32C64.1060603@redhat.com> (raw)
In-Reply-To: <CAPnjgZ1FHRxEv5o_jPG+udqLZQOoTGSWO5qPY+4_Zjb1=JZHUQ@mail.gmail.com>

Hi,

On 05-02-15 03:52, Simon Glass wrote:
> Hi Hans,
>
> On 4 February 2015 at 01:47, Hans de Goede <hdegoede@redhat.com> wrote:
>> Hi,
>>
>> On 04-02-15 05:18, Simon Glass wrote:
>>>
>>> (What does FEL stand for?)
>>>
>>> This is an attempt to make sunxi's FEL code fit with the normal U-Boot
>>> boot sequence instead of creating its own.
>>>
>>> Most of the FEL special-case code is removed, although I may have gone too
>>> far with my changes to generate a u-boot-sunxi-with-spl.bin file even when
>>> FEL is enabled. This may not be possible since the MMC stack makes SPL
>>> too large for FEL anyway, although it may be possible for other boot
>>> mediums.
>>>
>>> This series is available at u-boot-dm, branch sunxi-working.
>>
>>
>> Thanks for working on this, I like the direction where this is heading,
>> if Albert is ok with the start.S changes this would be an excellent way
>> to fix FEL mode for v2015.04.
>
> OK great!
>
>>
>> WRT you going to far when also building a u-boot-sunxi-with-spl.bin when
>> FEL is enabled, your right, AFAIK that binary will not work when written
>> to sdcard because FEL mode has a CONFIG_SPL_TEXT_BASE of 0x2000 where
>> as non FEL has CONFIG_SPL_TEXT_BASE set to 0x0020.
>>
>> But we've discussed in the past to use 0x2000 everywhere, this would mean
>> moving the stack pointer to 0x2000 instead of 0x8000, and then making
>> the normal SPL fit between 0x2000 - 0x8000.
>>
>> All you need to change then AFAIK is tools/mksunxiboot.c so that the jump
>> instruction which is the first 32 bit word of the header it generates jumps
>> to 0x2000 rather then 0x0020, and then we may be able to have a unified
>> build which does both FEL and regular SPL for sdcard boot.
>
> Interesting, I wondered about that.
>
> I see your other series too so this could all fit quite nicely. I'll
> focus on getting this series through with Albert's help.

Great, thanks for working on this. About the having one unified SPL,
I believe that that is probably best left as something to do for
v2015.07, and that we should just try to get your work to fix FEL mode
to work in for v2015.04, do you agree ?

I'm asking because I'm wondering what the plan is wrt merging my
SPL size reduction series, we could in theory add that to v2015.04,
but it can also wait, and from a merge window pov it should wait.

If we want to add it to v2015.04 anyways I can take it upstream in its
entirety through u-boot-sunxi, or you can first merge the 2
malloc_simple patches through u-boot-dm.

Regards,

Hans

  reply	other threads:[~2015-02-05  8:40 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-04  4:18 [U-Boot] [PATCH 1/5] arm: Use r2 instead of r0 in start.S Simon Glass
2015-02-04  4:18 ` [U-Boot] [PATCH 2/5] arm: Allow reset init to be controlled Simon Glass
2015-02-04  8:50   ` Albert ARIBAUD
2015-02-05  2:51     ` Simon Glass
2015-02-04  4:18 ` [U-Boot] [PATCH 3/5] arm: Allow lr to be saved by board code Simon Glass
2015-02-04  8:59   ` Albert ARIBAUD
2015-02-05  2:51     ` Simon Glass
2015-02-05 14:15       ` Albert ARIBAUD
2015-02-05 14:19         ` Simon Glass
2015-02-05 21:40       ` Tom Rini
2015-02-04  4:18 ` [U-Boot] [PATCH 4/5] arm: spl: Provide for a board-specific loader Simon Glass
2015-02-04  4:18 ` [U-Boot] [PATCH 5/5] RFC: sunxi: WIP FEL support Simon Glass
2015-02-04  8:47   ` Hans de Goede
2015-02-04  9:04     ` Albert ARIBAUD
2015-02-05  2:52     ` Simon Glass
2015-02-05  8:40       ` Hans de Goede [this message]
2015-02-05 10:21   ` Siarhei Siamashka
2015-02-06  5:45     ` Simon Glass
2015-02-04  8:57 ` [U-Boot] [PATCH 1/5] arm: Use r2 instead of r0 in start.S Albert ARIBAUD
2015-02-05  2:51   ` Simon Glass

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=54D32C64.1060603@redhat.com \
    --to=hdegoede@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox