All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Kemnade <andreas@kemnade.info>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: "Kory Maincent (TI)" <kory.maincent@bootlin.com>,
	Aaro Koskinen <aaro.koskinen@iki.fi>,
	Kevin Hilman <khilman@baylibre.com>,
	Roger Quadros <rogerq@kernel.org>,
	Tony Lindgren <tony@atomide.com>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Russell King <linux@armlinux.org.uk>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	dri-devel@lists.freedesktop.org,
	Luca Ceresoli <luca.ceresoli@bootlin.com>,
	Bajjuri Praneeth <praneeth@ti.com>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	Louis Chauvet <louis.chauvet@bootlin.com>
Subject: Re: [PATCH 1/5] ARM: dts: ti: Enable overlays for all DTB files
Date: Mon, 16 Feb 2026 17:22:07 +0100	[thread overview]
Message-ID: <20260216172207.1b34324e@kemnade.info> (raw)
In-Reply-To: <9D7424B6-2022-4B03-9B8C-455CD3E3FBF1@goldelico.com>

On Fri, 13 Feb 2026 06:33:46 +0100
"H. Nikolaus Schaller" <hns@goldelico.com> wrote:

> Hi,
> 
> > Am 12.02.2026 um 23:19 schrieb Andreas Kemnade <andreas@kemnade.info>:
> > 
> > On Thu, 12 Feb 2026 17:55:43 +0100
> > "H. Nikolaus Schaller" <hns@goldelico.com> wrote:
> >   
> >>> Am 12.02.2026 um 17:47 schrieb Andreas Kemnade <andreas@kemnade.info>:
> >>> 
> >>> On Thu, 12 Feb 2026 16:49:43 +0100
> >>> "H. Nikolaus Schaller" <hns@goldelico.com> wrote:
> >>>   
> >>>>> Am 12.02.2026 um 16:26 schrieb Kory Maincent (TI) <kory.maincent@bootlin.com>:
> >>>>> 
> >>>>> Allow overlays to be applied to any DTB. This adds around ~40% to the
> >>>>> total size of the DTB files on average.      
> >>>> 
> >>>> Is this unconditionally enabled or can it be turned off by some CONFIG? We have
> >>>> our own defconfig so I would not worry if if is enabled in omap2plus_defconfig
> >>>> and disabled in ours.
> >>>> 
> >>>> We have several devices where the boot loader can't handle overlays (never touch
> >>>> a working boot-loader :) So this seems to only contribute to build and load time
> >>>> without benefit.
> >>>>   
> >>> As long as you do not add overlays, the bootloader does not care. I would
> >>> like to simply carry around the 1-bit mmc overlay for one broken board.
> >>> That would help me. So I think there is a benefit but nobody forces
> >>> you to use it.    
> >> 
> >> Well, it does not force to use the really good feature, but it forces to add
> >> ~40% more file size and some more compile time, if I understand it correctly.
> >>   
> > Compile time, hardly measurable even if you just do make dtbs.
> > 
> > Size on disk:
> > a) if it lives around in a /boot partitions with kernels and initrams in it,
> >   then we are around 1% more space needed.  
> 
> Ah, I see. I was too focussed on the "adds around ~40% to the total size of the DTB files".
> 
> For the Letux arm distro all DTBs are around 8.1 MB at the moment so it will grow not that much
> (there are non-TI devices included).
> 
> So you are right, it is ~1% of the total if the kernel image is counted.
> 
> Therefore, space should not be something we should be too concerned about (although I remember
> discussions for driver code where every single byte did count).
> 
> On the other hand this increases load time from (sometimes slow) µSD for a specific DTB by 40%.
> That should at least be discussed.
> 
so also 1% of total loading time. And symbols can be stripped away during
e.g. deployment.
fdtput -r some_board.dtb /__symbols__

I am not strongly against it. It is just we are doing something unusual here.
Are there reasons why it is unusual I do not understand?
I think it can be useful for every board in some situations. For the
ones with can carry a head, they are the most useful.

What was the reasoning to have it enabled for k3 for every board? K3 and Broadcom-64bit
seem to be the only arch doing that. Broadcom mostly many RPis, some routers. 

Regards,
Andreas


  reply	other threads:[~2026-02-16 16:22 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-12 15:26 [PATCH 0/5] Add support for Seeed Studio BeagleBone HDMI cape overlay Kory Maincent (TI)
2026-02-12 15:26 ` [PATCH 1/5] ARM: dts: ti: Enable overlays for all DTB files Kory Maincent (TI)
2026-02-12 15:42   ` Krzysztof Kozlowski
2026-02-12 16:55     ` Kory Maincent
2026-02-12 15:49   ` H. Nikolaus Schaller
2026-02-12 16:47     ` Andreas Kemnade
2026-02-12 16:55       ` H. Nikolaus Schaller
2026-02-12 22:19         ` Andreas Kemnade
2026-02-13  3:16           ` Robert Nelson
2026-02-16 15:11             ` Kory Maincent
2026-02-17  8:58               ` H. Nikolaus Schaller
2026-02-17  9:34                 ` Andreas Kemnade
2026-02-17  9:36                 ` Kory Maincent
2026-02-18  7:28                   ` H. Nikolaus Schaller
2026-02-13  5:33           ` H. Nikolaus Schaller
2026-02-16 16:22             ` Andreas Kemnade [this message]
2026-02-12 15:26 ` [PATCH 2/5] ARM: multi_v7_defconfig: Enable TILCDC DRM driver Kory Maincent (TI)
2026-02-12 15:26 ` [PATCH 3/5] ARM: omap2plus_defconfig: Enable ITE IT66121 driver Kory Maincent (TI)
2026-02-12 15:26 ` [PATCH 4/5] ARM: multi_v7_defconfig: " Kory Maincent (TI)
2026-02-12 15:40   ` Krzysztof Kozlowski
2026-02-12 16:58     ` Kory Maincent
2026-02-12 15:26 ` [PATCH 5/5] ARM: dts: am335x: Add Seeed Studio BeagleBone HDMI cape overlay Kory Maincent (TI)

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=20260216172207.1b34324e@kemnade.info \
    --to=andreas@kemnade.info \
    --cc=aaro.koskinen@iki.fi \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hns@goldelico.com \
    --cc=khilman@baylibre.com \
    --cc=kory.maincent@bootlin.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=louis.chauvet@bootlin.com \
    --cc=luca.ceresoli@bootlin.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=praneeth@ti.com \
    --cc=robh@kernel.org \
    --cc=rogerq@kernel.org \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=tony@atomide.com \
    --cc=tzimmermann@suse.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.