From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM: keystone: Pass SPI MTD partition table via kernel command line
Date: Tue, 21 Mar 2017 20:25:20 -0400 [thread overview]
Message-ID: <20170322002520.GG19897@bill-the-cat> (raw)
In-Reply-To: <81837249-0486-1981-86b4-cf6f08f599d5@ti.com>
On Tue, Mar 21, 2017 at 01:52:07PM -0500, Franklin S Cooper Jr wrote:
>
>
> On 03/20/2017 11:57 PM, Vignesh R wrote:
> >
> >
> > On Saturday 18 March 2017 08:04 PM, Tom Rini wrote:
> >>>> And can we not move towards the "normal" method of SPL loading
> >>>> the u-boot.img (or FIT) from? I guess the current architecture here is
> >>>> confusing me.
> >>> This has been same for all k2 platforms. I guess we have single image so
> >>> that user don't have to bother flashing multiple images for spi boot
> >>> given the fact that all other boot modes have single image.
> >>>
> >>>> Regardless, I still see the DT problem as the bigger one long term, and
> >>>> dra7xx shows that. And I agree we need to re-size how the flash is
> >>>> partitioned.
> >>> True.
> >> The next question is, given that Franklin is talking about being able to
> >> load the right DTB for any K2 platform basically, is the layout in
> >> https://patchwork.ozlabs.org/patch/736498/ really looking like it will
> >> be enough?
> >>
> >
> > I will leave Franklin to comment here. I dont think he has plans to do
> > changes for all K2 platforms (I guess his plans are mostly limited to K2G)
>
> I'm not sure if there is any real solution other than providing a
> generous amount of storage for U-boot in the flash memory. I don't think
> atleast within our TI SDK use cases we even use the misc partition. So I
> don't see a reason why we couldn't give U-boot's partition 3 or 4 MB of
> space.
OK. But what about a dedicated place where the right (base) board DTB
could reside?
> Also has there been any thoughts of compressing dtbs? These dtbs are
> relatively massive and compressed they are around 1/5 the size.
There has not yet been. My first thought is about decompression time,
but maybe that won't matter.
>
> Personally I'm not a fan of U-boot performing all these fix ups before
> passing things to the kernel. It forces so much coupling between
> bootloader versions and kernel. And things become more painful when
> changes in the kernel causes U-boot fix ups to break.
To the final point, I cannot find the video from the device tree BoF at
ELC, but as we talked about there, to some degree overlays/fixups will
have to be done in Linux. But by the same token, a lot of overlay
applications will be done prior to the kernel as well. When in doubt,
this shouldn't be done in C, but in user replaceable/updatable scripts.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170321/aae9dc6b/attachment.sig>
next prev parent reply other threads:[~2017-03-22 0:25 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-04 12:47 [U-Boot] [PATCH] ARM: keystone: Pass SPI MTD partition table via kernel command line Vignesh R
2017-03-04 17:09 ` Tom Rini
2017-03-06 9:46 ` Vignesh R
2017-03-08 14:41 ` Tom Rini
2017-03-09 9:58 ` Vignesh R
2017-03-10 18:02 ` Tom Rini
2017-03-14 11:41 ` Vignesh R
2017-03-15 20:48 ` Tom Rini
2017-03-18 12:14 ` Vignesh R
2017-03-18 14:34 ` Tom Rini
2017-03-21 4:57 ` Vignesh R
2017-03-21 18:52 ` Franklin S Cooper Jr
2017-03-22 0:25 ` Tom Rini [this message]
2017-03-30 17:17 ` Franklin S Cooper Jr
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=20170322002520.GG19897@bill-the-cat \
--to=trini@konsulko.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