Skip to contentSkip to conversation

output: Drop display output class support

PATCH In review Jean Delvare 1 participant 2 messages Mar 17, 2014 – Mar 19, 2014

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 4/4 output: Drop display output class support

It was only ever used by the ACPI video driver, and that only use case vanished over 3 years ago (see commit 677bd810, "ACPI video: remove output switching control".) So this is dead code and I guess we can remove it now.

Cc Zhang Rui <[email protected]>
Cc Len Brown <[email protected]>
Cc "Rafael J. Wysocki" <[email protected]>
---
 drivers/video/Kconfig        |    6 -
 drivers/video/Makefile       |    2 
 drivers/video/output.c       |  133 -------------------------------------------
 include/linux/video_output.h |   57 ------------------
 4 files changed, 198 deletions(-)
drivers/video/Kconfig +0 -6
@@ -27,12 +27,6 @@ config VGASTATE
        tristate
        default n
 
-config VIDEO_OUTPUT_CONTROL
-	tristate "Lowlevel video output switch controls"
-	help
-	  This framework adds support for low-level control of the video 
-	  output switch.
-
 config VIDEOMODE_HELPERS
 	bool
 
drivers/video/Makefile +0 -2
@@ -172,8 +172,6 @@ obj-$(CONFIG_FB_SIMPLE)           += sim
 # the test framebuffer is last
 obj-$(CONFIG_FB_VIRTUAL)          += vfb.o
 
-#video output switch sysfs driver
-obj-$(CONFIG_VIDEO_OUTPUT_CONTROL) += output.o
 obj-$(CONFIG_VIDEOMODE_HELPERS) += display_timing.o videomode.o
 ifeq ($(CONFIG_OF),y)
 obj-$(CONFIG_VIDEOMODE_HELPERS) += of_display_timing.o of_videomode.o
drivers/video/output.c deleted +0 -133

133 lines not on this page · raw message

include/linux/video_output.h deleted +0 -57

57 lines not on this page · raw message

Signature · 2 lines
Jean Delvare
SUSE L3 Support
#1 lore raw
Quoted 9 lines · Jean Delvare

It was only ever used by the ACPI video driver, and that only use case vanished over 3 years ago (see commit 677bd810, "ACPI video: remove output switching control".) So this is dead code and I guess we can remove it now.

Cc Zhang Rui <[email protected]>
Cc Len Brown <[email protected]>
Cc "Rafael J. Wysocki" <[email protected]>
Quoted 7 lines
---
 drivers/video/Kconfig        |    6 -
 drivers/video/Makefile       |    2 
 drivers/video/output.c       |  133 -------------------------------------------
 include/linux/video_output.h |   57 ------------------
 4 files changed, 198 deletions(-)

(...)

Rafael, wait a minute. I just remembered that this last patch depends on another clean-up patch of mine which is not upstream yet:
[PATCH 1/2] fujitsu-laptop: Drop unneeded include

List platform-driver-x86 doesn't seem to be archived anywhere so I can't point you to it. I will resend the patch with you in Cc so that you can include it in the series.

Thanks,

Signature · 2 lines
Jean Delvare
SUSE L3 Support