From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH 6/9] drm: bridge/dw_hdmi: adjust pixel clock values in N calculation Date: Sat, 5 Sep 2015 01:27:33 +0100 Message-ID: <20150905002733.GJ21084@n2100.arm.linux.org.uk> References: <20150808160936.GN7557@n2100.arm.linux.org.uk> <20150904212401.GI21084@n2100.arm.linux.org.uk> 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: Doug Anderson Cc: Fabio Estevam , alsa-devel@alsa-project.org, "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Jaroslav Kysela , "open list:ARM/Rockchip SoC..." , Mark Brown , Yakir Yang , Andy Yan , "linux-arm-kernel@lists.infradead.org" List-Id: alsa-devel@alsa-project.org T24gRnJpLCBTZXAgMDQsIDIwMTUgYXQgMDQ6NTA6MDNQTSAtMDcwMCwgRG91ZyBBbmRlcnNvbiB3 cm90ZToKPiBSdXNzZWxsLAo+IAo+IE9uIEZyaSwgU2VwIDQsIDIwMTUgYXQgMjoyNCBQTSwgUnVz c2VsbCBLaW5nIC0gQVJNIExpbnV4Cj4gPGxpbnV4QGFybS5saW51eC5vcmcudWs+IHdyb3RlOgo+ ID4+IEluIHlvdXIgY2FzZSB5b3UncmUgcHJvYmFibHkgbWFraW5nIHRoZSB2YWx1ZSB0aGF0IExp bnV4Cj4gPj4gYXNrZWQgeW91IHRvIG1ha2UsIEFLQSAyNS4xNzUwMDAgTUh6Lgo+ID4KPiA+IC4u LiB3aGljaCBpcyB0aGUgc3BlYyB2YWx1ZS4KPiAKPiBUaGlzIGlzIHdoZXJlIHdlJ3JlIG5vdCBv biB0aGUgc2FtZSBwYWdlLiAgV2hlcmUgaW4gdGhlIHNwZWMgZG9lcyBpdAo+IHNheSAyNS4xNzUw MCBNSHo/ICBJIHNlZSBpbiB0aGUgc3BlYzoKPiAgMjUuMiAvIDEuMDAxCgpTZWN0aW9uIDQgb2Yg Q0VBLTg2MS1CLCB3aGljaCBkZWZpbmVzIHRoZSB2aWRlbyBjbG9jayByYXRlcyBhbmQgdGhlaXIK YWNjdXJhY3kgb2YgMC41JS4KCj4gLi4uYW5kIHRoaXMgaXMgYSBjcnVjaWFsIGRpZmZlcmVuY2Ug aGVyZS4gIFBsZWFzZSBkb3VibGUtY2hlY2sgbXkgbWF0aCwgYnV0Ogo+IAo+ICgyNTE3NTAwMCAq IDQ1NzYpIC8gKDEyOCAqIDMyMDAwLikKPiA9PiAyODEyNS4xOTUzMTI1Cj4gCj4gKDI1MTc0ODI1 ICogNDU3NikgLyAoMTI4ICogMzIwMDAuKQo+ID0+IDI4MTI1LjAKPiAKPiBUaGlzIGNhbGN1bGF0 aW9uIGlzIHdoYXQgbGVkIHRvIG15IGJlbGllZiB0aGF0IHRoZSBnb2FsIGhlcmUgaXMgdG8KPiBt YWtlIGFuIGludGVncmFsIENUUy4gIElmIHlvdSBoYXZlIDI1LjE3NSBNSFogY2xvY2sgYW5kIE4g b2YgNDU3NiB5b3UKPiBfd2lsbCBub3RfIGhhdmUgYW4gaW50ZWdyYWwgQ1RTLiAgSWYgeW91IGlu c3RlYWQgaGF2ZSAyNS4xNzQ4MjUgTUh6Cj4gY2xvY2sgYW5kIE4gb2YgNDU3NiB5b3UgX3dpbGxf IGhhdmUgYW4gaW50ZWdyYWwgQ1RTLgoKUmlnaHQsIGJ1dCAyNS4xNzUgaXMgY2xvc2UgZW5vdWdo IHRvIDI1LjE3NDgyNS4gIERvIHRoaXMgY2FsY3VsYXRpb246CgoyNTE3NTAwMCAqIDQ1NzYgLyAy ODEyNSAvIDEyOAoKVGhhdCdsbCBnaXZlIHlvdSB0aGUgcmVzdWx0aW5nIGF1ZGlvIHNhbXBsZSBy YXRlLCB3aGljaCBpcyAzMjAwMC4yMjJIei4KVGhhdCdzIGFuIGVycm9yIG9mLi4uIDAuMDAwNjkl LCB3aGljaCBpcyBwcm9iYWJseSBhcm91bmQgdGhlIHR5cGljYWwKZXJyb3Igb2YgeW91ciBhdmVy YWdlIGNyeXN0YWwgb3NjaWxsYXRvci4gIFJlYWxseSBub3Qgd29ydGggYm90aGVyaW5nCndpdGgu Cgo+IFNhaWQgYW5vdGhlciB3YXk6Cj4gCj4gMS4gVGhlIHJlYXNvbiAyNTE3NDgyNSBIeiBoYXMg YSBkaWZmZXJlbnQgTiBpcyB0byBtYWtlIGFuIGludGVncmFsIENUUy4KPiAKPiAyLiBJZiB5b3Ug YXJlIGluZGVlZCBtYWtpbmcgMjUxNzUwMDAgdGhlbiB0aGVyZSBpcyBubyBuZWVkIGZvciBhCj4g ZGlmZmVyZW50IE4gdG8gbWFrZSBhbiBpbnRlZ3JhbCBDVFMKPiAKPiAzLiBJZiB5b3UgdXNlIDQ1 NzYgZm9yIE4gYnV0IHlvdSdyZSBtYWtpbmcgMjUxNzUwMDAgSHosIHlvdSBlbmQgdXAgaW4KPiBh IF93b3JzZV8gcG9zaXRpb24gdGhhbiBpZiB5b3UgdXNlIHRoZSBzdGFuZGFyZCA0MDk2IGZvciBO LgoKVG90YWwgcnViYmlzaC4gIFNvcnJ5LCBidXQgaXQgaXMuCgpGb2xsb3cgdGhlIGNvZGUuICBQ aXhlbCBjbG9jayBpcyAyNTE3NTAwMC4gIEZvciAzMmtIeiwgTiB3aWxsIGJlIDQ1NzYuCjI1MTc1 MDAwICogNDU3NiA9IDEuMTUyMDA4ZTExLiAgRGl2aWRlIHRoYXQgYnkgdGhlIGF1ZGlvIGNsb2Nr IHJhdGUKKDEyOCAqIDMyMDAwKSBnaXZlcyAyODEyNS4xOTUzMS4gIFNpbmNlIHdlJ3JlIHVzaW5n IGludGVnZXIgZGl2aXNpb24sCnRoYXQgZ2V0cyByb3VuZGVkIGRvd24gdG8gMjgxMjUuCgpEUk0g dXNlcyBhIGNsb2NrIHJhdGUgb2YgIjI1MTc1IiB0byByZXByZXNlbnQgMjUuMi8xLjAwMSBtb2Rl cy4gIFNvLAppZiB5b3VyIGhhcmR3YXJlIHNldHMgYSB2aWRlbyBjbG9jayByYXRlIG9mIDI1LjJN SHovMS4wMDEsIHRoZW4geW91CmVuZCB1cCB3aXRoIGEgc2FtcGxlIHJhdGUgb2YgZXhhY3RseSAz MmtIei4gIElmIHlvdSBzZXQgZXhhY3RseQoyNS4xNzVNSHosIHlvdSBlbmQgdXAgd2l0aCBhbiBh cHByb3hpbWF0ZSAzMmtIeiBzYW1wbGUgcmF0ZSAtIG9uZQp3aGljaCBpcyAwLjAwMDY5JSBpbiBl cnJvciwgd2hpY2ggaXMgKGV4Y2x1c2UgdGhlIGxhbmd1YWdlKSBmdWNrIGFsbApkaWZmZXJlbnQg ZnJvbSBleGFjdGx5IDMya0h6LgoKQXJlIHlvdSBfcmVhbGx5XyBnb2luZyB0byBjb250aW51ZSBh cmd1aW5nIG92ZXIgYSAwLjAwMDY5JSBlcnJvcj8KSWYgeW91IGFyZSwgSSdtIG5vdCBnb2luZyB0 byBsaXN0ZW4gYW55bW9yZSAtIGl0J3Mgc29vIGRhbW5lZCBzbWFsbAp0aGF0IGl0J3Mgbm90IHdv cnRoIGJvdGhlcmluZyB3aXRoLiAgQXQgYWxsLgoKVGhlIG9ubHkgdGltZSB0aGF0IHlvdSdkIG5l ZWQgdG8gd29ycnkgYWJvdXQgaXQgaXMgaWYgeW91IHdhbnRlZCBhCnN1cGVyLWFjY3VyYXRlIHN5 c3RlbSwgYW5kIGZvciB0aGF0IHlvdSdkIG5lZWQgYW4gYXRvbWljIGNsb2NrIHRvCnNvdXJjZSB5 b3VyIHN5c3RlbSBjbG9ja3MgdG8gcmVkdWNlIGFnaW5nIGVmZmVjdHMsIHRlbXBlcmF0dXJlCmlu ZHVjZWQgZHJpZnQsIGV0YywgbWF5YmUgbG9ja2luZyB0aGUgYXRvbWljIGNsb2NrIHRvIGEgbmF0 aW9uYWwKZnJlcXVlbmN5IHN0YW5kYXJkIGxpa2UgdGhlIEFudGhvcm4gTVNGIDYwa0h6IHRyYW5z bWl0dGVyIHNpZ25hbApicm9hZGNhc3QgYnkgdGhlIFVLIE5hdGlvbmFsIFBoeXNpY3MgTGFib3Jh dG9yeS4KCj4gPj4gQXMgYSBmaXJzdCBzdGVwIEknZCBzdWdnZXN0IGp1c3QgcmVtb3ZpbmcgYWxs IHRoZSBzcGVjaWFsIGNhc2VzIGFuZAo+ID4+IGFkZCBhIGNvbW1lbnQuICBGcm9tIHJlYWwgd29y bGQgdGVzdGluZyBpdCBkb2Vzbid0IHNlZW0gdGVycmlibHkKPiA+PiBjcml0aWNhbCB0byBiZSBz bGlnaHRseSBvZmYgb24gQ1RTLiAgLi4uYW5kIGluIGFueSBjYXNlIGZvciBhbnkgY2xvY2sKPiA+ PiByYXRlcyBleGNlcHQgdGhlIHNtYWxsIGhhbmRmdWwgaW4gdGhlIEhETUkgc3BlYyB3ZSdsbCBi ZSBzbGlnaHRseSBvZmYKPiA+PiBvbiBDVFMgYW55d2F5Li4uCj4gPgo+ID4gVGhleSdyZSBub3Qg InNwZWNpYWwgY2FzZXMiIG1hZGUgdXAgdG8gZml0IHNvbWV0aGluZyAtIHRoZXkncmUgZnJvbSB0 aGUKPiA+IHRhYmxlcyBpbiB0aGUgSERNSSBzcGVjaWZpY2F0aW9uLgo+IAo+IFRoZXkgYXJlIGRl ZmluaXRlbHkgInNwZWNpYWwgY2FzZXMiLiAgVGhlcmUgaXMgYSBnZW5lcmFsIHJ1bGUgaW4gdGhl Cj4gY29kZSB5b3UgcG9zdGVkIChhaW0gZm9yIDEyOCAqIGZyZXEpIGFuZCB0aGVzZSBhcmUgY2Fz ZXMgZm9yIGNlcnRhaW4KPiBjbG9ja3MgdGhhdCBhcmUgYW4gZXhjZXB0aW9uIHRvIHRoZSBnZW5l cmFsIHJ1bGUuICBBS0EgdGhleSBhcmUKPiBzcGVjaWFsIGNhc2VzLgoKU29ycnksIEkgZGlzYWdy ZWUgd2l0aCB5b3UuCgo+ID4gVGhhdCBhc3N1bWVzIHRoYXQgdGhlIGF1ZGlvIGFuZCB2aWRlbyBj bG9ja3MgYXJlIGNvaGVyZW50LiAgT24gaU1YNgo+ID4gaGFyZHdhcmUgdXNpbmcgdGhpcywgdGhl IGF1ZGlvIGlzIGNsb2NrZWQgYXQgdGhlIHJhdGUgZGVmaW5lZCBieSB0aGUKPiA+IFRETVMgY2xv Y2sgYW5kIHRoZSBDVFMvTiB2YWx1ZXMuCj4gCj4gSSdsbCBhZG1pdCBJIGhhdmVuJ3QgbG9va2Vk IGF0IHRoZSBhdWRpbyBzZWN0aW9uIG9mIGR3X2hkbWkgbXVjaCwgYnV0Cj4gSSdkIGltYWdpbmUg dGhhdCBmb3IgYWxsIHVzZXJzIG9mIHRoaXMgY29udHJvbGxlciAvIFBIWSB0aGUgYXVkaW8gYW5k Cj4gdmlkZW8gY2xvY2tzIGFyZSBjb2hlcmVudC4KCk5vdCBpZiB0aGUgYXVkaW8gY2xvY2sgY29t ZXMgZnJvbSBhbiBJMlMgbWFzdGVyIHJhdGhlciB0aGFuIGJlaW5nCnNvdXJjZWQgZnJvbSB0aGUg SERNSSBibG9jay4KCj4gSSB0aGluayBpbiB0aGUgcGVyZmVjdCB3b3JsZCB3ZSdkIGJlIGFibGUg dG8gZ2VuZXJhdGUgZXhhY3RseQo+IDI1MTc0ODI1LjE3NDgyNTE3NyBIeiBhbmQgd2UnZCB1c2Ug YWxsIHRoZSByYXRlcyBmcm9tIHRoZSBIRE1JIHNwZWMuCgpUbyBnZW5lcmF0ZSBzb21ldGhpbmcg b2YgdGhhdCBhY2N1cmFjeSwgeW91J2QgbmVlZCBzb21ldGhpbmcgbGlrZSBhCmNhZXNpdW0gZm91 bnRhaW4gYXRvbWljIGNsb2NrLgoKPiBhbmQgd2UnZCBnZXQgc3BvdCBvbiAzMiBrSHogYXVkaW8u ICAuLi5idXQgSSdtIHNpbXBseSBzYXlpbmcgdGhhdAo+IHdlJ3JlIG5vdCBpbiB0aGF0IHBlcmZl Y3Qgd29ybGQgeWV0Lgo+IAo+IEFsc28gbm90ZSB0aGF0IHRoZXJlIGFyZSBtYW55IG1hbnkgcmF0 ZXMgbm90IGluIHRoZSBIRE1JIHNwZWMgdGhhdAo+IGNvdWxkIGJlbmVmaXQgZnJvbSBzaW1pbGFy IG9wdGltaXphdGlvbiBvZiB0cnlpbmcgdG8gYWRqdXN0IE4gdG8gbWFrZQo+IGFuIGludGVncmFs IENUUy4KCk5vdyBnbyBhbmQgbG9vayBhdCB0aGUgSERNSSBzcGVjLCB3aGVyZSBpdCBnaXZlcyB0 aGUgQ1RTIHZhbHVlIGZvcgo3NC4yNS8xLjAwMSBmb3IgMzJrSHouICBUaGF0IGNhbid0IGJlIHJl cHJlc2VudGVkIGJ5IGFuIGludGVnZXIgQ1RTCnZhbHVlLCBzbyB1c2luZyB0aGlzIGhhcmR3YXJl LCB3ZSBjYW4ndCBnZW5lcmF0ZSB0aGF0IHNhbXBsZSByYXRlCndpdGhvdXQgYW4gZXJyb3IuICBX ZSdkIHVzZSBhIGZpeGVkIENUUyB2YWx1ZSBvZiAyMTA5MzcgaW5zdGVhZCwgd2hpY2gKd29ya3Mg b3V0IGF0IGEgMC4wMDAyNCUgZXJyb3IuICBBZ2Fpbiwgbm90IHdvcnRoIHdvcnJ5aW5nIGFib3V0 LgoKCj4gCj4gLS0tCj4gCj4gQXMgYSBzaWRlIG5vdGU6IEkgcmVhbGl6ZWQgb25lIHBhcnQgb2Yg dGhlIEhETUkgc3BlYyB0aGF0IGlzbid0IHRyeWluZwo+IHRvIG1ha2UgYW4gaW50ZWdyYWwgdmFs dWUgYnV0IHN0aWxsIHVzZXMgYSBkaWZmZXJlbnQgdmFsdWUgZm9yIE46IDI5Nwo+IE1Iei4gIEZy b20gdGhlIERlc2lnbldhcmUgc3BlYyBJIGhhdmUgaXQgYXBwZWFycyB0aGF0IDU5NCBNSHogaXMK PiBzaW1pbGFyLiAgRm9yIHRob3NlIGNhc2VzIGl0IGxvb2tzIGxpa2Ugd2UgaGF2ZToKCjI5N01I eiBfZG9lc18gd29yay4KCjI5NzAwMDAwMCAqIDMwNzIgLyAyMjI3NTAgPSAxMjggKiAzMjAwMCBl eGFjdGx5LgoKPiAKPiBpZiAocGl4ZWxfY2xrID09IDI5NzAwMDAwMCkgewo+ICAgc3dpdGNoIChm cmVxKSB7Cj4gICBjYXNlIDMyMDAwOgo+ICAgICByZXR1cm4gKDEyOCAqIGZyZXEpIC8gMTMzMzsK ClBsdWcgdGhlIG51bWJlcnMgaW4uICAxMjggKiAzMjAwMCAvIDEzMzMgPSAzMDcyLjk2IGJ1dCBi ZWNhdXNlIHdlJ3JlIHVzaW5nCmludGVnZXIgbWF0aCwgdGhhdCdzIDMwNzIuICBXaGljaCBqdXN0 IGhhcHBlbnMgdG8gYmUgdGhlIHZhbHVlIGluIHRoZSBIRE1JCnNwZWMuCgo+ICAgY2FzZSA0NDEw MDoKPiAgIGNhc2UgNDgwMDA6Cj4gICBjYXNlIDg4MjAwOgo+ICAgY2FzZSA5NjAwMDoKPiAgIGNh c2UgMTc2NDAwOgo+ICAgICByZXR1cm4gKDEyOCAqIGZyZXEpIC8gMTIwMDsKCkRvIHRoZSBtYXRo IGFnYWluLiAgWW91IGdldCB0aGUgc3BlYyBmaWd1cmVzIGZvciBOLgoKLS0gCkZUVEMgYnJvYWRi YW5kIGZvciAwLjhtaWxlIGxpbmU6IGN1cnJlbnRseSBhdCA5LjZNYnBzIGRvd24gNDAwa2JwcyB1 cAphY2NvcmRpbmcgdG8gc3BlZWR0ZXN0Lm5ldC4KX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9kcmktZGV2ZWwK