From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 4/5] drm/connector: Split out orientation quirk detection Date: Tue, 11 Jun 2019 11:08:03 +0300 Message-ID: <87zhmoy270.fsf@intel.com> References: <20190611040350.90064-1-dbasehore@chromium.org> <20190611040350.90064-5-dbasehore@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20190611040350.90064-5-dbasehore@chromium.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: linux-kernel@vger.kernel.org Cc: Mark Rutland , Hans de Goede , David Airlie , dri-devel@lists.freedesktop.org, Sam Ravnborg , Maxime Ripard , CK Hu , devicetree@vger.kernel.org, intel-gfx@lists.freedesktop.org, Derek Basehore , Rob Herring , linux-mediatek@lists.infradead.org, Matthias Brugger , linux-arm-kernel@lists.infradead.org, Philipp Zabel List-Id: linux-mediatek@lists.infradead.org T24gTW9uLCAxMCBKdW4gMjAxOSwgRGVyZWsgQmFzZWhvcmUgPGRiYXNlaG9yZUBjaHJvbWl1bS5v cmc+IHdyb3RlOgo+IFRoaXMgcmVtb3ZlcyB0aGUgb3JpZW50YXRpb24gcXVpcmsgZGV0ZWN0aW9u IGZyb20gdGhlIGNvZGUgdG8gYWRkCj4gYW4gb3JpZW50YXRpb24gcHJvcGVydHkgdG8gYSBwYW5l bC4gVGhpcyBpcyB1c2VkIG9ubHkgZm9yIGxlZ2FjeSB4ODYKPiBzeXN0ZW1zLCB5ZXQgd2UnZCBs aWtlIHRvIHN0YXJ0IHVzaW5nIHRoaXMgb24gZGV2aWNldHJlZSBzeXN0ZW1zIHdoZXJlCj4gcXVp cmsgZGV0ZWN0aW9uIGxpa2UgdGhpcyBpcyBub3QgbmVlZGVkLgoKTm90IG5lZWRlZCwgYnV0IG5v IGhhcm0gZG9uZSBlaXRoZXIsIHJpZ2h0PwoKSSBndWVzcyBJJ2xsIGRlZmVyIGp1ZGdlbWVudCBv biB0aGlzIHRvIEhhbnMgYW5kIFZpbGxlIChDYydkKS4KClNpZGUgbm90ZSwgSSdtIGFib3V0IHRv IGFwcGx5IHNvbWUgKG1pbm9yKSBjb25mbGljdGluZyBjaGFuZ2VzIGluIG91cgotbmV4dCBhcyBz b29uIGFzIEkgZ2V0IENJIHJlc3VsdHMgb24gaXQuCgoKQlIsCkphbmkuCgoKPgo+IFNpZ25lZC1v ZmYtYnk6IERlcmVrIEJhc2Vob3JlIDxkYmFzZWhvcmVAY2hyb21pdW0ub3JnPgo+IC0tLQo+ICBk cml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5jIHwgMTYgKysrKy0tLS0tLS0tLS0tLQo+ICBk cml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jIHwgMTQgKysrKysrKysrKystLS0KPiAgZHJp dmVycy9ncHUvZHJtL2k5MTUvdmx2X2RzaS5jICB8IDE0ICsrKysrKysrKystLS0tCj4gIGluY2x1 ZGUvZHJtL2RybV9jb25uZWN0b3IuaCAgICAgfCAgMiArLQo+ICA0IGZpbGVzIGNoYW5nZWQsIDI2 IGluc2VydGlvbnMoKyksIDIwIGRlbGV0aW9ucygtKQo+Cj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMv Z3B1L2RybS9kcm1fY29ubmVjdG9yLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2Nvbm5lY3Rvci5j Cj4gaW5kZXggZTE3NTg2YWFhODBmLi41OGEwOWI2NTAyOGIgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVy cy9ncHUvZHJtL2RybV9jb25uZWN0b3IuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fY29u bmVjdG9yLmMKPiBAQCAtMTg5NCwzMSArMTg5NCwyMyBAQCBFWFBPUlRfU1lNQk9MKGRybV9jb25u ZWN0b3Jfc2V0X3Zycl9jYXBhYmxlX3Byb3BlcnR5KTsKPiAgICogZHJtX2Nvbm5lY3Rvcl9pbml0 X3BhbmVsX29yaWVudGF0aW9uX3Byb3BlcnR5IC0KPiAgICoJaW5pdGlhbGl6ZSB0aGUgY29ubmVj dGVycyBwYW5lbF9vcmllbnRhdGlvbiBwcm9wZXJ0eQo+ICAgKiBAY29ubmVjdG9yOiBjb25uZWN0 b3IgZm9yIHdoaWNoIHRvIGluaXQgdGhlIHBhbmVsLW9yaWVudGF0aW9uIHByb3BlcnR5Lgo+IC0g KiBAd2lkdGg6IHdpZHRoIGluIHBpeGVscyBvZiB0aGUgcGFuZWwsIHVzZWQgZm9yIHBhbmVsIHF1 aXJrIGRldGVjdGlvbgo+IC0gKiBAaGVpZ2h0OiBoZWlnaHQgaW4gcGl4ZWxzIG9mIHRoZSBwYW5l bCwgdXNlZCBmb3IgcGFuZWwgcXVpcmsgZGV0ZWN0aW9uCj4gICAqCj4gICAqIFRoaXMgZnVuY3Rp b24gc2hvdWxkIG9ubHkgYmUgY2FsbGVkIGZvciBidWlsdC1pbiBwYW5lbHMsIGFmdGVyIHNldHRp bmcKPiAgICogY29ubmVjdG9yLT5kaXNwbGF5X2luZm8ucGFuZWxfb3JpZW50YXRpb24gZmlyc3Qg KGlmIGtub3duKS4KPiAgICoKPiAtICogVGhpcyBmdW5jdGlvbiB3aWxsIGNoZWNrIGZvciBwbGF0 Zm9ybSBzcGVjaWZpYyAoZS5nLiBETUkgYmFzZWQpIHF1aXJrcwo+IC0gKiBvdmVycmlkaW5nIGRp c3BsYXlfaW5mby5wYW5lbF9vcmllbnRhdGlvbiBmaXJzdCwgdGhlbiBpZiBwYW5lbF9vcmllbnRh dGlvbgo+IC0gKiBpcyBub3QgRFJNX01PREVfUEFORUxfT1JJRU5UQVRJT05fVU5LTk9XTiBpdCB3 aWxsIGF0dGFjaCB0aGUKPiAtICogInBhbmVsIG9yaWVudGF0aW9uIiBwcm9wZXJ0eSB0byB0aGUg Y29ubmVjdG9yLgo+ICsgKiBUaGlzIGZ1bmN0aW9uIHdpbGwgY2hlY2sgaWYgdGhlIHBhbmVsX29y aWVudGF0aW9uIGlzIG5vdAo+ICsgKiBEUk1fTU9ERV9QQU5FTF9PUklFTlRBVElPTl9VTktOT1dO LiBJZiBub3QsIGl0IHdpbGwgYXR0YWNoIHRoZSAicGFuZWwKPiArICogb3JpZW50YXRpb24iIHBy b3BlcnR5IHRvIHRoZSBjb25uZWN0b3IuCj4gICAqCj4gICAqIFJldHVybnM6Cj4gICAqIFplcm8g b24gc3VjY2VzcywgbmVnYXRpdmUgZXJybm8gb24gZmFpbHVyZS4KPiAgICovCj4gIGludCBkcm1f Y29ubmVjdG9yX2luaXRfcGFuZWxfb3JpZW50YXRpb25fcHJvcGVydHkoCj4gLQlzdHJ1Y3QgZHJt X2Nvbm5lY3RvciAqY29ubmVjdG9yLCBpbnQgd2lkdGgsIGludCBoZWlnaHQpCj4gKwlzdHJ1Y3Qg ZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9yKQo+ICB7Cj4gIAlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2 ID0gY29ubmVjdG9yLT5kZXY7Cj4gIAlzdHJ1Y3QgZHJtX2Rpc3BsYXlfaW5mbyAqaW5mbyA9ICZj b25uZWN0b3ItPmRpc3BsYXlfaW5mbzsKPiAgCXN0cnVjdCBkcm1fcHJvcGVydHkgKnByb3A7Cj4g LQlpbnQgb3JpZW50YXRpb25fcXVpcms7Cj4gLQo+IC0Jb3JpZW50YXRpb25fcXVpcmsgPSBkcm1f Z2V0X3BhbmVsX29yaWVudGF0aW9uX3F1aXJrKHdpZHRoLCBoZWlnaHQpOwo+IC0JaWYgKG9yaWVu dGF0aW9uX3F1aXJrICE9IERSTV9NT0RFX1BBTkVMX09SSUVOVEFUSU9OX1VOS05PV04pCj4gLQkJ aW5mby0+cGFuZWxfb3JpZW50YXRpb24gPSBvcmllbnRhdGlvbl9xdWlyazsKPiAgCj4gIAlpZiAo aW5mby0+cGFuZWxfb3JpZW50YXRpb24gPT0gRFJNX01PREVfUEFORUxfT1JJRU5UQVRJT05fVU5L Tk9XTikKPiAgCQlyZXR1cm4gMDsKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUv aW50ZWxfZHAuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMKPiBpbmRleCBiMDk5 YTlkYzI4ZmQuLjcyYWIwOTBlYTk3YSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9kcC5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHAuYwo+IEBA IC00MCw2ICs0MCw3IEBACj4gICNpbmNsdWRlIDxkcm0vZHJtX2VkaWQuaD4KPiAgI2luY2x1ZGUg PGRybS9kcm1faGRjcC5oPgo+ICAjaW5jbHVkZSA8ZHJtL2RybV9wcm9iZV9oZWxwZXIuaD4KPiAr I2luY2x1ZGUgPGRybS9kcm1fdXRpbHMuaD4KPiAgI2luY2x1ZGUgPGRybS9pOTE1X2RybS5oPgo+ ICAKPiAgI2luY2x1ZGUgImk5MTVfZGVidWdmcy5oIgo+IEBAIC03MjgxLDkgKzcyODIsMTYgQEAg c3RhdGljIGJvb2wgaW50ZWxfZWRwX2luaXRfY29ubmVjdG9yKHN0cnVjdCBpbnRlbF9kcCAqaW50 ZWxfZHAsCj4gIAlpbnRlbF9jb25uZWN0b3ItPnBhbmVsLmJhY2tsaWdodC5wb3dlciA9IGludGVs X2VkcF9iYWNrbGlnaHRfcG93ZXI7Cj4gIAlpbnRlbF9wYW5lbF9zZXR1cF9iYWNrbGlnaHQoY29u bmVjdG9yLCBwaXBlKTsKPiAgCj4gLQlpZiAoZml4ZWRfbW9kZSkKPiAtCQlkcm1fY29ubmVjdG9y X2luaXRfcGFuZWxfb3JpZW50YXRpb25fcHJvcGVydHkoCj4gLQkJCWNvbm5lY3RvciwgZml4ZWRf bW9kZS0+aGRpc3BsYXksIGZpeGVkX21vZGUtPnZkaXNwbGF5KTsKPiArCWlmIChmaXhlZF9tb2Rl KSB7Cj4gKwkJaW50IG9yaWVudGF0aW9uID0gZHJtX2dldF9wYW5lbF9vcmllbnRhdGlvbl9xdWly aygKPiArCQkJCWZpeGVkX21vZGUtPmhkaXNwbGF5LCBmaXhlZF9tb2RlLT52ZGlzcGxheSk7Cj4g Kwo+ICsJCWlmIChvcmllbnRhdGlvbiAhPSBEUk1fTU9ERV9QQU5FTF9PUklFTlRBVElPTl9VTktO T1dOKQo+ICsJCQljb25uZWN0b3ItPmRpc3BsYXlfaW5mby5wYW5lbF9vcmllbnRhdGlvbiA9Cj4g KwkJCQlvcmllbnRhdGlvbjsKPiArCj4gKwkJZHJtX2Nvbm5lY3Rvcl9pbml0X3BhbmVsX29yaWVu dGF0aW9uX3Byb3BlcnR5KGNvbm5lY3Rvcik7Cj4gKwl9Cj4gIAo+ICAJcmV0dXJuIHRydWU7Cj4g IAo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS92bHZfZHNpLmMgYi9kcml2ZXJz L2dwdS9kcm0vaTkxNS92bHZfZHNpLmMKPiBpbmRleCBiZmUyODkxZWFjMzcuLjI3Zjg2YTc4N2Y2 MCAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS92bHZfZHNpLmMKPiArKysgYi9k cml2ZXJzL2dwdS9kcm0vaTkxNS92bHZfZHNpLmMKPiBAQCAtMzAsNiArMzAsNyBAQAo+ICAjaW5j bHVkZSA8ZHJtL2RybV9jcnRjLmg+Cj4gICNpbmNsdWRlIDxkcm0vZHJtX2VkaWQuaD4KPiAgI2lu Y2x1ZGUgPGRybS9kcm1fbWlwaV9kc2kuaD4KPiArI2luY2x1ZGUgPGRybS9kcm1fdXRpbHMuaD4K PiAgCj4gICNpbmNsdWRlICJpOTE1X2Rydi5oIgo+ICAjaW5jbHVkZSAiaW50ZWxfYXRvbWljLmgi Cj4gQEAgLTE2NTAsNiArMTY1MSw3IEBAIHN0YXRpYyB2b2lkIGludGVsX2RzaV9hZGRfcHJvcGVy dGllcyhzdHJ1Y3QgaW50ZWxfY29ubmVjdG9yICpjb25uZWN0b3IpCj4gIAo+ICAJaWYgKGNvbm5l Y3Rvci0+cGFuZWwuZml4ZWRfbW9kZSkgewo+ICAJCXUzMiBhbGxvd2VkX3NjYWxlcnM7Cj4gKwkJ aW50IG9yaWVudGF0aW9uOwo+ICAKPiAgCQlhbGxvd2VkX3NjYWxlcnMgPSBCSVQoRFJNX01PREVf U0NBTEVfQVNQRUNUKSB8IEJJVChEUk1fTU9ERV9TQ0FMRV9GVUxMU0NSRUVOKTsKPiAgCQlpZiAo IUhBU19HTUNIKGRldl9wcml2KSkKPiBAQCAtMTY2MCwxMiArMTY2MiwxNiBAQCBzdGF0aWMgdm9p ZCBpbnRlbF9kc2lfYWRkX3Byb3BlcnRpZXMoc3RydWN0IGludGVsX2Nvbm5lY3RvciAqY29ubmVj dG9yKQo+ICAKPiAgCQljb25uZWN0b3ItPmJhc2Uuc3RhdGUtPnNjYWxpbmdfbW9kZSA9IERSTV9N T0RFX1NDQUxFX0FTUEVDVDsKPiAgCj4gLQkJY29ubmVjdG9yLT5iYXNlLmRpc3BsYXlfaW5mby5w YW5lbF9vcmllbnRhdGlvbiA9Cj4gLQkJCXZsdl9kc2lfZ2V0X3BhbmVsX29yaWVudGF0aW9uKGNv bm5lY3Rvcik7Cj4gLQkJZHJtX2Nvbm5lY3Rvcl9pbml0X3BhbmVsX29yaWVudGF0aW9uX3Byb3Bl cnR5KAo+IC0JCQkJJmNvbm5lY3Rvci0+YmFzZSwKPiArCQlvcmllbnRhdGlvbiA9IGRybV9nZXRf cGFuZWxfb3JpZW50YXRpb25fcXVpcmsoCj4gIAkJCQljb25uZWN0b3ItPnBhbmVsLmZpeGVkX21v ZGUtPmhkaXNwbGF5LAo+ICAJCQkJY29ubmVjdG9yLT5wYW5lbC5maXhlZF9tb2RlLT52ZGlzcGxh eSk7Cj4gKwkJaWYgKG9yaWVudGF0aW9uICE9IERSTV9NT0RFX1BBTkVMX09SSUVOVEFUSU9OX1VO S05PV04pCj4gKwkJCWNvbm5lY3Rvci0+YmFzZS5kaXNwbGF5X2luZm8ucGFuZWxfb3JpZW50YXRp b24gPSBvcmllbnRhdGlvbjsKPiArCQllbHNlCj4gKwkJCWNvbm5lY3Rvci0+YmFzZS5kaXNwbGF5 X2luZm8ucGFuZWxfb3JpZW50YXRpb24gPQo+ICsJCQkJdmx2X2RzaV9nZXRfcGFuZWxfb3JpZW50 YXRpb24oY29ubmVjdG9yKTsKPiArCj4gKwkJZHJtX2Nvbm5lY3Rvcl9pbml0X3BhbmVsX29yaWVu dGF0aW9uX3Byb3BlcnR5KCZjb25uZWN0b3ItPmJhc2UpOwo+ICAJfQo+ICB9Cj4gIAo+IGRpZmYg LS1naXQgYS9pbmNsdWRlL2RybS9kcm1fY29ubmVjdG9yLmggYi9pbmNsdWRlL2RybS9kcm1fY29u bmVjdG9yLmgKPiBpbmRleCA0N2U3NDliNzRlNWYuLmMyOTkyZjdhMGRkNSAxMDA2NDQKPiAtLS0g YS9pbmNsdWRlL2RybS9kcm1fY29ubmVjdG9yLmgKPiArKysgYi9pbmNsdWRlL2RybS9kcm1fY29u bmVjdG9yLmgKPiBAQCAtMTM3MCw3ICsxMzcwLDcgQEAgdm9pZCBkcm1fY29ubmVjdG9yX3NldF9s aW5rX3N0YXR1c19wcm9wZXJ0eShzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9yLAo+ICB2 b2lkIGRybV9jb25uZWN0b3Jfc2V0X3Zycl9jYXBhYmxlX3Byb3BlcnR5KAo+ICAJCXN0cnVjdCBk cm1fY29ubmVjdG9yICpjb25uZWN0b3IsIGJvb2wgY2FwYWJsZSk7Cj4gIGludCBkcm1fY29ubmVj dG9yX2luaXRfcGFuZWxfb3JpZW50YXRpb25fcHJvcGVydHkoCj4gLQlzdHJ1Y3QgZHJtX2Nvbm5l Y3RvciAqY29ubmVjdG9yLCBpbnQgd2lkdGgsIGludCBoZWlnaHQpOwo+ICsJc3RydWN0IGRybV9j b25uZWN0b3IgKmNvbm5lY3Rvcik7Cj4gIGludCBkcm1fY29ubmVjdG9yX2F0dGFjaF9tYXhfYnBj X3Byb3BlcnR5KHN0cnVjdCBkcm1fY29ubmVjdG9yICpjb25uZWN0b3IsCj4gIAkJCQkJICBpbnQg bWluLCBpbnQgbWF4KTsKCi0tIApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBTb3VyY2UgR3JhcGhp Y3MgQ2VudGVyCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f CkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpo dHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeA== 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=-6.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,T_DKIMWL_WL_HIGH autolearn=unavailable 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 7980AC4321A for ; Tue, 11 Jun 2019 08:05:27 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 43169208E3 for ; Tue, 11 Jun 2019 08:05:27 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="P1KAPs6d" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 43169208E3 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=VrZLw/bSxDj2iifoVDliBqb7+yxprcumAUj1GSVmN7w=; b=P1KAPs6dqkkc7m zFtsqnt0s27iWP+86KT1ubDRQd49zi0g4UrM6S8fo9ogQJPII5iuT7yRnf5PrPRxku1jMyPgBvOhp UfsaC9ANsK3cgYochPUkFOb1yEKSgtbIjSFJVrPR8fmS/KCTClEdwFHl7cRfyKQ9AIbUPdKPVQlP3 YPz83Mxke/vRKzVMpOeJs4sleJCnJD5q0bgtZTT5cDuBoelspAE9lkkEsVs8M+ORhx2FNeVKGyZi6 8+MjAjO+JgemURa/q6gb8pl6qMfI2wh0rzHgiIS2RpNBWxJuLXAcg8ycNX2tTmNFcLxO0A236yqK8 G00GECbvPhzeIckDEXMg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1habmP-0000LA-Fb; Tue, 11 Jun 2019 08:05:17 +0000 Received: from mga12.intel.com ([192.55.52.136]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1habmL-0008Tt-Kh; Tue, 11 Jun 2019 08:05:15 +0000 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Jun 2019 01:05:08 -0700 X-ExtLoop1: 1 Received: from jnikula-mobl3.fi.intel.com (HELO localhost) ([10.237.66.150]) by fmsmga001.fm.intel.com with ESMTP; 11 Jun 2019 01:05:02 -0700 From: Jani Nikula To: Derek Basehore , linux-kernel@vger.kernel.org Subject: Re: [PATCH 4/5] drm/connector: Split out orientation quirk detection In-Reply-To: <20190611040350.90064-5-dbasehore@chromium.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20190611040350.90064-1-dbasehore@chromium.org> <20190611040350.90064-5-dbasehore@chromium.org> Date: Tue, 11 Jun 2019 11:08:03 +0300 Message-ID: <87zhmoy270.fsf@intel.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190611_010513_691907_5A14321A X-CRM114-Status: GOOD ( 22.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Hans de Goede , David Airlie , Joonas Lahtinen , dri-devel@lists.freedesktop.org, Thierry Reding , Sam Ravnborg , Ville =?utf-8?B?U3lyasOkbMOk?= , Maxime Ripard , CK Hu , devicetree@vger.kernel.org, Daniel Vetter , intel-gfx@lists.freedesktop.org, Derek Basehore , Maarten Lankhorst , Rob Herring , linux-mediatek@lists.infradead.org, Rodrigo Vivi , Matthias Brugger , Sean Paul , linux-arm-kernel@lists.infradead.org, Philipp Zabel Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 10 Jun 2019, Derek Basehore wrote: > This removes the orientation quirk detection from the code to add > an orientation property to a panel. This is used only for legacy x86 > systems, yet we'd like to start using this on devicetree systems where > quirk detection like this is not needed. Not needed, but no harm done either, right? I guess I'll defer judgement on this to Hans and Ville (Cc'd). Side note, I'm about to apply some (minor) conflicting changes in our -next as soon as I get CI results on it. BR, Jani. > > Signed-off-by: Derek Basehore > --- > drivers/gpu/drm/drm_connector.c | 16 ++++------------ > drivers/gpu/drm/i915/intel_dp.c | 14 +++++++++++--- > drivers/gpu/drm/i915/vlv_dsi.c | 14 ++++++++++---- > include/drm/drm_connector.h | 2 +- > 4 files changed, 26 insertions(+), 20 deletions(-) > > diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c > index e17586aaa80f..58a09b65028b 100644 > --- a/drivers/gpu/drm/drm_connector.c > +++ b/drivers/gpu/drm/drm_connector.c > @@ -1894,31 +1894,23 @@ EXPORT_SYMBOL(drm_connector_set_vrr_capable_property); > * drm_connector_init_panel_orientation_property - > * initialize the connecters panel_orientation property > * @connector: connector for which to init the panel-orientation property. > - * @width: width in pixels of the panel, used for panel quirk detection > - * @height: height in pixels of the panel, used for panel quirk detection > * > * This function should only be called for built-in panels, after setting > * connector->display_info.panel_orientation first (if known). > * > - * This function will check for platform specific (e.g. DMI based) quirks > - * overriding display_info.panel_orientation first, then if panel_orientation > - * is not DRM_MODE_PANEL_ORIENTATION_UNKNOWN it will attach the > - * "panel orientation" property to the connector. > + * This function will check if the panel_orientation is not > + * DRM_MODE_PANEL_ORIENTATION_UNKNOWN. If not, it will attach the "panel > + * orientation" property to the connector. > * > * Returns: > * Zero on success, negative errno on failure. > */ > int drm_connector_init_panel_orientation_property( > - struct drm_connector *connector, int width, int height) > + struct drm_connector *connector) > { > struct drm_device *dev = connector->dev; > struct drm_display_info *info = &connector->display_info; > struct drm_property *prop; > - int orientation_quirk; > - > - orientation_quirk = drm_get_panel_orientation_quirk(width, height); > - if (orientation_quirk != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > - info->panel_orientation = orientation_quirk; > > if (info->panel_orientation == DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > return 0; > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index b099a9dc28fd..72ab090ea97a 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -40,6 +40,7 @@ > #include > #include > #include > +#include > #include > > #include "i915_debugfs.h" > @@ -7281,9 +7282,16 @@ static bool intel_edp_init_connector(struct intel_dp *intel_dp, > intel_connector->panel.backlight.power = intel_edp_backlight_power; > intel_panel_setup_backlight(connector, pipe); > > - if (fixed_mode) > - drm_connector_init_panel_orientation_property( > - connector, fixed_mode->hdisplay, fixed_mode->vdisplay); > + if (fixed_mode) { > + int orientation = drm_get_panel_orientation_quirk( > + fixed_mode->hdisplay, fixed_mode->vdisplay); > + > + if (orientation != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > + connector->display_info.panel_orientation = > + orientation; > + > + drm_connector_init_panel_orientation_property(connector); > + } > > return true; > > diff --git a/drivers/gpu/drm/i915/vlv_dsi.c b/drivers/gpu/drm/i915/vlv_dsi.c > index bfe2891eac37..27f86a787f60 100644 > --- a/drivers/gpu/drm/i915/vlv_dsi.c > +++ b/drivers/gpu/drm/i915/vlv_dsi.c > @@ -30,6 +30,7 @@ > #include > #include > #include > +#include > > #include "i915_drv.h" > #include "intel_atomic.h" > @@ -1650,6 +1651,7 @@ static void intel_dsi_add_properties(struct intel_connector *connector) > > if (connector->panel.fixed_mode) { > u32 allowed_scalers; > + int orientation; > > allowed_scalers = BIT(DRM_MODE_SCALE_ASPECT) | BIT(DRM_MODE_SCALE_FULLSCREEN); > if (!HAS_GMCH(dev_priv)) > @@ -1660,12 +1662,16 @@ static void intel_dsi_add_properties(struct intel_connector *connector) > > connector->base.state->scaling_mode = DRM_MODE_SCALE_ASPECT; > > - connector->base.display_info.panel_orientation = > - vlv_dsi_get_panel_orientation(connector); > - drm_connector_init_panel_orientation_property( > - &connector->base, > + orientation = drm_get_panel_orientation_quirk( > connector->panel.fixed_mode->hdisplay, > connector->panel.fixed_mode->vdisplay); > + if (orientation != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > + connector->base.display_info.panel_orientation = orientation; > + else > + connector->base.display_info.panel_orientation = > + vlv_dsi_get_panel_orientation(connector); > + > + drm_connector_init_panel_orientation_property(&connector->base); > } > } > > diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > index 47e749b74e5f..c2992f7a0dd5 100644 > --- a/include/drm/drm_connector.h > +++ b/include/drm/drm_connector.h > @@ -1370,7 +1370,7 @@ void drm_connector_set_link_status_property(struct drm_connector *connector, > void drm_connector_set_vrr_capable_property( > struct drm_connector *connector, bool capable); > int drm_connector_init_panel_orientation_property( > - struct drm_connector *connector, int width, int height); > + struct drm_connector *connector); > int drm_connector_attach_max_bpc_property(struct drm_connector *connector, > int min, int max); -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel 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=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS 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 C3DC2C43218 for ; Tue, 11 Jun 2019 08:05:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9587521721 for ; Tue, 11 Jun 2019 08:05:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404574AbfFKIFJ (ORCPT ); Tue, 11 Jun 2019 04:05:09 -0400 Received: from mga02.intel.com ([134.134.136.20]:33802 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404389AbfFKIFJ (ORCPT ); Tue, 11 Jun 2019 04:05:09 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Jun 2019 01:05:08 -0700 X-ExtLoop1: 1 Received: from jnikula-mobl3.fi.intel.com (HELO localhost) ([10.237.66.150]) by fmsmga001.fm.intel.com with ESMTP; 11 Jun 2019 01:05:02 -0700 From: Jani Nikula To: Derek Basehore , linux-kernel@vger.kernel.org Cc: Thierry Reding , Sam Ravnborg , David Airlie , Daniel Vetter , Rob Herring , Mark Rutland , Maarten Lankhorst , Maxime Ripard , Sean Paul , Joonas Lahtinen , Rodrigo Vivi , CK Hu , Philipp Zabel , Matthias Brugger , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, intel-gfx@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Derek Basehore , Hans de Goede , Ville =?utf-8?B?U3lyasOkbMOk?= Subject: Re: [PATCH 4/5] drm/connector: Split out orientation quirk detection In-Reply-To: <20190611040350.90064-5-dbasehore@chromium.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20190611040350.90064-1-dbasehore@chromium.org> <20190611040350.90064-5-dbasehore@chromium.org> Date: Tue, 11 Jun 2019 11:08:03 +0300 Message-ID: <87zhmoy270.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 10 Jun 2019, Derek Basehore wrote: > This removes the orientation quirk detection from the code to add > an orientation property to a panel. This is used only for legacy x86 > systems, yet we'd like to start using this on devicetree systems where > quirk detection like this is not needed. Not needed, but no harm done either, right? I guess I'll defer judgement on this to Hans and Ville (Cc'd). Side note, I'm about to apply some (minor) conflicting changes in our -next as soon as I get CI results on it. BR, Jani. > > Signed-off-by: Derek Basehore > --- > drivers/gpu/drm/drm_connector.c | 16 ++++------------ > drivers/gpu/drm/i915/intel_dp.c | 14 +++++++++++--- > drivers/gpu/drm/i915/vlv_dsi.c | 14 ++++++++++---- > include/drm/drm_connector.h | 2 +- > 4 files changed, 26 insertions(+), 20 deletions(-) > > diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c > index e17586aaa80f..58a09b65028b 100644 > --- a/drivers/gpu/drm/drm_connector.c > +++ b/drivers/gpu/drm/drm_connector.c > @@ -1894,31 +1894,23 @@ EXPORT_SYMBOL(drm_connector_set_vrr_capable_property); > * drm_connector_init_panel_orientation_property - > * initialize the connecters panel_orientation property > * @connector: connector for which to init the panel-orientation property. > - * @width: width in pixels of the panel, used for panel quirk detection > - * @height: height in pixels of the panel, used for panel quirk detection > * > * This function should only be called for built-in panels, after setting > * connector->display_info.panel_orientation first (if known). > * > - * This function will check for platform specific (e.g. DMI based) quirks > - * overriding display_info.panel_orientation first, then if panel_orientation > - * is not DRM_MODE_PANEL_ORIENTATION_UNKNOWN it will attach the > - * "panel orientation" property to the connector. > + * This function will check if the panel_orientation is not > + * DRM_MODE_PANEL_ORIENTATION_UNKNOWN. If not, it will attach the "panel > + * orientation" property to the connector. > * > * Returns: > * Zero on success, negative errno on failure. > */ > int drm_connector_init_panel_orientation_property( > - struct drm_connector *connector, int width, int height) > + struct drm_connector *connector) > { > struct drm_device *dev = connector->dev; > struct drm_display_info *info = &connector->display_info; > struct drm_property *prop; > - int orientation_quirk; > - > - orientation_quirk = drm_get_panel_orientation_quirk(width, height); > - if (orientation_quirk != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > - info->panel_orientation = orientation_quirk; > > if (info->panel_orientation == DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > return 0; > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index b099a9dc28fd..72ab090ea97a 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -40,6 +40,7 @@ > #include > #include > #include > +#include > #include > > #include "i915_debugfs.h" > @@ -7281,9 +7282,16 @@ static bool intel_edp_init_connector(struct intel_dp *intel_dp, > intel_connector->panel.backlight.power = intel_edp_backlight_power; > intel_panel_setup_backlight(connector, pipe); > > - if (fixed_mode) > - drm_connector_init_panel_orientation_property( > - connector, fixed_mode->hdisplay, fixed_mode->vdisplay); > + if (fixed_mode) { > + int orientation = drm_get_panel_orientation_quirk( > + fixed_mode->hdisplay, fixed_mode->vdisplay); > + > + if (orientation != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > + connector->display_info.panel_orientation = > + orientation; > + > + drm_connector_init_panel_orientation_property(connector); > + } > > return true; > > diff --git a/drivers/gpu/drm/i915/vlv_dsi.c b/drivers/gpu/drm/i915/vlv_dsi.c > index bfe2891eac37..27f86a787f60 100644 > --- a/drivers/gpu/drm/i915/vlv_dsi.c > +++ b/drivers/gpu/drm/i915/vlv_dsi.c > @@ -30,6 +30,7 @@ > #include > #include > #include > +#include > > #include "i915_drv.h" > #include "intel_atomic.h" > @@ -1650,6 +1651,7 @@ static void intel_dsi_add_properties(struct intel_connector *connector) > > if (connector->panel.fixed_mode) { > u32 allowed_scalers; > + int orientation; > > allowed_scalers = BIT(DRM_MODE_SCALE_ASPECT) | BIT(DRM_MODE_SCALE_FULLSCREEN); > if (!HAS_GMCH(dev_priv)) > @@ -1660,12 +1662,16 @@ static void intel_dsi_add_properties(struct intel_connector *connector) > > connector->base.state->scaling_mode = DRM_MODE_SCALE_ASPECT; > > - connector->base.display_info.panel_orientation = > - vlv_dsi_get_panel_orientation(connector); > - drm_connector_init_panel_orientation_property( > - &connector->base, > + orientation = drm_get_panel_orientation_quirk( > connector->panel.fixed_mode->hdisplay, > connector->panel.fixed_mode->vdisplay); > + if (orientation != DRM_MODE_PANEL_ORIENTATION_UNKNOWN) > + connector->base.display_info.panel_orientation = orientation; > + else > + connector->base.display_info.panel_orientation = > + vlv_dsi_get_panel_orientation(connector); > + > + drm_connector_init_panel_orientation_property(&connector->base); > } > } > > diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > index 47e749b74e5f..c2992f7a0dd5 100644 > --- a/include/drm/drm_connector.h > +++ b/include/drm/drm_connector.h > @@ -1370,7 +1370,7 @@ void drm_connector_set_link_status_property(struct drm_connector *connector, > void drm_connector_set_vrr_capable_property( > struct drm_connector *connector, bool capable); > int drm_connector_init_panel_orientation_property( > - struct drm_connector *connector, int width, int height); > + struct drm_connector *connector); > int drm_connector_attach_max_bpc_property(struct drm_connector *connector, > int min, int max); -- Jani Nikula, Intel Open Source Graphics Center