From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sam Ravnborg Subject: Re: [PATCH 11/33] fbdev/sh_mobile: remove sh_mobile_lcdc_display_notify Date: Sat, 25 May 2019 17:01:59 +0200 Message-ID: <20190525150159.GA27341@ravnborg.org> References: <20190524085354.27411-1-daniel.vetter@ffwll.ch> <20190524085354.27411-12-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20190524085354.27411-12-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: DRI Development , Intel Graphics Development , LKML , Geert Uytterhoeven List-Id: dri-devel@lists.freedesktop.org SGkgRGFuaWVsCgo+IEl0J3MgZGVhZCBjb2RlLCBhbmQgcmVtb3ZpbmcgaXQgYXZvaWRzIG1lIGhh dmluZyB0byB1bmRlcnN0YW5kCj4gd2hhdCBpdCdzIGRvaW5nIHdpdGggbG9ja19mYl9pbmZvLgoK SSBwdXNoZWQgdGhlIHNlcmllcyB0aHJvdWdoIG15IGJ1aWxkIHRlc3RzIHdoaWNoIGluY2x1ZGUg dGhlIHNoCmFyY2hpdGVjdHVyZS4KCk9uZSBlcnJvciBhbmQgb25lIHdhcm5pbmcgd2FzIHRyaWdn ZXJlZCBmcm9tIHNoX21vYmlsZV9sY2RjZmIuYy4KVGhlIHJlc3Qgd2FzIGZpbmUuCgpUaGUgcGF0 Y2ggYmVsb3cgcmVtb3ZlZCB0aGUgc29sZSB1c2VyIG9mCnNoX21vYmlsZV9sY2RjX211c3RfcmVj b25maWd1cmUoKSBzbyB0aGlzIHRyaWdnZXJzIGEgd2FybmluZy4KCkFuZCBJIGFsc28gZ2V0IHRo ZSBmb2xsb3dpbmcgZXJyb3I6CmRyaXZlcnMvdmlkZW8vZmJkZXYvc2hfbW9iaWxlX2xjZGNmYi5j OiBJbiBmdW5jdGlvbiDigJhzaF9tb2JpbGVfZmJfcmVjb25maWfigJk6CmRyaXZlcnMvdmlkZW8v ZmJkZXYvc2hfbW9iaWxlX2xjZGNmYi5jOjE4MDA6MjogZXJyb3I6IGltcGxpY2l0IGRlY2xhcmF0 aW9uIG9mIGZ1bmN0aW9uIOKAmGZiY29uX3VwZGF0ZV92Y3PigJk7IGRpZCB5b3UgbWVhbiDigJhm aWxlX3VwZGF0ZV90aW1l4oCZPyBbLVdlcnJvcj1pbXBsaWNpdC1mdW5jdGlvbi1kZWNsYXJhdGlv bl0KICBmYmNvbl91cGRhdGVfdmNzKGluZm8sIHRydWUpOwogIF5+fn5+fn5+fn5+fn5+fn4KICBm aWxlX3VwZGF0ZV90aW1lCgpJIGRpZCBub3QgY2hlY2sgYnV0IGFzc3VtZSB0aGUgZXJyb3Igd2Fz IHRyaWdnZXJlZCBpbiBwYXRjaCAyOCB3aGVyZQpmYmNvbl91cGRhdGVfdmNzKCkgaW4gaW50cm9k dWNlZC4KCgpCb3RoIGFyZSB0cml2aWFsbHkgZml4ZWQgYnkgYXBwZW5kZWQgcGF0Y2guCgoJU2Ft CgpkaWZmIC0tZ2l0IGEvZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVfbGNkY2ZiLmMgYi9k cml2ZXJzL3ZpZGVvL2ZiZGV2L3NoX21vYmlsZV9sY2RjZmIuYwppbmRleCBiYjFhNjEwZDAzNjMu LmI4NDU0NDI0OTEwZCAxMDA2NDQKLS0tIGEvZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVf bGNkY2ZiLmMKKysrIGIvZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVfbGNkY2ZiLmMKQEAg LTE1LDYgKzE1LDcgQEAKICNpbmNsdWRlIDxsaW51eC9jdHlwZS5oPgogI2luY2x1ZGUgPGxpbnV4 L2RtYS1tYXBwaW5nLmg+CiAjaW5jbHVkZSA8bGludXgvZGVsYXkuaD4KKyNpbmNsdWRlIDxsaW51 eC9mYmNvbi5oPgogI2luY2x1ZGUgPGxpbnV4L2dwaW8uaD4KICNpbmNsdWRlIDxsaW51eC9pbml0 Lmg+CiAjaW5jbHVkZSA8bGludXgvaW50ZXJydXB0Lmg+CkBAIC01MzMsMjUgKzUzNCw2IEBAIHN0 YXRpYyB2b2lkIHNoX21vYmlsZV9sY2RjX2Rpc3BsYXlfb2ZmKHN0cnVjdCBzaF9tb2JpbGVfbGNk Y19jaGFuICpjaCkKIAkJY2gtPnR4X2Rldi0+b3BzLT5kaXNwbGF5X29mZihjaC0+dHhfZGV2KTsK IH0KIAotc3RhdGljIGJvb2wKLXNoX21vYmlsZV9sY2RjX211c3RfcmVjb25maWd1cmUoc3RydWN0 IHNoX21vYmlsZV9sY2RjX2NoYW4gKmNoLAotCQkJCWNvbnN0IHN0cnVjdCBmYl92aWRlb21vZGUg Km5ld19tb2RlKQotewotCWRldl9kYmcoY2gtPmluZm8tPmRldiwgIk9sZCAldXgldSwgbmV3ICV1 eCV1XG4iLAotCQljaC0+ZGlzcGxheS5tb2RlLnhyZXMsIGNoLT5kaXNwbGF5Lm1vZGUueXJlcywK LQkJbmV3X21vZGUtPnhyZXMsIG5ld19tb2RlLT55cmVzKTsKLQotCS8qIEl0IGNhbiBiZSBhIGRp ZmZlcmVudCBtb25pdG9yIHdpdGggYW4gZXF1YWwgdmlkZW8tbW9kZSAqLwotCWlmIChmYl9tb2Rl X2lzX2VxdWFsKCZjaC0+ZGlzcGxheS5tb2RlLCBuZXdfbW9kZSkpCi0JCXJldHVybiBmYWxzZTsK LQotCWRldl9kYmcoY2gtPmluZm8tPmRldiwgIlN3aXRjaGluZyAldSAtPiAldSBsaW5lc1xuIiwK LQkJY2gtPmRpc3BsYXkubW9kZS55cmVzLCBuZXdfbW9kZS0+eXJlcyk7Ci0JY2gtPmRpc3BsYXku bW9kZSA9ICpuZXdfbW9kZTsKLQotCXJldHVybiB0cnVlOwotfQotCiBzdGF0aWMgaW50IHNoX21v YmlsZV9sY2RjX2NoZWNrX3ZhcihzdHJ1Y3QgZmJfdmFyX3NjcmVlbmluZm8gKnZhciwKIAkJCQkg ICAgc3RydWN0IGZiX2luZm8gKmluZm8pOwogCgogCj4gU2lnbmVkLW9mZi1ieTogRGFuaWVsIFZl dHRlciA8ZGFuaWVsLnZldHRlckBmZndsbC5jaD4KPiBSZXZpZXdlZC1ieTogR2VlcnQgVXl0dGVy aG9ldmVuIDxnZWVydCtyZW5lc2FzQGdsaWRlci5iZT4KPiBDYzogR2VlcnQgVXl0dGVyaG9ldmVu IDxnZWVydCtyZW5lc2FzQGdsaWRlci5iZT4KPiBDYzogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZl dHRlckBmZndsbC5jaD4KPiAtLS0KPiAgZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVfbGNk Y2ZiLmMgfCA2MyAtLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+ICBkcml2ZXJzL3ZpZGVvL2Zi ZGV2L3NoX21vYmlsZV9sY2RjZmIuaCB8ICA1IC0tCj4gIDIgZmlsZXMgY2hhbmdlZCwgNjggZGVs ZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvdmlkZW8vZmJkZXYvc2hfbW9iaWxl X2xjZGNmYi5jIGIvZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVfbGNkY2ZiLmMKPiBpbmRl eCBkYzQ2YmUzOGM5NzAuLmM1OTI0ZjVlOThjNiAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL3ZpZGVv L2ZiZGV2L3NoX21vYmlsZV9sY2RjZmIuYwo+ICsrKyBiL2RyaXZlcnMvdmlkZW8vZmJkZXYvc2hf bW9iaWxlX2xjZGNmYi5jCj4gQEAgLTU1Niw2NyArNTU2LDYgQEAgc2hfbW9iaWxlX2xjZGNfbXVz dF9yZWNvbmZpZ3VyZShzdHJ1Y3Qgc2hfbW9iaWxlX2xjZGNfY2hhbiAqY2gsCj4gIHN0YXRpYyBp bnQgc2hfbW9iaWxlX2xjZGNfY2hlY2tfdmFyKHN0cnVjdCBmYl92YXJfc2NyZWVuaW5mbyAqdmFy LAo+ICAJCQkJICAgIHN0cnVjdCBmYl9pbmZvICppbmZvKTsKPiAgCj4gLXN0YXRpYyBpbnQgc2hf bW9iaWxlX2xjZGNfZGlzcGxheV9ub3RpZnkoc3RydWN0IHNoX21vYmlsZV9sY2RjX2NoYW4gKmNo LAo+IC0JCQkJCSBlbnVtIHNoX21vYmlsZV9sY2RjX2VudGl0eV9ldmVudCBldmVudCwKPiAtCQkJ CQkgY29uc3Qgc3RydWN0IGZiX3ZpZGVvbW9kZSAqbW9kZSwKPiAtCQkJCQkgY29uc3Qgc3RydWN0 IGZiX21vbnNwZWNzICptb25zcGVjKQo+IC17Cj4gLQlzdHJ1Y3QgZmJfaW5mbyAqaW5mbyA9IGNo LT5pbmZvOwo+IC0Jc3RydWN0IGZiX3Zhcl9zY3JlZW5pbmZvIHZhcjsKPiAtCWludCByZXQgPSAw Owo+IC0KPiAtCXN3aXRjaCAoZXZlbnQpIHsKPiAtCWNhc2UgU0hfTU9CSUxFX0xDRENfRVZFTlRf RElTUExBWV9DT05ORUNUOgo+IC0JCS8qIEhETUkgcGx1ZyBpbiAqLwo+IC0JCWNvbnNvbGVfbG9j aygpOwo+IC0JCWlmIChsb2NrX2ZiX2luZm8oaW5mbykpIHsKPiAtCj4gLQo+IC0JCQljaC0+ZGlz cGxheS53aWR0aCA9IG1vbnNwZWMtPm1heF94ICogMTA7Cj4gLQkJCWNoLT5kaXNwbGF5LmhlaWdo dCA9IG1vbnNwZWMtPm1heF95ICogMTA7Cj4gLQo+IC0JCQlpZiAoIXNoX21vYmlsZV9sY2RjX211 c3RfcmVjb25maWd1cmUoY2gsIG1vZGUpICYmCj4gLQkJCSAgICBpbmZvLT5zdGF0ZSA9PSBGQklO Rk9fU1RBVEVfUlVOTklORykgewo+IC0JCQkJLyogRmlyc3QgYWN0aXZhdGlvbiB3aXRoIHRoZSBk ZWZhdWx0IG1vbml0b3IuCj4gLQkJCQkgKiBKdXN0IHR1cm4gb24sIGlmIHdlIHJ1biBhIHJlc3Vt ZSBoZXJlLCB0aGUKPiAtCQkJCSAqIGxvZ28gZGlzYXBwZWFycy4KPiAtCQkJCSAqLwo+IC0JCQkJ aW5mby0+dmFyLndpZHRoID0gY2gtPmRpc3BsYXkud2lkdGg7Cj4gLQkJCQlpbmZvLT52YXIuaGVp Z2h0ID0gY2gtPmRpc3BsYXkuaGVpZ2h0Owo+IC0JCQkJc2hfbW9iaWxlX2xjZGNfZGlzcGxheV9v bihjaCk7Cj4gLQkJCX0gZWxzZSB7Cj4gLQkJCQkvKiBOZXcgbW9uaXRvciBvciBoYXZlIHRvIHdh a2UgdXAgKi8KPiAtCQkJCWZiX3NldF9zdXNwZW5kKGluZm8sIDApOwo+IC0JCQl9Cj4gLQo+IC0K PiAtCQkJdW5sb2NrX2ZiX2luZm8oaW5mbyk7Cj4gLQkJfQo+IC0JCWNvbnNvbGVfdW5sb2NrKCk7 Cj4gLQkJYnJlYWs7Cj4gLQo+IC0JY2FzZSBTSF9NT0JJTEVfTENEQ19FVkVOVF9ESVNQTEFZX0RJ U0NPTk5FQ1Q6Cj4gLQkJLyogSERNSSBkaXNjb25uZWN0ICovCj4gLQkJY29uc29sZV9sb2NrKCk7 Cj4gLQkJaWYgKGxvY2tfZmJfaW5mbyhpbmZvKSkgewo+IC0JCQlmYl9zZXRfc3VzcGVuZChpbmZv LCAxKTsKPiAtCQkJdW5sb2NrX2ZiX2luZm8oaW5mbyk7Cj4gLQkJfQo+IC0JCWNvbnNvbGVfdW5s b2NrKCk7Cj4gLQkJYnJlYWs7Cj4gLQo+IC0JY2FzZSBTSF9NT0JJTEVfTENEQ19FVkVOVF9ESVNQ TEFZX01PREU6Cj4gLQkJLyogVmFsaWRhdGUgYSBwcm9wb3NlZCBuZXcgbW9kZSAqLwo+IC0JCWZi X3ZpZGVvbW9kZV90b192YXIoJnZhciwgbW9kZSk7Cj4gLQkJdmFyLmJpdHNfcGVyX3BpeGVsID0g aW5mby0+dmFyLmJpdHNfcGVyX3BpeGVsOwo+IC0JCXZhci5ncmF5c2NhbGUgPSBpbmZvLT52YXIu Z3JheXNjYWxlOwo+IC0JCXJldCA9IHNoX21vYmlsZV9sY2RjX2NoZWNrX3ZhcigmdmFyLCBpbmZv KTsKPiAtCQlicmVhazsKPiAtCX0KPiAtCj4gLQlyZXR1cm4gcmV0Owo+IC19Cj4gLQo+ICAvKiAt LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLQo+ICAgKiBGb3JtYXQgaGVscGVycwo+ICAgKi8KPiBAQCAtMjU0 MCw4ICsyNDc5LDYgQEAgc2hfbW9iaWxlX2xjZGNfY2hhbm5lbF9pbml0KHN0cnVjdCBzaF9tb2Jp bGVfbGNkY19jaGFuICpjaCkKPiAgCXVuc2lnbmVkIGludCBtYXhfc2l6ZTsKPiAgCXVuc2lnbmVk IGludCBpOwo+ICAKPiAtCWNoLT5ub3RpZnkgPSBzaF9tb2JpbGVfbGNkY19kaXNwbGF5X25vdGlm eTsKPiAtCj4gIAkvKiBWYWxpZGF0ZSB0aGUgZm9ybWF0LiAqLwo+ICAJZm9ybWF0ID0gc2hfbW9i aWxlX2Zvcm1hdF9pbmZvKGNmZy0+Zm91cmNjKTsKPiAgCWlmIChmb3JtYXQgPT0gTlVMTCkgewo+ IGRpZmYgLS1naXQgYS9kcml2ZXJzL3ZpZGVvL2ZiZGV2L3NoX21vYmlsZV9sY2RjZmIuaCBiL2Ry aXZlcnMvdmlkZW8vZmJkZXYvc2hfbW9iaWxlX2xjZGNmYi5oCj4gaW5kZXggYjhlNDdhOGJkOGFi Li41ODk0MDAzNzIwOTggMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2Jp bGVfbGNkY2ZiLmgKPiArKysgYi9kcml2ZXJzL3ZpZGVvL2ZiZGV2L3NoX21vYmlsZV9sY2RjZmIu aAo+IEBAIC04NywxMSArODcsNiBAQCBzdHJ1Y3Qgc2hfbW9iaWxlX2xjZGNfY2hhbiB7Cj4gIAl1 bnNpZ25lZCBsb25nIGJhc2VfYWRkcl9jOwo+ICAJdW5zaWduZWQgaW50IGxpbmVfc2l6ZTsKPiAg Cj4gLQlpbnQgKCpub3RpZnkpKHN0cnVjdCBzaF9tb2JpbGVfbGNkY19jaGFuICpjaCwKPiAtCQkg ICAgICBlbnVtIHNoX21vYmlsZV9sY2RjX2VudGl0eV9ldmVudCBldmVudCwKPiAtCQkgICAgICBj b25zdCBzdHJ1Y3QgZmJfdmlkZW9tb2RlICptb2RlLAo+IC0JCSAgICAgIGNvbnN0IHN0cnVjdCBm Yl9tb25zcGVjcyAqbW9uc3BlYyk7Cj4gLQo+ICAJLyogQmFja2xpZ2h0ICovCj4gIAlzdHJ1Y3Qg YmFja2xpZ2h0X2RldmljZSAqYmw7Cj4gIAl1bnNpZ25lZCBpbnQgYmxfYnJpZ2h0bmVzczsKPiAt LSAKPiAyLjIwLjEKPiAKPiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwo+IGRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKPiBkcmktZGV2ZWxAbGlzdHMuZnJlZWRl c2t0b3Aub3JnCj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5m by9kcmktZGV2ZWwKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BDDA2C07542 for ; Sat, 25 May 2019 15:02:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 82CA52168B for ; Sat, 25 May 2019 15:02:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727135AbfEYPCH (ORCPT ); Sat, 25 May 2019 11:02:07 -0400 Received: from asavdk4.altibox.net ([109.247.116.15]:37976 "EHLO asavdk4.altibox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727002AbfEYPCH (ORCPT ); Sat, 25 May 2019 11:02:07 -0400 Received: from ravnborg.org (unknown [158.248.194.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by asavdk4.altibox.net (Postfix) with ESMTPS id 7C9228036E; Sat, 25 May 2019 17:02:02 +0200 (CEST) Date: Sat, 25 May 2019 17:01:59 +0200 From: Sam Ravnborg To: Daniel Vetter Cc: LKML , Intel Graphics Development , Geert Uytterhoeven , DRI Development Subject: Re: [PATCH 11/33] fbdev/sh_mobile: remove sh_mobile_lcdc_display_notify Message-ID: <20190525150159.GA27341@ravnborg.org> References: <20190524085354.27411-1-daniel.vetter@ffwll.ch> <20190524085354.27411-12-daniel.vetter@ffwll.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20190524085354.27411-12-daniel.vetter@ffwll.ch> User-Agent: Mutt/1.10.1 (2018-07-13) X-CMAE-Score: 0 X-CMAE-Analysis: v=2.3 cv=VcLZwmh9 c=1 sm=1 tr=0 a=UWs3HLbX/2nnQ3s7vZ42gw==:117 a=UWs3HLbX/2nnQ3s7vZ42gw==:17 a=jpOVt7BSZ2e4Z31A5e1TngXxSK0=:19 a=IkcTkHD0fZMA:10 a=e5mUnYsNAAAA:8 a=48fi2nN1bOFThYuLmmEA:9 a=FjwzjW-X6r35EzHo:21 a=pi2SNfIRU6eJWu1B:21 a=QEXdDO2ut3YA:10 a=Vxmtnl_E_bksehYqCbjh:22 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Daniel > It's dead code, and removing it avoids me having to understand > what it's doing with lock_fb_info. I pushed the series through my build tests which include the sh architecture. One error and one warning was triggered from sh_mobile_lcdcfb.c. The rest was fine. The patch below removed the sole user of sh_mobile_lcdc_must_reconfigure() so this triggers a warning. And I also get the following error: drivers/video/fbdev/sh_mobile_lcdcfb.c: In function ‘sh_mobile_fb_reconfig’: drivers/video/fbdev/sh_mobile_lcdcfb.c:1800:2: error: implicit declaration of function ‘fbcon_update_vcs’; did you mean ‘file_update_time’? [-Werror=implicit-function-declaration] fbcon_update_vcs(info, true); ^~~~~~~~~~~~~~~~ file_update_time I did not check but assume the error was triggered in patch 28 where fbcon_update_vcs() in introduced. Both are trivially fixed by appended patch. Sam diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c index bb1a610d0363..b8454424910d 100644 --- a/drivers/video/fbdev/sh_mobile_lcdcfb.c +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.c @@ -15,6 +15,7 @@ #include #include #include +#include #include #include #include @@ -533,25 +534,6 @@ static void sh_mobile_lcdc_display_off(struct sh_mobile_lcdc_chan *ch) ch->tx_dev->ops->display_off(ch->tx_dev); } -static bool -sh_mobile_lcdc_must_reconfigure(struct sh_mobile_lcdc_chan *ch, - const struct fb_videomode *new_mode) -{ - dev_dbg(ch->info->dev, "Old %ux%u, new %ux%u\n", - ch->display.mode.xres, ch->display.mode.yres, - new_mode->xres, new_mode->yres); - - /* It can be a different monitor with an equal video-mode */ - if (fb_mode_is_equal(&ch->display.mode, new_mode)) - return false; - - dev_dbg(ch->info->dev, "Switching %u -> %u lines\n", - ch->display.mode.yres, new_mode->yres); - ch->display.mode = *new_mode; - - return true; -} - static int sh_mobile_lcdc_check_var(struct fb_var_screeninfo *var, struct fb_info *info); > Signed-off-by: Daniel Vetter > Reviewed-by: Geert Uytterhoeven > Cc: Geert Uytterhoeven > Cc: Daniel Vetter > --- > drivers/video/fbdev/sh_mobile_lcdcfb.c | 63 -------------------------- > drivers/video/fbdev/sh_mobile_lcdcfb.h | 5 -- > 2 files changed, 68 deletions(-) > > diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c > index dc46be38c970..c5924f5e98c6 100644 > --- a/drivers/video/fbdev/sh_mobile_lcdcfb.c > +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.c > @@ -556,67 +556,6 @@ sh_mobile_lcdc_must_reconfigure(struct sh_mobile_lcdc_chan *ch, > static int sh_mobile_lcdc_check_var(struct fb_var_screeninfo *var, > struct fb_info *info); > > -static int sh_mobile_lcdc_display_notify(struct sh_mobile_lcdc_chan *ch, > - enum sh_mobile_lcdc_entity_event event, > - const struct fb_videomode *mode, > - const struct fb_monspecs *monspec) > -{ > - struct fb_info *info = ch->info; > - struct fb_var_screeninfo var; > - int ret = 0; > - > - switch (event) { > - case SH_MOBILE_LCDC_EVENT_DISPLAY_CONNECT: > - /* HDMI plug in */ > - console_lock(); > - if (lock_fb_info(info)) { > - > - > - ch->display.width = monspec->max_x * 10; > - ch->display.height = monspec->max_y * 10; > - > - if (!sh_mobile_lcdc_must_reconfigure(ch, mode) && > - info->state == FBINFO_STATE_RUNNING) { > - /* First activation with the default monitor. > - * Just turn on, if we run a resume here, the > - * logo disappears. > - */ > - info->var.width = ch->display.width; > - info->var.height = ch->display.height; > - sh_mobile_lcdc_display_on(ch); > - } else { > - /* New monitor or have to wake up */ > - fb_set_suspend(info, 0); > - } > - > - > - unlock_fb_info(info); > - } > - console_unlock(); > - break; > - > - case SH_MOBILE_LCDC_EVENT_DISPLAY_DISCONNECT: > - /* HDMI disconnect */ > - console_lock(); > - if (lock_fb_info(info)) { > - fb_set_suspend(info, 1); > - unlock_fb_info(info); > - } > - console_unlock(); > - break; > - > - case SH_MOBILE_LCDC_EVENT_DISPLAY_MODE: > - /* Validate a proposed new mode */ > - fb_videomode_to_var(&var, mode); > - var.bits_per_pixel = info->var.bits_per_pixel; > - var.grayscale = info->var.grayscale; > - ret = sh_mobile_lcdc_check_var(&var, info); > - break; > - } > - > - return ret; > -} > - > /* ----------------------------------------------------------------------------- > * Format helpers > */ > @@ -2540,8 +2479,6 @@ sh_mobile_lcdc_channel_init(struct sh_mobile_lcdc_chan *ch) > unsigned int max_size; > unsigned int i; > > - ch->notify = sh_mobile_lcdc_display_notify; > - > /* Validate the format. */ > format = sh_mobile_format_info(cfg->fourcc); > if (format == NULL) { > diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.h b/drivers/video/fbdev/sh_mobile_lcdcfb.h > index b8e47a8bd8ab..589400372098 100644 > --- a/drivers/video/fbdev/sh_mobile_lcdcfb.h > +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.h > @@ -87,11 +87,6 @@ struct sh_mobile_lcdc_chan { > unsigned long base_addr_c; > unsigned int line_size; > > - int (*notify)(struct sh_mobile_lcdc_chan *ch, > - enum sh_mobile_lcdc_entity_event event, > - const struct fb_videomode *mode, > - const struct fb_monspecs *monspec); > - > /* Backlight */ > struct backlight_device *bl; > unsigned int bl_brightness; > -- > 2.20.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel