* [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
@ 2016-02-15 12:57 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Changes v1->v2: incorporated Daniel comments
* added signed-off-by line to all patches
* threading enabled due to e-mail server constraints
* common blurb lines to all patches
* add reviewed by Alex, to patches 4 and 14
Carlos Palminha (17):
drm/virtio: removed optional dummy encoder mode_fixup function.
drm/udl: removed optional dummy encoder mode_fixup function.
drm/exynos: removed optional dummy encoder mode_fixup function.
drm/amdgpu: removed optional dummy encoder mode_fixup function.
drm/ast: removed optional dummy encoder mode_fixup function.
drm/bochs: removed optional dummy encoder mode_fixup function.
drm/cirrus: removed optional dummy encoder mode_fixup function.
drm/exynos: removed optional dummy encoder mode_fixup function.
drm/gma500: removed optional dummy encoder mode_fixup function.
drm/imx: removed optional dummy encoder mode_fixup function.
drm/msm/mdp: removed optional dummy encoder mode_fixup function.
drm/mgag200: removed optional dummy encoder mode_fixup function.
drm/qxl: removed optional dummy encoder mode_fixup function.
drm/radeon: removed optional dummy encoder mode_fixup function.
drm/rockchip: removed optional dummy encoder mode_fixup function.
drm/sti: removed optional dummy encoder mode_fixup function.
drm/tilcdc: removed optional dummy encoder mode_fixup function.
drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
drivers/gpu/drm/ast/ast_mode.c | 8 --------
drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
drivers/gpu/drm/gma500/gma_display.c | 7 -------
drivers/gpu/drm/gma500/gma_display.h | 3 ---
drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
drivers/gpu/drm/imx/imx-ldb.c | 8 --------
drivers/gpu/drm/imx/imx-tve.c | 8 --------
drivers/gpu/drm/imx/parallel-display.c | 8 --------
drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
drivers/gpu/drm/udl/udl_encoder.c | 8 --------
drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
33 files changed, 244 deletions(-)
--
2.5.0
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
@ 2016-02-15 12:57 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Changes v1->v2: incorporated Daniel comments
* added signed-off-by line to all patches
* threading enabled due to e-mail server constraints
* common blurb lines to all patches
* add reviewed by Alex, to patches 4 and 14
Carlos Palminha (17):
drm/virtio: removed optional dummy encoder mode_fixup function.
drm/udl: removed optional dummy encoder mode_fixup function.
drm/exynos: removed optional dummy encoder mode_fixup function.
drm/amdgpu: removed optional dummy encoder mode_fixup function.
drm/ast: removed optional dummy encoder mode_fixup function.
drm/bochs: removed optional dummy encoder mode_fixup function.
drm/cirrus: removed optional dummy encoder mode_fixup function.
drm/exynos: removed optional dummy encoder mode_fixup function.
drm/gma500: removed optional dummy encoder mode_fixup function.
drm/imx: removed optional dummy encoder mode_fixup function.
drm/msm/mdp: removed optional dummy encoder mode_fixup function.
drm/mgag200: removed optional dummy encoder mode_fixup function.
drm/qxl: removed optional dummy encoder mode_fixup function.
drm/radeon: removed optional dummy encoder mode_fixup function.
drm/rockchip: removed optional dummy encoder mode_fixup function.
drm/sti: removed optional dummy encoder mode_fixup function.
drm/tilcdc: removed optional dummy encoder mode_fixup function.
drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
drivers/gpu/drm/ast/ast_mode.c | 8 --------
drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
drivers/gpu/drm/gma500/gma_display.c | 7 -------
drivers/gpu/drm/gma500/gma_display.h | 3 ---
drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
drivers/gpu/drm/imx/imx-ldb.c | 8 --------
drivers/gpu/drm/imx/imx-tve.c | 8 --------
drivers/gpu/drm/imx/parallel-display.c | 8 --------
drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
drivers/gpu/drm/udl/udl_encoder.c | 8 --------
drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
33 files changed, 244 deletions(-)
--
2.5.0
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 01/17] drm/virtio: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:57 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index a165f03..429aa31 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -282,13 +282,6 @@ static const struct drm_crtc_helper_funcs virtio_gpu_crtc_helper_funcs = {
.atomic_check = virtio_gpu_crtc_atomic_check,
};
-static bool virtio_gpu_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void virtio_gpu_enc_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -362,7 +355,6 @@ virtio_gpu_best_encoder(struct drm_connector *connector)
}
static const struct drm_encoder_helper_funcs virtio_gpu_enc_helper_funcs = {
- .mode_fixup = virtio_gpu_enc_mode_fixup,
.mode_set = virtio_gpu_enc_mode_set,
.enable = virtio_gpu_enc_enable,
.disable = virtio_gpu_enc_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 01/17] drm/virtio: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
(?)
(?)
@ 2016-02-15 12:57 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index a165f03..429aa31 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -282,13 +282,6 @@ static const struct drm_crtc_helper_funcs virtio_gpu_crtc_helper_funcs = {
.atomic_check = virtio_gpu_crtc_atomic_check,
};
-static bool virtio_gpu_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void virtio_gpu_enc_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -362,7 +355,6 @@ virtio_gpu_best_encoder(struct drm_connector *connector)
}
static const struct drm_encoder_helper_funcs virtio_gpu_enc_helper_funcs = {
- .mode_fixup = virtio_gpu_enc_mode_fixup,
.mode_set = virtio_gpu_enc_mode_set,
.enable = virtio_gpu_enc_enable,
.disable = virtio_gpu_enc_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 01/17] drm/virtio: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:57 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio/virtgpu_display.c
index a165f03..429aa31 100644
--- a/drivers/gpu/drm/virtio/virtgpu_display.c
+++ b/drivers/gpu/drm/virtio/virtgpu_display.c
@@ -282,13 +282,6 @@ static const struct drm_crtc_helper_funcs virtio_gpu_crtc_helper_funcs = {
.atomic_check = virtio_gpu_crtc_atomic_check,
};
-static bool virtio_gpu_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void virtio_gpu_enc_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -362,7 +355,6 @@ virtio_gpu_best_encoder(struct drm_connector *connector)
}
static const struct drm_encoder_helper_funcs virtio_gpu_enc_helper_funcs = {
- .mode_fixup = virtio_gpu_enc_mode_fixup,
.mode_set = virtio_gpu_enc_mode_set,
.enable = virtio_gpu_enc_enable,
.disable = virtio_gpu_enc_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 02/17] drm/udl: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:57 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/udl/udl_encoder.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/udl/udl_encoder.c b/drivers/gpu/drm/udl/udl_encoder.c
index a181a64..59a4b34 100644
--- a/drivers/gpu/drm/udl/udl_encoder.c
+++ b/drivers/gpu/drm/udl/udl_encoder.c
@@ -26,13 +26,6 @@ static void udl_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool udl_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void udl_encoder_prepare(struct drm_encoder *encoder)
{
}
@@ -54,7 +47,6 @@ udl_encoder_dpms(struct drm_encoder *encoder, int mode)
static const struct drm_encoder_helper_funcs udl_helper_funcs = {
.dpms = udl_encoder_dpms,
- .mode_fixup = udl_mode_fixup,
.prepare = udl_encoder_prepare,
.mode_set = udl_encoder_mode_set,
.commit = udl_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 02/17] drm/udl: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (3 preceding siblings ...)
(?)
@ 2016-02-15 12:57 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/udl/udl_encoder.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/udl/udl_encoder.c b/drivers/gpu/drm/udl/udl_encoder.c
index a181a64..59a4b34 100644
--- a/drivers/gpu/drm/udl/udl_encoder.c
+++ b/drivers/gpu/drm/udl/udl_encoder.c
@@ -26,13 +26,6 @@ static void udl_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool udl_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void udl_encoder_prepare(struct drm_encoder *encoder)
{
}
@@ -54,7 +47,6 @@ udl_encoder_dpms(struct drm_encoder *encoder, int mode)
static const struct drm_encoder_helper_funcs udl_helper_funcs = {
.dpms = udl_encoder_dpms,
- .mode_fixup = udl_mode_fixup,
.prepare = udl_encoder_prepare,
.mode_set = udl_encoder_mode_set,
.commit = udl_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 02/17] drm/udl: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:57 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:57 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/udl/udl_encoder.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/udl/udl_encoder.c b/drivers/gpu/drm/udl/udl_encoder.c
index a181a64..59a4b34 100644
--- a/drivers/gpu/drm/udl/udl_encoder.c
+++ b/drivers/gpu/drm/udl/udl_encoder.c
@@ -26,13 +26,6 @@ static void udl_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool udl_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void udl_encoder_prepare(struct drm_encoder *encoder)
{
}
@@ -54,7 +47,6 @@ udl_encoder_dpms(struct drm_encoder *encoder, int mode)
static const struct drm_encoder_helper_funcs udl_helper_funcs = {
.dpms = udl_encoder_dpms,
- .mode_fixup = udl_mode_fixup,
.prepare = udl_encoder_prepare,
.mode_set = udl_encoder_mode_set,
.commit = udl_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:58 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:58 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
index e977a81..736115c 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
@@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dsi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
}
static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
- .mode_fixup = exynos_dsi_mode_fixup,
.mode_set = exynos_dsi_mode_set,
.enable = exynos_dsi_enable,
.disable = exynos_dsi_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:58 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:58 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
index e977a81..736115c 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
@@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dsi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
}
static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
- .mode_fixup = exynos_dsi_mode_fixup,
.mode_set = exynos_dsi_mode_set,
.enable = exynos_dsi_enable,
.disable = exynos_dsi_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 04/17] drm/amdgpu: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:58 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:58 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
3 files changed, 24 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
index 093599a..3483018 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
@@ -3624,16 +3624,8 @@ dce_v10_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v10_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v10_0_ext_helper_funcs = {
.dpms = dce_v10_0_ext_dpms,
- .mode_fixup = dce_v10_0_ext_mode_fixup,
.prepare = dce_v10_0_ext_prepare,
.mode_set = dce_v10_0_ext_mode_set,
.commit = dce_v10_0_ext_commit,
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
index 8e67249..36deea1 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
@@ -3619,16 +3619,8 @@ dce_v11_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v11_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v11_0_ext_helper_funcs = {
.dpms = dce_v11_0_ext_dpms,
- .mode_fixup = dce_v11_0_ext_mode_fixup,
.prepare = dce_v11_0_ext_prepare,
.mode_set = dce_v11_0_ext_mode_set,
.commit = dce_v11_0_ext_commit,
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
index d0e128c..25dd8b6 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
@@ -3554,16 +3554,8 @@ dce_v8_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v8_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v8_0_ext_helper_funcs = {
.dpms = dce_v8_0_ext_dpms,
- .mode_fixup = dce_v8_0_ext_mode_fixup,
.prepare = dce_v8_0_ext_prepare,
.mode_set = dce_v8_0_ext_mode_set,
.commit = dce_v8_0_ext_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 04/17] drm/amdgpu: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:58 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:58 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
3 files changed, 24 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
index 093599a..3483018 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v10_0.c
@@ -3624,16 +3624,8 @@ dce_v10_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v10_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v10_0_ext_helper_funcs = {
.dpms = dce_v10_0_ext_dpms,
- .mode_fixup = dce_v10_0_ext_mode_fixup,
.prepare = dce_v10_0_ext_prepare,
.mode_set = dce_v10_0_ext_mode_set,
.commit = dce_v10_0_ext_commit,
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
index 8e67249..36deea1 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v11_0.c
@@ -3619,16 +3619,8 @@ dce_v11_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v11_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v11_0_ext_helper_funcs = {
.dpms = dce_v11_0_ext_dpms,
- .mode_fixup = dce_v11_0_ext_mode_fixup,
.prepare = dce_v11_0_ext_prepare,
.mode_set = dce_v11_0_ext_mode_set,
.commit = dce_v11_0_ext_commit,
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
index d0e128c..25dd8b6 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v8_0.c
@@ -3554,16 +3554,8 @@ dce_v8_0_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool dce_v8_0_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs dce_v8_0_ext_helper_funcs = {
.dpms = dce_v8_0_ext_dpms,
- .mode_fixup = dce_v8_0_ext_mode_fixup,
.prepare = dce_v8_0_ext_prepare,
.mode_set = dce_v8_0_ext_mode_set,
.commit = dce_v8_0_ext_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 05/17] drm/ast: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/ast/ast_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
index 0123458..f221e2d 100644
--- a/drivers/gpu/drm/ast/ast_mode.c
+++ b/drivers/gpu/drm/ast/ast_mode.c
@@ -710,13 +710,6 @@ static void ast_encoder_dpms(struct drm_encoder *encoder, int mode)
}
-static bool ast_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void ast_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -736,7 +729,6 @@ static void ast_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs ast_enc_helper_funcs = {
.dpms = ast_encoder_dpms,
- .mode_fixup = ast_mode_fixup,
.prepare = ast_encoder_prepare,
.commit = ast_encoder_commit,
.mode_set = ast_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 05/17] drm/ast: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (6 preceding siblings ...)
(?)
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/ast/ast_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
index 0123458..f221e2d 100644
--- a/drivers/gpu/drm/ast/ast_mode.c
+++ b/drivers/gpu/drm/ast/ast_mode.c
@@ -710,13 +710,6 @@ static void ast_encoder_dpms(struct drm_encoder *encoder, int mode)
}
-static bool ast_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void ast_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -736,7 +729,6 @@ static void ast_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs ast_enc_helper_funcs = {
.dpms = ast_encoder_dpms,
- .mode_fixup = ast_mode_fixup,
.prepare = ast_encoder_prepare,
.commit = ast_encoder_commit,
.mode_set = ast_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 05/17] drm/ast: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:59 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/ast/ast_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
index 0123458..f221e2d 100644
--- a/drivers/gpu/drm/ast/ast_mode.c
+++ b/drivers/gpu/drm/ast/ast_mode.c
@@ -710,13 +710,6 @@ static void ast_encoder_dpms(struct drm_encoder *encoder, int mode)
}
-static bool ast_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void ast_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -736,7 +729,6 @@ static void ast_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs ast_enc_helper_funcs = {
.dpms = ast_encoder_dpms,
- .mode_fixup = ast_mode_fixup,
.prepare = ast_encoder_prepare,
.commit = ast_encoder_commit,
.mode_set = ast_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 06/17] drm/bochs: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/bochs/bochs_kms.c b/drivers/gpu/drm/bochs/bochs_kms.c
index 2849f1b..317c27f 100644
--- a/drivers/gpu/drm/bochs/bochs_kms.c
+++ b/drivers/gpu/drm/bochs/bochs_kms.c
@@ -152,13 +152,6 @@ static void bochs_crtc_init(struct drm_device *dev)
drm_crtc_helper_add(crtc, &bochs_helper_funcs);
}
-static bool bochs_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void bochs_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -179,7 +172,6 @@ static void bochs_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs bochs_encoder_helper_funcs = {
.dpms = bochs_encoder_dpms,
- .mode_fixup = bochs_encoder_mode_fixup,
.mode_set = bochs_encoder_mode_set,
.prepare = bochs_encoder_prepare,
.commit = bochs_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 06/17] drm/bochs: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (8 preceding siblings ...)
(?)
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/bochs/bochs_kms.c b/drivers/gpu/drm/bochs/bochs_kms.c
index 2849f1b..317c27f 100644
--- a/drivers/gpu/drm/bochs/bochs_kms.c
+++ b/drivers/gpu/drm/bochs/bochs_kms.c
@@ -152,13 +152,6 @@ static void bochs_crtc_init(struct drm_device *dev)
drm_crtc_helper_add(crtc, &bochs_helper_funcs);
}
-static bool bochs_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void bochs_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -179,7 +172,6 @@ static void bochs_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs bochs_encoder_helper_funcs = {
.dpms = bochs_encoder_dpms,
- .mode_fixup = bochs_encoder_mode_fixup,
.mode_set = bochs_encoder_mode_set,
.prepare = bochs_encoder_prepare,
.commit = bochs_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 06/17] drm/bochs: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:59 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/bochs/bochs_kms.c b/drivers/gpu/drm/bochs/bochs_kms.c
index 2849f1b..317c27f 100644
--- a/drivers/gpu/drm/bochs/bochs_kms.c
+++ b/drivers/gpu/drm/bochs/bochs_kms.c
@@ -152,13 +152,6 @@ static void bochs_crtc_init(struct drm_device *dev)
drm_crtc_helper_add(crtc, &bochs_helper_funcs);
}
-static bool bochs_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void bochs_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -179,7 +172,6 @@ static void bochs_encoder_commit(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs bochs_encoder_helper_funcs = {
.dpms = bochs_encoder_dpms,
- .mode_fixup = bochs_encoder_mode_fixup,
.mode_set = bochs_encoder_mode_set,
.prepare = bochs_encoder_prepare,
.commit = bochs_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 07/17] drm/cirrus: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c b/drivers/gpu/drm/cirrus/cirrus_mode.c
index 4a02854..432ce94 100644
--- a/drivers/gpu/drm/cirrus/cirrus_mode.c
+++ b/drivers/gpu/drm/cirrus/cirrus_mode.c
@@ -430,14 +430,6 @@ void cirrus_crtc_fb_gamma_get(struct drm_crtc *crtc, u16 *red, u16 *green,
*blue = cirrus_crtc->lut_b[regno];
}
-
-static bool cirrus_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void cirrus_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -466,7 +458,6 @@ static void cirrus_encoder_destroy(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs cirrus_encoder_helper_funcs = {
.dpms = cirrus_encoder_dpms,
- .mode_fixup = cirrus_encoder_mode_fixup,
.mode_set = cirrus_encoder_mode_set,
.prepare = cirrus_encoder_prepare,
.commit = cirrus_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 07/17] drm/cirrus: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:59 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/cirrus/cirrus_mode.c b/drivers/gpu/drm/cirrus/cirrus_mode.c
index 4a02854..432ce94 100644
--- a/drivers/gpu/drm/cirrus/cirrus_mode.c
+++ b/drivers/gpu/drm/cirrus/cirrus_mode.c
@@ -430,14 +430,6 @@ void cirrus_crtc_fb_gamma_get(struct drm_crtc *crtc, u16 *red, u16 *green,
*blue = cirrus_crtc->lut_b[regno];
}
-
-static bool cirrus_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void cirrus_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -466,7 +458,6 @@ static void cirrus_encoder_destroy(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs cirrus_encoder_helper_funcs = {
.dpms = cirrus_encoder_dpms,
- .mode_fixup = cirrus_encoder_mode_fixup,
.mode_set = cirrus_encoder_mode_set,
.prepare = cirrus_encoder_prepare,
.commit = cirrus_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 08/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
3 files changed, 24 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c b/drivers/gpu/drm/exynos/exynos_dp_core.c
index 673164b..9fd12c62 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
@@ -1155,13 +1155,6 @@ static int exynos_dp_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dp_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dp_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1177,7 +1170,6 @@ static void exynos_dp_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dp_encoder_helper_funcs = {
- .mode_fixup = exynos_dp_mode_fixup,
.mode_set = exynos_dp_mode_set,
.enable = exynos_dp_enable,
.disable = exynos_dp_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_dpi.c b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
index 05350ae..75e570f 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dpi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
@@ -128,13 +128,6 @@ static int exynos_dpi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dpi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dpi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -162,7 +155,6 @@ static void exynos_dpi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dpi_encoder_helper_funcs = {
- .mode_fixup = exynos_dpi_mode_fixup,
.mode_set = exynos_dpi_mode_set,
.enable = exynos_dpi_enable,
.disable = exynos_dpi_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
index 62ac4e5..65108cb 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
@@ -410,13 +410,6 @@ static int vidi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_vidi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_vidi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -432,7 +425,6 @@ static void exynos_vidi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_vidi_encoder_helper_funcs = {
- .mode_fixup = exynos_vidi_mode_fixup,
.mode_set = exynos_vidi_mode_set,
.enable = exynos_vidi_enable,
.disable = exynos_vidi_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 08/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (11 preceding siblings ...)
(?)
@ 2016-02-15 12:59 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
3 files changed, 24 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c b/drivers/gpu/drm/exynos/exynos_dp_core.c
index 673164b..9fd12c62 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
@@ -1155,13 +1155,6 @@ static int exynos_dp_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dp_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dp_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1177,7 +1170,6 @@ static void exynos_dp_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dp_encoder_helper_funcs = {
- .mode_fixup = exynos_dp_mode_fixup,
.mode_set = exynos_dp_mode_set,
.enable = exynos_dp_enable,
.disable = exynos_dp_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_dpi.c b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
index 05350ae..75e570f 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dpi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
@@ -128,13 +128,6 @@ static int exynos_dpi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dpi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dpi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -162,7 +155,6 @@ static void exynos_dpi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dpi_encoder_helper_funcs = {
- .mode_fixup = exynos_dpi_mode_fixup,
.mode_set = exynos_dpi_mode_set,
.enable = exynos_dpi_enable,
.disable = exynos_dpi_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
index 62ac4e5..65108cb 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
@@ -410,13 +410,6 @@ static int vidi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_vidi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_vidi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -432,7 +425,6 @@ static void exynos_vidi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_vidi_encoder_helper_funcs = {
- .mode_fixup = exynos_vidi_mode_fixup,
.mode_set = exynos_vidi_mode_set,
.enable = exynos_vidi_enable,
.disable = exynos_vidi_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 08/17] drm/exynos: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 12:59 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 12:59 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
3 files changed, 24 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_dp_core.c b/drivers/gpu/drm/exynos/exynos_dp_core.c
index 673164b..9fd12c62 100644
--- a/drivers/gpu/drm/exynos/exynos_dp_core.c
+++ b/drivers/gpu/drm/exynos/exynos_dp_core.c
@@ -1155,13 +1155,6 @@ static int exynos_dp_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dp_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dp_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1177,7 +1170,6 @@ static void exynos_dp_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dp_encoder_helper_funcs = {
- .mode_fixup = exynos_dp_mode_fixup,
.mode_set = exynos_dp_mode_set,
.enable = exynos_dp_enable,
.disable = exynos_dp_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_dpi.c b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
index 05350ae..75e570f 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_dpi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_dpi.c
@@ -128,13 +128,6 @@ static int exynos_dpi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_dpi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_dpi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -162,7 +155,6 @@ static void exynos_dpi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_dpi_encoder_helper_funcs = {
- .mode_fixup = exynos_dpi_mode_fixup,
.mode_set = exynos_dpi_mode_set,
.enable = exynos_dpi_enable,
.disable = exynos_dpi_disable,
diff --git a/drivers/gpu/drm/exynos/exynos_drm_vidi.c b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
index 62ac4e5..65108cb 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_vidi.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_vidi.c
@@ -410,13 +410,6 @@ static int vidi_create_connector(struct drm_encoder *encoder)
return 0;
}
-static bool exynos_vidi_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void exynos_vidi_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -432,7 +425,6 @@ static void exynos_vidi_disable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs exynos_vidi_encoder_helper_funcs = {
- .mode_fixup = exynos_vidi_mode_fixup,
.mode_set = exynos_vidi_mode_set,
.enable = exynos_vidi_enable,
.disable = exynos_vidi_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
drivers/gpu/drm/gma500/gma_display.c | 7 -------
drivers/gpu/drm/gma500/gma_display.h | 3 ---
drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
5 files changed, 13 deletions(-)
diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
index d0717a8..b837e7a 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
@@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
.dpms = cdv_intel_crt_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.commit = gma_encoder_commit,
.mode_set = cdv_intel_crt_mode_set,
diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
index ddf2d77..28f9d90 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
@@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
.dpms = cdv_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = cdv_hdmi_mode_set,
.commit = gma_encoder_commit,
diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
index ff17af4..9270821 100644
--- a/drivers/gpu/drm/gma500/gma_display.c
+++ b/drivers/gpu/drm/gma500/gma_display.c
@@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
return 0;
}
-bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
const struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
index ed569d8..78b9f98 100644
--- a/drivers/gpu/drm/gma500/gma_display.h
+++ b/drivers/gpu/drm/gma500/gma_display.h
@@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
extern void gma_encoder_prepare(struct drm_encoder *encoder);
extern void gma_encoder_commit(struct drm_encoder *encoder);
extern void gma_encoder_destroy(struct drm_encoder *encoder);
-extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode);
/* Common clock related functions */
extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
index 2d18499..8b2eb32 100644
--- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
+++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
@@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
.dpms = oaktrail_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = oaktrail_hdmi_mode_set,
.commit = gma_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (13 preceding siblings ...)
(?)
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
drivers/gpu/drm/gma500/gma_display.c | 7 -------
drivers/gpu/drm/gma500/gma_display.h | 3 ---
drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
5 files changed, 13 deletions(-)
diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
index d0717a8..b837e7a 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
@@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
.dpms = cdv_intel_crt_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.commit = gma_encoder_commit,
.mode_set = cdv_intel_crt_mode_set,
diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
index ddf2d77..28f9d90 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
@@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
.dpms = cdv_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = cdv_hdmi_mode_set,
.commit = gma_encoder_commit,
diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
index ff17af4..9270821 100644
--- a/drivers/gpu/drm/gma500/gma_display.c
+++ b/drivers/gpu/drm/gma500/gma_display.c
@@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
return 0;
}
-bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
const struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
index ed569d8..78b9f98 100644
--- a/drivers/gpu/drm/gma500/gma_display.h
+++ b/drivers/gpu/drm/gma500/gma_display.h
@@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
extern void gma_encoder_prepare(struct drm_encoder *encoder);
extern void gma_encoder_commit(struct drm_encoder *encoder);
extern void gma_encoder_destroy(struct drm_encoder *encoder);
-extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode);
/* Common clock related functions */
extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
index 2d18499..8b2eb32 100644
--- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
+++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
@@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
.dpms = oaktrail_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = oaktrail_hdmi_mode_set,
.commit = gma_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:00 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
drivers/gpu/drm/gma500/gma_display.c | 7 -------
drivers/gpu/drm/gma500/gma_display.h | 3 ---
drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
5 files changed, 13 deletions(-)
diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
index d0717a8..b837e7a 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
@@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
.dpms = cdv_intel_crt_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.commit = gma_encoder_commit,
.mode_set = cdv_intel_crt_mode_set,
diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
index ddf2d77..28f9d90 100644
--- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
+++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
@@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
.dpms = cdv_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = cdv_hdmi_mode_set,
.commit = gma_encoder_commit,
diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
index ff17af4..9270821 100644
--- a/drivers/gpu/drm/gma500/gma_display.c
+++ b/drivers/gpu/drm/gma500/gma_display.c
@@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
return 0;
}
-bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
const struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
index ed569d8..78b9f98 100644
--- a/drivers/gpu/drm/gma500/gma_display.h
+++ b/drivers/gpu/drm/gma500/gma_display.h
@@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
extern void gma_encoder_prepare(struct drm_encoder *encoder);
extern void gma_encoder_commit(struct drm_encoder *encoder);
extern void gma_encoder_destroy(struct drm_encoder *encoder);
-extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode);
/* Common clock related functions */
extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
index 2d18499..8b2eb32 100644
--- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
+++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
@@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
.dpms = oaktrail_hdmi_dpms,
- .mode_fixup = gma_encoder_mode_fixup,
.prepare = gma_encoder_prepare,
.mode_set = oaktrail_hdmi_mode_set,
.commit = gma_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
drivers/gpu/drm/imx/imx-ldb.c | 8 --------
drivers/gpu/drm/imx/imx-tve.c | 8 --------
drivers/gpu/drm/imx/parallel-display.c | 8 --------
4 files changed, 32 deletions(-)
diff --git a/drivers/gpu/drm/imx/dw_hdmi-imx.c b/drivers/gpu/drm/imx/dw_hdmi-imx.c
index 063825f..21d6158 100644
--- a/drivers/gpu/drm/imx/dw_hdmi-imx.c
+++ b/drivers/gpu/drm/imx/dw_hdmi-imx.c
@@ -109,13 +109,6 @@ static void dw_hdmi_imx_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool dw_hdmi_imx_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adj_mode)
-{
- return true;
-}
-
static void dw_hdmi_imx_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adj_mode)
@@ -138,7 +131,6 @@ static void dw_hdmi_imx_encoder_prepare(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs dw_hdmi_imx_encoder_helper_funcs = {
- .mode_fixup = dw_hdmi_imx_encoder_mode_fixup,
.mode_set = dw_hdmi_imx_encoder_mode_set,
.prepare = dw_hdmi_imx_encoder_prepare,
.commit = dw_hdmi_imx_encoder_commit,
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c
index 22ac482..024d613 100644
--- a/drivers/gpu/drm/imx/imx-ldb.c
+++ b/drivers/gpu/drm/imx/imx-ldb.c
@@ -139,13 +139,6 @@ static void imx_ldb_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool imx_ldb_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_ldb_set_clock(struct imx_ldb *ldb, int mux, int chno,
unsigned long serial_clk, unsigned long di_clk)
{
@@ -376,7 +369,6 @@ static const struct drm_encoder_funcs imx_ldb_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_ldb_encoder_helper_funcs = {
.dpms = imx_ldb_encoder_dpms,
- .mode_fixup = imx_ldb_encoder_mode_fixup,
.prepare = imx_ldb_encoder_prepare,
.commit = imx_ldb_encoder_commit,
.mode_set = imx_ldb_encoder_mode_set,
diff --git a/drivers/gpu/drm/imx/imx-tve.c b/drivers/gpu/drm/imx/imx-tve.c
index 292349f..ae7a9fb 100644
--- a/drivers/gpu/drm/imx/imx-tve.c
+++ b/drivers/gpu/drm/imx/imx-tve.c
@@ -286,13 +286,6 @@ static void imx_tve_encoder_dpms(struct drm_encoder *encoder, int mode)
dev_err(tve->dev, "failed to disable TVOUT: %d\n", ret);
}
-static bool imx_tve_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_tve_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_tve *tve = enc_to_tve(encoder);
@@ -379,7 +372,6 @@ static const struct drm_encoder_funcs imx_tve_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_tve_encoder_helper_funcs = {
.dpms = imx_tve_encoder_dpms,
- .mode_fixup = imx_tve_encoder_mode_fixup,
.prepare = imx_tve_encoder_prepare,
.mode_set = imx_tve_encoder_mode_set,
.commit = imx_tve_encoder_commit,
diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
index 0ffef17..363e2c7 100644
--- a/drivers/gpu/drm/imx/parallel-display.c
+++ b/drivers/gpu/drm/imx/parallel-display.c
@@ -112,13 +112,6 @@ static void imx_pd_encoder_dpms(struct drm_encoder *encoder, int mode)
drm_panel_enable(imxpd->panel);
}
-static bool imx_pd_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_pd_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_parallel_display *imxpd = enc_to_imxpd(encoder);
@@ -166,7 +159,6 @@ static const struct drm_encoder_funcs imx_pd_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_pd_encoder_helper_funcs = {
.dpms = imx_pd_encoder_dpms,
- .mode_fixup = imx_pd_encoder_mode_fixup,
.prepare = imx_pd_encoder_prepare,
.commit = imx_pd_encoder_commit,
.mode_set = imx_pd_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (15 preceding siblings ...)
(?)
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
drivers/gpu/drm/imx/imx-ldb.c | 8 --------
drivers/gpu/drm/imx/imx-tve.c | 8 --------
drivers/gpu/drm/imx/parallel-display.c | 8 --------
4 files changed, 32 deletions(-)
diff --git a/drivers/gpu/drm/imx/dw_hdmi-imx.c b/drivers/gpu/drm/imx/dw_hdmi-imx.c
index 063825f..21d6158 100644
--- a/drivers/gpu/drm/imx/dw_hdmi-imx.c
+++ b/drivers/gpu/drm/imx/dw_hdmi-imx.c
@@ -109,13 +109,6 @@ static void dw_hdmi_imx_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool dw_hdmi_imx_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adj_mode)
-{
- return true;
-}
-
static void dw_hdmi_imx_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adj_mode)
@@ -138,7 +131,6 @@ static void dw_hdmi_imx_encoder_prepare(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs dw_hdmi_imx_encoder_helper_funcs = {
- .mode_fixup = dw_hdmi_imx_encoder_mode_fixup,
.mode_set = dw_hdmi_imx_encoder_mode_set,
.prepare = dw_hdmi_imx_encoder_prepare,
.commit = dw_hdmi_imx_encoder_commit,
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c
index 22ac482..024d613 100644
--- a/drivers/gpu/drm/imx/imx-ldb.c
+++ b/drivers/gpu/drm/imx/imx-ldb.c
@@ -139,13 +139,6 @@ static void imx_ldb_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool imx_ldb_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_ldb_set_clock(struct imx_ldb *ldb, int mux, int chno,
unsigned long serial_clk, unsigned long di_clk)
{
@@ -376,7 +369,6 @@ static const struct drm_encoder_funcs imx_ldb_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_ldb_encoder_helper_funcs = {
.dpms = imx_ldb_encoder_dpms,
- .mode_fixup = imx_ldb_encoder_mode_fixup,
.prepare = imx_ldb_encoder_prepare,
.commit = imx_ldb_encoder_commit,
.mode_set = imx_ldb_encoder_mode_set,
diff --git a/drivers/gpu/drm/imx/imx-tve.c b/drivers/gpu/drm/imx/imx-tve.c
index 292349f..ae7a9fb 100644
--- a/drivers/gpu/drm/imx/imx-tve.c
+++ b/drivers/gpu/drm/imx/imx-tve.c
@@ -286,13 +286,6 @@ static void imx_tve_encoder_dpms(struct drm_encoder *encoder, int mode)
dev_err(tve->dev, "failed to disable TVOUT: %d\n", ret);
}
-static bool imx_tve_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_tve_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_tve *tve = enc_to_tve(encoder);
@@ -379,7 +372,6 @@ static const struct drm_encoder_funcs imx_tve_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_tve_encoder_helper_funcs = {
.dpms = imx_tve_encoder_dpms,
- .mode_fixup = imx_tve_encoder_mode_fixup,
.prepare = imx_tve_encoder_prepare,
.mode_set = imx_tve_encoder_mode_set,
.commit = imx_tve_encoder_commit,
diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
index 0ffef17..363e2c7 100644
--- a/drivers/gpu/drm/imx/parallel-display.c
+++ b/drivers/gpu/drm/imx/parallel-display.c
@@ -112,13 +112,6 @@ static void imx_pd_encoder_dpms(struct drm_encoder *encoder, int mode)
drm_panel_enable(imxpd->panel);
}
-static bool imx_pd_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_pd_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_parallel_display *imxpd = enc_to_imxpd(encoder);
@@ -166,7 +159,6 @@ static const struct drm_encoder_funcs imx_pd_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_pd_encoder_helper_funcs = {
.dpms = imx_pd_encoder_dpms,
- .mode_fixup = imx_pd_encoder_mode_fixup,
.prepare = imx_pd_encoder_prepare,
.commit = imx_pd_encoder_commit,
.mode_set = imx_pd_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:00 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
drivers/gpu/drm/imx/imx-ldb.c | 8 --------
drivers/gpu/drm/imx/imx-tve.c | 8 --------
drivers/gpu/drm/imx/parallel-display.c | 8 --------
4 files changed, 32 deletions(-)
diff --git a/drivers/gpu/drm/imx/dw_hdmi-imx.c b/drivers/gpu/drm/imx/dw_hdmi-imx.c
index 063825f..21d6158 100644
--- a/drivers/gpu/drm/imx/dw_hdmi-imx.c
+++ b/drivers/gpu/drm/imx/dw_hdmi-imx.c
@@ -109,13 +109,6 @@ static void dw_hdmi_imx_encoder_disable(struct drm_encoder *encoder)
{
}
-static bool dw_hdmi_imx_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adj_mode)
-{
- return true;
-}
-
static void dw_hdmi_imx_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adj_mode)
@@ -138,7 +131,6 @@ static void dw_hdmi_imx_encoder_prepare(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs dw_hdmi_imx_encoder_helper_funcs = {
- .mode_fixup = dw_hdmi_imx_encoder_mode_fixup,
.mode_set = dw_hdmi_imx_encoder_mode_set,
.prepare = dw_hdmi_imx_encoder_prepare,
.commit = dw_hdmi_imx_encoder_commit,
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c
index 22ac482..024d613 100644
--- a/drivers/gpu/drm/imx/imx-ldb.c
+++ b/drivers/gpu/drm/imx/imx-ldb.c
@@ -139,13 +139,6 @@ static void imx_ldb_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool imx_ldb_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_ldb_set_clock(struct imx_ldb *ldb, int mux, int chno,
unsigned long serial_clk, unsigned long di_clk)
{
@@ -376,7 +369,6 @@ static const struct drm_encoder_funcs imx_ldb_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_ldb_encoder_helper_funcs = {
.dpms = imx_ldb_encoder_dpms,
- .mode_fixup = imx_ldb_encoder_mode_fixup,
.prepare = imx_ldb_encoder_prepare,
.commit = imx_ldb_encoder_commit,
.mode_set = imx_ldb_encoder_mode_set,
diff --git a/drivers/gpu/drm/imx/imx-tve.c b/drivers/gpu/drm/imx/imx-tve.c
index 292349f..ae7a9fb 100644
--- a/drivers/gpu/drm/imx/imx-tve.c
+++ b/drivers/gpu/drm/imx/imx-tve.c
@@ -286,13 +286,6 @@ static void imx_tve_encoder_dpms(struct drm_encoder *encoder, int mode)
dev_err(tve->dev, "failed to disable TVOUT: %d\n", ret);
}
-static bool imx_tve_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_tve_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_tve *tve = enc_to_tve(encoder);
@@ -379,7 +372,6 @@ static const struct drm_encoder_funcs imx_tve_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_tve_encoder_helper_funcs = {
.dpms = imx_tve_encoder_dpms,
- .mode_fixup = imx_tve_encoder_mode_fixup,
.prepare = imx_tve_encoder_prepare,
.mode_set = imx_tve_encoder_mode_set,
.commit = imx_tve_encoder_commit,
diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c
index 0ffef17..363e2c7 100644
--- a/drivers/gpu/drm/imx/parallel-display.c
+++ b/drivers/gpu/drm/imx/parallel-display.c
@@ -112,13 +112,6 @@ static void imx_pd_encoder_dpms(struct drm_encoder *encoder, int mode)
drm_panel_enable(imxpd->panel);
}
-static bool imx_pd_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void imx_pd_encoder_prepare(struct drm_encoder *encoder)
{
struct imx_parallel_display *imxpd = enc_to_imxpd(encoder);
@@ -166,7 +159,6 @@ static const struct drm_encoder_funcs imx_pd_encoder_funcs = {
static const struct drm_encoder_helper_funcs imx_pd_encoder_helper_funcs = {
.dpms = imx_pd_encoder_dpms,
- .mode_fixup = imx_pd_encoder_mode_fixup,
.prepare = imx_pd_encoder_prepare,
.commit = imx_pd_encoder_commit,
.mode_set = imx_pd_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 11/17] drm/msm/mdp: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
5 files changed, 41 deletions(-)
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
index 2f57e94..106f0e7 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
@@ -47,13 +47,6 @@ static const struct drm_encoder_funcs mdp4_dsi_encoder_funcs = {
.destroy = mdp4_dsi_encoder_destroy,
};
-static bool mdp4_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_dsi_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -163,7 +156,6 @@ static void mdp4_dsi_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_dsi_encoder_helper_funcs = {
- .mode_fixup = mdp4_dsi_encoder_mode_fixup,
.mode_set = mdp4_dsi_encoder_mode_set,
.disable = mdp4_dsi_encoder_disable,
.enable = mdp4_dsi_encoder_enable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
index a21df54..35ad78a 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
@@ -94,13 +94,6 @@ static const struct drm_encoder_funcs mdp4_dtv_encoder_funcs = {
.destroy = mdp4_dtv_encoder_destroy,
};
-static bool mdp4_dtv_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_dtv_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -234,7 +227,6 @@ static void mdp4_dtv_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_dtv_encoder_helper_funcs = {
- .mode_fixup = mdp4_dtv_encoder_mode_fixup,
.mode_set = mdp4_dtv_encoder_mode_set,
.enable = mdp4_dtv_encoder_enable,
.disable = mdp4_dtv_encoder_disable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
index cd63fed..bc3d8e7 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
@@ -260,13 +260,6 @@ static void setup_phy(struct drm_encoder *encoder)
mdp4_write(mdp4_kms, REG_MDP4_LVDS_PHY_CFG0, lvds_phy_cfg0);
}
-static bool mdp4_lcdc_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_lcdc_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -430,7 +423,6 @@ static void mdp4_lcdc_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_lcdc_encoder_helper_funcs = {
- .mode_fixup = mdp4_lcdc_encoder_mode_fixup,
.mode_set = mdp4_lcdc_encoder_mode_set,
.disable = mdp4_lcdc_encoder_disable,
.enable = mdp4_lcdc_encoder_enable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
index 1aa21db..69094cb 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
@@ -188,13 +188,6 @@ static const struct drm_encoder_funcs mdp5_cmd_encoder_funcs = {
.destroy = mdp5_cmd_encoder_destroy,
};
-static bool mdp5_cmd_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp5_cmd_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -256,7 +249,6 @@ static void mdp5_cmd_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp5_cmd_encoder_helper_funcs = {
- .mode_fixup = mdp5_cmd_encoder_mode_fixup,
.mode_set = mdp5_cmd_encoder_mode_set,
.disable = mdp5_cmd_encoder_disable,
.enable = mdp5_cmd_encoder_enable,
@@ -340,4 +332,3 @@ fail:
return ERR_PTR(ret);
}
-
diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
index 0d737ca..1d95f9f 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
@@ -112,13 +112,6 @@ static const struct drm_encoder_funcs mdp5_encoder_funcs = {
.destroy = mdp5_encoder_destroy,
};
-static bool mdp5_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp5_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -287,7 +280,6 @@ static void mdp5_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp5_encoder_helper_funcs = {
- .mode_fixup = mdp5_encoder_mode_fixup,
.mode_set = mdp5_encoder_mode_set,
.disable = mdp5_encoder_disable,
.enable = mdp5_encoder_enable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 11/17] drm/msm/mdp: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:00 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
5 files changed, 41 deletions(-)
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
index 2f57e94..106f0e7 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c
@@ -47,13 +47,6 @@ static const struct drm_encoder_funcs mdp4_dsi_encoder_funcs = {
.destroy = mdp4_dsi_encoder_destroy,
};
-static bool mdp4_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_dsi_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -163,7 +156,6 @@ static void mdp4_dsi_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_dsi_encoder_helper_funcs = {
- .mode_fixup = mdp4_dsi_encoder_mode_fixup,
.mode_set = mdp4_dsi_encoder_mode_set,
.disable = mdp4_dsi_encoder_disable,
.enable = mdp4_dsi_encoder_enable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
index a21df54..35ad78a 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c
@@ -94,13 +94,6 @@ static const struct drm_encoder_funcs mdp4_dtv_encoder_funcs = {
.destroy = mdp4_dtv_encoder_destroy,
};
-static bool mdp4_dtv_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_dtv_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -234,7 +227,6 @@ static void mdp4_dtv_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_dtv_encoder_helper_funcs = {
- .mode_fixup = mdp4_dtv_encoder_mode_fixup,
.mode_set = mdp4_dtv_encoder_mode_set,
.enable = mdp4_dtv_encoder_enable,
.disable = mdp4_dtv_encoder_disable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
index cd63fed..bc3d8e7 100644
--- a/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c
@@ -260,13 +260,6 @@ static void setup_phy(struct drm_encoder *encoder)
mdp4_write(mdp4_kms, REG_MDP4_LVDS_PHY_CFG0, lvds_phy_cfg0);
}
-static bool mdp4_lcdc_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp4_lcdc_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -430,7 +423,6 @@ static void mdp4_lcdc_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp4_lcdc_encoder_helper_funcs = {
- .mode_fixup = mdp4_lcdc_encoder_mode_fixup,
.mode_set = mdp4_lcdc_encoder_mode_set,
.disable = mdp4_lcdc_encoder_disable,
.enable = mdp4_lcdc_encoder_enable,
diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
index 1aa21db..69094cb 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c
@@ -188,13 +188,6 @@ static const struct drm_encoder_funcs mdp5_cmd_encoder_funcs = {
.destroy = mdp5_cmd_encoder_destroy,
};
-static bool mdp5_cmd_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp5_cmd_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -256,7 +249,6 @@ static void mdp5_cmd_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp5_cmd_encoder_helper_funcs = {
- .mode_fixup = mdp5_cmd_encoder_mode_fixup,
.mode_set = mdp5_cmd_encoder_mode_set,
.disable = mdp5_cmd_encoder_disable,
.enable = mdp5_cmd_encoder_enable,
@@ -340,4 +332,3 @@ fail:
return ERR_PTR(ret);
}
-
diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
index 0d737ca..1d95f9f 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c
@@ -112,13 +112,6 @@ static const struct drm_encoder_funcs mdp5_encoder_funcs = {
.destroy = mdp5_encoder_destroy,
};
-static bool mdp5_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mdp5_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -287,7 +280,6 @@ static void mdp5_encoder_enable(struct drm_encoder *encoder)
}
static const struct drm_encoder_helper_funcs mdp5_encoder_helper_funcs = {
- .mode_fixup = mdp5_encoder_mode_fixup,
.mode_set = mdp5_encoder_mode_set,
.disable = mdp5_encoder_disable,
.enable = mdp5_encoder_enable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 12/17] drm/mgag200: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c
index dc13c48..af8b4c1 100644
--- a/drivers/gpu/drm/mgag200/mgag200_mode.c
+++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
@@ -1479,13 +1479,6 @@ void mga_crtc_fb_gamma_get(struct drm_crtc *crtc, u16 *red, u16 *green,
* These functions are analagous to those in the CRTC code, but are intended
* to handle any encoder-specific limitations
*/
-static bool mga_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mga_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1515,7 +1508,6 @@ static void mga_encoder_destroy(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs mga_encoder_helper_funcs = {
.dpms = mga_encoder_dpms,
- .mode_fixup = mga_encoder_mode_fixup,
.mode_set = mga_encoder_mode_set,
.prepare = mga_encoder_prepare,
.commit = mga_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 12/17] drm/mgag200: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (19 preceding siblings ...)
(?)
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c
index dc13c48..af8b4c1 100644
--- a/drivers/gpu/drm/mgag200/mgag200_mode.c
+++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
@@ -1479,13 +1479,6 @@ void mga_crtc_fb_gamma_get(struct drm_crtc *crtc, u16 *red, u16 *green,
* These functions are analagous to those in the CRTC code, but are intended
* to handle any encoder-specific limitations
*/
-static bool mga_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mga_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1515,7 +1508,6 @@ static void mga_encoder_destroy(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs mga_encoder_helper_funcs = {
.dpms = mga_encoder_dpms,
- .mode_fixup = mga_encoder_mode_fixup,
.mode_set = mga_encoder_mode_set,
.prepare = mga_encoder_prepare,
.commit = mga_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 12/17] drm/mgag200: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:00 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c b/drivers/gpu/drm/mgag200/mgag200_mode.c
index dc13c48..af8b4c1 100644
--- a/drivers/gpu/drm/mgag200/mgag200_mode.c
+++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
@@ -1479,13 +1479,6 @@ void mga_crtc_fb_gamma_get(struct drm_crtc *crtc, u16 *red, u16 *green,
* These functions are analagous to those in the CRTC code, but are intended
* to handle any encoder-specific limitations
*/
-static bool mga_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void mga_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -1515,7 +1508,6 @@ static void mga_encoder_destroy(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs mga_encoder_helper_funcs = {
.dpms = mga_encoder_dpms,
- .mode_fixup = mga_encoder_mode_fixup,
.mode_set = mga_encoder_mode_set,
.prepare = mga_encoder_prepare,
.commit = mga_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 13/17] drm/qxl: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c
index 8627651..43e5f50 100644
--- a/drivers/gpu/drm/qxl/qxl_display.c
+++ b/drivers/gpu/drm/qxl/qxl_display.c
@@ -734,14 +734,6 @@ static void qxl_enc_dpms(struct drm_encoder *encoder, int mode)
DRM_DEBUG("\n");
}
-static bool qxl_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- DRM_DEBUG("\n");
- return true;
-}
-
static void qxl_enc_prepare(struct drm_encoder *encoder)
{
DRM_DEBUG("\n");
@@ -864,7 +856,6 @@ static struct drm_encoder *qxl_best_encoder(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs qxl_enc_helper_funcs = {
.dpms = qxl_enc_dpms,
- .mode_fixup = qxl_enc_mode_fixup,
.prepare = qxl_enc_prepare,
.mode_set = qxl_enc_mode_set,
.commit = qxl_enc_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 13/17] drm/qxl: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (20 preceding siblings ...)
(?)
@ 2016-02-15 13:00 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c
index 8627651..43e5f50 100644
--- a/drivers/gpu/drm/qxl/qxl_display.c
+++ b/drivers/gpu/drm/qxl/qxl_display.c
@@ -734,14 +734,6 @@ static void qxl_enc_dpms(struct drm_encoder *encoder, int mode)
DRM_DEBUG("\n");
}
-static bool qxl_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- DRM_DEBUG("\n");
- return true;
-}
-
static void qxl_enc_prepare(struct drm_encoder *encoder)
{
DRM_DEBUG("\n");
@@ -864,7 +856,6 @@ static struct drm_encoder *qxl_best_encoder(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs qxl_enc_helper_funcs = {
.dpms = qxl_enc_dpms,
- .mode_fixup = qxl_enc_mode_fixup,
.prepare = qxl_enc_prepare,
.mode_set = qxl_enc_mode_set,
.commit = qxl_enc_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 13/17] drm/qxl: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:00 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:00 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c
index 8627651..43e5f50 100644
--- a/drivers/gpu/drm/qxl/qxl_display.c
+++ b/drivers/gpu/drm/qxl/qxl_display.c
@@ -734,14 +734,6 @@ static void qxl_enc_dpms(struct drm_encoder *encoder, int mode)
DRM_DEBUG("\n");
}
-static bool qxl_enc_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- DRM_DEBUG("\n");
- return true;
-}
-
static void qxl_enc_prepare(struct drm_encoder *encoder)
{
DRM_DEBUG("\n");
@@ -864,7 +856,6 @@ static struct drm_encoder *qxl_best_encoder(struct drm_connector *connector)
static const struct drm_encoder_helper_funcs qxl_enc_helper_funcs = {
.dpms = qxl_enc_dpms,
- .mode_fixup = qxl_enc_mode_fixup,
.prepare = qxl_enc_prepare,
.mode_set = qxl_enc_mode_set,
.commit = qxl_enc_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 14/17] drm/radeon: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
index 01b20e1..1603751 100644
--- a/drivers/gpu/drm/radeon/atombios_encoders.c
+++ b/drivers/gpu/drm/radeon/atombios_encoders.c
@@ -2623,16 +2623,8 @@ radeon_atom_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool radeon_atom_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs radeon_atom_ext_helper_funcs = {
.dpms = radeon_atom_ext_dpms,
- .mode_fixup = radeon_atom_ext_mode_fixup,
.prepare = radeon_atom_ext_prepare,
.mode_set = radeon_atom_ext_mode_set,
.commit = radeon_atom_ext_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 14/17] drm/radeon: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (23 preceding siblings ...)
(?)
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
index 01b20e1..1603751 100644
--- a/drivers/gpu/drm/radeon/atombios_encoders.c
+++ b/drivers/gpu/drm/radeon/atombios_encoders.c
@@ -2623,16 +2623,8 @@ radeon_atom_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool radeon_atom_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs radeon_atom_ext_helper_funcs = {
.dpms = radeon_atom_ext_dpms,
- .mode_fixup = radeon_atom_ext_mode_fixup,
.prepare = radeon_atom_ext_prepare,
.mode_set = radeon_atom_ext_mode_set,
.commit = radeon_atom_ext_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 14/17] drm/radeon: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:01 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
index 01b20e1..1603751 100644
--- a/drivers/gpu/drm/radeon/atombios_encoders.c
+++ b/drivers/gpu/drm/radeon/atombios_encoders.c
@@ -2623,16 +2623,8 @@ radeon_atom_ext_dpms(struct drm_encoder *encoder, int mode)
}
-static bool radeon_atom_ext_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static const struct drm_encoder_helper_funcs radeon_atom_ext_helper_funcs = {
.dpms = radeon_atom_ext_dpms,
- .mode_fixup = radeon_atom_ext_mode_fixup,
.prepare = radeon_atom_ext_prepare,
.mode_set = radeon_atom_ext_mode_set,
.commit = radeon_atom_ext_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
index f8f8f29..7766206 100644
--- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
+++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
@@ -875,13 +875,6 @@ static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
clk_disable_unprepare(dsi->pclk);
}
-static bool dw_mipi_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
{
struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
@@ -931,7 +924,6 @@ static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
static struct drm_encoder_helper_funcs
dw_mipi_dsi_encoder_helper_funcs = {
- .mode_fixup = dw_mipi_dsi_encoder_mode_fixup,
.commit = dw_mipi_dsi_encoder_commit,
.mode_set = dw_mipi_dsi_encoder_mode_set,
.disable = dw_mipi_dsi_encoder_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (25 preceding siblings ...)
(?)
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
index f8f8f29..7766206 100644
--- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
+++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
@@ -875,13 +875,6 @@ static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
clk_disable_unprepare(dsi->pclk);
}
-static bool dw_mipi_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
{
struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
@@ -931,7 +924,6 @@ static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
static struct drm_encoder_helper_funcs
dw_mipi_dsi_encoder_helper_funcs = {
- .mode_fixup = dw_mipi_dsi_encoder_mode_fixup,
.commit = dw_mipi_dsi_encoder_commit,
.mode_set = dw_mipi_dsi_encoder_mode_set,
.disable = dw_mipi_dsi_encoder_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:01 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
index f8f8f29..7766206 100644
--- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
+++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
@@ -875,13 +875,6 @@ static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
clk_disable_unprepare(dsi->pclk);
}
-static bool dw_mipi_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
{
struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
@@ -931,7 +924,6 @@ static void dw_mipi_dsi_encoder_commit(struct drm_encoder *encoder)
static struct drm_encoder_helper_funcs
dw_mipi_dsi_encoder_helper_funcs = {
- .mode_fixup = dw_mipi_dsi_encoder_mode_fixup,
.commit = dw_mipi_dsi_encoder_commit,
.mode_set = dw_mipi_dsi_encoder_mode_set,
.disable = dw_mipi_dsi_encoder_disable,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
index f2afcf5..24a3735 100644
--- a/drivers/gpu/drm/sti/sti_tvout.c
+++ b/drivers/gpu/drm/sti/sti_tvout.c
@@ -440,13 +440,6 @@ static void sti_tvout_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool sti_tvout_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void sti_tvout_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -486,7 +479,6 @@ static void sti_dvo_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_dvo_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_dvo_encoder_commit,
@@ -540,7 +532,6 @@ static void sti_hda_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hda_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hda_encoder_commit,
@@ -589,7 +580,6 @@ static void sti_hdmi_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hdmi_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hdmi_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (26 preceding siblings ...)
(?)
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
index f2afcf5..24a3735 100644
--- a/drivers/gpu/drm/sti/sti_tvout.c
+++ b/drivers/gpu/drm/sti/sti_tvout.c
@@ -440,13 +440,6 @@ static void sti_tvout_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool sti_tvout_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void sti_tvout_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -486,7 +479,6 @@ static void sti_dvo_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_dvo_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_dvo_encoder_commit,
@@ -540,7 +532,6 @@ static void sti_hda_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hda_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hda_encoder_commit,
@@ -589,7 +580,6 @@ static void sti_hdmi_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hdmi_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hdmi_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:01 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
index f2afcf5..24a3735 100644
--- a/drivers/gpu/drm/sti/sti_tvout.c
+++ b/drivers/gpu/drm/sti/sti_tvout.c
@@ -440,13 +440,6 @@ static void sti_tvout_encoder_dpms(struct drm_encoder *encoder, int mode)
{
}
-static bool sti_tvout_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- return true;
-}
-
static void sti_tvout_encoder_mode_set(struct drm_encoder *encoder,
struct drm_display_mode *mode,
struct drm_display_mode *adjusted_mode)
@@ -486,7 +479,6 @@ static void sti_dvo_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_dvo_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_dvo_encoder_commit,
@@ -540,7 +532,6 @@ static void sti_hda_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hda_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hda_encoder_commit,
@@ -589,7 +580,6 @@ static void sti_hdmi_encoder_disable(struct drm_encoder *encoder)
static const struct drm_encoder_helper_funcs sti_hdmi_encoder_helper_funcs = {
.dpms = sti_tvout_encoder_dpms,
- .mode_fixup = sti_tvout_encoder_mode_fixup,
.mode_set = sti_tvout_encoder_mode_set,
.prepare = sti_tvout_encoder_prepare,
.commit = sti_hdmi_encoder_commit,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 17/17] drm/tilcdc: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: CARLOS.PALMINHA, airlied, jingoohan1, kgene, k.kozlowski,
patrik.r.jakobsson, p.zabel, mark.yao, heiko, benjamin.gaignard,
vincent.abriou, kraxel, linux-arm-kernel, linux-samsung-soc,
linux-rockchip, virtualization
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
2 files changed, 18 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
index 4dda6e2..8dcf02a 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
@@ -70,14 +70,6 @@ static void panel_encoder_dpms(struct drm_encoder *encoder, int mode)
mode == DRM_MODE_DPMS_ON ? 1 : 0);
}
-static bool panel_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void panel_encoder_prepare(struct drm_encoder *encoder)
{
struct panel_encoder *panel_encoder = to_panel_encoder(encoder);
@@ -103,7 +95,6 @@ static const struct drm_encoder_funcs panel_encoder_funcs = {
static const struct drm_encoder_helper_funcs panel_encoder_helper_funcs = {
.dpms = panel_encoder_dpms,
- .mode_fixup = panel_encoder_mode_fixup,
.prepare = panel_encoder_prepare,
.commit = panel_encoder_commit,
.mode_set = panel_encoder_mode_set,
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
index 5052a8a..1c23017 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
@@ -80,14 +80,6 @@ static void tfp410_encoder_dpms(struct drm_encoder *encoder, int mode)
tfp410_encoder->dpms = mode;
}
-static bool tfp410_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void tfp410_encoder_prepare(struct drm_encoder *encoder)
{
tfp410_encoder_dpms(encoder, DRM_MODE_DPMS_OFF);
@@ -112,7 +104,6 @@ static const struct drm_encoder_funcs tfp410_encoder_funcs = {
static const struct drm_encoder_helper_funcs tfp410_encoder_helper_funcs = {
.dpms = tfp410_encoder_dpms,
- .mode_fixup = tfp410_encoder_mode_fixup,
.prepare = tfp410_encoder_prepare,
.commit = tfp410_encoder_commit,
.mode_set = tfp410_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 17/17] drm/tilcdc: removed optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (28 preceding siblings ...)
(?)
@ 2016-02-15 13:01 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, CARLOS.PALMINHA, patrik.r.jakobsson,
virtualization, linux-rockchip, kgene, p.zabel, vincent.abriou,
linux-arm-kernel, mark.yao
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
2 files changed, 18 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
index 4dda6e2..8dcf02a 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
@@ -70,14 +70,6 @@ static void panel_encoder_dpms(struct drm_encoder *encoder, int mode)
mode == DRM_MODE_DPMS_ON ? 1 : 0);
}
-static bool panel_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void panel_encoder_prepare(struct drm_encoder *encoder)
{
struct panel_encoder *panel_encoder = to_panel_encoder(encoder);
@@ -103,7 +95,6 @@ static const struct drm_encoder_funcs panel_encoder_funcs = {
static const struct drm_encoder_helper_funcs panel_encoder_helper_funcs = {
.dpms = panel_encoder_dpms,
- .mode_fixup = panel_encoder_mode_fixup,
.prepare = panel_encoder_prepare,
.commit = panel_encoder_commit,
.mode_set = panel_encoder_mode_set,
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
index 5052a8a..1c23017 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
@@ -80,14 +80,6 @@ static void tfp410_encoder_dpms(struct drm_encoder *encoder, int mode)
tfp410_encoder->dpms = mode;
}
-static bool tfp410_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void tfp410_encoder_prepare(struct drm_encoder *encoder)
{
tfp410_encoder_dpms(encoder, DRM_MODE_DPMS_OFF);
@@ -112,7 +104,6 @@ static const struct drm_encoder_funcs tfp410_encoder_funcs = {
static const struct drm_encoder_helper_funcs tfp410_encoder_helper_funcs = {
.dpms = tfp410_encoder_dpms,
- .mode_fixup = tfp410_encoder_mode_fixup,
.prepare = tfp410_encoder_prepare,
.commit = tfp410_encoder_commit,
.mode_set = tfp410_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* [PATCH v2 17/17] drm/tilcdc: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:01 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 13:01 UTC (permalink / raw)
To: linux-arm-kernel
mode_fixup function for encoder drivers became optional with patch
http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
This patch set nukes all the dummy mode_fixup implementations.
(made on top of Daniel topic/drm-misc branch)
Signed-off-by: Carlos Palminha <palminha@synopsys.com>
---
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
2 files changed, 18 deletions(-)
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
index 4dda6e2..8dcf02a 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c
@@ -70,14 +70,6 @@ static void panel_encoder_dpms(struct drm_encoder *encoder, int mode)
mode == DRM_MODE_DPMS_ON ? 1 : 0);
}
-static bool panel_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void panel_encoder_prepare(struct drm_encoder *encoder)
{
struct panel_encoder *panel_encoder = to_panel_encoder(encoder);
@@ -103,7 +95,6 @@ static const struct drm_encoder_funcs panel_encoder_funcs = {
static const struct drm_encoder_helper_funcs panel_encoder_helper_funcs = {
.dpms = panel_encoder_dpms,
- .mode_fixup = panel_encoder_mode_fixup,
.prepare = panel_encoder_prepare,
.commit = panel_encoder_commit,
.mode_set = panel_encoder_mode_set,
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
index 5052a8a..1c23017 100644
--- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
+++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c
@@ -80,14 +80,6 @@ static void tfp410_encoder_dpms(struct drm_encoder *encoder, int mode)
tfp410_encoder->dpms = mode;
}
-static bool tfp410_encoder_mode_fixup(struct drm_encoder *encoder,
- const struct drm_display_mode *mode,
- struct drm_display_mode *adjusted_mode)
-{
- /* nothing needed */
- return true;
-}
-
static void tfp410_encoder_prepare(struct drm_encoder *encoder)
{
tfp410_encoder_dpms(encoder, DRM_MODE_DPMS_OFF);
@@ -112,7 +104,6 @@ static const struct drm_encoder_funcs tfp410_encoder_funcs = {
static const struct drm_encoder_helper_funcs tfp410_encoder_helper_funcs = {
.dpms = tfp410_encoder_dpms,
- .mode_fixup = tfp410_encoder_mode_fixup,
.prepare = tfp410_encoder_prepare,
.commit = tfp410_encoder_commit,
.mode_set = tfp410_encoder_mode_set,
--
2.5.0
^ permalink raw reply related [flat|nested] 74+ messages in thread
* Re: [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 12:58 ` Carlos Palminha
@ 2016-02-15 13:49 ` Patrik Jakobsson
-1 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 13:49 UTC (permalink / raw)
To: Carlos Palminha
Cc: dri-devel, David Airlie, jingoohan1, kgene, k.kozlowski, p.zabel,
mark.yao, heiko, benjamin.gaignard, vincent.abriou, kraxel,
linux-arm-kernel, linux-samsung-soc, linux-rockchip,
virtualization
Hi Carlos
Any particular reason why this patch isn't squashed with patch 8/17?
Thanks
Patrik
On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> index e977a81..736115c 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
> return 0;
> }
>
> -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> }
>
> static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
> - .mode_fixup = exynos_dsi_mode_fixup,
> .mode_set = exynos_dsi_mode_set,
> .enable = exynos_dsi_enable,
> .disable = exynos_dsi_disable,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 12:58 ` Carlos Palminha
(?)
@ 2016-02-15 13:49 ` Patrik Jakobsson
-1 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 13:49 UTC (permalink / raw)
To: Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
David Airlie, jingoohan1, dri-devel, virtualization,
linux-rockchip, kgene, p.zabel, vincent.abriou, linux-arm-kernel,
mark.yao
Hi Carlos
Any particular reason why this patch isn't squashed with patch 8/17?
Thanks
Patrik
On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> index e977a81..736115c 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
> return 0;
> }
>
> -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> }
>
> static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
> - .mode_fixup = exynos_dsi_mode_fixup,
> .mode_set = exynos_dsi_mode_set,
> .enable = exynos_dsi_enable,
> .disable = exynos_dsi_disable,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 13:49 ` Patrik Jakobsson
0 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 13:49 UTC (permalink / raw)
To: linux-arm-kernel
Hi Carlos
Any particular reason why this patch isn't squashed with patch 8/17?
Thanks
Patrik
On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> index e977a81..736115c 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
> return 0;
> }
>
> -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> }
>
> static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
> - .mode_fixup = exynos_dsi_mode_fixup,
> .mode_set = exynos_dsi_mode_set,
> .enable = exynos_dsi_enable,
> .disable = exynos_dsi_disable,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-15 14:19 ` Christian König
-1 siblings, 0 replies; 74+ messages in thread
From: Christian König @ 2016-02-15 14:19 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, jingoohan1, virtualization,
linux-rockchip, kgene, benjamin.gaignard, vincent.abriou,
linux-arm-kernel
Am 15.02.2016 um 13:57 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
You can add a Reviewed-by: Christian König <christian.koenig@amd.com> to
patch 4 and 14 as well.
Nice cleanup. Thanks,
Christian.
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (31 preceding siblings ...)
(?)
@ 2016-02-15 14:19 ` Christian König
-1 siblings, 0 replies; 74+ messages in thread
From: Christian König @ 2016-02-15 14:19 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, jingoohan1, virtualization,
linux-rockchip, kgene, benjamin.gaignard, vincent.abriou,
linux-arm-kernel
Am 15.02.2016 um 13:57 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
You can add a Reviewed-by: Christian König <christian.koenig@amd.com> to
patch 4 and 14 as well.
Nice cleanup. Thanks,
Christian.
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
@ 2016-02-15 14:19 ` Christian König
0 siblings, 0 replies; 74+ messages in thread
From: Christian König @ 2016-02-15 14:19 UTC (permalink / raw)
To: linux-arm-kernel
Am 15.02.2016 um 13:57 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
You can add a Reviewed-by: Christian K?nig <christian.koenig@amd.com> to
patch 4 and 14 as well.
Nice cleanup. Thanks,
Christian.
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
2016-02-15 13:00 ` Carlos Palminha
@ 2016-02-15 14:24 ` Patrik Jakobsson
-1 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 14:24 UTC (permalink / raw)
To: Carlos Palminha
Cc: dri-devel, David Airlie, jingoohan1, kgene, k.kozlowski, p.zabel,
mark.yao, heiko, benjamin.gaignard, vincent.abriou, kraxel,
linux-arm-kernel, linux-samsung-soc, linux-rockchip,
virtualization
On Mon, Feb 15, 2016 at 2:00 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
> ---
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> 5 files changed, 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> index d0717a8..b837e7a 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> @@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
>
> static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
> .dpms = cdv_intel_crt_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .commit = gma_encoder_commit,
> .mode_set = cdv_intel_crt_mode_set,
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> index ddf2d77..28f9d90 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> @@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
> .dpms = cdv_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = cdv_hdmi_mode_set,
> .commit = gma_encoder_commit,
> diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
> index ff17af4..9270821 100644
> --- a/drivers/gpu/drm/gma500/gma_display.c
> +++ b/drivers/gpu/drm/gma500/gma_display.c
> @@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
> return 0;
> }
>
> -bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
> const struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
> index ed569d8..78b9f98 100644
> --- a/drivers/gpu/drm/gma500/gma_display.h
> +++ b/drivers/gpu/drm/gma500/gma_display.h
> @@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
> extern void gma_encoder_prepare(struct drm_encoder *encoder);
> extern void gma_encoder_commit(struct drm_encoder *encoder);
> extern void gma_encoder_destroy(struct drm_encoder *encoder);
> -extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode);
>
> /* Common clock related functions */
> extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
> diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> index 2d18499..8b2eb32 100644
> --- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> +++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> @@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
> .dpms = oaktrail_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = oaktrail_hdmi_mode_set,
> .commit = gma_encoder_commit,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
2016-02-15 13:00 ` Carlos Palminha
(?)
@ 2016-02-15 14:24 ` Patrik Jakobsson
-1 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 14:24 UTC (permalink / raw)
To: Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
David Airlie, jingoohan1, dri-devel, virtualization,
linux-rockchip, kgene, p.zabel, vincent.abriou, linux-arm-kernel,
mark.yao
On Mon, Feb 15, 2016 at 2:00 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
> ---
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> 5 files changed, 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> index d0717a8..b837e7a 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> @@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
>
> static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
> .dpms = cdv_intel_crt_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .commit = gma_encoder_commit,
> .mode_set = cdv_intel_crt_mode_set,
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> index ddf2d77..28f9d90 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> @@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
> .dpms = cdv_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = cdv_hdmi_mode_set,
> .commit = gma_encoder_commit,
> diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
> index ff17af4..9270821 100644
> --- a/drivers/gpu/drm/gma500/gma_display.c
> +++ b/drivers/gpu/drm/gma500/gma_display.c
> @@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
> return 0;
> }
>
> -bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
> const struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
> index ed569d8..78b9f98 100644
> --- a/drivers/gpu/drm/gma500/gma_display.h
> +++ b/drivers/gpu/drm/gma500/gma_display.h
> @@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
> extern void gma_encoder_prepare(struct drm_encoder *encoder);
> extern void gma_encoder_commit(struct drm_encoder *encoder);
> extern void gma_encoder_destroy(struct drm_encoder *encoder);
> -extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode);
>
> /* Common clock related functions */
> extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
> diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> index 2d18499..8b2eb32 100644
> --- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> +++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> @@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
> .dpms = oaktrail_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = oaktrail_hdmi_mode_set,
> .commit = gma_encoder_commit,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 09/17] drm/gma500: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 14:24 ` Patrik Jakobsson
0 siblings, 0 replies; 74+ messages in thread
From: Patrik Jakobsson @ 2016-02-15 14:24 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Feb 15, 2016 at 2:00 PM, Carlos Palminha
<CARLOS.PALMINHA@synopsys.com> wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Reviewed-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
> ---
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> 5 files changed, 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_crt.c b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> index d0717a8..b837e7a 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_crt.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_crt.c
> @@ -217,7 +217,6 @@ static int cdv_intel_crt_set_property(struct drm_connector *connector,
>
> static const struct drm_encoder_helper_funcs cdv_intel_crt_helper_funcs = {
> .dpms = cdv_intel_crt_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .commit = gma_encoder_commit,
> .mode_set = cdv_intel_crt_mode_set,
> diff --git a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> index ddf2d77..28f9d90 100644
> --- a/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> +++ b/drivers/gpu/drm/gma500/cdv_intel_hdmi.c
> @@ -255,7 +255,6 @@ static void cdv_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs cdv_hdmi_helper_funcs = {
> .dpms = cdv_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = cdv_hdmi_mode_set,
> .commit = gma_encoder_commit,
> diff --git a/drivers/gpu/drm/gma500/gma_display.c b/drivers/gpu/drm/gma500/gma_display.c
> index ff17af4..9270821 100644
> --- a/drivers/gpu/drm/gma500/gma_display.c
> +++ b/drivers/gpu/drm/gma500/gma_display.c
> @@ -478,13 +478,6 @@ int gma_crtc_cursor_move(struct drm_crtc *crtc, int x, int y)
> return 0;
> }
>
> -bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode)
> -{
> - return true;
> -}
> -
> bool gma_crtc_mode_fixup(struct drm_crtc *crtc,
> const struct drm_display_mode *mode,
> struct drm_display_mode *adjusted_mode)
> diff --git a/drivers/gpu/drm/gma500/gma_display.h b/drivers/gpu/drm/gma500/gma_display.h
> index ed569d8..78b9f98 100644
> --- a/drivers/gpu/drm/gma500/gma_display.h
> +++ b/drivers/gpu/drm/gma500/gma_display.h
> @@ -90,9 +90,6 @@ extern void gma_crtc_restore(struct drm_crtc *crtc);
> extern void gma_encoder_prepare(struct drm_encoder *encoder);
> extern void gma_encoder_commit(struct drm_encoder *encoder);
> extern void gma_encoder_destroy(struct drm_encoder *encoder);
> -extern bool gma_encoder_mode_fixup(struct drm_encoder *encoder,
> - const struct drm_display_mode *mode,
> - struct drm_display_mode *adjusted_mode);
>
> /* Common clock related functions */
> extern const struct gma_limit_t *gma_limit(struct drm_crtc *crtc, int refclk);
> diff --git a/drivers/gpu/drm/gma500/oaktrail_hdmi.c b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> index 2d18499..8b2eb32 100644
> --- a/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> +++ b/drivers/gpu/drm/gma500/oaktrail_hdmi.c
> @@ -601,7 +601,6 @@ static void oaktrail_hdmi_destroy(struct drm_connector *connector)
>
> static const struct drm_encoder_helper_funcs oaktrail_hdmi_helper_funcs = {
> .dpms = oaktrail_hdmi_dpms,
> - .mode_fixup = gma_encoder_mode_fixup,
> .prepare = gma_encoder_prepare,
> .mode_set = oaktrail_hdmi_mode_set,
> .commit = gma_encoder_commit,
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 13:49 ` Patrik Jakobsson
@ 2016-02-15 15:18 ` Carlos Palminha
-1 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 15:18 UTC (permalink / raw)
To: Patrik Jakobsson, Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
David Airlie, jingoohan1, dri-devel, virtualization,
linux-rockchip, kgene, p.zabel, vincent.abriou, linux-arm-kernel,
mark.yao
Hi Patrik,
No particular reason... i was grepping along the development and probably didn't noticed that exynos was not completely nuked... :)
Regards,
C.Palminha
On 15-02-2016 13:49, Patrik Jakobsson wrote:
> Hi Carlos
>
> Any particular reason why this patch isn't squashed with patch 8/17?
>
> Thanks
> Patrik
>
>
> On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
> <CARLOS.PALMINHA@synopsys.com> wrote:
>> mode_fixup function for encoder drivers became optional with patch
>> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>>
>> This patch set nukes all the dummy mode_fixup implementations.
>>
>> (made on top of Daniel topic/drm-misc branch)
>>
>> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
>> ---
>> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
>> 1 file changed, 8 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> index e977a81..736115c 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
>> return 0;
>> }
>>
>> -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
>> - const struct drm_display_mode *mode,
>> - struct drm_display_mode *adjusted_mode)
>> -{
>> - return true;
>> -}
>> -
>> static void exynos_dsi_mode_set(struct drm_encoder *encoder,
>> struct drm_display_mode *mode,
>> struct drm_display_mode *adjusted_mode)
>> @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
>> }
>>
>> static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
>> - .mode_fixup = exynos_dsi_mode_fixup,
>> .mode_set = exynos_dsi_mode_set,
>> .enable = exynos_dsi_enable,
>> .disable = exynos_dsi_disable,
>> --
>> 2.5.0
>>
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 15:18 ` Carlos Palminha
0 siblings, 0 replies; 74+ messages in thread
From: Carlos Palminha @ 2016-02-15 15:18 UTC (permalink / raw)
To: linux-arm-kernel
Hi Patrik,
No particular reason... i was grepping along the development and probably didn't noticed that exynos was not completely nuked... :)
Regards,
C.Palminha
On 15-02-2016 13:49, Patrik Jakobsson wrote:
> Hi Carlos
>
> Any particular reason why this patch isn't squashed with patch 8/17?
>
> Thanks
> Patrik
>
>
> On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
> <CARLOS.PALMINHA@synopsys.com> wrote:
>> mode_fixup function for encoder drivers became optional with patch
>> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>>
>> This patch set nukes all the dummy mode_fixup implementations.
>>
>> (made on top of Daniel topic/drm-misc branch)
>>
>> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
>> ---
>> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
>> 1 file changed, 8 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> index e977a81..736115c 100644
>> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
>> @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
>> return 0;
>> }
>>
>> -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
>> - const struct drm_display_mode *mode,
>> - struct drm_display_mode *adjusted_mode)
>> -{
>> - return true;
>> -}
>> -
>> static void exynos_dsi_mode_set(struct drm_encoder *encoder,
>> struct drm_display_mode *mode,
>> struct drm_display_mode *adjusted_mode)
>> @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
>> }
>>
>> static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
>> - .mode_fixup = exynos_dsi_mode_fixup,
>> .mode_set = exynos_dsi_mode_set,
>> .enable = exynos_dsi_enable,
>> .disable = exynos_dsi_disable,
>> --
>> 2.5.0
>>
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
2016-02-15 13:00 ` Carlos Palminha
@ 2016-02-15 16:16 ` Philipp Zabel
-1 siblings, 0 replies; 74+ messages in thread
From: Philipp Zabel @ 2016-02-15 16:16 UTC (permalink / raw)
To: Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, jingoohan1, virtualization,
linux-rockchip, kgene, dri-devel, benjamin.gaignard,
vincent.abriou, linux-arm-kernel
Am Montag, den 15.02.2016, 13:00 +0000 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
regards
Philipp
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
2016-02-15 13:00 ` Carlos Palminha
(?)
(?)
@ 2016-02-15 16:16 ` Philipp Zabel
-1 siblings, 0 replies; 74+ messages in thread
From: Philipp Zabel @ 2016-02-15 16:16 UTC (permalink / raw)
To: Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, heiko, airlied, jingoohan1,
patrik.r.jakobsson, virtualization, linux-rockchip, kgene,
dri-devel, benjamin.gaignard, vincent.abriou, linux-arm-kernel,
mark.yao
Am Montag, den 15.02.2016, 13:00 +0000 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
regards
Philipp
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 10/17] drm/imx: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 16:16 ` Philipp Zabel
0 siblings, 0 replies; 74+ messages in thread
From: Philipp Zabel @ 2016-02-15 16:16 UTC (permalink / raw)
To: linux-arm-kernel
Am Montag, den 15.02.2016, 13:00 +0000 schrieb Carlos Palminha:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
regards
Philipp
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
2016-02-15 13:01 ` Carlos Palminha
@ 2016-02-15 16:44 ` Vincent ABRIOU
-1 siblings, 0 replies; 74+ messages in thread
From: Vincent ABRIOU @ 2016-02-15 16:44 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, benjamin.gaignard, jingoohan1,
virtualization, linux-rockchip, kgene, linux-arm-kernel
On 02/15/2016 02:01 PM, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
Acked-by: Vincent Abriou <vincent.abriou@st.com>
BR
Vincent
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
2016-02-15 13:01 ` Carlos Palminha
(?)
@ 2016-02-15 16:44 ` Vincent ABRIOU
-1 siblings, 0 replies; 74+ messages in thread
From: Vincent ABRIOU @ 2016-02-15 16:44 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, patrik.r.jakobsson, virtualization,
linux-rockchip, kgene, p.zabel, linux-arm-kernel, mark.yao
On 02/15/2016 02:01 PM, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
Acked-by: Vincent Abriou <vincent.abriou@st.com>
BR
Vincent
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 16/17] drm/sti: removed optional dummy encoder mode_fixup function.
@ 2016-02-15 16:44 ` Vincent ABRIOU
0 siblings, 0 replies; 74+ messages in thread
From: Vincent ABRIOU @ 2016-02-15 16:44 UTC (permalink / raw)
To: linux-arm-kernel
On 02/15/2016 02:01 PM, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
Acked-by: Vincent Abriou <vincent.abriou@st.com>
BR
Vincent
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
2016-02-15 13:01 ` Carlos Palminha
@ 2016-02-16 0:39 ` Mark yao
-1 siblings, 0 replies; 74+ messages in thread
From: Mark yao @ 2016-02-16 0:39 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, benjamin.gaignard, jingoohan1,
virtualization, linux-rockchip, kgene, vincent.abriou,
linux-arm-kernel
On 2016年02月15日 21:01, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
>
Acked-by: Mark Yao <mark.yao@rock-chips.com>
Thanks
Mark
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
2016-02-15 13:01 ` Carlos Palminha
(?)
(?)
@ 2016-02-16 0:39 ` Mark yao
-1 siblings, 0 replies; 74+ messages in thread
From: Mark yao @ 2016-02-16 0:39 UTC (permalink / raw)
To: Carlos Palminha, dri-devel
Cc: k.kozlowski, linux-samsung-soc, heiko, benjamin.gaignard,
airlied, jingoohan1, patrik.r.jakobsson, virtualization,
linux-rockchip, kgene, p.zabel, vincent.abriou, linux-arm-kernel
On 2016年02月15日 21:01, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
>
Acked-by: Mark Yao <mark.yao@rock-chips.com>
Thanks
Mark
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 15/17] drm/rockchip: removed optional dummy encoder mode_fixup function.
@ 2016-02-16 0:39 ` Mark yao
0 siblings, 0 replies; 74+ messages in thread
From: Mark yao @ 2016-02-16 0:39 UTC (permalink / raw)
To: linux-arm-kernel
On 2016?02?15? 21:01, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> ---
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> 1 file changed, 8 deletions(-)
>
>
Acked-by: Mark Yao <mark.yao@rock-chips.com>
Thanks
Mark
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
2016-02-15 13:49 ` Patrik Jakobsson
@ 2016-02-16 14:31 ` Daniel Vetter
-1 siblings, 0 replies; 74+ messages in thread
From: Daniel Vetter @ 2016-02-16 14:31 UTC (permalink / raw)
To: Patrik Jakobsson
Cc: k.kozlowski, linux-samsung-soc, jingoohan1, Carlos Palminha,
dri-devel, virtualization, linux-rockchip, kgene,
benjamin.gaignard, vincent.abriou, linux-arm-kernel
On Mon, Feb 15, 2016 at 02:49:09PM +0100, Patrik Jakobsson wrote:
> Hi Carlos
>
> Any particular reason why this patch isn't squashed with patch 8/17?
I've squashed them while applying.
-Daniel
>
> Thanks
> Patrik
>
>
> On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
> <CARLOS.PALMINHA@synopsys.com> wrote:
> > mode_fixup function for encoder drivers became optional with patch
> > http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
> >
> > This patch set nukes all the dummy mode_fixup implementations.
> >
> > (made on top of Daniel topic/drm-misc branch)
> >
> > Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> > ---
> > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> > 1 file changed, 8 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > index e977a81..736115c 100644
> > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
> > return 0;
> > }
> >
> > -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
> > - const struct drm_display_mode *mode,
> > - struct drm_display_mode *adjusted_mode)
> > -{
> > - return true;
> > -}
> > -
> > static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> > struct drm_display_mode *mode,
> > struct drm_display_mode *adjusted_mode)
> > @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> > }
> >
> > static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
> > - .mode_fixup = exynos_dsi_mode_fixup,
> > .mode_set = exynos_dsi_mode_set,
> > .enable = exynos_dsi_enable,
> > .disable = exynos_dsi_disable,
> > --
> > 2.5.0
> >
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 03/17] drm/exynos: removed optional dummy encoder mode_fixup function.
@ 2016-02-16 14:31 ` Daniel Vetter
0 siblings, 0 replies; 74+ messages in thread
From: Daniel Vetter @ 2016-02-16 14:31 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Feb 15, 2016 at 02:49:09PM +0100, Patrik Jakobsson wrote:
> Hi Carlos
>
> Any particular reason why this patch isn't squashed with patch 8/17?
I've squashed them while applying.
-Daniel
>
> Thanks
> Patrik
>
>
> On Mon, Feb 15, 2016 at 1:58 PM, Carlos Palminha
> <CARLOS.PALMINHA@synopsys.com> wrote:
> > mode_fixup function for encoder drivers became optional with patch
> > http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
> >
> > This patch set nukes all the dummy mode_fixup implementations.
> >
> > (made on top of Daniel topic/drm-misc branch)
> >
> > Signed-off-by: Carlos Palminha <palminha@synopsys.com>
> > ---
> > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> > 1 file changed, 8 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > index e977a81..736115c 100644
> > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > @@ -1597,13 +1597,6 @@ static int exynos_dsi_create_connector(struct drm_encoder *encoder)
> > return 0;
> > }
> >
> > -static bool exynos_dsi_mode_fixup(struct drm_encoder *encoder,
> > - const struct drm_display_mode *mode,
> > - struct drm_display_mode *adjusted_mode)
> > -{
> > - return true;
> > -}
> > -
> > static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> > struct drm_display_mode *mode,
> > struct drm_display_mode *adjusted_mode)
> > @@ -1623,7 +1616,6 @@ static void exynos_dsi_mode_set(struct drm_encoder *encoder,
> > }
> >
> > static const struct drm_encoder_helper_funcs exynos_dsi_encoder_helper_funcs = {
> > - .mode_fixup = exynos_dsi_mode_fixup,
> > .mode_set = exynos_dsi_mode_set,
> > .enable = exynos_dsi_enable,
> > .disable = exynos_dsi_disable,
> > --
> > 2.5.0
> >
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
@ 2016-02-16 14:38 ` Daniel Vetter
-1 siblings, 0 replies; 74+ messages in thread
From: Daniel Vetter @ 2016-02-16 14:38 UTC (permalink / raw)
To: Carlos Palminha
Cc: dri-devel, k.kozlowski, linux-samsung-soc, benjamin.gaignard,
jingoohan1, virtualization, linux-rockchip, kgene,
vincent.abriou, linux-arm-kernel
On Mon, Feb 15, 2016 at 12:57:04PM +0000, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
I've thrown them all into a topic branch for soaking, and plan to merge
into drm-misc in the next few days.
Thanks, Daniel
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
> --
> 2.5.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 74+ messages in thread
* Re: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
2016-02-15 12:57 ` Carlos Palminha
` (33 preceding siblings ...)
(?)
@ 2016-02-16 14:38 ` Daniel Vetter
-1 siblings, 0 replies; 74+ messages in thread
From: Daniel Vetter @ 2016-02-16 14:38 UTC (permalink / raw)
To: Carlos Palminha
Cc: k.kozlowski, linux-samsung-soc, jingoohan1, dri-devel,
virtualization, linux-rockchip, kgene, benjamin.gaignard,
vincent.abriou, linux-arm-kernel
On Mon, Feb 15, 2016 at 12:57:04PM +0000, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha@synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
I've thrown them all into a topic branch for soaking, and plan to merge
into drm-misc in the next few days.
Thanks, Daniel
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
> --
> 2.5.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 74+ messages in thread
* [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function.
@ 2016-02-16 14:38 ` Daniel Vetter
0 siblings, 0 replies; 74+ messages in thread
From: Daniel Vetter @ 2016-02-16 14:38 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Feb 15, 2016 at 12:57:04PM +0000, Carlos Palminha wrote:
> mode_fixup function for encoder drivers became optional with patch
> http://patchwork.freedesktop.org/patch/msgid/1455106522-32307-1-git-send-email-palminha at synopsys.com
>
> This patch set nukes all the dummy mode_fixup implementations.
>
> (made on top of Daniel topic/drm-misc branch)
>
> Changes v1->v2: incorporated Daniel comments
> * added signed-off-by line to all patches
> * threading enabled due to e-mail server constraints
> * common blurb lines to all patches
> * add reviewed by Alex, to patches 4 and 14
I've thrown them all into a topic branch for soaking, and plan to merge
into drm-misc in the next few days.
Thanks, Daniel
>
> Carlos Palminha (17):
> drm/virtio: removed optional dummy encoder mode_fixup function.
> drm/udl: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/amdgpu: removed optional dummy encoder mode_fixup function.
> drm/ast: removed optional dummy encoder mode_fixup function.
> drm/bochs: removed optional dummy encoder mode_fixup function.
> drm/cirrus: removed optional dummy encoder mode_fixup function.
> drm/exynos: removed optional dummy encoder mode_fixup function.
> drm/gma500: removed optional dummy encoder mode_fixup function.
> drm/imx: removed optional dummy encoder mode_fixup function.
> drm/msm/mdp: removed optional dummy encoder mode_fixup function.
> drm/mgag200: removed optional dummy encoder mode_fixup function.
> drm/qxl: removed optional dummy encoder mode_fixup function.
> drm/radeon: removed optional dummy encoder mode_fixup function.
> drm/rockchip: removed optional dummy encoder mode_fixup function.
> drm/sti: removed optional dummy encoder mode_fixup function.
> drm/tilcdc: removed optional dummy encoder mode_fixup function.
>
> drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 --------
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 --------
> drivers/gpu/drm/ast/ast_mode.c | 8 --------
> drivers/gpu/drm/bochs/bochs_kms.c | 8 --------
> drivers/gpu/drm/cirrus/cirrus_mode.c | 9 ---------
> drivers/gpu/drm/exynos/exynos_dp_core.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dpi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 8 --------
> drivers/gpu/drm/exynos/exynos_drm_vidi.c | 8 --------
> drivers/gpu/drm/gma500/cdv_intel_crt.c | 1 -
> drivers/gpu/drm/gma500/cdv_intel_hdmi.c | 1 -
> drivers/gpu/drm/gma500/gma_display.c | 7 -------
> drivers/gpu/drm/gma500/gma_display.h | 3 ---
> drivers/gpu/drm/gma500/oaktrail_hdmi.c | 1 -
> drivers/gpu/drm/imx/dw_hdmi-imx.c | 8 --------
> drivers/gpu/drm/imx/imx-ldb.c | 8 --------
> drivers/gpu/drm/imx/imx-tve.c | 8 --------
> drivers/gpu/drm/imx/parallel-display.c | 8 --------
> drivers/gpu/drm/mgag200/mgag200_mode.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dsi_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_dtv_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp4/mdp4_lcdc_encoder.c | 8 --------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_cmd_encoder.c | 9 ---------
> drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 8 --------
> drivers/gpu/drm/qxl/qxl_display.c | 9 ---------
> drivers/gpu/drm/radeon/atombios_encoders.c | 8 --------
> drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 8 --------
> drivers/gpu/drm/sti/sti_tvout.c | 10 ----------
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 9 ---------
> drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 9 ---------
> drivers/gpu/drm/udl/udl_encoder.c | 8 --------
> drivers/gpu/drm/virtio/virtgpu_display.c | 8 --------
> 33 files changed, 244 deletions(-)
>
> --
> 2.5.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 74+ messages in thread
end of thread, other threads:[~2016-02-16 14:38 UTC | newest]
Thread overview: 74+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-15 12:57 [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function Carlos Palminha
2016-02-15 12:57 ` Carlos Palminha
2016-02-15 12:57 ` [PATCH v2 01/17] drm/virtio: removed " Carlos Palminha
2016-02-15 12:57 ` Carlos Palminha
2016-02-15 12:57 ` Carlos Palminha
2016-02-15 12:57 ` [PATCH v2 02/17] drm/udl: " Carlos Palminha
2016-02-15 12:57 ` Carlos Palminha
2016-02-15 12:57 ` Carlos Palminha
2016-02-15 12:58 ` [PATCH v2 03/17] drm/exynos: " Carlos Palminha
2016-02-15 12:58 ` Carlos Palminha
2016-02-15 13:49 ` Patrik Jakobsson
2016-02-15 13:49 ` Patrik Jakobsson
2016-02-15 13:49 ` Patrik Jakobsson
2016-02-15 15:18 ` Carlos Palminha
2016-02-15 15:18 ` Carlos Palminha
2016-02-16 14:31 ` Daniel Vetter
2016-02-16 14:31 ` Daniel Vetter
2016-02-15 12:58 ` [PATCH v2 04/17] drm/amdgpu: " Carlos Palminha
2016-02-15 12:58 ` Carlos Palminha
2016-02-15 12:59 ` [PATCH v2 05/17] drm/ast: " Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` [PATCH v2 06/17] drm/bochs: " Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` [PATCH v2 07/17] drm/cirrus: " Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` [PATCH v2 08/17] drm/exynos: " Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 12:59 ` Carlos Palminha
2016-02-15 13:00 ` [PATCH v2 09/17] drm/gma500: " Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 14:24 ` Patrik Jakobsson
2016-02-15 14:24 ` Patrik Jakobsson
2016-02-15 14:24 ` Patrik Jakobsson
2016-02-15 13:00 ` [PATCH v2 10/17] drm/imx: " Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 16:16 ` Philipp Zabel
2016-02-15 16:16 ` Philipp Zabel
2016-02-15 16:16 ` Philipp Zabel
2016-02-15 13:00 ` [PATCH v2 11/17] drm/msm/mdp: " Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` [PATCH v2 12/17] drm/mgag200: " Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` [PATCH v2 13/17] drm/qxl: " Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:00 ` Carlos Palminha
2016-02-15 13:01 ` [PATCH v2 14/17] drm/radeon: " Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 13:01 ` [PATCH v2 15/17] drm/rockchip: " Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-16 0:39 ` Mark yao
2016-02-16 0:39 ` Mark yao
2016-02-16 0:39 ` Mark yao
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 13:01 ` [PATCH v2 16/17] drm/sti: " Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 16:44 ` Vincent ABRIOU
2016-02-15 16:44 ` Vincent ABRIOU
2016-02-15 16:44 ` Vincent ABRIOU
2016-02-15 13:01 ` [PATCH v2 17/17] drm/tilcdc: " Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 13:01 ` Carlos Palminha
2016-02-15 14:19 ` [PATCH v2 00/17] drm encoders cleanup: nuke " Christian König
2016-02-15 14:19 ` Christian König
2016-02-15 14:19 ` Christian König
2016-02-16 14:38 ` Daniel Vetter
2016-02-16 14:38 ` Daniel Vetter
2016-02-16 14:38 ` Daniel Vetter
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.