From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liviu Dudau Subject: Re: [PATCH v2] drm: hdlcd: Fix the calculation of the scanout start address Date: Mon, 3 Apr 2017 11:31:34 +0100 Message-ID: <20170403103133.GL13355@e110455-lin.cambridge.arm.com> References: <20170222153036.GI31595@intel.com> <20170308163025.31098-1-Liviu.Dudau@arm.com> <20170331094937.GO7909@n2100.armlinux.org.uk> <20170331131830.GK13355@e110455-lin.cambridge.arm.com> <20170331134810.GA25244@n2100.armlinux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from foss.arm.com (foss.arm.com [217.140.101.70]) by gabe.freedesktop.org (Postfix) with ESMTP id 3D96C89F89 for ; Mon, 3 Apr 2017 10:31:36 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20170331134810.GA25244@n2100.armlinux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Russell King - ARM Linux Cc: LAKML , DRI devel List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBNYXIgMzEsIDIwMTcgYXQgMDI6NDg6MTBQTSArMDEwMCwgUnVzc2VsbCBLaW5nIC0g QVJNIExpbnV4IHdyb3RlOgo+IE9uIEZyaSwgTWFyIDMxLCAyMDE3IGF0IDAyOjE4OjMxUE0gKzAx MDAsIExpdml1IER1ZGF1IHdyb3RlOgo+ID4gT24gRnJpLCBNYXIgMzEsIDIwMTcgYXQgMTA6NDk6 MzhBTSArMDEwMCwgUnVzc2VsbCBLaW5nIC0gQVJNIExpbnV4IHdyb3RlOgo+ID4gPiBPbiBXZWQs IE1hciAwOCwgMjAxNyBhdCAwNDozMDoyNVBNICswMDAwLCBMaXZpdSBEdWRhdSB3cm90ZToKPiA+ ID4gPiBUaGUgY2FsY3VsYXRpb24gb2YgdGhlIGZyYW1lYnVmZmVyJ3Mgc3RhcnQgYWRkcmVzcyB3 YXMgd3JvbmdseSB1c2luZwo+ID4gPiA+IHRoZSBDUlRDJ3MgeCBhbmQgeSBwb3NpdGlvbiByYXRo ZXIgdGhhbiB0aGUgb25lIG9mIHRoZSBzb3VyY2UKPiA+ID4gPiBmcmFtZWJ1ZmZlci4gVG8gZml4 IHRoYXQgd2UgbmVlZCB0byB1cGRhdGUgdGhlIHBsYW5lX2NoZWNrIGNvZGUgdG8KPiA+ID4gPiBj YWxsIGRybV9wbGFuZV9oZWxwZXJfY2hlY2tfc3RhdGUoKSB0byBjbGlwIHRoZSBzcmMgYW5kIGRz dCBjb29yZGluYXRlcy4KPiA+ID4gPiBXaGlsZSB0aGVyZSBzbyBzb21lIG1pbm9yIGNsZWFudXAg b2YgcmVkdW5kYW50IGZyZWVpbmcgb2YKPiA+ID4gPiBkZXZtX2FsbG9jLWF0ZWQgbWVtb3J5Lgo+ ID4gPiAKPiA+ID4gVGhlIGZvbGxvd2luZyBzZXJpZXMgaXMgd2hhdCBJIGNhbWUgdXAgd2l0aCwg YWx0aG91Z2ggSSd2ZSBoYWQgbm8gdGltZQo+ID4gPiB0byB0ZXN0IGl0Lgo+ID4gCj4gPiBJJ20g YWZyYWlkIEknbSBnb2luZyB0byBOQUsgdGhpcyBzZXJpZXMuIEl0IHdvdWxkJ3ZlIGJlZW4gbmlj ZXIgZm9yIHlvdSB0bwo+ID4gY29tbWVudCBvbiB0aGUgdjIgcGF0Y2ggdGhhdCBJIGhhdmUgc2Vu dCByYXRoZXIgdGhhbiBnb2luZyBhcm91bmQgYW5kIGNvbWluZwo+ID4gYmFjayB3aXRoIGVmZmVj dGl2ZWx5IHRoZSBzYW1lIHRoaW5nIGJ1dCBzcGxpdCBpbnRvIDIgcGF0Y2hlcy4gSSdtIGhhdmlu Zwo+ID4gdHJvdWJsZSBhcHBseWluZyB5b3VyIHNlcmllcyB0byB0aGUgdjQuMTEtcmM0IChzcGVj aWFsbHkgMi8zKS4gQWxzbyAzLzMgaXMKPiA+IHN1cGVyZmx1b3VzLCBhcyB3ZSBkb24ndCBleHBv c2UgdGhlIERSTV9ST1RBVEUgcHJvcGVydHkgdG8gdXNlcnNwYWNlLgo+IAo+IFJhdGhlciB0aGFu IHRocm93aW5nIGFjY3VzYXRpb25zLCBsZXQncyBsb29rIGF0IHdoYXQgaGFwcGVuZWQuCgpObyBh Y2N1c2F0aW9ucyB3ZXJlIHRocm93biwganVzdCBleHBsYW5hdGlvbnMgb2YgbXkgZGVjaXNpb24u Cgo+IAo+IEkgcmVwb3J0ZWQgdGhlIGJ1ZyBvbiAxOHRoIE5vdmVtYmVyIDIwMTYgLSBJIHF1b3Rl Ogo+IAo+ICAgIFNvbWV0aGluZyBJIGFsc28gbm90aWNlZCBpcyB0aGlzOgo+IAo+ICAgICAgICAg c2Nhbm91dF9zdGFydCA9IGdlbS0+cGFkZHIgKyBwbGFuZS0+c3RhdGUtPmZiLT5vZmZzZXRzWzBd ICsKPiAgICAgICAgICAgICAgICAgcGxhbmUtPnN0YXRlLT5jcnRjX3kgKiBwbGFuZS0+c3RhdGUt PmZiLT5waXRjaGVzWzBdICsKPiAgICAgICAgICAgICAgICAgcGxhbmUtPnN0YXRlLT5jcnRjX3gg KiBicHAgLyA4Owo+IAo+ICAgIFN1cmVseSB0aGlzIHNob3VsZCBiZSB1c2luZyBzcmNfW3h5XSAo d2hpY2ggYXJlIHRoZSBwb3NpdGlvbiBpbiB0aGUKPiAgICBzb3VyY2UgLSBpb3csIG1lbW9yeSwg YW5kIG5vdCBjcnRjX1t4eV0gd2hpY2ggaXMgdGhlIHBvc2l0aW9uIG9uIHRoZQo+ICAgIENSVEMg ZGlzcGxheWVkIHdpbmRvdy4gIFRvIHB1dCBpdCBhbm90aGVyIHdheSwgdGhlIHNyY18qIGRlZmlu ZSB0aGUKPiAgICByZWdpb24gb2YgdGhlIHNvdXJjZSBtYXRlcmlhbCB0aGF0IGlzIG1hcHBlZCBv bnRvIGEgcmVjdGFuZ3VsYXIgYXJlYQo+ICAgIG9uIHRoZSBkaXNwbGF5IGRlZmluZWQgYnkgY3J0 Y18qLgo+IAo+IFRoaXMgZ290IGlnbm9yZWQsIGFuZCBvbiAyMXN0IE5vdmVtYmVyLCBJIGNhbWUg dXAgd2l0aCBhbiBpbml0aWFsIHBhdGNoCj4gdG8gc29sdmUgaXQgYXQgdGhlIHRpbWUsIGJ1dCB3 ZSB3ZXJlIGJ1c3kgZGlzY3Vzc2luZyB0aGUgYmFzZSBhZGRyZXNzCj4gaXNzdWUuCgpBcyBJJ3Zl IHJlcGxpZWQgdG8gdGhlIDIwdGggRmVicnVhcnkgZW1haWwsIEkgZGlkIG5vdCBpZ25vcmUgaXQs IGp1c3QgbG9zdAp0cmFjayBvZiBpdC4KCj4gCj4gSSBzZW50IGEgcmVtaW5kZXIgb24gMjB0aCBG ZWJydWFyeSBhYm91dCBpdCwgYW5kIHdlIGRpc2N1c3NlZCBpdCwgYW5kIEkKPiBzYWlkIGF0IHRo ZSB0aW1lIEkgZGlkIG5vdCBoYXZlIHRpbWUgdG8gdGVzdCB5b3VyIHBhdGNoLiAgVmlsbGUgY29t bWVudGVkCj4gb24geW91ciBwYXRjaCwgd2hpY2ggY29uZnVzZWQgbWUgYSBsaXR0bGUsIGJ1dCBo YXZpbmcgd29ya2VkIGl0IG91dCwgSQo+IHJld29ya2VkIHRoZSBwYXRjaCBmcm9tIDIxc3QgTm92 ZW1iZXIgdG8gZml4IHRoYXQsIGNyZWF0aW5nIHRoaXMgcGF0Y2gKPiBzZXJpZXMuCj4gCj4gSSBk aWQgbm90IHBvc3QgaXQsIGJlY2F1c2UgSSBoYWRuJ3QgdGVzdGVkIGl0IChzaW5jZSB0aGUgSnVu byByZXF1aXJlcwo+IGEgbG9uZy13aW5kZWQgd2F5IHRvIHVwZGF0ZSB0aGUga2VybmVsKSwgc28g SSBuZXZlciBnb3QgYXJvdW5kIHRvCj4gdGVzdGluZyB0aGlzLiAgU28sIHRoaXMgc2VyaWVzIHBy ZS1kYXRlcyB5b3VyIHYyIHBhdGNoIGJ5IGEgZ29vZCBmZXcKPiB3ZWVrcy4KClRoYXQgaW5mb3Jt YXRpb24gd2FzIHByaXZ5IHRvIHlvdSBhbmQgd291bGQndmUgYmVlbiBuaWNlIHRvIHNoYXJlIHdp dGggbWUuCgo+IAo+IFlvdSBwb3N0ZWQgeW91ciB2MiBwYXRjaCBvbiBNYXJjaCA4dGgsIGFuZCBJ J3ZlIG5vdCBoYWQgYSBjaGFuY2UgdG8KPiB0ZXN0IHRoYXQsIG5vciBoYXZlIEkgaGFkIGEgY2hh bmNlIHRvIHRlc3QgbXkgb3duIHRocmVlIHBhdGNoIHNlcmllcy4KPiAKPiBUb2RheSwgSSBub3Rp Y2VkIHRoYXQgSSBzdGlsbCBoYWQgdGhlIHRocmVlIHBhdGNoIHNlcmllcywgc28gSSB0aG91Z2h0 Cj4gSSBvdWdodCB0byBnZXQgaXQgb3V0IGluIHRoZSB3aWxkLgoKSGV5LCBsb29rLCBhIGNsYXNp YyBjYXNlIG9mIGNvbWVkeSBvZiBlcnJvcnMgd2hlbiBwZW9wbGUgZG9uJ3QgdGFsayB0byBlYWNo IG90aGVyISEhCgpTb3JyeSwgUnVzc2VsbCwgYnV0IEknbSBub3QgaW5zaWRlIHlvdXIgYnJhaW4g b3IgeW91ciBjb21wdXRlciwgc28gSSBkb24ndCBrbm93CmludGltYXRlbHkgdGhlIGhpc3Rvcnkg b2YgdGhpbmdzLiBUaGFua3MgZm9yIGVubGlnaHRlbmluZyBtZSBidXQgSSBhbHNvCnJlYWQgeW91 ciBzdG9yeSBhcyBhIGNsZWFyIGV4YW1wbGUgd2h5IGEgc21hbGwgcGFyYWdyYXBoIGFmdGVyIHRo ZSBjb21taXQKbG9nIGV4cGxhaW5pbmcgd2h5IHRoaXMgc2VyaWVzIGhhcyBiZWVuIHN1Ym1pdHRl ZCB3b3VsZCBoYXZlIGdvbmUgYSBsb25nCndheSBjbGVhcmluZyB0aGUgZm9nLgoKPiAKPiBOb3cs IGR1ZSB0byB0aGUgYW1vdW50IG9mIHBhdGNoZXMgSSBjYXJyeToKPiAKPiAkIGdpdCBsZyBvcmln aW4uLiB8IHdjIC1sCj4gNDkxCj4gCj4gSSB3b3JrIGFnYWluc3QgTGludXMnIHRyZWUgX29ubHlf LCBzbyBhbGwgcGF0Y2hlcyBJIHBvc3QgYXJlIGJhc2VkIG9uCj4gTGludXMnIGtlcm5lbCwgYW5k IG5vdCByYW5kb20gb3RoZXIgZ2l0IHRyZWVzLiAgSSBkbyBub3QgcmFuZG9tbHkgZmV0Y2gKPiBv dGhlciBnaXQgdHJlZXMgdG8gYmFzZSBwYXRjaGVzIG9uIHRoZW0sIGJlY2F1c2UgdGhhdCB3b3Vs ZCBjYXVzZSBtZQo+IGluc2FuZSBtZXJnaW5nIGlzc3VlcyBzbyB0aGF0IEkgY2FuIHRlc3QgaGFs ZiB0aGUgc3R1ZmYgSSdtIGNhcnJ5aW5nLgoKSSB1bmRlcnN0YW5kICh0byB0aGUgYmVzdCBvZiBt eSBhYmlsaXRpZXMpIHlvdXIgcG9zaXRpb24gYW5kIHRoZSBmYWN0IHRoYXQKYXMgYSBtYWludGFp bmVyIG9mIGEgbGFyZ2Ugc3Vic3lzdGVtIHlvdSBoYXZlIGEgc3BlY2lmaWMgd29ya2Zsb3cgdGhh dCB5b3UKZG9uJ3Qgd2FudCB0byBwb2x1dGUgd2l0aCBtaW5vciBleGNlcHRpb25zLiBJIHdvdWxk IGFsc28gYXNrIHlvdSB0byB1bmRlcnN0YW5kCnRoYXQgbm90IGV2ZXJ5b25lIHdvcmtzIGluIHRo ZSBzYW1lIHdheSBhcyB5b3UgYW5kIHRoYXQgb3RoZXIgbWFpbnRhaW5lcnMKYW5kIG90aGVyIHN1 YnN5c3RlbXMgaGF2ZSBkaWZmZXJlbnQgd29ya2Zsb3dzIGFuZCByZXF1aXJlbWVudHMuIEhhdmlu ZyBteQp0cmVlIGFzIHBhcnQgb2YgdGhlIERSTSBzdWJ0cmVlIG1lYW5zIHRoYXQgd2Ugd29yayBt b3N0bHkgb24gdGhlIHJlY2VudApMaW51cycgLXJjIHVwIHVudGlsIGFib3V0IC1yYzQgYW5kIHRo ZSBxdWlja2x5IHN3aXRjaCB0byBsaW51eC1uZXh0LiBTbyBvbmUKd2F5IG9mIGFwcHJvYWNoaW5n IHRoaXMgaXMgdG8gZHJvcCB0aGUgYXJjaC9hcm0gZnJhbWUgb2YgdGhvdWdodCB3aGVuCmNvbnRy aWJ1dGluZyB0byBvdGhlciB0cmVlcyBhbmQgYWRvcHQgdGhlaXIgd29ya2Zsb3cgKHlvdSBrbm93 LCB0aGUgIndoZW4KaW4gUm9tZSwgZG8gd2hhdCByb21hbnMgZG8iIGF0dGl0dWRlKS4gVGhlIG90 aGVyIHdheSBpcyB0byBnbyB0byBkaWZmZXJlbnQKbWFpbnRhaW5lcnMgYW5kIGFzayBmb3Igc3Bl Y2lhbCBzdGF0dXMgYW5kIHRlbGwgdGhlbSB0aGF0IHRoZWlyIHB1bnkgZHJpdmVycwphbmQgdHJl ZSBkb24ndCBtYXR0ZXIgdGhhdCBtdWNoIGNvbXBhcmVkIHRvIHlvdXIgbWlnaHR5IHdvcmtmbG93 IGFuZCB0aGV5CnNob3VsZCBhbGwgYmVuZCB0byB5b3VyIGdyZWF0bmVzcyAodGhlICJhbGwgeW91 ciBiYXNlcyBiZWxvbmcgdG8gdXMiIG1pbmRzZXQpLgoKPiAKPiBOb3csIGl0J3MgdHJ1ZSB0aGF0 IHRoZXkncmUgbm90IGFnYWluc3QgLXJjLCBidXQgYXJlIGN1cnJlbnRseSBhZ2FpbnN0Cj4gNC4x MCAtIGl0IHNlZW1zIHRoYXQgSSBtaXNzZWQgcmViYXNpbmcgX3RoaXNfIHBhcnRpY3VsYXIgYnJh bmNoIHRvCj4gNC4xMS1yYyB5ZXQsIGFsdGhvdWdoIG1vc3Qgb2YgbXkgb3RoZXIgYnJhbmNoZXMg YXJlLgoKQW5kIGhvdyB3b3VsZCB5b3UgaGF2ZSBoYW5kbGVkIHRoaXMgc2l0dWF0aW9uPyBBbm90 aGVyIG1haW50YWluZXIgc2VuZGluZwp5b3UgYSBwYXRjaHNldCBiYXNlZCBvbiBhbiBvbGRlciB0 cmVlIHRoYXQgZG9lc24ndCBtYXRjaCB5b3VyIGN1cnJlbnRseQpwdWJsaXNoZWQgb25lPyBXb3Vs ZCB5b3UgaGF2ZSBnb25lIHRvIHRoZSB0cm91YmxlIG9mIHJlYmFzaW5nIHRoZWlyIHdvcmssCm9y IGFzayB0aGVtIGRvIGdldCBiYWNrIHRvIHlvdSB3aXRoIGEgYmV0dGVyIHNlcmllcz8KCj4gCj4g V2hhdCB3YXMgSSBzbyBidXN5IHdpdGggd2hlbiB5b3UgcG9zdGVkIHlvdXIgcGF0Y2ggb24gdGhl IDh0aCBNYXJjaD8KPiBPaCB5ZXMsIHRoYXQgd2FzIHRoZSB3ZWVrIF9hZnRlcl8gdGhlIG1lcmdl IHdpbmRvdyBjbG9zZWQsIGFuZCB3YXMgdGhlCj4gd2VlayBJIHdhcyBkb2luZyB0aGUgbWFzcyBy ZWJhc2Ugb2YgYWJvdXQgNTAwIHBhdGNoZXMuCgpJJ3ZlIHNlZW4gcGVvcGxlIHNlbmRpbmcgZW1h aWxzIChub3Qgd2hpbGUgdGhleSBhcmUgYnVzeSwgOHRoIE1hcmNoIHdhcyAzCndlZWtzIHByaW9y IHRvIHRoaXMgd2hvbGUgdGhyZWFkKSBzYXlpbmcgIkknbSBzdGlsbCBpbnRlcmVzdGVkIGluIHRo aXMsIEkKd2lsbCB0ZXN0IGl0IHdoZW4gSSBnZXQgYSBjaGFuY2UiLiBPciBzb21ldGhpbmcgbW9y ZSBwZXJzb25hbCwgbGlrZSAiSQphbHJlYWR5IGhhZCBzb21ldGhpbmcgaW4gbXkgdHJlZSwgaWYg SSBkb24ndCBnZXQgYXJvdW5kIHRlc3RpbmcgeW91cgpwYXRjaCB0aGVuIEkgd2lsbCBwdWJsaXNo IG1pbmUgYmFzZWQgb24gYW4gb2xkIHRyZWUsIHNvIGJlIHByZXBhcmVkIgoKPiAKPiBTb3JyeSBJ IGRpZG4ndCBnZXQgYXJvdW5kIHRvIHRlc3RpbmcgeW91ciBwYXRjaCwgYW5kIHNvcnJ5IGZvciBl dmVudHVhbGx5Cj4gZ2V0dGluZyBhcm91bmQgdG8gcG9zdGluZyBteSBwYXRjaGVzLiAgT2J2aW91 c2x5LCBJIHNob3VsZCBqdXN0IGZ1Y2sgb2ZmCj4gYW5kIGRvIHNvbWV0aGluZyBlbHNlLgoKUGxl YXNlIGRvbid0ISBJIGFtIGhhcHB5IHRoYXQgeW91IHNob3cgaW50ZXJlc3QgaW4gdGhpcyBkcml2 ZXIgYW5kIHlvdSBoZWxwCm1lIGltcHJvdmUgaXQsIGFuZCBJIGRvIG5vdCB3aXNoIHRvIHJlcGVh dCB0aGUgY3VycmVudCBjb25mbGljdCBpbiB0aGUgZnV0dXJlLgpJJ20ganVzdCBob3BpbmcgdGhh dCB3ZSBjYW4gaW1wcm92ZSB0aGUgZmxvdyBvZiBpbmZvcm1hdGlvbiBiZXR3ZWVuIHVzIGFuZApn aXZlIHlvdSBtb3JlIHBlYWNlIG9mIG1pbmQuCgpCZXN0IHJlZ2FyZHMsCkxpdml1CgoKPiAKPiAt LSAKPiBSTUsncyBQYXRjaCBzeXN0ZW06IGh0dHA6Ly93d3cuYXJtbGludXgub3JnLnVrL2RldmVs b3Blci9wYXRjaGVzLwo+IEZUVEMgYnJvYWRiYW5kIGZvciAwLjhtaWxlIGxpbmU6IGN1cnJlbnRs eSBhdCA5LjZNYnBzIGRvd24gNDAwa2JwcyB1cAo+IGFjY29yZGluZyB0byBzcGVlZHRlc3QubmV0 LgoKLS0gCj09PT09PT09PT09PT09PT09PT09CnwgSSB3b3VsZCBsaWtlIHRvIHwKfCBmaXggdGhl IHdvcmxkLCAgfAp8IGJ1dCB0aGV5J3JlIG5vdCB8CnwgZ2l2aW5nIG1lIHRoZSAgIHwKIFwgc291 cmNlIGNvZGUhICAvCiAgLS0tLS0tLS0tLS0tLS0tCiAgICDCr1xfKOODhClfL8KvCl9fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5n IGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVk ZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=