From: Javier Martinez Canillas <javierm@redhat.com> To: linux-kernel@vger.kernel.org Cc: linux-fbdev@vger.kernel.org, Dave Hansen <dave.hansen@linux.intel.com>, Liviu Dudau <liviu.dudau@arm.com>, dri-devel@lists.freedesktop.org, "H. Peter Anvin" <hpa@zytor.com>, Sam Ravnborg <sam@ravnborg.org>, Nipun Gupta <nipun.gupta@amd.com>, Helge Deller <deller@gmx.de>, x86@kernel.org, Javier Martinez Canillas <javierm@redhat.com>, Ingo Molnar <mingo@redhat.com>, Geert Uytterhoeven <geert@linux-m68k.org>, Thomas Zimmermann <tzimmermann@suse.de>, Arnd Bergmann <arnd@arndb.de>, Maxime Ripard <mripard@kernel.org>, Borislav Petkov <bp@alien8.de>, Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>, Thomas Gleixner <tglx@linutronix.de>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Oded Gabbay <ogabbay@kernel.org>, Randy Dunlap <rdunlap@infradead.org> Subject: [PATCH v4 0/5] Allow disabling all native fbdev drivers and only keeping DRM emulation Date: Tue, 4 Jul 2023 01:05:24 +0200 [thread overview] Message-ID: <20230703230534.997525-1-javierm@redhat.com> (raw) 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 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'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, #2 and #3 are cleanups to the "Graphics support" Kconfig menu, patch #4 does the FB symbol split and introduces the FB_CORE symbol and finally patch #5 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. This is a v4 of the patch-set that addresses issues pointed out by Arnd Bergmann, Thomas Zimmermann and Geert Uytterhoeven in the previous v3: https://lists.freedesktop.org/archives/dri-devel/2023-July/411689.html Changes in v4: - Fix menuconfig hierarchy that was broken in v3 (Arnd Bergmann). Changes in v3: - Really make a hidden symbol by removing the prompt (Arnd Bergmann). - Change FB_CORE to config instead of menuconfig (Arnd Bergmann). - Keep "depends on FB" for FIRMWARE_EDID (Arnd Bergmann). - Compile out fb_backlight.o and fbmon.o that are only needed for FB (Arnd Bergmann). - Make FB_DEVICE to depend on FB_CORE instead of selecting it. - Make the DRM symbol to select FB_CORE if DRM_FBDEV_EMULATION is enabled (Arnd Bergmann). - Also make DRM select FB_SYS_HELPERS_DEFERRED if DRM_FBDEV_EMULATION - Make DRM_FBDEV_EMULATION to depend on DRM instead of DRM_KMS_HELPER. Changes in v2: - Keep "depends on FB" for FB_DDC, FB_HECUBA, FB_SVGALIB, FB_MACMODES, FB_BACKLIGHT, FB_MODE_HELPERS and FB_TILEBLITTING (Arnd Bergmann). - Don't change the fb.o object name (Arnd Bergmann). - Make FB_CORE a non-visible Kconfig symbol instead (Thomas Zimmermann). - Make CONFIG_DRM_FBDEV_EMULATION to select FB_CORE (Thomas Zimmermann). Javier Martinez Canillas (5): video: Add auxiliary display drivers to Graphics support menu fbdev: Move core fbdev symbols to a separate Kconfig file drm/arm: Make ARM devices menu depend on DRM fbdev: Split frame buffer support in FB and FB_CORE symbols drm: Make FB_CORE to be selected if DRM fbdev emulation is enabled arch/x86/Makefile | 2 +- arch/x86/video/Makefile | 2 +- drivers/Kconfig | 2 - drivers/gpu/drm/Kconfig | 7 +- drivers/gpu/drm/arm/Kconfig | 1 + drivers/video/Kconfig | 2 + drivers/video/console/Kconfig | 2 +- drivers/video/fbdev/Kconfig | 213 ++---------------------------- drivers/video/fbdev/core/Kconfig | 206 +++++++++++++++++++++++++++++ drivers/video/fbdev/core/Makefile | 8 +- 10 files changed, 230 insertions(+), 215 deletions(-) create mode 100644 drivers/video/fbdev/core/Kconfig -- 2.41.0
WARNING: multiple messages have this Message-ID (diff)
From: Javier Martinez Canillas <javierm@redhat.com> To: linux-kernel@vger.kernel.org Cc: Geert Uytterhoeven <geert@linux-m68k.org>, Thomas Zimmermann <tzimmermann@suse.de>, Arnd Bergmann <arnd@arndb.de>, Javier Martinez Canillas <javierm@redhat.com>, Andy Shevchenko <andriy.shevchenko@linux.intel.com>, Borislav Petkov <bp@alien8.de>, Daniel Vetter <daniel@ffwll.ch>, Dave Hansen <dave.hansen@linux.intel.com>, David Airlie <airlied@gmail.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, "H. Peter Anvin" <hpa@zytor.com>, Helge Deller <deller@gmx.de>, Ingo Molnar <mingo@redhat.com>, Jacek Lawrynowicz <jacek.lawrynowicz@linux.intel.com>, Liviu Dudau <liviu.dudau@arm.com>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Maxime Ripard <mripard@kernel.org>, Nipun Gupta <nipun.gupta@amd.com>, Oded Gabbay <ogabbay@kernel.org>, Randy Dunlap <rdunlap@infradead.org>, Sam Ravnborg <sam@ravnborg.org>, Thomas Gleixner <tglx@linutronix.de>, dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, x86@kernel.org Subject: [PATCH v4 0/5] Allow disabling all native fbdev drivers and only keeping DRM emulation Date: Tue, 4 Jul 2023 01:05:24 +0200 [thread overview] Message-ID: <20230703230534.997525-1-javierm@redhat.com> (raw) 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 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'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, #2 and #3 are cleanups to the "Graphics support" Kconfig menu, patch #4 does the FB symbol split and introduces the FB_CORE symbol and finally patch #5 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. This is a v4 of the patch-set that addresses issues pointed out by Arnd Bergmann, Thomas Zimmermann and Geert Uytterhoeven in the previous v3: https://lists.freedesktop.org/archives/dri-devel/2023-July/411689.html Changes in v4: - Fix menuconfig hierarchy that was broken in v3 (Arnd Bergmann). Changes in v3: - Really make a hidden symbol by removing the prompt (Arnd Bergmann). - Change FB_CORE to config instead of menuconfig (Arnd Bergmann). - Keep "depends on FB" for FIRMWARE_EDID (Arnd Bergmann). - Compile out fb_backlight.o and fbmon.o that are only needed for FB (Arnd Bergmann). - Make FB_DEVICE to depend on FB_CORE instead of selecting it. - Make the DRM symbol to select FB_CORE if DRM_FBDEV_EMULATION is enabled (Arnd Bergmann). - Also make DRM select FB_SYS_HELPERS_DEFERRED if DRM_FBDEV_EMULATION - Make DRM_FBDEV_EMULATION to depend on DRM instead of DRM_KMS_HELPER. Changes in v2: - Keep "depends on FB" for FB_DDC, FB_HECUBA, FB_SVGALIB, FB_MACMODES, FB_BACKLIGHT, FB_MODE_HELPERS and FB_TILEBLITTING (Arnd Bergmann). - Don't change the fb.o object name (Arnd Bergmann). - Make FB_CORE a non-visible Kconfig symbol instead (Thomas Zimmermann). - Make CONFIG_DRM_FBDEV_EMULATION to select FB_CORE (Thomas Zimmermann). Javier Martinez Canillas (5): video: Add auxiliary display drivers to Graphics support menu fbdev: Move core fbdev symbols to a separate Kconfig file drm/arm: Make ARM devices menu depend on DRM fbdev: Split frame buffer support in FB and FB_CORE symbols drm: Make FB_CORE to be selected if DRM fbdev emulation is enabled arch/x86/Makefile | 2 +- arch/x86/video/Makefile | 2 +- drivers/Kconfig | 2 - drivers/gpu/drm/Kconfig | 7 +- drivers/gpu/drm/arm/Kconfig | 1 + drivers/video/Kconfig | 2 + drivers/video/console/Kconfig | 2 +- drivers/video/fbdev/Kconfig | 213 ++---------------------------- drivers/video/fbdev/core/Kconfig | 206 +++++++++++++++++++++++++++++ drivers/video/fbdev/core/Makefile | 8 +- 10 files changed, 230 insertions(+), 215 deletions(-) create mode 100644 drivers/video/fbdev/core/Kconfig -- 2.41.0
next reply other threads:[~2023-07-03 23:05 UTC|newest] Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-07-03 23:05 Javier Martinez Canillas [this message] 2023-07-03 23:05 ` [PATCH v4 0/5] Allow disabling all native fbdev drivers and only keeping DRM emulation Javier Martinez Canillas 2023-07-03 23:05 ` [PATCH v4 1/5] video: Add auxiliary display drivers to Graphics support menu Javier Martinez Canillas 2023-07-03 23:05 ` Javier Martinez Canillas 2023-07-04 7:37 ` Geert Uytterhoeven 2023-07-04 7:37 ` Geert Uytterhoeven 2023-07-04 7:54 ` Javier Martinez Canillas 2023-07-04 7:54 ` Javier Martinez Canillas 2023-07-04 8:01 ` Geert Uytterhoeven 2023-07-04 8:01 ` Geert Uytterhoeven 2023-07-04 8:05 ` Javier Martinez Canillas 2023-07-04 8:05 ` Javier Martinez Canillas 2023-07-03 23:05 ` [PATCH v4 2/5] fbdev: Move core fbdev symbols to a separate Kconfig file Javier Martinez Canillas 2023-07-03 23:05 ` Javier Martinez Canillas 2023-07-04 7:17 ` Andy Shevchenko 2023-07-04 7:17 ` Andy Shevchenko 2023-07-04 7:58 ` Javier Martinez Canillas 2023-07-04 7:58 ` Javier Martinez Canillas 2023-07-03 23:05 ` [PATCH v4 3/5] drm/arm: Make ARM devices menu depend on DRM Javier Martinez Canillas 2023-07-03 23:05 ` Javier Martinez Canillas 2023-07-04 16:12 ` Liviu Dudau 2023-07-04 16:12 ` Liviu Dudau 2023-07-10 16:57 ` Javier Martinez Canillas 2023-07-10 16:57 ` Javier Martinez Canillas 2023-07-03 23:05 ` [PATCH v4 4/5] fbdev: Split frame buffer support in FB and FB_CORE symbols Javier Martinez Canillas 2023-07-03 23:05 ` Javier Martinez Canillas 2023-07-04 7:20 ` Andy Shevchenko 2023-07-04 7:20 ` Andy Shevchenko 2023-07-04 7:21 ` Andy Shevchenko 2023-07-04 7:21 ` Andy Shevchenko 2023-07-04 7:56 ` Javier Martinez Canillas 2023-07-04 7:56 ` Javier Martinez Canillas 2023-07-03 23:05 ` [PATCH v4 5/5] drm: Make FB_CORE to be selected if DRM fbdev emulation is enabled Javier Martinez Canillas 2023-07-03 23:05 ` Javier Martinez Canillas
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=20230703230534.997525-1-javierm@redhat.com \ --to=javierm@redhat.com \ --cc=andriy.shevchenko@linux.intel.com \ --cc=arnd@arndb.de \ --cc=bp@alien8.de \ --cc=dave.hansen@linux.intel.com \ --cc=deller@gmx.de \ --cc=dri-devel@lists.freedesktop.org \ --cc=geert@linux-m68k.org \ --cc=gregkh@linuxfoundation.org \ --cc=hpa@zytor.com \ --cc=jacek.lawrynowicz@linux.intel.com \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=liviu.dudau@arm.com \ --cc=mingo@redhat.com \ --cc=mripard@kernel.org \ --cc=nipun.gupta@amd.com \ --cc=ogabbay@kernel.org \ --cc=rdunlap@infradead.org \ --cc=sam@ravnborg.org \ --cc=tglx@linutronix.de \ --cc=tzimmermann@suse.de \ --cc=x86@kernel.org \ /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: linkBe 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.