From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: [PATCH v5 2/2] drm/i915: Allow "max bpc" property to limit
 pipe_bpp
Date: Thu, 6 Sep 2018 14:36:40 +0300
Message-ID: <20180906113640.GV5565@intel.com>
References: <20180905201200.29414-1-radhakrishna.sripada@intel.com>
 <20180905201200.29414-2-radhakrishna.sripada@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
Return-path: <dri-devel-bounces@lists.freedesktop.org>
Content-Disposition: inline
In-Reply-To: <20180905201200.29414-2-radhakrishna.sripada@intel.com>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel" <dri-devel-bounces@lists.freedesktop.org>
To: Radhakrishna Sripada <radhakrishna.sripada@intel.com>
Cc: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>, intel-gfx@lists.freedesktop.org, Kishore Kadiyala <kishore.kadiyala@intel.com>, Manasi Navare <manasi.d.navare@intel.com>, dri-devel@lists.freedesktop.org, Rodrigo Vivi <rodrigo.vivi@intel.com>
List-Id: dri-devel@lists.freedesktop.org

T24gV2VkLCBTZXAgMDUsIDIwMTggYXQgMDE6MTI6MDBQTSAtMDcwMCwgUmFkaGFrcmlzaG5hIFNy
aXBhZGEgd3JvdGU6Cj4gVXNlIHRoZSBuZXdseSBhZGRlZCAibWF4IGJwYyIgY29ubmVjdG9yIHBy
b3BlcnR5IHRvIGxpbWl0IHBpcGUgYnBwLgo+IAo+IFYzOiBVc2UgZHJtX2Nvbm5lY3Rvcl9zdGF0
ZSB0byBhY2Nlc3MgdGhlICJtYXggYnBjIiBwcm9wZXJ0eQo+IFY0OiBJbml0aWFsaXplIHRoZSBk
cm0gcHJvcGVydHksIGFkZCBzdXVwb3J0IHRvIERQKFZpbGxlKQo+IFY1OiBVc2UgdGhlIHByb3Bl
cnR5IGluIHRoZSBjb25uZWN0b3IgYW5kIGZpeCBDSSBmYWlsdXJlKFZpbGxlKQo+IAo+IENjOiBW
aWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+IENjOiBSb2Ry
aWdvIFZpdmkgPHJvZHJpZ28udml2aUBpbnRlbC5jb20+Cj4gQ2M6IEtpc2hvcmUgS2FkaXlhbGEg
PGtpc2hvcmUua2FkaXlhbGFAaW50ZWwuY29tPgo+IENjOiBNYW5hc2kgTmF2YXJlIDxtYW5hc2ku
ZC5uYXZhcmVAaW50ZWwuY29tPgo+IENjOiBTdGFuaXNsYXYgTGlzb3Zza2l5IDxzdGFuaXNsYXYu
bGlzb3Zza2l5QGludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBSYWRoYWtyaXNobmEgU3JpcGFk
YSA8cmFkaGFrcmlzaG5hLnNyaXBhZGFAaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2ZXJzL2dwdS9k
cm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgfCAzMSArKysrKysrKysrKysrKysrKysrKysrKysrKysr
KysrCj4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2RwLmMgICAgICB8ICAxICsKPiAgZHJp
dmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHJ2LmggICAgIHwgIDIgKysKPiAgZHJpdmVycy9ncHUv
ZHJtL2k5MTUvaW50ZWxfaGRtaS5jICAgIHwgIDcgKysrKysrKwo+ICBkcml2ZXJzL2dwdS9kcm0v
aTkxNS9pbnRlbF9tb2Rlcy5jICAgfCAyMCArKysrKysrKysrKysrKysrKysrKwo+ICA1IGZpbGVz
IGNoYW5nZWQsIDYxIGluc2VydGlvbnMoKykKPiAKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv
ZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlz
cGxheS5jCj4gaW5kZXggMWJkMTRjNjFkYWI1Li5hODkwYWFkZTA5NGMgMTAwNjQ0Cj4gLS0tIGEv
ZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gKysrIGIvZHJpdmVycy9ncHUv
ZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gQEAgLTEwNzg3LDYgKzEwNzg3LDM0IEBAIGNvbm5l
Y3RlZF9zaW5rX2NvbXB1dGVfYnBwKHN0cnVjdCBpbnRlbF9jb25uZWN0b3IgKmNvbm5lY3RvciwK
PiAgCX0KPiAgfQo+ICAKPiArc3RhdGljIHZvaWQKPiArY29ubmVjdGVkX3NpbmtfbWF4X2JwcChz
dHJ1Y3QgZHJtX2Nvbm5lY3Rvcl9zdGF0ZSAqY29ubl9zdGF0ZSwKPiArCQkJICAgICBzdHJ1Y3Qg
aW50ZWxfY3J0Y19zdGF0ZSAqcGlwZV9jb25maWcpCj4gK3sKPiArCXN3aXRjaCAoY29ubl9zdGF0
ZS0+bWF4X2JwYykgewo+ICsJY2FzZSA4Ogo+ICsJY2FzZSA5Ogo+ICsJCXBpcGVfY29uZmlnLT5w
aXBlX2JwcCA9IDgqMzsKPiArCQlicmVhazsKPiArCWNhc2UgMTA6Cj4gKwljYXNlIDExOgo+ICsJ
CXBpcGVfY29uZmlnLT5waXBlX2JwcCA9IDEwKjM7Cj4gKwkJYnJlYWs7Cj4gKwljYXNlIDEyOgo+
ICsJY2FzZSAxMzoKPiArCWNhc2UgMTQ6Cj4gKwljYXNlIDE1Ogo+ICsJCXBpcGVfY29uZmlnLT5w
aXBlX2JwcCA9IDEyKjM7Cj4gKwkJYnJlYWs7Cj4gKwljYXNlIDE2Ogo+ICsJCXBpcGVfY29uZmln
LT5waXBlX2JwcCA9IDE2KjM7Cj4gKwkJYnJlYWs7Cj4gKwlkZWZhdWx0Ogo+ICsJCWJyZWFrOwo+
ICsJfQo+ICsJRFJNX0RFQlVHX0tNUygiTGltaXRpbmcgZGlzcGxheSBicHAgdG8gJWRcbiIsIHBp
cGVfY29uZmlnLT5waXBlX2JwcCk7Cj4gK30KPiArCj4gIHN0YXRpYyBpbnQKPiAgY29tcHV0ZV9i
YXNlbGluZV9waXBlX2JwcChzdHJ1Y3QgaW50ZWxfY3J0YyAqY3J0YywKPiAgCQkJICBzdHJ1Y3Qg
aW50ZWxfY3J0Y19zdGF0ZSAqcGlwZV9jb25maWcpCj4gQEAgLTEwODE1LDYgKzEwODQzLDkgQEAg
Y29tcHV0ZV9iYXNlbGluZV9waXBlX2JwcChzdHJ1Y3QgaW50ZWxfY3J0YyAqY3J0YywKPiAgCQlp
ZiAoY29ubmVjdG9yX3N0YXRlLT5jcnRjICE9ICZjcnRjLT5iYXNlKQo+ICAJCQljb250aW51ZTsK
PiAgCj4gKwkJaWYgKGNvbm5lY3Rvcl9zdGF0ZS0+bWF4X2JwYykKPiArCQkJY29ubmVjdGVkX3Np
bmtfbWF4X2JwcChjb25uZWN0b3Jfc3RhdGUsIHBpcGVfY29uZmlnKTsKPiArCj4gIAkJY29ubmVj
dGVkX3NpbmtfY29tcHV0ZV9icHAodG9faW50ZWxfY29ubmVjdG9yKGNvbm5lY3RvciksCj4gIAkJ
CQkJICAgcGlwZV9jb25maWcpOwo+ICAJfQo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0v
aTkxNS9pbnRlbF9kcC5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHAuYwo+IGluZGV4
IDQzNmMyMmRlMzNiNi4uMzk1NTc0NWE0ZDlmIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry
bS9pOTE1L2ludGVsX2RwLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcC5j
Cj4gQEAgLTU3MTksNiArNTcxOSw3IEBAIGludGVsX2RwX2FkZF9wcm9wZXJ0aWVzKHN0cnVjdCBp
bnRlbF9kcCAqaW50ZWxfZHAsIHN0cnVjdCBkcm1fY29ubmVjdG9yICpjb25uZWN0Cj4gIAkJaW50
ZWxfYXR0YWNoX2ZvcmNlX2F1ZGlvX3Byb3BlcnR5KGNvbm5lY3Rvcik7Cj4gIAo+ICAJaW50ZWxf
YXR0YWNoX2Jyb2FkY2FzdF9yZ2JfcHJvcGVydHkoY29ubmVjdG9yKTsKPiArCWludGVsX2F0dGFj
aF9tYXhfYnBjX3Byb3BlcnR5KGNvbm5lY3RvciwgOCwgMTYpOwoKSUlSQyBnbWNoIHBsYXRmb3Jt
cyBjYW4ndCBkbyBtb3JlIHRoYW4gMTBicGMsIGFuZCB0aGUgcmVzdAphcmUgbGltaXRlZCB0byAx
MmJwYy4KCj4gIAo+ICAJaWYgKGludGVsX2RwX2lzX2VkcChpbnRlbF9kcCkpIHsKPiAgCQl1MzIg
YWxsb3dlZF9zY2FsZXJzOwo+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRl
bF9kcnYuaCBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rydi5oCj4gaW5kZXggZjU3MzEy
MTUyMTBhLi5iM2M3MDNkYWNjOTIgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUv
aW50ZWxfZHJ2LmgKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kcnYuaAo+IEBA
IC0xODY5LDYgKzE4NjksOCBAQCBpbnQgaW50ZWxfZGRjX2dldF9tb2RlcyhzdHJ1Y3QgZHJtX2Nv
bm5lY3RvciAqYywgc3RydWN0IGkyY19hZGFwdGVyICphZGFwdGVyKTsKPiAgdm9pZCBpbnRlbF9h
dHRhY2hfZm9yY2VfYXVkaW9fcHJvcGVydHkoc3RydWN0IGRybV9jb25uZWN0b3IgKmNvbm5lY3Rv
cik7Cj4gIHZvaWQgaW50ZWxfYXR0YWNoX2Jyb2FkY2FzdF9yZ2JfcHJvcGVydHkoc3RydWN0IGRy
bV9jb25uZWN0b3IgKmNvbm5lY3Rvcik7Cj4gIHZvaWQgaW50ZWxfYXR0YWNoX2FzcGVjdF9yYXRp
b19wcm9wZXJ0eShzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9yKTsKPiArdm9pZCBpbnRl
bF9hdHRhY2hfbWF4X2JwY19wcm9wZXJ0eShzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29ubmVjdG9y
LCBpbnQgbWluLCBpbnQKPiArCQkJCSAgIG1heCk7Cj4gIAo+ICAKPiAgLyogaW50ZWxfb3Zlcmxh
eS5jICovCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hkbWkuYyBi
L2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hkbWkuYwo+IGluZGV4IGEyZGFiMGI2YmRlNi4u
ZTY0OWJiZjA3NjQyIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hk
bWkuYwo+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2hkbWkuYwo+IEBAIC0yMTA5
LDExICsyMTA5LDE4IEBAIHN0YXRpYyBjb25zdCBzdHJ1Y3QgZHJtX2VuY29kZXJfZnVuY3MgaW50
ZWxfaGRtaV9lbmNfZnVuY3MgPSB7Cj4gIHN0YXRpYyB2b2lkCj4gIGludGVsX2hkbWlfYWRkX3By
b3BlcnRpZXMoc3RydWN0IGludGVsX2hkbWkgKmludGVsX2hkbWksIHN0cnVjdCBkcm1fY29ubmVj
dG9yICpjb25uZWN0b3IpCj4gIHsKPiArCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJp
diA9IHRvX2k5MTUoY29ubmVjdG9yLT5kZXYpOwo+ICsKPiAgCWludGVsX2F0dGFjaF9mb3JjZV9h
dWRpb19wcm9wZXJ0eShjb25uZWN0b3IpOwo+ICAJaW50ZWxfYXR0YWNoX2Jyb2FkY2FzdF9yZ2Jf
cHJvcGVydHkoY29ubmVjdG9yKTsKPiAgCWludGVsX2F0dGFjaF9hc3BlY3RfcmF0aW9fcHJvcGVy
dHkoY29ubmVjdG9yKTsKPiAgCWRybV9jb25uZWN0b3JfYXR0YWNoX2NvbnRlbnRfdHlwZV9wcm9w
ZXJ0eShjb25uZWN0b3IpOwo+ICAJY29ubmVjdG9yLT5zdGF0ZS0+cGljdHVyZV9hc3BlY3RfcmF0
aW8gPSBIRE1JX1BJQ1RVUkVfQVNQRUNUX05PTkU7Cj4gKwo+ICsJaWYgKEhBU19HTUNIX0RJU1BM
QVkoZGV2X3ByaXYpKQo+ICsJCWludGVsX2F0dGFjaF9tYXhfYnBjX3Byb3BlcnR5KGNvbm5lY3Rv
ciwgOCwgOCk7CgpOb3Qgc3VyZSBleHBvc2luZyB0aGUgcHJvcCBtYWtlcyBtdWNoIHNlbnNlIHdo
ZW4geW91IGNhbid0IG1vZGlmeSBpdC4KCj4gKwllbHNlCj4gKwkJaW50ZWxfYXR0YWNoX21heF9i
cGNfcHJvcGVydHkoY29ubmVjdG9yLCA4LCAxMik7Cj4gIH0KPiAgCj4gIC8qCj4gZGlmZiAtLWdp
dCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX21vZGVzLmMgYi9kcml2ZXJzL2dwdS9kcm0v
aTkxNS9pbnRlbF9tb2Rlcy5jCj4gaW5kZXggY2E0NGJmMzY4ZTI0Li4xMmYxMjM4YmFkOGEgMTAw
NjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfbW9kZXMuYwo+ICsrKyBiL2Ry
aXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX21vZGVzLmMKPiBAQCAtMTMzLDMgKzEzMywyMyBAQCBp
bnRlbF9hdHRhY2hfYXNwZWN0X3JhdGlvX3Byb3BlcnR5KHN0cnVjdCBkcm1fY29ubmVjdG9yICpj
b25uZWN0b3IpCj4gIAkJCWNvbm5lY3Rvci0+ZGV2LT5tb2RlX2NvbmZpZy5hc3BlY3RfcmF0aW9f
cHJvcGVydHksCj4gIAkJCURSTV9NT0RFX1BJQ1RVUkVfQVNQRUNUX05PTkUpOwo+ICB9Cj4gKwo+
ICt2b2lkCj4gK2ludGVsX2F0dGFjaF9tYXhfYnBjX3Byb3BlcnR5KHN0cnVjdCBkcm1fY29ubmVj
dG9yICpjb25uZWN0b3IsIGludCBtaW4sIGludAo+ICsJCQkgICAgICAgbWF4KQo+ICt7Cj4gKwlz
dHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gY29ubmVjdG9yLT5kZXY7Cj4gKwlzdHJ1Y3QgZHJtX3By
b3BlcnR5ICpwcm9wOwo+ICsKPiArCXByb3AgPSBjb25uZWN0b3ItPm1heF9icGNfcHJvcGVydHk7
Cj4gKwlpZiAocHJvcCA9PSBOVUxMKSB7Cj4gKwkJcHJvcCA9IGRybV9wcm9wZXJ0eV9jcmVhdGVf
cmFuZ2UoZGV2LCAwLCAibWF4IGJwYyIsIG1pbiwgbWF4KTsKPiArCQlpZiAocHJvcCA9PSBOVUxM
KQo+ICsJCQlyZXR1cm47Cj4gKwo+ICsJCWNvbm5lY3Rvci0+bWF4X2JwY19wcm9wZXJ0eSA9IHBy
b3A7Cj4gKwl9Cj4gKwo+ICsJZHJtX29iamVjdF9hdHRhY2hfcHJvcGVydHkoJmNvbm5lY3Rvci0+
YmFzZSwgcHJvcCwgbWF4KTsKPiArCWNvbm5lY3Rvci0+c3RhdGUtPm1heF9icGMgPSBtYXg7Cj4g
K30KClRoZXJlJ3Mgbm90aGluZyBpOTE1IHNwZWNpZmljIGluIHRoaXMgZnVuY3Rpb24sIHNvIG1p
Z2h0IGFzIHdlbGwgbW92ZSBpdAp0byB0aGUgY29yZS4KCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50
ZWwKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRl
dmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8v
bGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==