All of lore.kernel.org
 help / color / mirror / Atom feed
From: Segher Boessenkool <segher@kernel.crashing.org>
To: Rob Herring <robh@kernel.org>
Cc: Stan Johnson <userm57@yahoo.com>,
	linuxppc-dev@lists.ozlabs.org, Finn Thain <fthain@linux-m68k.org>
Subject: Re: Excluded List for "#size-cells" warning
Date: Wed, 29 Oct 2025 02:29:23 -0500	[thread overview]
Message-ID: <aQHCU1pb7zT167rO@gate> (raw)
In-Reply-To: <CAL_JsqJ8amo2ggUVfL=YzRrA9UnqKfKCipn-_yOkahJKZVs5=A@mail.gmail.com>

On Tue, Oct 28, 2025 at 08:17:27PM -0500, Rob Herring wrote:
> On Tue, Oct 28, 2025 at 7:05 PM Stan Johnson <userm57@yahoo.com> wrote:
> >
> > -------- Forwarded Message --------
> > Subject: Excluded List for "#size-cells" warning
> > Date: Tue, 28 Oct 2025 10:00:25 -0600
> > From: Stan Johnson <userm57@yahoo.com>
> > To: debian-powerpc@lists.debian.org
> > CC: Finn Thain <fthain@linux-m68k.org>, Christophe Leroy
> > <christophe.leroy@csgroup.eu>
> >
> > Hello,
> >
> > On a PowerBook G3 Pismo running the latest Debian SID, dmesg reports the
> > warning shown below. I've also seen the warning on PowerBook Lombard and
> > Wallstreet systems. I haven't checked PowerBook 3400c or Kanga.
> 
> Can you send me a dump of the device tree on these systems:
> 
> dtc -O dts /proc/device-tree
> 
> We've been fixing up these cases such as in commit 7e67ef889c9a
> ("powerpc/prom_init: Fixup missing #size-cells on PowerBook6,7")

And of course it is perfectly fine for an actual Open Firmware to *not*
repeat the defaults.  As the documentation (the main IEEE 1275 thing)
says: "A missing “#size-cells” property signifies the default value of
one."  There are many other places in OF geared towards this default
btw, take for example the "reg" word, that silently assumes your node's
#size-cells is 1, and does completely the wrong thing if not.

Flattened device trees are a fine thing, but the gratuitous ways it
differs from OF, are not.


Segher


  reply	other threads:[~2025-10-29  7:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <70d76dcf-c7d4-43c5-c169-07054a37195c@yahoo.com>
2025-10-28 23:55 ` Fwd: Excluded List for "#size-cells" warning Stan Johnson
2025-10-29  1:17   ` Rob Herring
2025-10-29  7:29     ` Segher Boessenkool [this message]
2025-10-29 17:00       ` Stan Johnson
2025-10-30 22:14         ` Stan Johnson
2025-12-08 16:25           ` Stanley J. Johnson
2025-12-08 16:29             ` Rob Herring

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=aQHCU1pb7zT167rO@gate \
    --to=segher@kernel.crashing.org \
    --cc=fthain@linux-m68k.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=robh@kernel.org \
    --cc=userm57@yahoo.com \
    /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.