From: Rob Herring <robh@kernel.org>
To: "Michal Suchánek" <msuchanek@suse.de>
Cc: linux-fbdev@vger.kernel.org,
Linux regressions mailing list <regressions@lists.linux.dev>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
Cyril Brulebois <cyril@debamax.com>,
stable@vger.kernel.org, Thomas Zimmermann <tzimmermann@suse.de>,
Salvatore Bonaccorso <carnil@debian.org>
Subject: Re: [PATCH 1/2] fbdev/offb: Update expected device name
Date: Thu, 15 Jun 2023 14:09:01 -0600 [thread overview]
Message-ID: <20230615200901.GA1572644-robh@kernel.org> (raw)
In-Reply-To: <20230615132107.GA9196@kitsune.suse.cz>
On Thu, Jun 15, 2023 at 03:21:07PM +0200, Michal Suchánek wrote:
> Hello,
>
> On Thu, Jun 15, 2023 at 03:06:28PM +0200, Thomas Zimmermann wrote:
> > Hi
> >
> > Am 15.06.23 um 15:03 schrieb Linux regression tracking (Thorsten Leemhuis):
> > > On 16.04.23 14:34, Salvatore Bonaccorso wrote:
> > > >
> > > > On Wed, Apr 12, 2023 at 11:55:08AM +0200, Cyril Brulebois wrote:
> > > > > Since commit 241d2fb56a18 ("of: Make OF framebuffer device names unique"),
> > > > > as spotted by Frédéric Bonnard, the historical "of-display" device is
> > > > > gone: the updated logic creates "of-display.0" instead, then as many
> > > > > "of-display.N" as required.
> > > > >
> > > > > This means that offb no longer finds the expected device, which prevents
> > > > > the Debian Installer from setting up its interface, at least on ppc64el.
> > > > >
> > > > > It might be better to iterate on all possible nodes, but updating the
> > > > > hardcoded device from "of-display" to "of-display.0" is confirmed to fix
> > > > > the Debian Installer at the very least.
>
> At the time this was proposed it was said that "of-display", is wrong,
> and that "of-display.0" must be used for the first device instead, and
> if something breaks an alias can be provided.
>
> So how does one provide an alias so that offb can find "of-display.0" as
> "of-display"?
I'm not aware of any way. There isn't because device names and paths are
not considered ABI. There are mechanisms for getting stable class device
indices (e.g. i2c0, mmcblk0, fb0, fb1, etc.) though not implemented for
fbN (and please don't add it).
In any case, this should be an easy fix. Though if "linux,opened" or
"linux,boot-display" is not set, then you'd still get "of-display.0":
diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index 78ae84187449..e46482cef9c7 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -553,7 +553,7 @@ static int __init of_platform_default_populate_init(void)
if (!of_get_property(node, "linux,opened", NULL) ||
!of_get_property(node, "linux,boot-display", NULL))
continue;
- dev = of_platform_device_create(node, "of-display.0", NULL);
+ dev = of_platform_device_create(node, "of-display", NULL);
of_node_put(node);
if (WARN_ON(!dev))
return -ENOMEM;
next prev parent reply other threads:[~2023-06-15 20:09 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-12 9:55 Updating hardcoded "of-display" devices Cyril Brulebois
2023-04-12 9:55 ` [PATCH 1/2] fbdev/offb: Update expected device name Cyril Brulebois
2023-04-16 12:34 ` Salvatore Bonaccorso
2023-06-15 13:03 ` Linux regression tracking (Thorsten Leemhuis)
2023-06-15 13:06 ` Thomas Zimmermann
2023-06-15 13:21 ` Michal Suchánek
2023-06-15 20:09 ` Rob Herring [this message]
2023-06-15 21:19 ` Cyril Brulebois
2023-06-20 6:24 ` Helge Deller
2023-06-20 8:04 ` Michal Suchánek
2023-06-15 13:12 ` Cyril Brulebois
2023-04-22 9:50 ` Helge Deller
2023-04-24 7:31 ` Geert Uytterhoeven
2023-07-10 16:36 ` Michal Suchánek
2023-04-12 9:55 ` [PATCH 2/2] drm/ofdrm: " Cyril Brulebois
2023-04-24 7:33 ` Geert Uytterhoeven
2023-04-24 9:07 ` Thomas Zimmermann
2023-04-24 9:35 ` Helge Deller
2023-05-22 9:37 ` Linux regression tracking (Thorsten Leemhuis)
2023-05-22 11:44 ` Michal Suchánek
2023-07-10 16:36 ` Michal Suchánek
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=20230615200901.GA1572644-robh@kernel.org \
--to=robh@kernel.org \
--cc=carnil@debian.org \
--cc=cyril@debamax.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=msuchanek@suse.de \
--cc=regressions@lists.linux.dev \
--cc=stable@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).