All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/21] drm: make fbdev support really optional
@ 2016-07-15 12:47 Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 01/21] drm/arc: " Tobias Jakobi
                   ` (21 more replies)
  0 siblings, 22 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:47 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Hello,

as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).

With best wishes,
Tobias


Tobias Jakobi (21):
  drm/arc: make fbdev support really optional
  drm/armada: make fbdev support really optional
  drm/omap: make fbdev support really optional
  drm/mgag200: make fbdev support really optional
  drm/cirrus: make fbdev support really optional
  drm/qxl: make fbdev support really optional
  drm/imx: make fbdev support really optional
  drm/shmobile: make fbdev support really optional
  drm/rockchip: make fbdev support really optional
  drm/gma500: make fbdev support really optional
  drm/rcar-du: make fbdev support really optional
  drm/tilcdc: make fbdev support really optional
  drm/fsl-dcu: make fbdev support really optional
  drm/bochs: make fbdev support really optional
  drm/vmwgfx: make fbdev support really optional
  drm/udl: make fbdev support really optional
  drm/atmel-hlcdc: make fbdev support really optional
  drm/virtio: make fbdev support really optional
  drm/ast: make fbdev support really optional
  drm/arm: make fbdev support really optional
  drm/nouveau: make fbdev support really optional

 drivers/gpu/drm/arc/Kconfig         | 1 -
 drivers/gpu/drm/arm/Kconfig         | 1 -
 drivers/gpu/drm/armada/Kconfig      | 4 ----
 drivers/gpu/drm/ast/Kconfig         | 4 ----
 drivers/gpu/drm/atmel-hlcdc/Kconfig | 1 -
 drivers/gpu/drm/bochs/Kconfig       | 4 ----
 drivers/gpu/drm/cirrus/Kconfig      | 4 ----
 drivers/gpu/drm/fsl-dcu/Kconfig     | 5 -----
 drivers/gpu/drm/gma500/Kconfig      | 4 ----
 drivers/gpu/drm/imx/Kconfig         | 1 -
 drivers/gpu/drm/mgag200/Kconfig     | 4 ----
 drivers/gpu/drm/nouveau/Kconfig     | 6 ------
 drivers/gpu/drm/omapdrm/Kconfig     | 5 -----
 drivers/gpu/drm/qxl/Kconfig         | 5 -----
 drivers/gpu/drm/rcar-du/Kconfig     | 1 -
 drivers/gpu/drm/rockchip/Kconfig    | 4 ----
 drivers/gpu/drm/shmobile/Kconfig    | 1 -
 drivers/gpu/drm/tilcdc/Kconfig      | 1 -
 drivers/gpu/drm/udl/Kconfig         | 5 -----
 drivers/gpu/drm/virtio/Kconfig      | 4 ----
 drivers/gpu/drm/vmwgfx/Kconfig      | 4 ----
 21 files changed, 69 deletions(-)

-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH 01/21] drm/arc: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
@ 2016-07-15 12:47 ` Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 02/21] drm/armada: " Tobias Jakobi
                   ` (20 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:47 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling ARC DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/arc/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/arc/Kconfig b/drivers/gpu/drm/arc/Kconfig
index f9a13b6..f47d88b 100644
--- a/drivers/gpu/drm/arc/Kconfig
+++ b/drivers/gpu/drm/arc/Kconfig
@@ -2,7 +2,6 @@ config DRM_ARCPGU
 	tristate "ARC PGU"
 	depends on DRM && OF
 	select DRM_KMS_CMA_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_KMS_HELPER
 	help
 	  Choose this option if you have an ARC PGU controller.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 02/21] drm/armada: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 01/21] drm/arc: " Tobias Jakobi
@ 2016-07-15 12:47 ` Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 03/21] drm/omap: " Tobias Jakobi
                   ` (19 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:47 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Marvell Armada DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/armada/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/armada/Kconfig b/drivers/gpu/drm/armada/Kconfig
index eb773e9..15f3ecf 100644
--- a/drivers/gpu/drm/armada/Kconfig
+++ b/drivers/gpu/drm/armada/Kconfig
@@ -1,11 +1,7 @@
 config DRM_ARMADA
 	tristate "DRM support for Marvell Armada SoCs"
 	depends on DRM && HAVE_CLK && ARM
-	select FB_CFB_FILLRECT
-	select FB_CFB_COPYAREA
-	select FB_CFB_IMAGEBLIT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	help
 	  Support the "LCD" controllers found on the Marvell Armada 510
 	  devices.  There are two controllers on the device, each controller
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 03/21] drm/omap: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 01/21] drm/arc: " Tobias Jakobi
  2016-07-15 12:47 ` [PATCH 02/21] drm/armada: " Tobias Jakobi
@ 2016-07-15 12:47 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 04/21] drm/mgag200: " Tobias Jakobi
                   ` (18 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:47 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling OMAP DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/omapdrm/Kconfig | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig
index 336ad4d..556f81f 100644
--- a/drivers/gpu/drm/omapdrm/Kconfig
+++ b/drivers/gpu/drm/omapdrm/Kconfig
@@ -4,11 +4,6 @@ config DRM_OMAP
 	depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
 	select OMAP2_DSS
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
-	select FB_SYS_FOPS
 	default n
 	help
 	  DRM display driver for OMAP2/3/4 based boards.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 04/21] drm/mgag200: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (2 preceding siblings ...)
  2016-07-15 12:47 ` [PATCH 03/21] drm/omap: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 05/21] drm/cirrus: " Tobias Jakobi
                   ` (17 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling MGA G200 DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/mgag200/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/mgag200/Kconfig b/drivers/gpu/drm/mgag200/Kconfig
index 3a1c5fb..520e5e6 100644
--- a/drivers/gpu/drm/mgag200/Kconfig
+++ b/drivers/gpu/drm/mgag200/Kconfig
@@ -1,11 +1,7 @@
 config DRM_MGAG200
 	tristate "Kernel modesetting driver for MGA G200 server engines"
 	depends on DRM && PCI
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
 	help
 	 This is a KMS driver for the MGA G200 server chips, it
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 05/21] drm/cirrus: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (3 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 04/21] drm/mgag200: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 06/21] drm/qxl: " Tobias Jakobi
                   ` (16 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling QEMU Cirrus DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/cirrus/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/cirrus/Kconfig b/drivers/gpu/drm/cirrus/Kconfig
index 9864559..04b3c16 100644
--- a/drivers/gpu/drm/cirrus/Kconfig
+++ b/drivers/gpu/drm/cirrus/Kconfig
@@ -1,11 +1,7 @@
 config DRM_CIRRUS_QEMU
 	tristate "Cirrus driver for QEMU emulated device"
 	depends on DRM && PCI
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
 	help
 	 This is a KMS driver for emulated cirrus device in qemu.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 06/21] drm/qxl: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (4 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 05/21] drm/cirrus: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 07/21] drm/imx: " Tobias Jakobi
                   ` (15 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling QXL DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/qxl/Kconfig | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/gpu/drm/qxl/Kconfig b/drivers/gpu/drm/qxl/Kconfig
index 38c2bb7..da45b11 100644
--- a/drivers/gpu/drm/qxl/Kconfig
+++ b/drivers/gpu/drm/qxl/Kconfig
@@ -1,12 +1,7 @@
 config DRM_QXL
 	tristate "QXL virtual GPU"
 	depends on DRM && PCI
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
-	select FB_DEFERRED_IO
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
 	select CRC32
 	help
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 07/21] drm/imx: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (5 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 06/21] drm/qxl: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 08/21] drm/shmobile: " Tobias Jakobi
                   ` (14 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Freescale i.MX DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/imx/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/imx/Kconfig b/drivers/gpu/drm/imx/Kconfig
index a1844b5..f2c9ae8 100644
--- a/drivers/gpu/drm/imx/Kconfig
+++ b/drivers/gpu/drm/imx/Kconfig
@@ -1,7 +1,6 @@
 config DRM_IMX
 	tristate "DRM Support for Freescale i.MX"
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select VIDEOMODE_HELPERS
 	select DRM_GEM_CMA_HELPER
 	select DRM_KMS_CMA_HELPER
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 08/21] drm/shmobile: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (6 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 07/21] drm/imx: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 09/21] drm/rockchip: " Tobias Jakobi
                   ` (13 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling SH Mobile DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/shmobile/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/shmobile/Kconfig b/drivers/gpu/drm/shmobile/Kconfig
index 8d17d00..c987c82 100644
--- a/drivers/gpu/drm/shmobile/Kconfig
+++ b/drivers/gpu/drm/shmobile/Kconfig
@@ -6,7 +6,6 @@ config DRM_SHMOBILE
 	select BACKLIGHT_CLASS_DEVICE
 	select BACKLIGHT_LCD_SUPPORT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_KMS_CMA_HELPER
 	select DRM_GEM_CMA_HELPER
 	help
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 09/21] drm/rockchip: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (7 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 08/21] drm/shmobile: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 15:01   ` Sean Paul
  2016-07-15 12:48 ` [PATCH 10/21] drm/gma500: " Tobias Jakobi
                   ` (12 subsequent siblings)
  21 siblings, 1 reply; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Rockchip DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/rockchip/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/rockchip/Kconfig b/drivers/gpu/drm/rockchip/Kconfig
index d30bdc3..6f0ef56 100644
--- a/drivers/gpu/drm/rockchip/Kconfig
+++ b/drivers/gpu/drm/rockchip/Kconfig
@@ -3,11 +3,7 @@ config DRM_ROCKCHIP
 	depends on DRM && ROCKCHIP_IOMMU
 	depends on RESET_CONTROLLER
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_PANEL
-	select FB_CFB_FILLRECT
-	select FB_CFB_COPYAREA
-	select FB_CFB_IMAGEBLIT
 	select VIDEOMODE_HELPERS
 	help
 	  Choose this option if you have a Rockchip soc chipset.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 10/21] drm/gma500: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (8 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 09/21] drm/rockchip: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 11/21] drm/rcar-du: " Tobias Jakobi
                   ` (11 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Intel GMA DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/gma500/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/gma500/Kconfig b/drivers/gpu/drm/gma500/Kconfig
index 17f928e..8906d67 100644
--- a/drivers/gpu/drm/gma500/Kconfig
+++ b/drivers/gpu/drm/gma500/Kconfig
@@ -1,11 +1,7 @@
 config DRM_GMA500
 	tristate "Intel GMA5/600 KMS Framebuffer"
 	depends on DRM && PCI && X86
-	select FB_CFB_COPYAREA
-	select FB_CFB_FILLRECT
-	select FB_CFB_IMAGEBLIT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
 	# GMA500 depends on ACPI_VIDEO when ACPI is enabled, just like i915
 	select ACPI_VIDEO if ACPI
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 11/21] drm/rcar-du: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (9 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 10/21] drm/gma500: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 15:01   ` Sean Paul
  2016-07-15 12:48 ` [PATCH 12/21] drm/tilcdc: " Tobias Jakobi
                   ` (10 subsequent siblings)
  21 siblings, 1 reply; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling R-Car DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/rcar-du/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
index 7fc3ca5..4c2fd05 100644
--- a/drivers/gpu/drm/rcar-du/Kconfig
+++ b/drivers/gpu/drm/rcar-du/Kconfig
@@ -6,7 +6,6 @@ config DRM_RCAR_DU
 	select DRM_KMS_HELPER
 	select DRM_KMS_CMA_HELPER
 	select DRM_GEM_CMA_HELPER
-	select DRM_KMS_FB_HELPER
 	select VIDEOMODE_HELPERS
 	help
 	  Choose this option if you have an R-Car chipset.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 12/21] drm/tilcdc: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (10 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 11/21] drm/rcar-du: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 13/21] drm/fsl-dcu: " Tobias Jakobi
                   ` (9 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling TI LCDC DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/tilcdc/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/tilcdc/Kconfig b/drivers/gpu/drm/tilcdc/Kconfig
index f60a1ec..28fed7e 100644
--- a/drivers/gpu/drm/tilcdc/Kconfig
+++ b/drivers/gpu/drm/tilcdc/Kconfig
@@ -2,7 +2,6 @@ config DRM_TILCDC
 	tristate "DRM Support for TI LCDC Display Controller"
 	depends on DRM && OF && ARM
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_KMS_CMA_HELPER
 	select DRM_GEM_CMA_HELPER
 	select VIDEOMODE_HELPERS
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 13/21] drm/fsl-dcu: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (11 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 12/21] drm/tilcdc: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 14/21] drm/bochs: " Tobias Jakobi
                   ` (8 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Freescale DCU DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/fsl-dcu/Kconfig | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/gpu/drm/fsl-dcu/Kconfig b/drivers/gpu/drm/fsl-dcu/Kconfig
index b9c714d..14a72c4 100644
--- a/drivers/gpu/drm/fsl-dcu/Kconfig
+++ b/drivers/gpu/drm/fsl-dcu/Kconfig
@@ -5,12 +5,7 @@ config DRM_FSL_DCU
 	select BACKLIGHT_LCD_SUPPORT
 	select DRM_KMS_HELPER
 	select DRM_KMS_CMA_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_PANEL
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
-	select FB_SYS_FOPS
 	select REGMAP_MMIO
 	select VIDEOMODE_HELPERS
 	help
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 14/21] drm/bochs: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (12 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 13/21] drm/fsl-dcu: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 15/21] drm/vmwgfx: " Tobias Jakobi
                   ` (7 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Bochs DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/bochs/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/bochs/Kconfig b/drivers/gpu/drm/bochs/Kconfig
index 5f8b0c2..f739763 100644
--- a/drivers/gpu/drm/bochs/Kconfig
+++ b/drivers/gpu/drm/bochs/Kconfig
@@ -2,10 +2,6 @@ config DRM_BOCHS
 	tristate "DRM Support for bochs dispi vga interface (qemu stdvga)"
 	depends on DRM && PCI
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
 	select DRM_TTM
 	help
 	  Choose this option for qemu.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 15/21] drm/vmwgfx: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (13 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 14/21] drm/bochs: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 16/21] drm/udl: " Tobias Jakobi
                   ` (6 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling VMware Virtual DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/vmwgfx/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/vmwgfx/Kconfig b/drivers/gpu/drm/vmwgfx/Kconfig
index b49445d..e57ce65 100644
--- a/drivers/gpu/drm/vmwgfx/Kconfig
+++ b/drivers/gpu/drm/vmwgfx/Kconfig
@@ -1,10 +1,6 @@
 config DRM_VMWGFX
 	tristate "DRM driver for VMware Virtual GPU"
 	depends on DRM && PCI && X86
-	select FB_DEFERRED_IO
-	select FB_CFB_FILLRECT
-	select FB_CFB_COPYAREA
-	select FB_CFB_IMAGEBLIT
 	select DRM_TTM
 	# Only needed for the transitional use of drm_crtc_init - can be removed
 	# again once vmwgfx sets up the primary plane itself.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 16/21] drm/udl: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (14 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 15/21] drm/vmwgfx: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 17/21] drm/atmel-hlcdc: " Tobias Jakobi
                   ` (5 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling USB DisplayLink DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/udl/Kconfig | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/gpu/drm/udl/Kconfig b/drivers/gpu/drm/udl/Kconfig
index 613ab06..1616ec4 100644
--- a/drivers/gpu/drm/udl/Kconfig
+++ b/drivers/gpu/drm/udl/Kconfig
@@ -4,12 +4,7 @@ config DRM_UDL
 	depends on USB_SUPPORT
 	depends on USB_ARCH_HAS_HCD
 	select USB
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
-	select FB_DEFERRED_IO
 	select DRM_KMS_HELPER
-        select DRM_KMS_FB_HELPER
 	help
 	  This is a KMS driver for the USB displaylink video adapters.
           Say M/Y to add support for these devices via drm/kms interfaces.
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 17/21] drm/atmel-hlcdc: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (15 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 16/21] drm/udl: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 15:01   ` Sean Paul
  2016-07-15 12:48 ` [PATCH 18/21] drm/virtio: " Tobias Jakobi
                   ` (4 subsequent siblings)
  21 siblings, 1 reply; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling ATMEL HLCDC DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/atmel-hlcdc/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/atmel-hlcdc/Kconfig b/drivers/gpu/drm/atmel-hlcdc/Kconfig
index 99b4f06..32bcc4b 100644
--- a/drivers/gpu/drm/atmel-hlcdc/Kconfig
+++ b/drivers/gpu/drm/atmel-hlcdc/Kconfig
@@ -3,7 +3,6 @@ config DRM_ATMEL_HLCDC
 	depends on DRM && OF && COMMON_CLK && MFD_ATMEL_HLCDC && ARM
 	select DRM_GEM_CMA_HELPER
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_KMS_CMA_HELPER
 	select DRM_PANEL
 	help
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 18/21] drm/virtio: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (16 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 17/21] drm/atmel-hlcdc: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 19/21] drm/ast: " Tobias Jakobi
                   ` (3 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling VirtIO DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/virtio/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/virtio/Kconfig b/drivers/gpu/drm/virtio/Kconfig
index 9983ead..e1afc3d 100644
--- a/drivers/gpu/drm/virtio/Kconfig
+++ b/drivers/gpu/drm/virtio/Kconfig
@@ -1,11 +1,7 @@
 config DRM_VIRTIO_GPU
 	tristate "Virtio GPU driver"
 	depends on DRM && VIRTIO
-	select FB_SYS_FILLRECT
-	select FB_SYS_COPYAREA
-	select FB_SYS_IMAGEBLIT
         select DRM_KMS_HELPER
-        select DRM_KMS_FB_HELPER
         select DRM_TTM
 	help
 	   This is the virtual GPU driver for virtio.  It can be used with
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 19/21] drm/ast: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (17 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 18/21] drm/virtio: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 20/21] drm/arm: " Tobias Jakobi
                   ` (2 subsequent siblings)
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling AST Server DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/ast/Kconfig | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/ast/Kconfig b/drivers/gpu/drm/ast/Kconfig
index 8a784c4..15f6ce7 100644
--- a/drivers/gpu/drm/ast/Kconfig
+++ b/drivers/gpu/drm/ast/Kconfig
@@ -2,11 +2,7 @@ config DRM_AST
 	tristate "AST server chips"
 	depends on DRM && PCI
 	select DRM_TTM
-	select FB_SYS_COPYAREA
-	select FB_SYS_FILLRECT
-	select FB_SYS_IMAGEBLIT
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
 	help
 	 Say yes for experimental AST GPU driver. Do not enable
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 20/21] drm/arm: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (18 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 19/21] drm/ast: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 12:48 ` [PATCH 21/21] drm/nouveau: " Tobias Jakobi
  2016-07-15 13:21 ` [PATCH 00/21] drm: " Emil Velikov
  21 siblings, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling ARM HDLCD DRM support automatically pulls
in fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/arm/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/arm/Kconfig b/drivers/gpu/drm/arm/Kconfig
index eaed454..ff402500 100644
--- a/drivers/gpu/drm/arm/Kconfig
+++ b/drivers/gpu/drm/arm/Kconfig
@@ -9,7 +9,6 @@ config DRM_HDLCD
 	depends on COMMON_CLK
 	select DRM_ARM
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_KMS_CMA_HELPER
 	help
 	  Choose this option if you have an ARM High Definition Colour LCD
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH 21/21] drm/nouveau: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (19 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 20/21] drm/arm: " Tobias Jakobi
@ 2016-07-15 12:48 ` Tobias Jakobi
  2016-07-15 15:01   ` Sean Paul
  2016-07-15 13:21 ` [PATCH 00/21] drm: " Emil Velikov
  21 siblings, 1 reply; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 12:48 UTC (permalink / raw)
  To: dri-devel; +Cc: Tobias Jakobi

Currently enabling Nouveau DRM support automatically pulls in
fbdev dependency. However this dep is unnecessary since
DRM core already handles this for us (DRM_FBDEV_EMULATION).

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
---
 drivers/gpu/drm/nouveau/Kconfig | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig
index 5ab13e7..2922a82 100644
--- a/drivers/gpu/drm/nouveau/Kconfig
+++ b/drivers/gpu/drm/nouveau/Kconfig
@@ -3,13 +3,7 @@ config DRM_NOUVEAU
 	depends on DRM && PCI
         select FW_LOADER
 	select DRM_KMS_HELPER
-	select DRM_KMS_FB_HELPER
 	select DRM_TTM
-	select FB_CFB_FILLRECT
-	select FB_CFB_COPYAREA
-	select FB_CFB_IMAGEBLIT
-	select FB
-	select FRAMEBUFFER_CONSOLE if !EXPERT
 	select FB_BACKLIGHT if DRM_NOUVEAU_BACKLIGHT
 	select ACPI_VIDEO if ACPI && X86 && BACKLIGHT_CLASS_DEVICE && INPUT
 	select X86_PLATFORM_DEVICES if ACPI && X86
-- 
2.7.3

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
                   ` (20 preceding siblings ...)
  2016-07-15 12:48 ` [PATCH 21/21] drm/nouveau: " Tobias Jakobi
@ 2016-07-15 13:21 ` Emil Velikov
  2016-07-15 13:37   ` Tobias Jakobi
  21 siblings, 1 reply; 32+ messages in thread
From: Emil Velikov @ 2016-07-15 13:21 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: ML dri-devel

On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> Hello,
>
> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
>
Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
vmwgfx still references the cfb_* API), so they should be updated as
well.

A few quick greps (check the exported symbols in drm_fb_helper.c and
you'll see the respective fbdev/wrapper functions) and some build
warning/errors will show you the way.

Regards,
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 13:21 ` [PATCH 00/21] drm: " Emil Velikov
@ 2016-07-15 13:37   ` Tobias Jakobi
  2016-07-15 14:17     ` Emil Velikov
  0 siblings, 1 reply; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 13:37 UTC (permalink / raw)
  To: Emil Velikov; +Cc: ML dri-devel

Hello Emil,

Emil Velikov wrote:
> On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>> Hello,
>>
>> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
>>
> Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
> vmwgfx still references the cfb_* API), so they should be updated as
> well.
I can only do this on ARM atm, so I suggest to just drop insufficient
patches.

With best wishes,
Tobias

> A few quick greps (check the exported symbols in drm_fb_helper.c and
> you'll see the respective fbdev/wrapper functions) and some build
> warning/errors will show you the way.
> 
> Regards,
> Emil
> 

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 13:37   ` Tobias Jakobi
@ 2016-07-15 14:17     ` Emil Velikov
  2016-07-15 14:37       ` Sean Paul
  2016-07-15 14:47       ` Tobias Jakobi
  0 siblings, 2 replies; 32+ messages in thread
From: Emil Velikov @ 2016-07-15 14:17 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: ML dri-devel

On 15 July 2016 at 14:37, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> Hello Emil,
>
> Emil Velikov wrote:
>> On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>>> Hello,
>>>
>>> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
>>>
>> Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
>> vmwgfx still references the cfb_* API), so they should be updated as
>> well.
> I can only do this on ARM atm, so I suggest to just drop insufficient
> patches.
>
Without checking (build and/or grep) it'll be a bit hard to figure out
which patches are insufficient. At the end of the day it's up-to you
and DRM maintainer(s).

Thanks for sorting out these. The negative diffstat is always appreciated ;-)
-Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 14:17     ` Emil Velikov
@ 2016-07-15 14:37       ` Sean Paul
  2016-07-18  7:13         ` Daniel Vetter
  2016-07-15 14:47       ` Tobias Jakobi
  1 sibling, 1 reply; 32+ messages in thread
From: Sean Paul @ 2016-07-15 14:37 UTC (permalink / raw)
  To: Emil Velikov; +Cc: Tobias Jakobi, ML dri-devel

On Fri, Jul 15, 2016 at 10:17 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
> On 15 July 2016 at 14:37, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>> Hello Emil,
>>
>> Emil Velikov wrote:
>>> On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>>>> Hello,
>>>>
>>>> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
>>>>
>>> Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
>>> vmwgfx still references the cfb_* API), so they should be updated as
>>> well.
>> I can only do this on ARM atm, so I suggest to just drop insufficient
>> patches.
>>
> Without checking (build and/or grep) it'll be a bit hard to figure out
> which patches are insufficient. At the end of the day it's up-to you
> and DRM maintainer(s).
>

I've compile tested atmel/nouveau/rcar/rockchip, I'll merge those in drm-misc.

Since I can't test the others easily, I'll leave them for now.

Sean



> Thanks for sorting out these. The negative diffstat is always appreciated ;-)
> -Emil
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 14:17     ` Emil Velikov
  2016-07-15 14:37       ` Sean Paul
@ 2016-07-15 14:47       ` Tobias Jakobi
  1 sibling, 0 replies; 32+ messages in thread
From: Tobias Jakobi @ 2016-07-15 14:47 UTC (permalink / raw)
  To: Emil Velikov; +Cc: ML dri-devel

Emil Velikov wrote:
> On 15 July 2016 at 14:37, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>> Hello Emil,
>>
>> Emil Velikov wrote:
>>> On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>>>> Hello,
>>>>
>>>> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
>>>>
>>> Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
>>> vmwgfx still references the cfb_* API), so they should be updated as
>>> well.
>> I can only do this on ARM atm, so I suggest to just drop insufficient
>> patches.
>>
> Without checking (build and/or grep) it'll be a bit hard to figure out
> which patches are insufficient. At the end of the day it's up-to you
> and DRM maintainer(s).
I kind of was expecting this, so I asked Daniel on IRC if I was supposed
to only inspect the Kconfigs. Seems like this doesn't suffice in every
case. :(

- Tobias

> 
> Thanks for sorting out these. The negative diffstat is always appreciated ;-)
> -Emil
> 

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 17/21] drm/atmel-hlcdc: make fbdev support really optional
  2016-07-15 12:48 ` [PATCH 17/21] drm/atmel-hlcdc: " Tobias Jakobi
@ 2016-07-15 15:01   ` Sean Paul
  0 siblings, 0 replies; 32+ messages in thread
From: Sean Paul @ 2016-07-15 15:01 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: dri-devel

On Fri, Jul 15, 2016 at 02:48:13PM +0200, Tobias Jakobi wrote:
> Currently enabling ATMEL HLCDC DRM support automatically pulls
> in fbdev dependency. However this dep is unnecessary since
> DRM core already handles this for us (DRM_FBDEV_EMULATION).
> 
> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>

Applied to drm-misc.

> ---
>  drivers/gpu/drm/atmel-hlcdc/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/atmel-hlcdc/Kconfig b/drivers/gpu/drm/atmel-hlcdc/Kconfig
> index 99b4f06..32bcc4b 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/Kconfig
> +++ b/drivers/gpu/drm/atmel-hlcdc/Kconfig
> @@ -3,7 +3,6 @@ config DRM_ATMEL_HLCDC
>  	depends on DRM && OF && COMMON_CLK && MFD_ATMEL_HLCDC && ARM
>  	select DRM_GEM_CMA_HELPER
>  	select DRM_KMS_HELPER
> -	select DRM_KMS_FB_HELPER
>  	select DRM_KMS_CMA_HELPER
>  	select DRM_PANEL
>  	help
> -- 
> 2.7.3
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 21/21] drm/nouveau: make fbdev support really optional
  2016-07-15 12:48 ` [PATCH 21/21] drm/nouveau: " Tobias Jakobi
@ 2016-07-15 15:01   ` Sean Paul
  0 siblings, 0 replies; 32+ messages in thread
From: Sean Paul @ 2016-07-15 15:01 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: dri-devel

On Fri, Jul 15, 2016 at 02:48:17PM +0200, Tobias Jakobi wrote:
> Currently enabling Nouveau DRM support automatically pulls in
> fbdev dependency. However this dep is unnecessary since
> DRM core already handles this for us (DRM_FBDEV_EMULATION).
> 
> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>

Applied to drm-misc.

> ---
>  drivers/gpu/drm/nouveau/Kconfig | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig
> index 5ab13e7..2922a82 100644
> --- a/drivers/gpu/drm/nouveau/Kconfig
> +++ b/drivers/gpu/drm/nouveau/Kconfig
> @@ -3,13 +3,7 @@ config DRM_NOUVEAU
>  	depends on DRM && PCI
>          select FW_LOADER
>  	select DRM_KMS_HELPER
> -	select DRM_KMS_FB_HELPER
>  	select DRM_TTM
> -	select FB_CFB_FILLRECT
> -	select FB_CFB_COPYAREA
> -	select FB_CFB_IMAGEBLIT
> -	select FB
> -	select FRAMEBUFFER_CONSOLE if !EXPERT
>  	select FB_BACKLIGHT if DRM_NOUVEAU_BACKLIGHT
>  	select ACPI_VIDEO if ACPI && X86 && BACKLIGHT_CLASS_DEVICE && INPUT
>  	select X86_PLATFORM_DEVICES if ACPI && X86
> -- 
> 2.7.3
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 09/21] drm/rockchip: make fbdev support really optional
  2016-07-15 12:48 ` [PATCH 09/21] drm/rockchip: " Tobias Jakobi
@ 2016-07-15 15:01   ` Sean Paul
  0 siblings, 0 replies; 32+ messages in thread
From: Sean Paul @ 2016-07-15 15:01 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: dri-devel

On Fri, Jul 15, 2016 at 02:48:05PM +0200, Tobias Jakobi wrote:
> Currently enabling Rockchip DRM support automatically pulls
> in fbdev dependency. However this dep is unnecessary since
> DRM core already handles this for us (DRM_FBDEV_EMULATION).
> 
> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>

Applied to drm-misc.

> ---
>  drivers/gpu/drm/rockchip/Kconfig | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/rockchip/Kconfig b/drivers/gpu/drm/rockchip/Kconfig
> index d30bdc3..6f0ef56 100644
> --- a/drivers/gpu/drm/rockchip/Kconfig
> +++ b/drivers/gpu/drm/rockchip/Kconfig
> @@ -3,11 +3,7 @@ config DRM_ROCKCHIP
>  	depends on DRM && ROCKCHIP_IOMMU
>  	depends on RESET_CONTROLLER
>  	select DRM_KMS_HELPER
> -	select DRM_KMS_FB_HELPER
>  	select DRM_PANEL
> -	select FB_CFB_FILLRECT
> -	select FB_CFB_COPYAREA
> -	select FB_CFB_IMAGEBLIT
>  	select VIDEOMODE_HELPERS
>  	help
>  	  Choose this option if you have a Rockchip soc chipset.
> -- 
> 2.7.3
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 11/21] drm/rcar-du: make fbdev support really optional
  2016-07-15 12:48 ` [PATCH 11/21] drm/rcar-du: " Tobias Jakobi
@ 2016-07-15 15:01   ` Sean Paul
  0 siblings, 0 replies; 32+ messages in thread
From: Sean Paul @ 2016-07-15 15:01 UTC (permalink / raw)
  To: Tobias Jakobi; +Cc: dri-devel

On Fri, Jul 15, 2016 at 02:48:07PM +0200, Tobias Jakobi wrote:
> Currently enabling R-Car DRM support automatically pulls in
> fbdev dependency. However this dep is unnecessary since
> DRM core already handles this for us (DRM_FBDEV_EMULATION).
> 
> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>

Applied to drm-misc.

> ---
>  drivers/gpu/drm/rcar-du/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
> index 7fc3ca5..4c2fd05 100644
> --- a/drivers/gpu/drm/rcar-du/Kconfig
> +++ b/drivers/gpu/drm/rcar-du/Kconfig
> @@ -6,7 +6,6 @@ config DRM_RCAR_DU
>  	select DRM_KMS_HELPER
>  	select DRM_KMS_CMA_HELPER
>  	select DRM_GEM_CMA_HELPER
> -	select DRM_KMS_FB_HELPER
>  	select VIDEOMODE_HELPERS
>  	help
>  	  Choose this option if you have an R-Car chipset.
> -- 
> 2.7.3
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH 00/21] drm: make fbdev support really optional
  2016-07-15 14:37       ` Sean Paul
@ 2016-07-18  7:13         ` Daniel Vetter
  0 siblings, 0 replies; 32+ messages in thread
From: Daniel Vetter @ 2016-07-18  7:13 UTC (permalink / raw)
  To: Sean Paul; +Cc: Tobias Jakobi, Emil Velikov, ML dri-devel

On Fri, Jul 15, 2016 at 10:37:45AM -0400, Sean Paul wrote:
> On Fri, Jul 15, 2016 at 10:17 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
> > On 15 July 2016 at 14:37, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> >> Hello Emil,
> >>
> >> Emil Velikov wrote:
> >>> On 15 July 2016 at 13:47, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> >>>> Hello,
> >>>>
> >>>> as request by Daniel here (http://www.spinics.net/lists/dri-devel/msg112592.html) I went ahead and also cleaned up the other Kconfig files. I have mostly compile tested the changes on an ARMv7 system (COMPILE_TEST).
> >>>>
> >>> Yay :-) IIRC some drivers don't fully use the fb_helpers (for example
> >>> vmwgfx still references the cfb_* API), so they should be updated as
> >>> well.
> >> I can only do this on ARM atm, so I suggest to just drop insufficient
> >> patches.
> >>
> > Without checking (build and/or grep) it'll be a bit hard to figure out
> > which patches are insufficient. At the end of the day it's up-to you
> > and DRM maintainer(s).
> >
> 
> I've compile tested atmel/nouveau/rcar/rockchip, I'll merge those in drm-misc.
> 
> Since I can't test the others easily, I'll leave them for now.

vmwgfx seems to be the only special case (it doesn't use any of the kms
helpers at all, so that's normal). I merged all the others.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2016-07-18  7:13 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-15 12:47 [PATCH 00/21] drm: make fbdev support really optional Tobias Jakobi
2016-07-15 12:47 ` [PATCH 01/21] drm/arc: " Tobias Jakobi
2016-07-15 12:47 ` [PATCH 02/21] drm/armada: " Tobias Jakobi
2016-07-15 12:47 ` [PATCH 03/21] drm/omap: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 04/21] drm/mgag200: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 05/21] drm/cirrus: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 06/21] drm/qxl: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 07/21] drm/imx: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 08/21] drm/shmobile: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 09/21] drm/rockchip: " Tobias Jakobi
2016-07-15 15:01   ` Sean Paul
2016-07-15 12:48 ` [PATCH 10/21] drm/gma500: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 11/21] drm/rcar-du: " Tobias Jakobi
2016-07-15 15:01   ` Sean Paul
2016-07-15 12:48 ` [PATCH 12/21] drm/tilcdc: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 13/21] drm/fsl-dcu: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 14/21] drm/bochs: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 15/21] drm/vmwgfx: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 16/21] drm/udl: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 17/21] drm/atmel-hlcdc: " Tobias Jakobi
2016-07-15 15:01   ` Sean Paul
2016-07-15 12:48 ` [PATCH 18/21] drm/virtio: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 19/21] drm/ast: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 20/21] drm/arm: " Tobias Jakobi
2016-07-15 12:48 ` [PATCH 21/21] drm/nouveau: " Tobias Jakobi
2016-07-15 15:01   ` Sean Paul
2016-07-15 13:21 ` [PATCH 00/21] drm: " Emil Velikov
2016-07-15 13:37   ` Tobias Jakobi
2016-07-15 14:17     ` Emil Velikov
2016-07-15 14:37       ` Sean Paul
2016-07-18  7:13         ` Daniel Vetter
2016-07-15 14:47       ` Tobias Jakobi

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.