From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Lisovskiy, Stanislav" Subject: Re: [PATCH v12 2/2] drm/i915: Allow "max bpc" property to limit pipe_bpp Date: Thu, 11 Oct 2018 07:55:15 +0000 Message-ID: References: <20181011001252.19370-1-radhakrishna.sripada@intel.com> <20181011001252.19370-2-radhakrishna.sripada@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20181011001252.19370-2-radhakrishna.sripada@intel.com> Content-Language: en-US Content-ID: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Sripada, Radhakrishna" , "intel-gfx@lists.freedesktop.org" Cc: "daniel.vetter@ffwll.ch" , "Kadiyala, Kishore" , "Navare, Manasi D" , "dri-devel@lists.freedesktop.org" , "Vivi, Rodrigo" List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCAyMDE4LTEwLTEwIGF0IDE3OjEyIC0wNzAwLCBSYWRoYWtyaXNobmEgU3JpcGFkYSB3 cm90ZToNCj4gVXNlIHRoZSBuZXdseSBhZGRlZCAibWF4IGJwYyIgY29ubmVjdG9yIHByb3BlcnR5 IHRvIGxpbWl0IHBpcGUgYnBwLg0KPiANCj4gVjM6IFVzZSBkcm1fY29ubmVjdG9yX3N0YXRlIHRv IGFjY2VzcyB0aGUgIm1heCBicGMiIHByb3BlcnR5DQo+IFY0OiBJbml0aWFsaXplIHRoZSBkcm0g cHJvcGVydHksIGFkZCBzdXVwb3J0IHRvIERQKFZpbGxlKQ0KPiBWNTogVXNlIHRoZSBwcm9wZXJ0 eSBpbiB0aGUgY29ubmVjdG9yIGFuZCBmaXggQ0kgZmFpbHVyZShWaWxsZSkNCj4gVjY6IFVzZSB0 aGUgY29yZSBmdW5jdGlvbiB0byBhdHRhY2ggbWF4X2JwYyBwcm9wZXJ0eSwgcmVtb3ZlIHRoZQ0K PiByZWR1bmRhbnQNCj4gICAgIGNsYW1waW5nIG9mIHBpcGUgYnBwIGJhc2VkIG9uIGNvbm5lY3Rv ciBpbmZvDQo+IFY3OiBGaXggQ2hlY2twYXRjaCB3YXJuaW5ncw0KPiBWOTogQ2xlYW51cCBjb25u ZWN0ZWRfc2lua19tYXhfYnBwIGFuZCBmaXggaW5pdGlhbCB2YWx1ZSBpbiBEUChWaWxsZSkNCj4g VjEyOiBGaXggZGVidWcgbWVzc2FnZShWaWxsZSkNCj4gDQo+IENjOiBWaWxsZSBTeXJqw6Rsw6Qg PHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPg0KPiBDYzogRGFuaWVsIFZldHRlciA8ZGFu aWVsLnZldHRlckBmZndsbC5jaD4NCj4gQ2M6IFJvZHJpZ28gVml2aSA8cm9kcmlnby52aXZpQGlu dGVsLmNvbT4NCj4gQ2M6IEtpc2hvcmUgS2FkaXlhbGEgPGtpc2hvcmUua2FkaXlhbGFAaW50ZWwu Y29tPg0KPiBDYzogTWFuYXNpIE5hdmFyZSA8bWFuYXNpLmQubmF2YXJlQGludGVsLmNvbT4NCj4g Q2M6IFN0YW5pc2xhdiBMaXNvdnNraXkgPHN0YW5pc2xhdi5saXNvdnNraXlAaW50ZWwuY29tPg0K PiBTaWduZWQtb2ZmLWJ5OiBSYWRoYWtyaXNobmEgU3JpcGFkYSA8cmFkaGFrcmlzaG5hLnNyaXBh ZGFAaW50ZWwuY29tPg0KPiAtLS0NCj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3Bs YXkuYyB8IDQ4ICsrKysrKysrKysrKysrKysrKysrKy0tLS0tDQo+IC0tLS0tLS0tLS0NCj4gIGRy aXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMgICAgICB8ICA0ICsrKw0KPiAgZHJpdmVycy9n cHUvZHJtL2k5MTUvaW50ZWxfaGRtaS5jICAgIHwgIDUgKysrKw0KPiAgMyBmaWxlcyBjaGFuZ2Vk LCAzNyBpbnNlcnRpb25zKCspLCAyMCBkZWxldGlvbnMoLSkNCj4gDQo+IGRpZmYgLS1naXQgYS9k cml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMNCj4gYi9kcml2ZXJzL2dwdS9kcm0v aTkxNS9pbnRlbF9kaXNwbGF5LmMNCj4gaW5kZXggYTE0NWVmYmE5MTU3Li5hNTk3YzQ0MTBmNWQg MTAwNjQ0DQo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3BsYXkuYw0KPiAr KysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMNCj4gQEAgLTEwODY5LDMw ICsxMDg2OSwzOCBAQCBzdGF0aWMgdm9pZA0KPiBpbnRlbF9tb2Rlc2V0X3VwZGF0ZV9jb25uZWN0 b3JfYXRvbWljX3N0YXRlKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpDQo+ICAJZHJtX2Nvbm5lY3Rv cl9saXN0X2l0ZXJfZW5kKCZjb25uX2l0ZXIpOw0KPiAgfQ0KPiAgDQo+IC1zdGF0aWMgdm9pZA0K PiAtY29ubmVjdGVkX3NpbmtfY29tcHV0ZV9icHAoc3RydWN0IGludGVsX2Nvbm5lY3RvciAqY29u bmVjdG9yLA0KPiAtCQkJICAgc3RydWN0IGludGVsX2NydGNfc3RhdGUgKnBpcGVfY29uZmlnKQ0K PiArc3RhdGljIGludA0KPiArY29ubmVjdGVkX3NpbmtfbWF4X2JwcChjb25zdCBzdHJ1Y3QgZHJt X2Nvbm5lY3Rvcl9zdGF0ZSAqY29ubl9zdGF0ZSwNCj4gKwkJICAgICAgIHN0cnVjdCBpbnRlbF9j cnRjX3N0YXRlICpwaXBlX2NvbmZpZykNCj4gIHsNCj4gLQljb25zdCBzdHJ1Y3QgZHJtX2Rpc3Bs YXlfaW5mbyAqaW5mbyA9ICZjb25uZWN0b3ItDQo+ID5iYXNlLmRpc3BsYXlfaW5mbzsNCj4gLQlp bnQgYnBwID0gcGlwZV9jb25maWctPnBpcGVfYnBwOw0KPiArCWludCBicHA7DQo+ICsJc3RydWN0 IGRybV9kaXNwbGF5X2luZm8gKmluZm8gPSAmY29ubl9zdGF0ZS0+Y29ubmVjdG9yLQ0KPiA+ZGlz cGxheV9pbmZvOw0KPiAgDQo+IC0JRFJNX0RFQlVHX0tNUygiW0NPTk5FQ1RPUjolZDolc10gY2hl Y2tpbmcgZm9yIHNpbmsgYnBwDQo+IGNvbnN0cmFpbnNcbiIsDQo+IC0JCSAgICAgIGNvbm5lY3Rv ci0+YmFzZS5iYXNlLmlkLA0KPiAtCQkgICAgICBjb25uZWN0b3ItPmJhc2UubmFtZSk7DQo+ICsJ YnBwID0gbWluKHBpcGVfY29uZmlnLT5waXBlX2JwcCwgY29ubl9zdGF0ZS0+bWF4X2JwYyAqIDMp Ow0KPiAgDQo+IC0JLyogRG9uJ3QgdXNlIGFuIGludmFsaWQgRURJRCBicGMgdmFsdWUgKi8NCj4g LQlpZiAoaW5mby0+YnBjICE9IDAgJiYgaW5mby0+YnBjICogMyA8IGJwcCkgew0KPiAtCQlEUk1f REVCVUdfS01TKCJjbGFtcGluZyBkaXNwbGF5IGJwcCAod2FzICVkKSB0byBFRElEDQo+IHJlcG9y dGVkIG1heCBvZiAlZFxuIiwNCj4gLQkJCSAgICAgIGJwcCwgaW5mby0+YnBjICogMyk7DQo+IC0J CXBpcGVfY29uZmlnLT5waXBlX2JwcCA9IGluZm8tPmJwYyAqIDM7DQo+ICsJc3dpdGNoIChjb25u X3N0YXRlLT5tYXhfYnBjKSB7DQo+ICsJY2FzZSA2IC4uLiA3Og0KPiArCQlwaXBlX2NvbmZpZy0+ cGlwZV9icHAgPSA2ICogMzsNCj4gKwljYXNlIDggLi4uIDk6DQo+ICsJCXBpcGVfY29uZmlnLT5w aXBlX2JwcCA9IDggKiAzOw0KPiArCQlicmVhazsNCj4gKwljYXNlIDEwIC4uLiAxMToNCj4gKwkJ cGlwZV9jb25maWctPnBpcGVfYnBwID0gMTAgKiAzOw0KPiArCQlicmVhazsNCj4gKwljYXNlIDEy Og0KPiArCQlwaXBlX2NvbmZpZy0+cGlwZV9icHAgPSAxMiAqIDM7DQo+ICsJCWJyZWFrOw0KPiAr CWRlZmF1bHQ6DQo+ICsJCXJldHVybiAtRUlOVkFMOw0KPiAgCX0NCg0KV2h5IGRvIHdlIG5lZWQg dGhpcyBhc3NpZ25tZW50IGhlcmU/IEkgbWVhbiB5b3UgaGF2ZSBhbHJlYWR5IGRldGVybWluZWQN CnRoZSBtaW5pbXVtIHZhbHVlIHdoaWNoIGlzIHN0b3JlZCBpbiBicHAsIHdoaWNoIHNob3VsZCBi ZSB1c2VkIGFuZCB0aGVuDQpjaGVjayBhZ2FpbiwgdGhhdCBpdCBpcyBub3QgZXF1YWwgYW5kIG9u bHkgdGhlbiB1c2UgaXQuIA0KQ291bGQgaXQgYmUganVzdCBhcyBzaW1wbGUgYXMgdGhpcyhvciBJ IHNpbXBseSBtaXN1bmRlcnN0YW5kIHNvbWV0aGluZw0KaGVyZSk6DQoNCnN3aXRjaCAoY29ubl9z dGF0ZS0+bWF4X2JwYykgew0KY2FzZSA2IC4uLiA3Og0KCWJwcCA9IDYgKiAzOw0KLi4uDQpkZWZh dWx0Og0KCXJldHVybiAtRUlOVkFMOw0KfQ0KDQpzbyBoZXJlIHdlIHNldCBicHAoaW5zdGVhZCBv ZiBwaXBlX2JwcCkgdG8gY29ycmVjdCB2YWx1ZSBvciByZXR1cm4NCkVJTlZBTC4NCkFuZCB0aGVu IHdlIHNpbXBseSBhc3NpZ24gcGlwZV9icHAgdG8gbWluaW11bSBvZiB0aG9zZSBhbmQgcmV0dXJu Og0KDQpwaXBlX2JwcCA9IG1pbihwaXBlX2NvbmZpZy0+cGlwZV9icHAsIGJwcCk7DQoNCnJldHVy biAwOw0KDQpBbHNvIHRoYXQgd2F5IGdldCBicHAgdmFsdWVzIGFsc28gY2hlY2sgdG8gY29ycmVz cG9uZCB0byB0aGUgcmFuZ2VzDQpnaXZlbiBpbiBjYXNlcyBsYWJlbHMsIHdoaWxlIGluaXRpYWwg ZXhwcmVzc2lvbiANCm1pbihwaXBlX2NvbmZpZy0+cGlwZV9icHAsIGNvbm5fc3RhdGUtPm1heF9i cGMgKiAzKSBjb3VsZCBwb3NzaWJseSBnaXZlDQo3ICogMyBidXQgbm90IDYgKiAzKGFzIHNwZWNp ZmllZCBpbiAiY2FzZSA2IC4uIDciKSBmb3IgDQpjb25uX3N0YXRlLT5tYXhfYnBjID0gNy4NCg0K U28gdGhlbiB0aGVyZSBpcyBubyBuZWVkIGluIG5leHQgYWRkaXRpb25hbCBhc3NpZ25tZW50IGFu ZCBjaGVjazoNCg0KPiAgDQo+IC0JLyogQ2xhbXAgYnBwIHRvIDggb24gc2NyZWVucyB3aXRob3V0 IEVESUQgMS40ICovDQo+IC0JaWYgKGluZm8tPmJwYyA9PSAwICYmIGJwcCA+IDI0KSB7DQo+IC0J CURSTV9ERUJVR19LTVMoImNsYW1waW5nIGRpc3BsYXkgYnBwICh3YXMgJWQpIHRvDQo+IGRlZmF1 bHQgbGltaXQgb2YgMjRcbiIsDQo+IC0JCQkgICAgICBicHApOw0KPiAtCQlwaXBlX2NvbmZpZy0+ cGlwZV9icHAgPSAyNDsNCj4gKwlpZiAoYnBwICE9IHBpcGVfY29uZmlnLT5waXBlX2JwcCkgew0K PiArCQlEUk1fREVCVUdfS01TKCJMaW1pdGluZyBkaXNwbGF5IGJwcCB0byAlZCBpbnN0ZWFkIG9m DQo+IEVkaWQgYnBwICINCj4gKwkJCSAgICAgICIlZCwgcmVxdWVzdGVkIGJwcCAlZFxuIiwgYnBw LCAzICoNCj4gaW5mby0+YnBjLA0KPiArCQkJICAgICAgMyAqIGNvbm5fc3RhdGUtPm1heF9yZXF1 ZXN0ZWRfYnBjKTsNCj4gKwkJcGlwZV9jb25maWctPnBpcGVfYnBwID0gYnBwOw0KPiAgCX0NCj4g KwlyZXR1cm4gMDsNCj4gIH0NCj4gIA0KPiAgc3RhdGljIGludA0KPiBAQCAtMTA5MjMsOCArMTA5 MzEsOCBAQCBjb21wdXRlX2Jhc2VsaW5lX3BpcGVfYnBwKHN0cnVjdCBpbnRlbF9jcnRjDQo+ICpj cnRjLA0KPiAgCQlpZiAoY29ubmVjdG9yX3N0YXRlLT5jcnRjICE9ICZjcnRjLT5iYXNlKQ0KPiAg CQkJY29udGludWU7DQo+ICANCj4gLQkJY29ubmVjdGVkX3NpbmtfY29tcHV0ZV9icHAodG9faW50 ZWxfY29ubmVjdG9yKGNvbm5lYw0KPiB0b3IpLA0KPiAtCQkJCQkgICBwaXBlX2NvbmZpZyk7DQo+ ICsJCWlmIChjb25uZWN0ZWRfc2lua19tYXhfYnBwKGNvbm5lY3Rvcl9zdGF0ZSwNCj4gcGlwZV9j b25maWcpIDwgMCkNCj4gKwkJCXJldHVybiAtRUlOVkFMOw0KPiAgCX0NCj4gIA0KPiAgCXJldHVy biBicHA7DQo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jDQo+ IGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHAuYw0KPiBpbmRleCAwODU1Yjk3ODVmN2Iu Ljg2M2M4ODMyZmU4YiAxMDA2NDQNCj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxf ZHAuYw0KPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5jDQo+IEBAIC01NzA1 LDYgKzU3MDUsMTAgQEAgaW50ZWxfZHBfYWRkX3Byb3BlcnRpZXMoc3RydWN0IGludGVsX2RwDQo+ ICppbnRlbF9kcCwgc3RydWN0IGRybV9jb25uZWN0b3IgKmNvbm5lY3QNCj4gIAkJaW50ZWxfYXR0 YWNoX2ZvcmNlX2F1ZGlvX3Byb3BlcnR5KGNvbm5lY3Rvcik7DQo+ICANCj4gIAlpbnRlbF9hdHRh Y2hfYnJvYWRjYXN0X3JnYl9wcm9wZXJ0eShjb25uZWN0b3IpOw0KPiArCWlmIChIQVNfR01DSF9E SVNQTEFZKGRldl9wcml2KSkNCj4gKwkJZHJtX2Nvbm5lY3Rvcl9hdHRhY2hfbWF4X2JwY19wcm9w ZXJ0eShjb25uZWN0b3IsIDYsDQo+IDEwKTsNCj4gKwllbHNlIGlmIChJTlRFTF9HRU4oZGV2X3By aXYpID49IDUpDQo+ICsJCWRybV9jb25uZWN0b3JfYXR0YWNoX21heF9icGNfcHJvcGVydHkoY29u bmVjdG9yLCA2LA0KPiAxMik7DQo+ICANCj4gIAlpZiAoaW50ZWxfZHBfaXNfZWRwKGludGVsX2Rw KSkgew0KPiAgCQl1MzIgYWxsb3dlZF9zY2FsZXJzOw0KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9n cHUvZHJtL2k5MTUvaW50ZWxfaGRtaS5jDQo+IGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxf aGRtaS5jDQo+IGluZGV4IDJjNTNlZmM0NjNlNi4uMzE1OGFiMDg1YTMwIDEwMDY0NA0KPiAtLS0g YS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9oZG1pLmMNCj4gKysrIGIvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfaGRtaS5jDQo+IEBAIC0yMTAzLDExICsyMTAzLDE2IEBAIHN0YXRpYyBj b25zdCBzdHJ1Y3QgZHJtX2VuY29kZXJfZnVuY3MNCj4gaW50ZWxfaGRtaV9lbmNfZnVuY3MgPSB7 DQo+ICBzdGF0aWMgdm9pZA0KPiAgaW50ZWxfaGRtaV9hZGRfcHJvcGVydGllcyhzdHJ1Y3QgaW50 ZWxfaGRtaSAqaW50ZWxfaGRtaSwgc3RydWN0DQo+IGRybV9jb25uZWN0b3IgKmNvbm5lY3RvcikN Cj4gIHsNCj4gKwlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYgPSB0b19pOTE1KGNv bm5lY3Rvci0+ZGV2KTsNCj4gKw0KPiAgCWludGVsX2F0dGFjaF9mb3JjZV9hdWRpb19wcm9wZXJ0 eShjb25uZWN0b3IpOw0KPiAgCWludGVsX2F0dGFjaF9icm9hZGNhc3RfcmdiX3Byb3BlcnR5KGNv bm5lY3Rvcik7DQo+ICAJaW50ZWxfYXR0YWNoX2FzcGVjdF9yYXRpb19wcm9wZXJ0eShjb25uZWN0 b3IpOw0KPiAgCWRybV9jb25uZWN0b3JfYXR0YWNoX2NvbnRlbnRfdHlwZV9wcm9wZXJ0eShjb25u ZWN0b3IpOw0KPiAgCWNvbm5lY3Rvci0+c3RhdGUtPnBpY3R1cmVfYXNwZWN0X3JhdGlvID0NCj4g SERNSV9QSUNUVVJFX0FTUEVDVF9OT05FOw0KPiArDQo+ICsJaWYgKCFIQVNfR01DSF9ESVNQTEFZ KGRldl9wcml2KSkNCj4gKwkJZHJtX2Nvbm5lY3Rvcl9hdHRhY2hfbWF4X2JwY19wcm9wZXJ0eShj b25uZWN0b3IsIDgsDQo+IDEyKTsNCj4gIH0NCj4gIA0KPiAgLyoNCi0tIA0KQmVzdCBSZWdhcmRz LA0KDQpMaXNvdnNraXkgU3RhbmlzbGF2Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZy ZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2RyaS1kZXZlbAo=