From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Christian_K=c3=b6nig?= Subject: Re: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function. Date: Mon, 15 Feb 2016 15:19:07 +0100 Message-ID: <56C1DE5B.4010909@vodafone.de> References: Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Carlos Palminha , dri-devel@lists.freedesktop.org Cc: k.kozlowski@samsung.com, linux-samsung-soc@vger.kernel.org, jingoohan1@gmail.com, virtualization@lists.linux-foundation.org, linux-rockchip@lists.infradead.org, kgene@kernel.org, benjamin.gaignard@linaro.org, vincent.abriou@st.com, linux-arm-kernel@lists.infradead.org List-Id: linux-rockchip.vger.kernel.org QW0gMTUuMDIuMjAxNiB1bSAxMzo1NyBzY2hyaWViIENhcmxvcyBQYWxtaW5oYToKPiBtb2RlX2Zp eHVwIGZ1bmN0aW9uIGZvciBlbmNvZGVyIGRyaXZlcnMgYmVjYW1lIG9wdGlvbmFsIHdpdGggcGF0 Y2gKPiBodHRwOi8vcGF0Y2h3b3JrLmZyZWVkZXNrdG9wLm9yZy9wYXRjaC9tc2dpZC8xNDU1MTA2 NTIyLTMyMzA3LTEtZ2l0LXNlbmQtZW1haWwtcGFsbWluaGFAc3lub3BzeXMuY29tCj4KPiBUaGlz IHBhdGNoIHNldCBudWtlcyBhbGwgdGhlIGR1bW15IG1vZGVfZml4dXAgaW1wbGVtZW50YXRpb25z Lgo+Cj4gKG1hZGUgb24gdG9wIG9mIERhbmllbCB0b3BpYy9kcm0tbWlzYyBicmFuY2gpCj4KPiBD aGFuZ2VzIHYxLT52MjogaW5jb3Jwb3JhdGVkIERhbmllbCBjb21tZW50cwo+ICogYWRkZWQgc2ln bmVkLW9mZi1ieSBsaW5lIHRvIGFsbCBwYXRjaGVzCj4gKiB0aHJlYWRpbmcgZW5hYmxlZCBkdWUg dG8gZS1tYWlsIHNlcnZlciBjb25zdHJhaW50cwo+ICogY29tbW9uIGJsdXJiIGxpbmVzIHRvIGFs bCBwYXRjaGVzCj4gKiBhZGQgcmV2aWV3ZWQgYnkgQWxleCwgdG8gcGF0Y2hlcyA0IGFuZCAxNAoK WW91IGNhbiBhZGQgYSBSZXZpZXdlZC1ieTogQ2hyaXN0aWFuIEvDtm5pZyA8Y2hyaXN0aWFuLmtv ZW5pZ0BhbWQuY29tPiB0byAKcGF0Y2ggNCBhbmQgMTQgYXMgd2VsbC4KCk5pY2UgY2xlYW51cC4g VGhhbmtzLApDaHJpc3RpYW4uCgo+Cj4gQ2FybG9zIFBhbG1pbmhhICgxNyk6Cj4gICAgZHJtL3Zp cnRpbzogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24u Cj4gICAgZHJtL3VkbDogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAg ZnVuY3Rpb24uCj4gICAgZHJtL2V4eW5vczogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVy IG1vZGVfZml4dXAgZnVuY3Rpb24uCj4gICAgZHJtL2FtZGdwdTogcmVtb3ZlZCBvcHRpb25hbCBk dW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24uCj4gICAgZHJtL2FzdDogcmVtb3ZlZCBv cHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24uCj4gICAgZHJtL2JvY2hz OiByZW1vdmVkIG9wdGlvbmFsIGR1bW15IGVuY29kZXIgbW9kZV9maXh1cCBmdW5jdGlvbi4KPiAg ICBkcm0vY2lycnVzOiByZW1vdmVkIG9wdGlvbmFsIGR1bW15IGVuY29kZXIgbW9kZV9maXh1cCBm dW5jdGlvbi4KPiAgICBkcm0vZXh5bm9zOiByZW1vdmVkIG9wdGlvbmFsIGR1bW15IGVuY29kZXIg bW9kZV9maXh1cCBmdW5jdGlvbi4KPiAgICBkcm0vZ21hNTAwOiByZW1vdmVkIG9wdGlvbmFsIGR1 bW15IGVuY29kZXIgbW9kZV9maXh1cCBmdW5jdGlvbi4KPiAgICBkcm0vaW14OiByZW1vdmVkIG9w dGlvbmFsIGR1bW15IGVuY29kZXIgbW9kZV9maXh1cCBmdW5jdGlvbi4KPiAgICBkcm0vbXNtL21k cDogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24uCj4g ICAgZHJtL21nYWcyMDA6IHJlbW92ZWQgb3B0aW9uYWwgZHVtbXkgZW5jb2RlciBtb2RlX2ZpeHVw IGZ1bmN0aW9uLgo+ICAgIGRybS9xeGw6IHJlbW92ZWQgb3B0aW9uYWwgZHVtbXkgZW5jb2RlciBt b2RlX2ZpeHVwIGZ1bmN0aW9uLgo+ICAgIGRybS9yYWRlb246IHJlbW92ZWQgb3B0aW9uYWwgZHVt bXkgZW5jb2RlciBtb2RlX2ZpeHVwIGZ1bmN0aW9uLgo+ICAgIGRybS9yb2NrY2hpcDogcmVtb3Zl ZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24uCj4gICAgZHJtL3N0 aTogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAgZnVuY3Rpb24uCj4g ICAgZHJtL3RpbGNkYzogcmVtb3ZlZCBvcHRpb25hbCBkdW1teSBlbmNvZGVyIG1vZGVfZml4dXAg ZnVuY3Rpb24uCj4KPiAgIGRyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2RjZV92MTBfMC5jICAg ICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9kY2Vf djExXzAuYyAgICAgICAgICAgfCAgOCAtLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL2FtZC9h bWRncHUvZGNlX3Y4XzAuYyAgICAgICAgICAgIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1 L2RybS9hc3QvYXN0X21vZGUuYyAgICAgICAgICAgICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICBk cml2ZXJzL2dwdS9kcm0vYm9jaHMvYm9jaHNfa21zLmMgICAgICAgICAgICAgICAgfCAgOCAtLS0t LS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL2NpcnJ1cy9jaXJydXNfbW9kZS5jICAgICAgICAgICAg IHwgIDkgLS0tLS0tLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vZXh5bm9zL2V4eW5vc19kcF9jb3Jl LmMgICAgICAgICAgfCAgOCAtLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL2V4eW5vcy9leHlu b3NfZHJtX2RwaS5jICAgICAgICAgIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS9l eHlub3MvZXh5bm9zX2RybV9kc2kuYyAgICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICBkcml2ZXJz L2dwdS9kcm0vZXh5bm9zL2V4eW5vc19kcm1fdmlkaS5jICAgICAgICAgfCAgOCAtLS0tLS0tLQo+ ICAgZHJpdmVycy9ncHUvZHJtL2dtYTUwMC9jZHZfaW50ZWxfY3J0LmMgICAgICAgICAgIHwgIDEg LQo+ICAgZHJpdmVycy9ncHUvZHJtL2dtYTUwMC9jZHZfaW50ZWxfaGRtaS5jICAgICAgICAgIHwg IDEgLQo+ICAgZHJpdmVycy9ncHUvZHJtL2dtYTUwMC9nbWFfZGlzcGxheS5jICAgICAgICAgICAg IHwgIDcgLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL2dtYTUwMC9nbWFfZGlzcGxheS5oICAg ICAgICAgICAgIHwgIDMgLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vZ21hNTAwL29ha3RyYWlsX2hk bWkuYyAgICAgICAgICAgfCAgMSAtCj4gICBkcml2ZXJzL2dwdS9kcm0vaW14L2R3X2hkbWktaW14 LmMgICAgICAgICAgICAgICAgfCAgOCAtLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL2lteC9p bXgtbGRiLmMgICAgICAgICAgICAgICAgICAgIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1 L2RybS9pbXgvaW14LXR2ZS5jICAgICAgICAgICAgICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICBk cml2ZXJzL2dwdS9kcm0vaW14L3BhcmFsbGVsLWRpc3BsYXkuYyAgICAgICAgICAgfCAgOCAtLS0t LS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL21nYWcyMDAvbWdhZzIwMF9tb2RlLmMgICAgICAgICAg IHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS9tc20vbWRwL21kcDQvbWRwNF9kc2lf ZW5jb2Rlci5jICB8ICA4IC0tLS0tLS0tCj4gICBkcml2ZXJzL2dwdS9kcm0vbXNtL21kcC9tZHA0 L21kcDRfZHR2X2VuY29kZXIuYyAgfCAgOCAtLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL21z bS9tZHAvbWRwNC9tZHA0X2xjZGNfZW5jb2Rlci5jIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMv Z3B1L2RybS9tc20vbWRwL21kcDUvbWRwNV9jbWRfZW5jb2Rlci5jICB8ICA5IC0tLS0tLS0tLQo+ ICAgZHJpdmVycy9ncHUvZHJtL21zbS9tZHAvbWRwNS9tZHA1X2VuY29kZXIuYyAgICAgIHwgIDgg LS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS9xeGwvcXhsX2Rpc3BsYXkuYyAgICAgICAgICAg ICAgICB8ICA5IC0tLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL3JhZGVvbi9hdG9tYmlvc19l bmNvZGVycy5jICAgICAgIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS9yb2NrY2hp cC9kdy1taXBpLWRzaS5jICAgICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICBkcml2ZXJzL2dwdS9k cm0vc3RpL3N0aV90dm91dC5jICAgICAgICAgICAgICAgICAgfCAxMCAtLS0tLS0tLS0tCj4gICBk cml2ZXJzL2dwdS9kcm0vdGlsY2RjL3RpbGNkY19wYW5lbC5jICAgICAgICAgICAgfCAgOSAtLS0t LS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS90aWxjZGMvdGlsY2RjX3RmcDQxMC5jICAgICAgICAg ICB8ICA5IC0tLS0tLS0tLQo+ICAgZHJpdmVycy9ncHUvZHJtL3VkbC91ZGxfZW5jb2Rlci5jICAg ICAgICAgICAgICAgIHwgIDggLS0tLS0tLS0KPiAgIGRyaXZlcnMvZ3B1L2RybS92aXJ0aW8vdmly dGdwdV9kaXNwbGF5LmMgICAgICAgICB8ICA4IC0tLS0tLS0tCj4gICAzMyBmaWxlcyBjaGFuZ2Vk LCAyNDQgZGVsZXRpb25zKC0pCj4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVk ZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: deathsimple@vodafone.de (=?UTF-8?Q?Christian_K=c3=b6nig?=) Date: Mon, 15 Feb 2016 15:19:07 +0100 Subject: [PATCH v2 00/17] drm encoders cleanup: nuke optional dummy encoder mode_fixup function. In-Reply-To: References: Message-ID: <56C1DE5B.4010909@vodafone.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 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(-) >