public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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>

  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