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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 02AD7C433ED for ; Mon, 17 May 2021 12:15:48 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 9B80261209 for ; Mon, 17 May 2021 12:15:47 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9B80261209 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=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 2B1C16E93F; Mon, 17 May 2021 12:15:47 +0000 (UTC) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6D84F6E93F; Mon, 17 May 2021 12:15:46 +0000 (UTC) IronPort-SDR: bomVbLXnIfuIDRK+VIpw6mWDlDoK5mqILvT2NypzweKs60Wz27rPZ+fU/Q1CsZW264BjOBEk+B /nn6D+FzXhpg== X-IronPort-AV: E=McAfee;i="6200,9189,9986"; a="285979920" X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="285979920" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:45 -0700 IronPort-SDR: 7Ht9fR84yA4Rq8Y8w3Q2gXOI2V6j3777dycbyy09H98pxYDgPaGBbYFG66HJ8MS8XTvkcZkwbY lnxhCVCbYQ5Q== X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="472382788" Received: from sudhirk1-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.50.192]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:39 -0700 From: Jani Nikula To: Kai-Heng Feng , Ville =?utf-8?B?U3lyasOk?= =?utf-8?B?bMOk?= In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20210426152420.359402-1-kai.heng.feng@canonical.com> Date: Mon, 17 May 2021 15:15:36 +0300 Message-ID: <87fsylh7yv.fsf@intel.com> MIME-Version: 1.0 Subject: Re: [Intel-gfx] [PATCH v3] drm/i915: Invoke another _DSM to enable MUX on HP Workstation laptops X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: David Airlie , open list , Chris Wilson , Takashi Iwai , "open list:DRM DRIVERS" , Dave Airlie , intel-gfx Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" T24gRnJpLCAxNCBNYXkgMjAyMSwgS2FpLUhlbmcgRmVuZyA8a2FpLmhlbmcuZmVuZ0BjYW5vbmlj YWwuY29tPiB3cm90ZToKPiBPbiBXZWQsIE1heSAxMiwgMjAyMSBhdCAyOjE5IEFNIFZpbGxlIFN5 cmrDpGzDpAo+IDx2aWxsZS5zeXJqYWxhQGxpbnV4LmludGVsLmNvbT4gd3JvdGU6Cj4+Cj4+IE9u IE1vbiwgQXByIDI2LCAyMDIxIGF0IDExOjI0OjEwUE0gKzA4MDAsIEthaS1IZW5nIEZlbmcgd3Jv dGU6Cj4+ID4gT24gSFAgRnVyeSBHNyBXb3Jrc3RhdGlvbnMsIGdyYXBoaWNzIG91dHB1dCBpcyBy ZS1yb3V0ZWQgZnJvbSBJbnRlbCBHRlgKPj4gPiB0byBkaXNjcmV0ZSBHRlggYWZ0ZXIgUzMuIFRo aXMgaXMgbm90IGRlc2lyYWJsZSwgYmVjYXVzZSB1c2Vyc3BhY2Ugd2lsbAo+PiA+IHRyZWF0IGNv bm5lY3RlZCBkaXNwbGF5IGFzIGEgbmV3IG9uZSwgbG9zaW5nIGRpc3BsYXkgc2V0dGluZ3MuCj4+ ID4KPj4gPiBUaGUgZXhwZWN0ZWQgYmVoYXZpb3IgaXMgdG8gbGV0IGRpc2NyZXRlIEdGWCBkcml2 ZXMgYWxsIGV4dGVybmFsCj4+ID4gZGlzcGxheXMuCj4+ID4KPj4gPiBUaGUgcGxhdGZvcm0gaW4g cXVlc3Rpb24gdXNlcyBBQ1BJIG1ldGhvZCBcX1NCLlBDSTAuSEdNRSB0byBlbmFibGUgTVVYLgo+ PiA+IFRoZSBtZXRob2QgaXMgaW5zaWRlIHRoZSBhbm90aGVyIF9EU00sIHNvIGFkZCB0aGUgX0RT TSBhbmQgY2FsbCBpdAo+PiA+IGFjY29yZGluZ2x5Lgo+PiA+Cj4+ID4gSSBhbHNvIHRlc3RlZCBz b21lIE1VWC1sZXNzIGFuZCBpR1BVIG9ubHkgbGFwdG9wcyB3aXRoIHRoYXQgX0RTTSwgbm8KPj4g PiByZWdyZXNzaW9uIHdhcyBmb3VuZC4KPj4gPgo+PiA+IHYzOgo+PiA+ICAtIFJlbW92ZSBCWFQg ZnJvbSBuYW1lcy4KPj4gPiAgLSBDaGFuZ2UgdGhlIHBhcmFtZXRlciB0eXBlLgo+PiA+ICAtIEZv bGQgdGhlIGZ1bmN0aW9uIGludG8gaW50ZWxfbW9kZXNldF9pbml0X2h3KCkuCj4+ID4KPj4gPiB2 MjoKPj4gPiAgLSBGb3J3YXJkIGRlY2xhcmUgc3RydWN0IHBjaV9kZXYuCj4+ID4KPj4gPiBDbG9z ZXM6IGh0dHBzOi8vZ2l0bGFiLmZyZWVkZXNrdG9wLm9yZy9kcm0vaW50ZWwvLS9pc3N1ZXMvMzEx Mwo+PiA+IFJlZmVyZW5jZXM6IGh0dHBzOi8vbG9yZS5rZXJuZWwub3JnL2ludGVsLWdmeC8xNDYw MDQwNzMyLTMxNDE3LTQtZ2l0LXNlbmQtZW1haWwtYW5pbWVzaC5tYW5uYUBpbnRlbC5jb20vCj4+ ID4gU2lnbmVkLW9mZi1ieTogS2FpLUhlbmcgRmVuZyA8a2FpLmhlbmcuZmVuZ0BjYW5vbmljYWwu Y29tPgo+PiA+IC0tLQo+PiA+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9kaXNwbGF5L2ludGVsX2Fj cGkuYyAgICB8IDE4ICsrKysrKysrKysrKysrKysrKwo+PiA+ICBkcml2ZXJzL2dwdS9kcm0vaTkx NS9kaXNwbGF5L2ludGVsX2FjcGkuaCAgICB8ICAzICsrKwo+PiA+ICBkcml2ZXJzL2dwdS9kcm0v aTkxNS9kaXNwbGF5L2ludGVsX2Rpc3BsYXkuYyB8ICAyICsrCj4+ID4gIDMgZmlsZXMgY2hhbmdl ZCwgMjMgaW5zZXJ0aW9ucygrKQo+PiA+Cj4+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2Ry bS9pOTE1L2Rpc3BsYXkvaW50ZWxfYWNwaS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvZGlzcGxh eS9pbnRlbF9hY3BpLmMKPj4gPiBpbmRleCA4MzNkMGMxYmU0ZjEuLmQwMDhkMzk3NjI2MSAxMDA2 NDQKPj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9kaXNwbGF5L2ludGVsX2FjcGkuYwo+ PiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2Rpc3BsYXkvaW50ZWxfYWNwaS5jCj4+ID4g QEAgLTEzLDEyICsxMywxNyBAQAo+PiA+ICAjaW5jbHVkZSAiaW50ZWxfZGlzcGxheV90eXBlcy5o Igo+PiA+Cj4+ID4gICNkZWZpbmUgSU5URUxfRFNNX1JFVklTSU9OX0lEIDEgLyogRm9yIENhbHBl bGxhIGFueXdheS4uLiAqLwo+PiA+ICsjZGVmaW5lIElOVEVMX0RTTV9GTl9QTEFURk9STV9NVVhf RU5BQkxFIDAgLyogTm8gYXJncyAqLwo+Pgo+PiBUaGlzIGJsb2NrIG9mIGRlZmluZXMgaXMgZm9y IHRoZSBvdGhlciBEU00uIFdlIGRvbid0IHdhbnQgdG8KPj4gbWl4IHRoZXNlIHVwLiBXZSBhbHNv IHdhbnQgdG8gbmFtZSBpdCBhY2NvcmRpbmcgdG8gdGhlIHNwZWMsCj4+IHNvIHNvbWV0aGluZyBs aWtlIEdFVF9CSU9TX0RBVEFfRlVOQ1NfU1VQUE9SVEVELiBTaW1pbGFybHkKPj4gZm9yIHRoZSBp bnRlbF9kc21fZW5hYmxlX211eCgpIHdyYXBwZXIgZnVuY3Rpb24uICsgaXQgbmVlZHMKPj4gYSBj b21tZW50IHRvIGRvY3VtZW50IHRoYXQgc29tZSBCSU9TZXMgYWJ1c2UgaXQgdG8gZG8gTVVYCj4+ IGluaXRpYWxpemF0aW9uIGFuZCB3aGF0bm90Lgo+Cj4gV2lsbCBkby4KPgo+Cj4+Cj4+IFdlIHNo b3VsZCBwZXJoYXBzIHJlbmFtZSBhbGwgdGhlIG9sZCBEU00gc3R1ZmYgdG8KPj4gc29tZXRoaW5n IGEgYml0IGxlc3MgZ2VuZXJpYyBhcyB3ZWxsLi4uCj4KPiBJIGNhbiByZW5hbWUgdGhlbSBhcyB3 ZWxsLiBCdXQgd2hhdCdzIHRoZSBuYW1pbmcgc2NoZW1lIHlvdSBwcmVmZXI/Cj4KPj4KPj4gPiAg I2RlZmluZSBJTlRFTF9EU01fRk5fUExBVEZPUk1fTVVYX0lORk8gMSAvKiBObyBhcmdzICovCj4+ ID4KPj4gPiAgc3RhdGljIGNvbnN0IGd1aWRfdCBpbnRlbF9kc21fZ3VpZCA9Cj4+ID4gICAgICAg R1VJRF9JTklUKDB4N2VkODczZDMsIDB4YzJkMCwgMHg0ZTRmLAo+PiA+ICAgICAgICAgICAgICAg ICAweGE4LCAweDU0LCAweDBmLCAweDEzLCAweDE3LCAweGIwLCAweDFjLCAweDJjKTsKPj4gPgo+ PiA+ICtzdGF0aWMgY29uc3QgZ3VpZF90IGludGVsX2RzbV9ndWlkMiA9Cj4+ID4gKyAgICAgR1VJ RF9JTklUKDB4M2U1YjQxYzYsIDB4ZWIxZCwgMHg0MjYwLAo+PiA+ICsgICAgICAgICAgICAgICAw eDlkLCAweDE1LCAweGM3LCAweDFmLCAweGJhLCAweGRhLCAweGU0LCAweDE0KTsKPj4gPiArCj4+ ID4gIHN0YXRpYyBjaGFyICppbnRlbF9kc21fcG9ydF9uYW1lKHU4IGlkKQo+PiA+ICB7Cj4+ID4g ICAgICAgc3dpdGNoIChpZCkgewo+PiA+IEBAIC0xNzYsNiArMTgxLDE5IEBAIHZvaWQgaW50ZWxf dW5yZWdpc3Rlcl9kc21faGFuZGxlcih2b2lkKQo+PiA+ICB7Cj4+ID4gIH0KPj4gPgo+PiA+ICt2 b2lkIGludGVsX2RzbV9lbmFibGVfbXV4KHN0cnVjdCBkcm1faTkxNV9wcml2YXRlICppOTE1KQo+ PiA+ICt7Cj4+ID4gKyAgICAgc3RydWN0IHBjaV9kZXYgKnBkZXYgPSBpOTE1LT5kcm0ucGRldjsK Pj4gPiArICAgICBhY3BpX2hhbmRsZSBkaGFuZGxlOwo+PiA+ICsKPj4gPiArICAgICBkaGFuZGxl ID0gQUNQSV9IQU5ETEUoJnBkZXYtPmRldik7Cj4+ID4gKyAgICAgaWYgKCFkaGFuZGxlKQo+PiA+ ICsgICAgICAgICAgICAgcmV0dXJuOwo+PiA+ICsKPj4gPiArICAgICBhY3BpX2V2YWx1YXRlX2Rz bShkaGFuZGxlLCAmaW50ZWxfZHNtX2d1aWQyLCBJTlRFTF9EU01fUkVWSVNJT05fSUQsCj4+ID4g KyAgICAgICAgICAgICAgICAgICAgICAgSU5URUxfRFNNX0ZOX1BMQVRGT1JNX01VWF9FTkFCTEUs IE5VTEwpOwo+PiA+ICt9Cj4+ID4gKwo+PiA+ICAvKgo+PiA+ICAgKiBBQ1BJIFNwZWNpZmljYXRp b24sIFJldmlzaW9uIDUuMCwgQXBwZW5kaXggQi4zLjIgX0RPRCAoRW51bWVyYXRlIEFsbCBEZXZp Y2VzCj4+ID4gICAqIEF0dGFjaGVkIHRvIHRoZSBEaXNwbGF5IEFkYXB0ZXIpLgo+PiA+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9kaXNwbGF5L2ludGVsX2FjcGkuaCBiL2RyaXZl cnMvZ3B1L2RybS9pOTE1L2Rpc3BsYXkvaW50ZWxfYWNwaS5oCj4+ID4gaW5kZXggZThiMDY4NjYx ZDIyLi5kZWYwMTNjZjYzMDggMTAwNjQ0Cj4+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUv ZGlzcGxheS9pbnRlbF9hY3BpLmgKPj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9kaXNw bGF5L2ludGVsX2FjcGkuaAo+PiA+IEBAIC0xMSwxMSArMTEsMTQgQEAgc3RydWN0IGRybV9pOTE1 X3ByaXZhdGU7Cj4+ID4gICNpZmRlZiBDT05GSUdfQUNQSQo+PiA+ICB2b2lkIGludGVsX3JlZ2lz dGVyX2RzbV9oYW5kbGVyKHZvaWQpOwo+PiA+ICB2b2lkIGludGVsX3VucmVnaXN0ZXJfZHNtX2hh bmRsZXIodm9pZCk7Cj4+ID4gK3ZvaWQgaW50ZWxfZHNtX2VuYWJsZV9tdXgoc3RydWN0IGRybV9p OTE1X3ByaXZhdGUgKmk5MTUpOwo+PiA+ICB2b2lkIGludGVsX2FjcGlfZGV2aWNlX2lkX3VwZGF0 ZShzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqaTkxNSk7Cj4+ID4gICNlbHNlCj4+ID4gIHN0YXRp YyBpbmxpbmUgdm9pZCBpbnRlbF9yZWdpc3Rlcl9kc21faGFuZGxlcih2b2lkKSB7IHJldHVybjsg fQo+PiA+ICBzdGF0aWMgaW5saW5lIHZvaWQgaW50ZWxfdW5yZWdpc3Rlcl9kc21faGFuZGxlcih2 b2lkKSB7IHJldHVybjsgfQo+PiA+ICBzdGF0aWMgaW5saW5lCj4+ID4gK3ZvaWQgaW50ZWxfZHNt X2VuYWJsZV9tdXgoc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmk5MTUpIHsgcmV0dXJuOyB9Cj4+ ID4gK3N0YXRpYyBpbmxpbmUKPj4gPiAgdm9pZCBpbnRlbF9hY3BpX2RldmljZV9pZF91cGRhdGUo c3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmk5MTUpIHsgcmV0dXJuOyB9Cj4+ID4gICNlbmRpZiAv KiBDT05GSUdfQUNQSSAqLwo+PiA+Cj4+ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9p OTE1L2Rpc3BsYXkvaW50ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvZGlzcGxh eS9pbnRlbF9kaXNwbGF5LmMKPj4gPiBpbmRleCBhMTBlMjYzODBlZjMuLmQ3OWRhZTM3MGIyMCAx MDA2NDQKPj4gPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9kaXNwbGF5L2ludGVsX2Rpc3Bs YXkuYwo+PiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2Rpc3BsYXkvaW50ZWxfZGlzcGxh eS5jCj4+ID4gQEAgLTExNDcyLDYgKzExNDcyLDggQEAgdm9pZCBpbnRlbF9tb2Rlc2V0X2luaXRf aHcoc3RydWN0IGRybV9pOTE1X3ByaXZhdGUgKmk5MTUpCj4+ID4gIHsKPj4gPiAgICAgICBzdHJ1 Y3QgaW50ZWxfY2RjbGtfc3RhdGUgKmNkY2xrX3N0YXRlOwo+PiA+Cj4+ID4gKyAgICAgaW50ZWxf ZHNtX2VuYWJsZV9tdXgoaTkxNSk7Cj4+ID4gKwo+Pgo+PiBUaGlzIHNob3VsZCBwcm9iYWJseSBi ZSBzb21ld2hlcmUgYXJvdW5kIHdoZXJlIHdlIGRvIGFsbCB0aGUgb3RoZXIKPj4gc2VtaSBBQ1BJ IHJlbGF0ZWQgaW5pdCAoT3BSZWdpb24vZXRjLikuCj4KPiBIbW0sIGJ1dCBKYW5pIHByZWZlcnMg dG8gcHV0IGl0IGluc2lkZSBpbnRlbF9tb2Rlc2V0XyooKSBoZWxwZXJzLiBCdXQKPiBJIGRvbid0 IHNlZSBhbnkgb3ByZWdpb24gcmVsYXRlZCBmdW5jdGlvbnMgYXJlIGJlaW5nIGNhbGxlZCBieQo+ IGludGVsX21vZGVzZXRfKigpIGhlbHBlcnMuIEFueSBzdWdnZXN0aW9uPwoKSSB0aGluayBJIG1h aW5seSB3YW50ZWQgaXQgaW4gaW50ZWxfZGlzcGxheS5jIGluc3RlYWQgb2YgYXQgdGhlIGRyaXZl cgp0b3AgbGV2ZWwuCgpCUiwKSmFuaS4KCgo+Cj4gS2FpLUhlbmcKPgo+Pgo+PiA+ICAgICAgIGlm ICghSEFTX0RJU1BMQVkoaTkxNSkpCj4+ID4gICAgICAgICAgICAgICByZXR1cm47Cj4+ID4KPj4g PiAtLQo+PiA+IDIuMzAuMgo+Pgo+PiAtLQo+PiBWaWxsZSBTeXJqw6Rsw6QKPj4gSW50ZWwKCi0t IApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBTb3VyY2UgR3JhcGhpY3MgQ2VudGVyCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5n IGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVk ZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLWdmeAo= 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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 4B226C433B4 for ; Mon, 17 May 2021 12:15:50 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 D280F61209 for ; Mon, 17 May 2021 12:15:49 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D280F61209 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=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 958F66E941; Mon, 17 May 2021 12:15:47 +0000 (UTC) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6D84F6E93F; Mon, 17 May 2021 12:15:46 +0000 (UTC) IronPort-SDR: bomVbLXnIfuIDRK+VIpw6mWDlDoK5mqILvT2NypzweKs60Wz27rPZ+fU/Q1CsZW264BjOBEk+B /nn6D+FzXhpg== X-IronPort-AV: E=McAfee;i="6200,9189,9986"; a="285979920" X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="285979920" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:45 -0700 IronPort-SDR: 7Ht9fR84yA4Rq8Y8w3Q2gXOI2V6j3777dycbyy09H98pxYDgPaGBbYFG66HJ8MS8XTvkcZkwbY lnxhCVCbYQ5Q== X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="472382788" Received: from sudhirk1-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.50.192]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:39 -0700 From: Jani Nikula To: Kai-Heng Feng , Ville =?utf-8?B?U3lyasOk?= =?utf-8?B?bMOk?= Subject: Re: [PATCH v3] drm/i915: Invoke another _DSM to enable MUX on HP Workstation laptops In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20210426152420.359402-1-kai.heng.feng@canonical.com> Date: Mon, 17 May 2021 15:15:36 +0300 Message-ID: <87fsylh7yv.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Karthik B S , David Airlie , open list , =?utf-8?Q?Jos=C3=A9?= Roberto de Souza , Chris Wilson , Manasi Navare , "open list:DRM DRIVERS" , Rodrigo Vivi , Dave Airlie , intel-gfx Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Fri, 14 May 2021, Kai-Heng Feng wrote: > On Wed, May 12, 2021 at 2:19 AM Ville Syrj=C3=A4l=C3=A4 > wrote: >> >> On Mon, Apr 26, 2021 at 11:24:10PM +0800, Kai-Heng Feng wrote: >> > On HP Fury G7 Workstations, graphics output is re-routed from Intel GFX >> > to discrete GFX after S3. This is not desirable, because userspace will >> > treat connected display as a new one, losing display settings. >> > >> > The expected behavior is to let discrete GFX drives all external >> > displays. >> > >> > The platform in question uses ACPI method \_SB.PCI0.HGME to enable MUX. >> > The method is inside the another _DSM, so add the _DSM and call it >> > accordingly. >> > >> > I also tested some MUX-less and iGPU only laptops with that _DSM, no >> > regression was found. >> > >> > v3: >> > - Remove BXT from names. >> > - Change the parameter type. >> > - Fold the function into intel_modeset_init_hw(). >> > >> > v2: >> > - Forward declare struct pci_dev. >> > >> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/3113 >> > References: https://lore.kernel.org/intel-gfx/1460040732-31417-4-git-s= end-email-animesh.manna@intel.com/ >> > Signed-off-by: Kai-Heng Feng >> > --- >> > drivers/gpu/drm/i915/display/intel_acpi.c | 18 ++++++++++++++++++ >> > drivers/gpu/drm/i915/display/intel_acpi.h | 3 +++ >> > drivers/gpu/drm/i915/display/intel_display.c | 2 ++ >> > 3 files changed, 23 insertions(+) >> > >> > diff --git a/drivers/gpu/drm/i915/display/intel_acpi.c b/drivers/gpu/d= rm/i915/display/intel_acpi.c >> > index 833d0c1be4f1..d008d3976261 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_acpi.c >> > +++ b/drivers/gpu/drm/i915/display/intel_acpi.c >> > @@ -13,12 +13,17 @@ >> > #include "intel_display_types.h" >> > >> > #define INTEL_DSM_REVISION_ID 1 /* For Calpella anyway... */ >> > +#define INTEL_DSM_FN_PLATFORM_MUX_ENABLE 0 /* No args */ >> >> This block of defines is for the other DSM. We don't want to >> mix these up. We also want to name it according to the spec, >> so something like GET_BIOS_DATA_FUNCS_SUPPORTED. Similarly >> for the intel_dsm_enable_mux() wrapper function. + it needs >> a comment to document that some BIOSes abuse it to do MUX >> initialization and whatnot. > > Will do. > > >> >> We should perhaps rename all the old DSM stuff to >> something a bit less generic as well... > > I can rename them as well. But what's the naming scheme you prefer? > >> >> > #define INTEL_DSM_FN_PLATFORM_MUX_INFO 1 /* No args */ >> > >> > static const guid_t intel_dsm_guid =3D >> > GUID_INIT(0x7ed873d3, 0xc2d0, 0x4e4f, >> > 0xa8, 0x54, 0x0f, 0x13, 0x17, 0xb0, 0x1c, 0x2c); >> > >> > +static const guid_t intel_dsm_guid2 =3D >> > + GUID_INIT(0x3e5b41c6, 0xeb1d, 0x4260, >> > + 0x9d, 0x15, 0xc7, 0x1f, 0xba, 0xda, 0xe4, 0x14); >> > + >> > static char *intel_dsm_port_name(u8 id) >> > { >> > switch (id) { >> > @@ -176,6 +181,19 @@ void intel_unregister_dsm_handler(void) >> > { >> > } >> > >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915) >> > +{ >> > + struct pci_dev *pdev =3D i915->drm.pdev; >> > + acpi_handle dhandle; >> > + >> > + dhandle =3D ACPI_HANDLE(&pdev->dev); >> > + if (!dhandle) >> > + return; >> > + >> > + acpi_evaluate_dsm(dhandle, &intel_dsm_guid2, INTEL_DSM_REVISION_= ID, >> > + INTEL_DSM_FN_PLATFORM_MUX_ENABLE, NULL); >> > +} >> > + >> > /* >> > * ACPI Specification, Revision 5.0, Appendix B.3.2 _DOD (Enumerate A= ll Devices >> > * Attached to the Display Adapter). >> > diff --git a/drivers/gpu/drm/i915/display/intel_acpi.h b/drivers/gpu/d= rm/i915/display/intel_acpi.h >> > index e8b068661d22..def013cf6308 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_acpi.h >> > +++ b/drivers/gpu/drm/i915/display/intel_acpi.h >> > @@ -11,11 +11,14 @@ struct drm_i915_private; >> > #ifdef CONFIG_ACPI >> > void intel_register_dsm_handler(void); >> > void intel_unregister_dsm_handler(void); >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915); >> > void intel_acpi_device_id_update(struct drm_i915_private *i915); >> > #else >> > static inline void intel_register_dsm_handler(void) { return; } >> > static inline void intel_unregister_dsm_handler(void) { return; } >> > static inline >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915) { return; } >> > +static inline >> > void intel_acpi_device_id_update(struct drm_i915_private *i915) { ret= urn; } >> > #endif /* CONFIG_ACPI */ >> > >> > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gp= u/drm/i915/display/intel_display.c >> > index a10e26380ef3..d79dae370b20 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_display.c >> > +++ b/drivers/gpu/drm/i915/display/intel_display.c >> > @@ -11472,6 +11472,8 @@ void intel_modeset_init_hw(struct drm_i915_pri= vate *i915) >> > { >> > struct intel_cdclk_state *cdclk_state; >> > >> > + intel_dsm_enable_mux(i915); >> > + >> >> This should probably be somewhere around where we do all the other >> semi ACPI related init (OpRegion/etc.). > > Hmm, but Jani prefers to put it inside intel_modeset_*() helpers. But > I don't see any opregion related functions are being called by > intel_modeset_*() helpers. Any suggestion? I think I mainly wanted it in intel_display.c instead of at the driver top level. BR, Jani. > > Kai-Heng > >> >> > if (!HAS_DISPLAY(i915)) >> > return; >> > >> > -- >> > 2.30.2 >> >> -- >> Ville Syrj=C3=A4l=C3=A4 >> Intel --=20 Jani Nikula, Intel Open Source Graphics Center 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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 00926C433B4 for ; Mon, 17 May 2021 12:16:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D219A61209 for ; Mon, 17 May 2021 12:16:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236911AbhEQMRL convert rfc822-to-8bit (ORCPT ); Mon, 17 May 2021 08:17:11 -0400 Received: from mga18.intel.com ([134.134.136.126]:11399 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234754AbhEQMRF (ORCPT ); Mon, 17 May 2021 08:17:05 -0400 IronPort-SDR: A4TtZ6VucI87sMggHQJHXSgmWLICemkZQtcHUF2sa17PTXpmmPY/Zye/PVtujDChoJlxdBJBKC 9fOQiiPL6SFA== X-IronPort-AV: E=McAfee;i="6200,9189,9986"; a="187854159" X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="187854159" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:45 -0700 IronPort-SDR: 7Ht9fR84yA4Rq8Y8w3Q2gXOI2V6j3777dycbyy09H98pxYDgPaGBbYFG66HJ8MS8XTvkcZkwbY lnxhCVCbYQ5Q== X-IronPort-AV: E=Sophos;i="5.82,307,1613462400"; d="scan'208";a="472382788" Received: from sudhirk1-mobl1.ger.corp.intel.com (HELO localhost) ([10.252.50.192]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2021 05:15:39 -0700 From: Jani Nikula To: Kai-Heng Feng , Ville =?utf-8?B?U3lyasOk?= =?utf-8?B?bMOk?= Cc: Joonas Lahtinen , Rodrigo Vivi , David Airlie , Daniel Vetter , Takashi Iwai , Manasi Navare , =?utf-8?Q?Jos=C3=A9?= Roberto de Souza , Chris Wilson , Imre Deak , Dave Airlie , Maarten Lankhorst , Karthik B S , Matt Roper , intel-gfx , "open list\:DRM DRIVERS" , open list Subject: Re: [PATCH v3] drm/i915: Invoke another _DSM to enable MUX on HP Workstation laptops In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20210426152420.359402-1-kai.heng.feng@canonical.com> Date: Mon, 17 May 2021 15:15:36 +0300 Message-ID: <87fsylh7yv.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 14 May 2021, Kai-Heng Feng wrote: > On Wed, May 12, 2021 at 2:19 AM Ville Syrjälä > wrote: >> >> On Mon, Apr 26, 2021 at 11:24:10PM +0800, Kai-Heng Feng wrote: >> > On HP Fury G7 Workstations, graphics output is re-routed from Intel GFX >> > to discrete GFX after S3. This is not desirable, because userspace will >> > treat connected display as a new one, losing display settings. >> > >> > The expected behavior is to let discrete GFX drives all external >> > displays. >> > >> > The platform in question uses ACPI method \_SB.PCI0.HGME to enable MUX. >> > The method is inside the another _DSM, so add the _DSM and call it >> > accordingly. >> > >> > I also tested some MUX-less and iGPU only laptops with that _DSM, no >> > regression was found. >> > >> > v3: >> > - Remove BXT from names. >> > - Change the parameter type. >> > - Fold the function into intel_modeset_init_hw(). >> > >> > v2: >> > - Forward declare struct pci_dev. >> > >> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/3113 >> > References: https://lore.kernel.org/intel-gfx/1460040732-31417-4-git-send-email-animesh.manna@intel.com/ >> > Signed-off-by: Kai-Heng Feng >> > --- >> > drivers/gpu/drm/i915/display/intel_acpi.c | 18 ++++++++++++++++++ >> > drivers/gpu/drm/i915/display/intel_acpi.h | 3 +++ >> > drivers/gpu/drm/i915/display/intel_display.c | 2 ++ >> > 3 files changed, 23 insertions(+) >> > >> > diff --git a/drivers/gpu/drm/i915/display/intel_acpi.c b/drivers/gpu/drm/i915/display/intel_acpi.c >> > index 833d0c1be4f1..d008d3976261 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_acpi.c >> > +++ b/drivers/gpu/drm/i915/display/intel_acpi.c >> > @@ -13,12 +13,17 @@ >> > #include "intel_display_types.h" >> > >> > #define INTEL_DSM_REVISION_ID 1 /* For Calpella anyway... */ >> > +#define INTEL_DSM_FN_PLATFORM_MUX_ENABLE 0 /* No args */ >> >> This block of defines is for the other DSM. We don't want to >> mix these up. We also want to name it according to the spec, >> so something like GET_BIOS_DATA_FUNCS_SUPPORTED. Similarly >> for the intel_dsm_enable_mux() wrapper function. + it needs >> a comment to document that some BIOSes abuse it to do MUX >> initialization and whatnot. > > Will do. > > >> >> We should perhaps rename all the old DSM stuff to >> something a bit less generic as well... > > I can rename them as well. But what's the naming scheme you prefer? > >> >> > #define INTEL_DSM_FN_PLATFORM_MUX_INFO 1 /* No args */ >> > >> > static const guid_t intel_dsm_guid = >> > GUID_INIT(0x7ed873d3, 0xc2d0, 0x4e4f, >> > 0xa8, 0x54, 0x0f, 0x13, 0x17, 0xb0, 0x1c, 0x2c); >> > >> > +static const guid_t intel_dsm_guid2 = >> > + GUID_INIT(0x3e5b41c6, 0xeb1d, 0x4260, >> > + 0x9d, 0x15, 0xc7, 0x1f, 0xba, 0xda, 0xe4, 0x14); >> > + >> > static char *intel_dsm_port_name(u8 id) >> > { >> > switch (id) { >> > @@ -176,6 +181,19 @@ void intel_unregister_dsm_handler(void) >> > { >> > } >> > >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915) >> > +{ >> > + struct pci_dev *pdev = i915->drm.pdev; >> > + acpi_handle dhandle; >> > + >> > + dhandle = ACPI_HANDLE(&pdev->dev); >> > + if (!dhandle) >> > + return; >> > + >> > + acpi_evaluate_dsm(dhandle, &intel_dsm_guid2, INTEL_DSM_REVISION_ID, >> > + INTEL_DSM_FN_PLATFORM_MUX_ENABLE, NULL); >> > +} >> > + >> > /* >> > * ACPI Specification, Revision 5.0, Appendix B.3.2 _DOD (Enumerate All Devices >> > * Attached to the Display Adapter). >> > diff --git a/drivers/gpu/drm/i915/display/intel_acpi.h b/drivers/gpu/drm/i915/display/intel_acpi.h >> > index e8b068661d22..def013cf6308 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_acpi.h >> > +++ b/drivers/gpu/drm/i915/display/intel_acpi.h >> > @@ -11,11 +11,14 @@ struct drm_i915_private; >> > #ifdef CONFIG_ACPI >> > void intel_register_dsm_handler(void); >> > void intel_unregister_dsm_handler(void); >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915); >> > void intel_acpi_device_id_update(struct drm_i915_private *i915); >> > #else >> > static inline void intel_register_dsm_handler(void) { return; } >> > static inline void intel_unregister_dsm_handler(void) { return; } >> > static inline >> > +void intel_dsm_enable_mux(struct drm_i915_private *i915) { return; } >> > +static inline >> > void intel_acpi_device_id_update(struct drm_i915_private *i915) { return; } >> > #endif /* CONFIG_ACPI */ >> > >> > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c >> > index a10e26380ef3..d79dae370b20 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_display.c >> > +++ b/drivers/gpu/drm/i915/display/intel_display.c >> > @@ -11472,6 +11472,8 @@ void intel_modeset_init_hw(struct drm_i915_private *i915) >> > { >> > struct intel_cdclk_state *cdclk_state; >> > >> > + intel_dsm_enable_mux(i915); >> > + >> >> This should probably be somewhere around where we do all the other >> semi ACPI related init (OpRegion/etc.). > > Hmm, but Jani prefers to put it inside intel_modeset_*() helpers. But > I don't see any opregion related functions are being called by > intel_modeset_*() helpers. Any suggestion? I think I mainly wanted it in intel_display.c instead of at the driver top level. BR, Jani. > > Kai-Heng > >> >> > if (!HAS_DISPLAY(i915)) >> > return; >> > >> > -- >> > 2.30.2 >> >> -- >> Ville Syrjälä >> Intel -- Jani Nikula, Intel Open Source Graphics Center