From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/i915/dp: Increase cdclk when DP audio is enabled with 4 lanes and HBR2 Date: Thu, 13 Oct 2016 21:44:14 +0300 Message-ID: <20161013184414.GT4329@intel.com> References: <1476381859-4614-1-git-send-email-dhinakaran.pandiyan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <1476381859-4614-1-git-send-email-dhinakaran.pandiyan@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Dhinakaran Pandiyan Cc: Jani Nikula , intel-gfx@freedesktop.org, Libin Yang , Jeeja KP List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBPY3QgMTMsIDIwMTYgYXQgMTE6MDQ6MTlBTSAtMDcwMCwgRGhpbmFrYXJhbiBQYW5k aXlhbiB3cm90ZToKPiBBY2NvcmRpbmcgdG8gQlNwZWMsIGNkY2xrIGhhcyB0byBiZSBub3QgbGVz cyB0aGFuIDQzMiBNSHogd2l0aCBEUCBhdWRpbwo+IGVuYWJsZWQsIHBvcnQgd2lkdGggeDQsIGFu ZCBsaW5rIHJhdGUgSEJSMiAoNS40IEdIeikKPiAKPiBIYXZpbmcgYSBsb3dlciBjZGNsayB0cmln Z2VycyBwaXBlIHVuZGVycnVucywgd2hpY2ggdGhlbiBsZWFkIHRvIGRpc3BsYXlzCj4gY29udGlu dW91c2x5IGN5Y2xpbmcgb2ZmIGFuZCBvbi4gVGhpcyBpcyBlc3NlbnRpYWwgZm9yIERQIE1TVCBh dWRpbyBhcyB0aGUKPiBsaW5rIGlzIHRyYWluZWQgYXQgSEJSMiBhbmQgNCBsYW5lcyBieSBkZWZh dWx0Lgo+IAo+IFRoaXMgc2hvdWxkIGZpeCBodHRwczovL2J1Z3MuZnJlZWRlc2t0b3Aub3JnL3No b3dfYnVnLmNnaT9pZD05NzkwNwo+IAo+IFNpZ25lZC1vZmYtYnk6IERoaW5ha2FyYW4gUGFuZGl5 YW4gPGRoaW5ha2FyYW4ucGFuZGl5YW5AaW50ZWwuY29tPgo+IC0tLQo+ICBkcml2ZXJzL2dwdS9k cm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgfCA0NyArKysrKysrKysrKysrKysrKysrKysrKysrKysr KysrKystLS0KPiAgMSBmaWxlIGNoYW5nZWQsIDQzIGluc2VydGlvbnMoKyksIDQgZGVsZXRpb25z KC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3BsYXku YyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2Rpc3BsYXkuYwo+IGluZGV4IGNmY2IwM2Yu LjZhMDUxODMgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxh eS5jCj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gQEAgLTY2 MDMsMTQgKzY2MDMsNDMgQEAgc3RhdGljIGludCB2YWxsZXl2aWV3X21vZGVzZXRfY2FsY19jZGNs ayhzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUpCj4gIAlyZXR1cm4gMDsKPiAgfQo+ICAK PiArc3RhdGljIGJvb2wgY2RjbGtfbWluX2Zvcl9kcF9hdWRpbyhzdHJ1Y3QgZHJtX2F0b21pY19z dGF0ZSAqc3RhdGUpCj4gK3sKPiArCj4gKwlzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKmNydGNfc3Rh dGU7Cj4gKwlzdHJ1Y3QgZHJtX2NydGMgKmNydGM7Cj4gKwlpbnQgaTsKPiArCj4gKwkvKiBCU3Bl YyBzYXlzICJEbyBub3QgdXNlIERpc3BsYXlQb3J0IHdpdGggQ0RDTEsgbGVzcyB0aGFuIDQzMiBN SHosCj4gKwkgKiBhdWRpbyBlbmFibGVkLCBwb3J0IHdpZHRoIHg0LCBhbmQgbGluayByYXRlIEhC UjIgKDUuNCBHSHopLCBvciBlbHNlCj4gKwkgKiB0aGVyZSBtYXkgYmUgYXVkaW8gY29ycnVwdGlv biBvciBzY3JlZW4gY29ycnVwdGlvbi4iCj4gKwkgKi8KPiArCj4gKwlmb3JfZWFjaF9jcnRjX2lu X3N0YXRlKHN0YXRlLCBjcnRjLCBjcnRjX3N0YXRlLCBpKSB7Cj4gKwkJc3RydWN0IGludGVsX2Ny dGNfc3RhdGUgKnBpcGVfY29uZmlnID0KPiArCQkJdG9faW50ZWxfY3J0Y19zdGF0ZShjcnRjX3N0 YXRlKTsKPiArCj4gKwkJcmV0dXJuIChpbnRlbF9jcnRjX2hhc19kcF9lbmNvZGVyKHBpcGVfY29u ZmlnKSAmJgo+ICsJCQlwaXBlX2NvbmZpZy0+aGFzX2F1ZGlvICYmCj4gKwkJCXBpcGVfY29uZmln LT5wb3J0X2Nsb2NrID09IDU0MDAwMCAmJgo+ICsJCQlwaXBlX2NvbmZpZy0+bGFuZV9jb3VudCA9 PSA0KTsKPiArCX0KClRoYXQncyBub3QgZ29vZCBlbm91Z2ggb24gYWNjb3VudCBvZiBjcnRjcyBu b3QgcGFydCBvZiB0aGUgc3RhdGUKcG90ZW50aWFsbHkgbmVlZGluZyB0aGUgd29ya2Fyb3VuZCBh cyB3ZWxsLiBIb3dldmVyLCBzaW5jZSB3ZSBvbmx5IGRvCnRoaXMgd2hlbiB0aGVyZSdzIGEgbW9k ZXNldCwgSSB0aGluayB3ZSdsbCBiZSBjb3ZlcmVkIGJ5IHRoZQpjb25uZWN0aW9uX211dGV4LCBh bmQgc28gd2Ugc2hvdWxkIGJlIGFibGUgdG8gcGVlayBhdCB0aGUgY3VycmVudCBzdGF0ZQpvZiBh bGwgY3J0Y3Mgd2l0aG91dCBncmFiYmluZyB0aGUgY29ycmVzcG9uZGluZyBjcnRjIGxvY2tzLgoK U28gSSB0aGluayB3ZSdkIGJlIE9LIHdpdGggc29tZXRoaW5nIGxpa2U6CgpXQVJOX09OKCFsb2Nr ZWQoY29ubmVjdGlvbl9tdXRleCkpOwoKZm9yX2VhY2hfaW50ZWxfY3J0YygpIHsKCS8qCgkgKiBQ ZWVraW5nIGF0IHRoZSBjdXJyZW50IHN0YXRlIGlzIHNhZmUgc2luY2UKCSAqIHdlIGNhbiBvbmx5 IGdldCBoZXJlIHdoaWxlIGhvbGRpbmcgdGhlCgkgKiBjb25uZWN0aW9uX211dGV4LgoJICovCglj cnRjX3N0YXRlID0gaW50ZWxfZ2V0X2V4aXN0aW5nX2NydGNfc3RhdGUoKTsKCWlmICghY3J0Y19z dGF0ZSkKCQljcnRjX3N0YXRlID0gdG9faW50ZWxfY3J0Y19zdGF0ZShjcnRjLT5iYXNlLnN0YXRl KTsKCgkuLi4KfQoKVGhlIG90aGVyIG9wdGlvbiB3b3VsZCBiZSB0byB0cmFjayB0aGUgbWluIGNk Y2xrIGZvciBlYWNoIHBpcGUgaW4gdGhlCnRvcCBsZXZlbCBzdGF0ZSBJIHN1cHBvc2UuIFdlIGFs cmVhZHkgZG8gdGhhdCBmb3IgdGhlIHBpeGVsIHJhdGUKYWN0dWFsbHkgc28gdGhhdCB3ZSBjYW4g Y2FsY3VsYXRlIHRoZSBjZGNsayB0byBiZWdpbiB3aXRoLiBIbW0uIE1heWJlCndlIHNob3VsZCBq dXN0IHN3aXRjaCB0byB0cmFja2luZyB0aGUgbWluIGNkY2xrIHBlciBwaXBlIGluc3RlYWQgb2Yg dGhlCnBpeGVsIHJhdGUuIE9yIGRpZCB3ZSBuZWVkIHRvIHRoZSBwaXhlbCByYXRlIGl0c2VsZiBm b3Igc29tZXRoaW5nIGVsc2UsCk1hYXJ0ZW4/CgpPciB3ZSBjb3VsZCBwZXJoYXBzIHJlcGxhY2Ug dGhlIHBpeGVsIHJhdGUvcGl4Y2xrIHRyYWNraW5nIHdpdGggdGhlIHBlZWsKYXBwcm9hY2ggZW50 aXJlbHkuIE5vdCBxdWl0ZSBzdXJlLiBXb3VsZCBoYXZlIHRvIHJlYWQgdGhlIGVudGlyZSB0aGlu Zwp0aHJvdWdoLgoKPiArCj4gKwlyZXR1cm4gZmFsc2U7Cj4gK30KPiArCj4gIHN0YXRpYyBpbnQg Ynh0X21vZGVzZXRfY2FsY19jZGNsayhzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUpCj4g IHsKPiAgCWludCBtYXhfcGl4Y2xrID0gaWxrX21heF9waXhlbF9yYXRlKHN0YXRlKTsKPiArCWlu dCBjZGNsaywgbWluX2NkY2xrID0gMDsKPiAgCXN0cnVjdCBpbnRlbF9hdG9taWNfc3RhdGUgKmlu dGVsX3N0YXRlID0KPiAgCQl0b19pbnRlbF9hdG9taWNfc3RhdGUoc3RhdGUpOwo+ICAKPiAtCWlu dGVsX3N0YXRlLT5jZGNsayA9IGludGVsX3N0YXRlLT5kZXZfY2RjbGsgPQo+IC0JCWJ4dF9jYWxj X2NkY2xrKG1heF9waXhjbGspOwo+ICsJaWYgKGNkY2xrX21pbl9mb3JfZHBfYXVkaW8oc3RhdGUp KQo+ICsJCW1pbl9jZGNsayA9IGJ4dF9jYWxjX2NkY2xrKDQzMjAwMCk7Cj4gKwo+ICsJY2RjbGsg PSBieHRfY2FsY19jZGNsayhtYXhfcGl4Y2xrKTsKPiArCWludGVsX3N0YXRlLT5jZGNsayA9IGlu dGVsX3N0YXRlLT5kZXZfY2RjbGsgPSBtYXgobWluX2NkY2xrLCBjZGNsayk7Cj4gIAo+ICAJaWYg KCFpbnRlbF9zdGF0ZS0+YWN0aXZlX2NydGNzKQo+ICAJCWludGVsX3N0YXRlLT5kZXZfY2RjbGsg PSBieHRfY2FsY19jZGNsaygwKTsKPiBAQCAtMTAzNzQsNyArMTA0MDMsMTAgQEAgc3RhdGljIGlu dCBicm9hZHdlbGxfbW9kZXNldF9jYWxjX2NkY2xrKHN0cnVjdCBkcm1fYXRvbWljX3N0YXRlICpz dGF0ZSkKPiAgCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdiA9IHRvX2k5MTUoc3Rh dGUtPmRldik7Cj4gIAlzdHJ1Y3QgaW50ZWxfYXRvbWljX3N0YXRlICppbnRlbF9zdGF0ZSA9IHRv X2ludGVsX2F0b21pY19zdGF0ZShzdGF0ZSk7Cj4gIAlpbnQgbWF4X3BpeGNsayA9IGlsa19tYXhf cGl4ZWxfcmF0ZShzdGF0ZSk7Cj4gLQlpbnQgY2RjbGs7Cj4gKwlpbnQgY2RjbGssIG1pbl9jZGNs ayA9IDA7Cj4gKwo+ICsJaWYgKGNkY2xrX21pbl9mb3JfZHBfYXVkaW8oc3RhdGUpKQo+ICsJCW1p bl9jZGNsayA9IGJyb2Fkd2VsbF9jYWxjX2NkY2xrKDQzMjAwMCk7Cj4gIAo+ICAJLyoKPiAgCSAq IEZJWE1FIHNob3VsZCBhbHNvIGFjY291bnQgZm9yIHBsYW5lIHJhdGlvCj4gQEAgLTEwMzgyLDYg KzEwNDE0LDggQEAgc3RhdGljIGludCBicm9hZHdlbGxfbW9kZXNldF9jYWxjX2NkY2xrKHN0cnVj dCBkcm1fYXRvbWljX3N0YXRlICpzdGF0ZSkKPiAgCSAqLwo+ICAJY2RjbGsgPSBicm9hZHdlbGxf Y2FsY19jZGNsayhtYXhfcGl4Y2xrKTsKPiAgCj4gKwljZGNsayA9CW1heChtaW5fY2RjbGssIGNk Y2xrKTsKPiArCj4gIAlpZiAoY2RjbGsgPiBkZXZfcHJpdi0+bWF4X2NkY2xrX2ZyZXEpIHsKPiAg CQlEUk1fREVCVUdfS01TKCJyZXF1ZXN0ZWQgY2RjbGsgKCVkIGtIeikgZXhjZWVkcyBtYXggKCVk IGtIeilcbiIsCj4gIAkJCSAgICAgIGNkY2xrLCBkZXZfcHJpdi0+bWF4X2NkY2xrX2ZyZXEpOwo+ IEBAIC0xMDQxMSw3ICsxMDQ0NSwxMCBAQCBzdGF0aWMgaW50IHNrbF9tb2Rlc2V0X2NhbGNfY2Rj bGsoc3RydWN0IGRybV9hdG9taWNfc3RhdGUgKnN0YXRlKQo+ICAJc3RydWN0IGRybV9pOTE1X3By aXZhdGUgKmRldl9wcml2ID0gdG9faTkxNShzdGF0ZS0+ZGV2KTsKPiAgCWNvbnN0IGludCBtYXhf cGl4Y2xrID0gaWxrX21heF9waXhlbF9yYXRlKHN0YXRlKTsKPiAgCWludCB2Y28gPSBpbnRlbF9z dGF0ZS0+Y2RjbGtfcGxsX3ZjbzsKPiAtCWludCBjZGNsazsKPiArCWludCBjZGNsaywgbWluX2Nk Y2xrID0gMDsKPiArCj4gKwlpZiAoY2RjbGtfbWluX2Zvcl9kcF9hdWRpbyhzdGF0ZSkpCj4gKwkJ bWluX2NkY2xrID0gc2tsX2NhbGNfY2RjbGsoNDMyMDAwLCB2Y28pOwo+ICAKPiAgCS8qCj4gIAkg KiBGSVhNRSBzaG91bGQgYWxzbyBhY2NvdW50IGZvciBwbGFuZSByYXRpbwo+IEBAIC0xMDQxOSw2 ICsxMDQ1Niw4IEBAIHN0YXRpYyBpbnQgc2tsX21vZGVzZXRfY2FsY19jZGNsayhzdHJ1Y3QgZHJt X2F0b21pY19zdGF0ZSAqc3RhdGUpCj4gIAkgKi8KPiAgCWNkY2xrID0gc2tsX2NhbGNfY2RjbGso bWF4X3BpeGNsaywgdmNvKTsKPiAgCj4gKwljZGNsayA9IG1heChtaW5fY2RjbGssIGNkY2xrKTsK PiArCj4gIAkvKgo+ICAJICogRklYTUUgbW92ZSB0aGUgY2RjbGsgY2FjbHVsYXRpb24gdG8KPiAg CSAqIGNvbXB1dGVfY29uZmlnKCkgc28gd2UgY2FuIGZhaWwgZ3JhY2VndWxseS4KPiAtLSAKPiAy LjcuNAoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1n ZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21h aWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==