All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 0/3] arm: tegra: apalis-tk1 and ext clock loopback
Date: Fri, 24 Mar 2017 22:22:37 +0000	[thread overview]
Message-ID: <1490394155.17461.14.camel@toradex.com> (raw)
In-Reply-To: <eddcecc5-cc5f-70de-bca5-49779056eb2c@suse.de>

Hi Alexander

On Tue, 2017-03-21 at 14:50 +0100, Alexander Graf wrote:
> 
> On 21/03/2017 14:37, Marcel Ziswiler wrote:
> > Hi Alex
> > 
> > On Tue, 2017-03-21 at 10:46 +0100, Alexander Graf wrote:
> > > 
> > > On 21/03/2017 10:29, Marcel Ziswiler wrote:
> > > > This series adds support for the Toradex Apalis TK1 and
> > > > introduces
> > > > a new
> > > > option to disable the external clock loopback on SDMMC3.
> > > > 
> > > > The whole series is also available here:
> > > > 
> > > > http://git.toradex.com/cgit/u-boot-toradex.git/log/?h=for-next
> > > > 
> > > > Changes in v4:
> > > > - Re-based.
> > > > - Drop patches 2 and 3:
> > > >     mmc: tegra: move CONFIG_TEGRA_MMC from headers to
> > > > defconfigs
> > > >     mmc: tegra: introduce CONFIG_TEGRA_MMC to Kconfig
> > > >   in favor of commit 1d2c0506d31a9997e5ffc22e90942902f673b107
> > > >     mmc: move more driver config options to Kconfig
> > > >   from Masahiro.
> > > > - Meld patch 6
> > > >     apalis-tk1: clean-up defconfig
> > > >   into first one.
> > > > - Add distro boot fallback.
> > > > - Enable FIT and device tree overlay support.
> > > > - Explicitly drop EFI loader support.
> > > 
> > > Any particular reason for this?
> > 
> > Yes, a waste of 20K which on some of our modules is rather crucial.
> > I
> > personally think force enabling EFI for everything was not too
> > smart an
> > idea but whatever.
> 
> Well, it's enabled by default for a good reason: Compatibility.

While I do agree about compatibility I don't quite see what this has to
do with EFI. I personally don't believe the BIOS/EFI disaster of the
x86 world is worth copying to ARM, sorry. Other distributions like
Fedora e.g. achieve the same with distro boot isn't it? After all isn't
the device tree the agreed upon hardware description in the ARM world?

> I want to move to a world where people can just take a random
> distribution and expect it to work, rather than waste weeks over
> weeks on fiddling around with random BSPs that nobody really wants
> only to realize 2 years down the road that their security footprint
> is en par with a swiss cheese.

Hey, nothing against Swiss cheese! And BTW the term Swiss cheese is
about as nonsense as the term Linux. There is really a huge collection
of different Swiss cheeses... Sorry, I just do get upset at times if
certain countries gastronomy people do ask whether I want my sandwich
with Swiss cheese wondering what they mean by that. But I guess those
chaps now do have their very own issues (;-p).

> It's also the preferred option to boot BSD for example.
> 
> So how bad are the 20k for you? In fact, I'm surprised it's 20k at
> all - 
> it used to be 10k. The overhead of DM should be much higher and I
> doubt 
> you want to disable that ;). Usually a full U-Boot build is ~500k,
> so 
> even at 20k we're talking about 4% - less than a compiler update
> usually 
> gets you.
> 
> If size really is such a big concern, moving to THUMB code is going
> to 
> buy you much more in storage savings than dropping (quite useful)
> UEFI 
> support.
> 
> If however there are real technical issues with it, I'm happy to
> hear 
> them and fix them as far as possible.

The technical issue is that I still don't quite see why U-Boot needs
EFI. I stopped using SuSE more than 15 years ago and never regretted
it, no pun intended.

But if it is really agreed upon mandatory to enable it I am happy to do
so as I would love to finally get that thing in after an almost 4
months endeavour.

> Alex

Cheers

Marcel

  reply	other threads:[~2017-03-24 22:22 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-21  9:29 [U-Boot] [PATCH v4 0/3] arm: tegra: apalis-tk1 and ext clock loopback Marcel Ziswiler
2017-03-21  9:29 ` [U-Boot] [PATCH v4 1/3] arm: tegra: initial support for apalis tk1 Marcel Ziswiler
2017-03-21  9:29 ` [U-Boot] [PATCH v4 2/3] mmc: tegra: allow disabling external clock loopback Marcel Ziswiler
2017-03-21 22:19   ` Jaehoon Chung
2017-03-21  9:29 ` [U-Boot] [PATCH v4 3/3] apalis-tk1: disable external clock loopback on SDMMC3 Marcel Ziswiler
2017-03-21  9:46 ` [U-Boot] [PATCH v4 0/3] arm: tegra: apalis-tk1 and ext clock loopback Alexander Graf
2017-03-21 13:37   ` Marcel Ziswiler
2017-03-21 13:50     ` Alexander Graf
2017-03-24 22:22       ` Marcel Ziswiler [this message]
2017-03-24 22:40         ` Alexander Graf
2017-03-24 23:52           ` Marcel Ziswiler
2017-04-01  4:22             ` Simon Glass
2017-04-03  9:02               ` Alexander Graf

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=1490394155.17461.14.camel@toradex.com \
    --to=marcel.ziswiler@toradex.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 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.