From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM: mx6: Enable MMC FS boot support
Date: Fri, 29 Apr 2016 00:37:51 +0200 [thread overview]
Message-ID: <572290BF.3030207@denx.de> (raw)
In-Reply-To: <20160428190216.GQ19598@bill-the-cat>
On 04/28/2016 09:02 PM, Tom Rini wrote:
> On Thu, Apr 28, 2016 at 08:29:47PM +0200, Marek Vasut wrote:
>> On 04/28/2016 08:06 PM, Tom Rini wrote:
>>> On Thu, Apr 28, 2016 at 03:40:45PM +0200, Marek Vasut wrote:
>>>> On 04/28/2016 03:36 PM, Stefano Babic wrote:
>>>>> Hi Marek,
>>>>>
>>>>> On 28/04/2016 13:03, Marek Vasut wrote:
>>>>>> On 04/28/2016 07:59 AM, Stefano Babic wrote:
>>>>>>> Hi Marek,
>>>>>>>
>>>>>>> On 28/04/2016 04:24, Peng Fan wrote:
>>>>>>>> Hi Marek,
>>>>>>>>
>>>>>>>> On Thu, Apr 28, 2016 at 01:06:07AM +0200, Marek Vasut wrote:
>>>>>>>>> Enable support for booting U-Boot image from filesystem instead of some
>>>>>>>>> random offset on the SD card. This makes the board usable by putting the
>>>>>>>>> u-boot.img to first partition of the SD card and writing the SPL this way:
>>>>>>>>> $ dd if=u-boot-with-spl.imx of=/dev/sdX seek=2 bs=512
>>>>>>>>
>>>>>>>> I once want to enable this for i.MX6UL, but was rejected.
>>>>>>>> Anyway I prefer load u-boot.img from filesystem.
>>>>>>>>
>>>>>>>
>>>>>>> Right - we have this discussion some times ago. The agreement we reach
>>>>>>> was to maintain SPL loading only from a raw image.
>>>>>>>
>>>>>>> https://www.mail-archive.com/u-boot at lists.denx.de/msg185432.html
>>>>>>
>>>>>> The feeling I get from the discussion you linked above and the
>>>>>> discussion here is that because user can be an idiot and delete
>>>>>> files at random, we should move back to the 80s
>>>>>
>>>>> No U-Boot in the 80s.
>>>>>
>>>>>> and store files
>>>>>> like on the casette tapes, at random offset. User can also delete
>>>>>> kernel image and yet, we store it on the filesystem. Maybe we should
>>>>>> also store the kernel image at another raw offset ... and hey, maybe
>>>>>> we should ditch filesystem altogether, just tar everything up and
>>>>>> store it at yet another offset, since user might delete files at
>>>>>> random, just imaging he'd delete libc ...
>>>>>
>>>>> wandboard has a market quite similar to the Raspi and yes, there is a
>>>>> lot of inexperienced people using it.
>>>>
>>>> Shall I prepare a patch which places kernel to yet another ad-hoc
>>>> location on the SD card then and tweak bootargs to use cramfs then?
>>>
>>> Users bricking their devices is a real problem. I don't object to
>>> adding support for many ways of doing things (we have it on TI boards
>>> today) enabled, but saying there's no use case nor reason to do non-FS
>>> installs is also missing the mark.
>>
>> I am convinced this patch does not remove the "fallback to legacy
>> behavior" though.
>
> Good. Then you need to shove this up and over to the right common
> location since all of the offsets are, or better be (or we have a
> problem here..) the same for all imx6 and probably falling back a while
> too. I know on TI parts the "magic" locations go back farther than
> U-Boot still supports. And I know that might take a little time which
> is fine, this isn't a patch that needs to be in for v2016.05.
>
Hm, grumble, the thing is falling apart the more I drill into it. There
will be a few more patches needed to deal with this raw legacy. Sigh ...
--
Best regards,
Marek Vasut
prev parent reply other threads:[~2016-04-28 22:37 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-27 23:06 [U-Boot] [PATCH] ARM: mx6: Enable MMC FS boot support Marek Vasut
2016-04-27 23:16 ` Tom Rini
2016-04-27 23:28 ` Marek Vasut
2016-04-27 23:32 ` Robert Nelson
2016-04-27 23:41 ` Marek Vasut
2016-04-27 23:49 ` Robert Nelson
2016-04-28 0:02 ` Marek Vasut
2016-04-28 0:54 ` Tom Rini
2016-04-28 1:06 ` Robert Nelson
2016-04-28 12:02 ` Tom Rini
2016-04-28 2:24 ` Peng Fan
2016-04-28 5:59 ` Stefano Babic
2016-04-28 11:03 ` Marek Vasut
2016-04-28 13:36 ` Stefano Babic
2016-04-28 13:40 ` Marek Vasut
2016-04-28 18:06 ` Tom Rini
2016-04-28 18:29 ` Marek Vasut
2016-04-28 19:02 ` Tom Rini
2016-04-28 22:37 ` Marek Vasut [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=572290BF.3030207@denx.de \
--to=marex@denx.de \
--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