From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 4/4] drm/i915/intel_dsi: Read back pclk set by GOP and use that as pclk Date: Mon, 9 Jul 2018 21:14:25 +0300 Message-ID: <20180709181425.GT5565@intel.com> References: <20180619201827.4257-1-hdegoede@redhat.com> <20180619201827.4257-5-hdegoede@redhat.com> <20180706141652.GK5565@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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Hans de Goede Cc: Hans de Goede , intel-gfx , dri-devel@lists.freedesktop.org, Rodrigo Vivi List-Id: dri-devel@lists.freedesktop.org T24gU2F0LCBKdWwgMDcsIDIwMTggYXQgMDg6MzI6MTZBTSArMDIwMCwgSGFucyBkZSBHb2VkZSB3 cm90ZToKPiBIaSwKPiAKPiBPbiAwNy8wNi8yMDE4IDA0OjE2IFBNLCBWaWxsZSBTeXJqw6Rsw6Qg d3JvdGU6Cj4gPiBPbiBUdWUsIEp1biAxOSwgMjAxOCBhdCAxMDoxODoyN1BNICswMjAwLCBIYW5z IGRlIEdvZWRlIHdyb3RlOgo+ID4+IE9uIEJZVCBhbmQgQ0hUIHRoZSBHT1Agc29tZXRpbWVzIGlu aXRpYWxpemVzIHRoZSBwY2xrIGF0IGEgKHNsaWdodGx5KQo+ID4+IGRpZmZlcmVudCBmcmVxdWVu Y3kgdGhlbiB0aGUgcGNsayB3aGljaCB3ZSd2ZSBjYWxjdWxhdGVkLgo+ID4+Cj4gPj4gVGhpcyBj b21taXQgbWFrZXMgdGhlIERTSSBjb2RlIHJlYWQtYmFjayB0aGUgcGNsayBzZXQgYnkgdGhlIEdP UCBhbmQKPiA+PiBpZiB0aGF0IGlzIHdpdGhpbiBhIHJlYXNvbmFibGUgbWFyZ2luIG9mIHRoZSBj YWxjdWxhdGVkIHBjbGssIHVzZXMKPiA+PiB0aGF0IGluc3RlYWQuCj4gPj4KPiA+PiBUaGlzIGZp eGVzIHRoZSBmaXJzdCBtb2Rlc2V0IGJlaW5nIGEgZnVsbCBtb2Rlc2V0IGluc3RlYWQgb2YgYQo+ ID4+IGZhc3QgbW9kZXNldCBvbiBzeXN0ZW1zIHdoZXJlIHRoZSBHT1AgcGNsayBpcyBkaWZmZXJl bnQuCj4gPj4KPiA+PiBTaWduZWQtb2ZmLWJ5OiBIYW5zIGRlIEdvZWRlIDxoZGVnb2VkZUByZWRo YXQuY29tPgo+ID4+IC0tLQo+ID4+ICAgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHNpX3Zi dC5jIHwgMTQgKysrKysrKysrKysrKysKPiA+PiAgIDEgZmlsZSBjaGFuZ2VkLCAxNCBpbnNlcnRp b25zKCspCj4gPj4KPiA+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxf ZHNpX3ZidC5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZHNpX3ZidC5jCj4gPj4gaW5k ZXggNGQ2ZmZhN2IzZTdiLi5kNGNjNjA5OTAxMmMgMTAwNjQ0Cj4gPj4gLS0tIGEvZHJpdmVycy9n cHUvZHJtL2k5MTUvaW50ZWxfZHNpX3ZidC5jCj4gPj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5 MTUvaW50ZWxfZHNpX3ZidC5jCj4gPj4gQEAgLTUxNyw2ICs1MTcsNyBAQCBib29sIGludGVsX2Rz aV92YnRfaW5pdChzdHJ1Y3QgaW50ZWxfZHNpICppbnRlbF9kc2ksIHUxNiBwYW5lbF9pZCkKPiA+ PiAgIAl1MzIgbXVsOwo+ID4+ICAgCXUxNiBidXJzdF9tb2RlX3JhdGlvOwo+ID4+ICAgCWVudW0g cG9ydCBwb3J0Owo+ID4+ICsJZW51bSBwaXBlIHBpcGU7Cj4gPj4gICAKPiA+PiAgIAlEUk1fREVC VUdfS01TKCJcbiIpOwo+ID4+ICAgCj4gPj4gQEAgLTU4Myw2ICs1ODQsMTkgQEAgYm9vbCBpbnRl bF9kc2lfdmJ0X2luaXQoc3RydWN0IGludGVsX2RzaSAqaW50ZWxfZHNpLCB1MTYgcGFuZWxfaWQp Cj4gPj4gICAJfSBlbHNlCj4gPj4gICAJCWJ1cnN0X21vZGVfcmF0aW8gPSAxMDA7Cj4gPj4gICAK PiA+PiArCS8qCj4gPj4gKwkgKiBPbiBCWVQgLyBDUkMgdGhlIEdPUCBzb21ldGltZXMgcGlja3Mg YSBzbGlnaHRseSBkaWZmZXJlbnQgcGNsaywKPiA+PiArCSAqIHJlYWQgYmFjayB0aGUgR09QIGNv bmZpZ3VyZWQgcGNsayBhbmQgcHJlZmVyIGl0IG92ZXIgb3Vycy4KPiA+PiArCSAqLwo+ID4+ICsJ aWYgKChJU19WQUxMRVlWSUVXKGRldl9wcml2KSB8fCBJU19DSEVSUllWSUVXKGRldl9wcml2KSkg JiYKPiA+PiArCSAgICBpbnRlbF9kc2lfZ2V0X2h3X3N0YXRlKCZpbnRlbF9kc2ktPmJhc2UsICZw aXBlKSkgewo+ID4+ICsJCXUzMiBnb3AgPSBpbnRlbF9kc2lfZ2V0X3BjbGsoJmludGVsX2RzaS0+ YmFzZSwgYnBwLCBOVUxMKTsKPiA+PiArCj4gPj4gKwkJRFJNX0RFQlVHX0tNUygiQ2FsY3VsYXRl ZCBwY2xrICVkIEdPUCAlZFxuIiwgcGNsaywgZ29wKTsKPiA+PiArCQlpZiAoZ29wID49IChwY2xr ICogOSAvIDEwKSAmJiBnb3AgPD0gKHBjbGsgKiAxMSAvIDEwKSkKPiA+PiArCQkJcGNsayA9IGdv cDsKPiA+PiArCX0KPiA+IAo+ID4gSXMgdGhlIGdvcCBhY3VhbGx5IHBpY2tpbmcgYSBkaWZmZXJl bnQgY2xvY2sgdGhhdCB3aGF0IHdlIHBpY2sgaW4gdGhlCj4gPiBlbmQsIG9yIGlzIGl0IGp1c3Qg dGhhdCB0aGUgdmFsdWUgaW4gdGhlIHZidCBkb2Vzbid0IHF1aXRlIG1hdGNoIHdoYXQgd2UKPiA+ IChhbmQgdGhlIGdvcCkgZW5kIHVwIHVzaW5nIG9uIGFjY291bnQgb2YgbGltaXRhdGlvbnMgb2Yg dGhlIHBsbD8KPiAKPiBJICp0aGluayogdGhlIEdPUCBpcyBwaWNraW5nIGEgZGlmZmVyZW50IGNs b2NrLCBJSVJDICgqKSBpdCBpcyBzb21ldGhpbmcgbGlrZQo+IDkwTWh6IGZvciB0aGUgR09QIGFu ZCB0aGUgVkJUIHNheXMgODdNaHogKGFuZCBvdXIgY2FsY3VsYXRpb25zIGxlYXZlIGl0Cj4gdW5t b2RpZmllZC4gV2l0aCB0aGlzIHBhdGNoIHdoaWNoIHB1dHMgcGNsayBhdCA5ME1oeiBvbiB0aGUg c3BlY2lmaWMKPiB0YWJsZXQgSSBkZXZlbG9wZWQgdGhpcyBvbiwgdGhlIFBMTCBzZXR0aW5ncyBj YWxjdWxhdGVkIGJ5IG91ciBQTEwgY29kZQo+IGVuZCB1cCBiZWluZyBleGFjdGx5IHRoZSBzYW1l IGFzIHRoZSBvbmNlIGNob3NlbiBieSB0aGUgR09QIG9uY2Ugd2UgaGF2ZQo+IHRoZSBwY2xrIHNl dCB0byA5ME1Iei4KPiAKPiBOb3RlIEkndmUgc2VlbiB0aGVzZSBzbWFsbCAoYW5kIHNvbWV0aW1l cyBzb21ld2hhdCBiaWdnZXIpIGRpZmZlcmVuY2VzCj4gYmV0d2VlbiBHT1AgYW5kIFZCVCBwY2xr IG9uIGEgbG90IG9mIGRldmljZXMsIG5vdCBqdXN0IHRoZSBvbmUgdGFibGV0Cj4gSSBkZXZlbG9w ZWQgaXQgb24uIFNpbmNlIHN1Ym1pdHRpbmcgdGhpcyBJJ3ZlIHJ1biB0aGlzIG9uIGF0IGxlYXN0 Cj4gNSBkaWZmZXJlbnQgQ0hUL0JZVCBkZXZpY2VzIGFuZCBpdCB3b3JrcyBhcyBhZHZlcnRpc2Vk IHNvIGZhci4KPiAKPiA+IEZvciB0aGF0IHBhcnRpY3VsYXIgcHJvYmxlbSBJIHRoaW5rIEkgaGFk IHBhdGNoZXMgbG9uZyBhZ28gdG8gZ28gdGhyb3VnaAo+ID4gdGhlIHBsbCBjb21wdXRhdGlvbiBk dXJpbmcgaW5pdCBzbyB0aGF0IHdlIGJhc2ljYWxseSBmaXggdXAgdGhlIHNsaWdodGx5Cj4gPiBi b2d1cyBjbG9jayBmcm9tIHRoZSB2YnQuCj4gCj4gV2UgZG8gZW5kIHVwIHdpdGggYSBzbGlnaHRs eSBkaWZmZXJlbnQgY2xvY2sgdGhlbiB0aGUgODdNSGh6IHdoZW4gZ29pbmcKPiB0aG91Z2ggdGhl IFBMTCBjYWxjdWxhdGlvbnMsIHNvbWV0aGluZyBsaWtlIDg2LjMzTUh6IG9yIHNvbWUgc3VjaCBm cm9tCj4gdGhlIHRvcCBvZiBteSBoZWFkLCBidXQgdGhlIHByb2JsZW0gaXMgbm90IHRoZSBwY2xr IG5vdCBtYXRjaGluZyB0aGUKPiBpbnRlbF9waXBlX2NvbmZpZ19jb21wYXJlKCkgZnVuY3Rpb24g ZG9lcyBub3QgbG9vayBhdCBpdCwgaXQgbG9va3MgYXQKPiBkc2lfcGxsLmN0cmwgZHNpX3BsbC5k aXYgYW5kIHRob3NlIGRvbid0IG1hdGNoLCB3aGVyZSBhcyB0aGV5IGRvIG1hdGNoCj4gaWYgd2Ug Zml4dXAgdGhlIFZCVCBjbG9jayB0byBiZSB0aGUgb25lIGNvbmZndXJlZCBieSB0aGUgR09QLgo+ IAo+ID4gQW55IGtpbmQgb2YgaGFjayB0aGF0IGludm9sdmVzIHJlYWRpbmcgb3V0IHRoZSBoYXJk d2FyZSBzdGF0ZSBzaG91bGQgZ28KPiA+IGludG8gc29tZXRoaW5nIGxpa2UgaW50ZWxfc2FuaXRp emVfZW5jb2RlcigpLiBBY3R1YWxseSBieSB0aGF0IHRpbWUgd2UKPiA+IGhhdmUgYWxyZWFkeSBy ZWFkIG91dCB0aGUgaHcgc3RhdGUsIHNvIGl0IHNob3VsZG4ndCByZXF1aXJlIGFueQo+ID4gbW9k aWZpY2F0aW9ucyB0byB0aGUgZXhpc3RpbmcgZHNpIGNvZGUgaXRzZWxmLgo+IAo+IEkgZG8gbm90 IHRoaW5rIHRoYXQgaW50ZWxfc2FuaXRpemUgZW5jb2RlciBpcyB0aGUgcmlnaHQgcGxhY2UgdG8g ZG8gdGhpczoKPiAKPiAqIEkgZG9uJ3Qgd2FudCB0byBtb2RpZnkgdGhlIHJlYWQtYmFjayBzdGF0 ZSwgSSB3YW50IHRvIG1vZGlmeSBvdXIKPiAgICBjYWxjdWxhdGVkICJuZXcvaWRlYWwiIHN0YXRl IHRvIG1hdGNoIHRoZSByZWFkLWJhY2sgc3RhdGUKCkkgd2Fzbid0IHN1Z2dlc3RpbmcgdGhhdC4g V2hhdCBJIG1lYW50IGlzIHRoYXQgeW91IGFscmVhZHkgaGF2ZSB0aGUKc3RhdGUgdGhlcmUgdG8g bG9vayBzbyB5b3UgZG9uJ3QgaGF2ZSB0byBoYWNrIHRoZSByZWFkb3V0IGZ1bmN0aW9ucwp0byBm dW5jdGlvbiB3aXRob3V0IGEgc3RhdGUgYmVpbmcgYXJvdW5kLgoKVGhhdCBzYWlkLCB3ZSBkbyBh bHJlYWR5IGhhdmUgaW50ZWxfZW5jb2Rlcl9jdXJyZW50X21vZGUoKSB3aGljaCBpcyBkb2luZwpz b21ldGhpbmcgc2ltaWxhciB0byB3aGF0IHlvdSdyZSBwcm9wb3NpbmcuIFNvIHByb2JhYmx5IHNo b3VsZCBqdXN0CnRyeSB0byByZXVzZSB0aGF0LgoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbApf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwg bWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK