[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <23clcxtgbzbsji2knwp47xdc5udj7lnhbvzsgqi3vklvmhdgjd@26ycx2ed77l4>
Date: Mon, 27 Oct 2025 18:19:12 +0100
From: Maxime Ripard <mripard@...nel.org>
To: Mario Limonciello <mario.limonciello@....com>
Cc: Thomas Zimmermann <tzimmermann@...e.de>,
Francesco Valla <francesco@...la.it>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Jonathan Corbet <corbet@....net>, Jocelyn Falempe <jfalempe@...hat.com>,
Javier Martinez Canillas <javierm@...hat.com>, Sam Ravnborg <sam@...nborg.org>, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org, linux-doc@...r.kernel.org, linux-embedded@...r.kernel.org
Subject: Re: [PATCH RFC 0/3] Add splash DRM client
On Mon, Oct 27, 2025 at 11:31:06AM -0500, Mario Limonciello wrote:
> On 10/27/25 11:28 AM, Maxime Ripard wrote:
> > On Mon, Oct 27, 2025 at 11:01:55AM -0500, Mario Limonciello wrote:
> > > On 10/27/25 7:35 AM, Thomas Zimmermann wrote:
> > > > > > - a very simple progress bar, which can be driven through sysfs;
> > > >
> > > > Once you have options to control these settings from user space, you
> > > > should do it in user space entirely. As Maxime suggested, please improve
> > > > plymouth for anything with animation.
> > > >
> > > > > > - a static image (optional).
> > > >
> > > > Board vendors often provide an image, see /sys/firmware/acpi/bgrt/. This
> > > > is a candidate for display, or the penguin or a custom image. Please
> > > > make it configurable by Kconfig. Again, if you need policy and
> > > > heuristics for deciding what to display, you better do this in user
> > > > space.
> > >
> > > I'd actually argue that the static image from BGRT should be the preferred
> > > priority. This can make for a nice hand off to Plymouth.
> > >
> > > The (UEFI) BIOS already will show this image as soon as the GOP driver is
> > > loaded. Bootloaders like GRUB by default will avoid showing anything or
> > > will overwrite with the exact same image in the same location. This can let
> > > the kernel do the same, and then the moment Plymouth takes over it could do
> > > the same.
> >
> > And BGRT isn't typically found on embedded systems at all, so I'm not
> > sure it's a sensible default, let alone a priority. At most a possible
>
> There are certainly embedded machines using UEFI and that have a BGRT.
Yes, indeed, hence the "typically".
> How about "Sensible default the top of the priority list if it exists"
How about we don't tell contributors what their priorities must be?
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)
Powered by blists - more mailing lists