Helge Deller [off-list ref] writes:

Hello Helge,

Hi Javier,

On 7/19/23 10:15, Javier Martinez Canillas wrote:

quoted

This patch series splits the fbdev core support in two different Kconfig
symbols: FB and FB_CORE. The motivation for this is to allow CONFIG_FB to
be disabled, while still having the the core fbdev support needed for the

One "the" too much.
(correcting just because this is a cover letter)

quoted

CONFIG_DRM_FBDEV_EMULATION to be enabled. The motivation is automatically
disabling all fbdev drivers instead of having to be disabled individually.

The reason for doing this is that now with simpledrm, there's no need for
the legacy fbdev (e.g: efifb or vesafb) drivers anymore and many distros
now disable them. > But it would simplify the config a lot fo have a single
Kconfig symbol to disable all fbdev drivers.

I suggest to rephrase this, e.g.:
The reason for doing this is that with simpledrm, mainstream Linux distributions
like Fedora, SUSE or Ubuntu can then more easily enable the DRM drivers only
by switching off legacy fbdev drivers with one Kconfig option (e.g. for efifb or
vesafb).

(note: there are quite many other distributions, e.g. debian and the derivates,
which still need the fbdev drivers for the various other architectures)

Right, I meant that there is no need anymore for the remaining fbdev
drivers that many distributions used to enable only to have fallback
fbcon/VT (like vesafb, efifb, simplefb, etc). But I'm OK with your
rephrasing and agree that's more accurate to cover all the distros.

quoted

I've built tested with possible combinations of CONFIG_FB, CONFIG_FB_CORE,
CONFIG_DRM_FBDEV_EMULATION and CONFIG_FB_DEVICE symbols set to 'y' or 'n'.

Patch #1 moves the auxdisplay drivers to "Graphics support" Kconfig menu,
patch #2 moves the core fbdev Kconfig symbols to a separate Kconfig file,
patch #3 does the FB symbol split and introduces the FB_CORE symbol and
finally patch #4 makes the DRM symbol to select FB_CORE if the DRM fbdev
emualtion support was enabled.

Since this series touches three subsystems (auxdisplay, fbdev and DRM),
I would like to merge it through DRM with the acks of these maintainers.

Sure. Ack from me.

The patch series look good otherwise. You may add:
Acked-by: Helge Deller <deller@gmx.de>

Thanks!

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat