From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/dp: Adjust i2c-over-aux retry count based on message size and i2c bus speed Date: Wed, 26 Aug 2015 14:43:25 +0300 Message-ID: <20150826114325.GN5176@intel.com> References: <1440514590-30837-1-git-send-email-ville.syrjala@linux.intel.com> <1440515414-31106-1-git-send-email-ville.syrjala@linux.intel.com> <1931029.CGeYQ37jkq@f19simon> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 016416EA23 for ; Wed, 26 Aug 2015 04:43:43 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1931029.CGeYQ37jkq@f19simon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: simon.farnsworth@onelan.com Cc: moosotc@gmail.com, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCBBdWcgMjYsIDIwMTUgYXQgMTI6MTE6NTZQTSArMDEwMCwgc2ltb24uZmFybnN3b3J0 aEBvbmVsYW4uY29tIHdyb3RlOgo+IE9uIFR1ZXNkYXkgMjUgQXVndXN0IDIwMTUgMTg6MTA6MTQg dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20gd3JvdGU6Cj4gPiBGcm9tOiBWaWxsZSBTeXJq w6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgo+ID4gCj4gPiBDYWxjdWxhdGUg dGhlIG51bWJlciBvZiByZXRyaWVzIHdlIHNob3VsZCBkbyBmb3IgZWFjaCBpMmMtb3Zlci1hdXgK PiA+IG1lc3NhZ2UgYmFzZWQgb24gdGhlIHRpbWUgaXQgdGFrZXMgdG8gcGVyZm9ybSB0aGUgaTJj IHRyYW5zZmVyIHZzLiB0aGUKPiA+IGF1eCB0cmFuc2Zlci4KPiA+IAo+ID4gVGhpcyBtb3N0bHkg bWF0Y2hlcyB3aGF0IHRoZSBEUCBzcGVjIHJlY29tbWVuZHMuIFRoZSBudW1iZXJzIGRpZG4ndCBj b21lCj4gPiBvdXQgZXhhY3RseSB0aGUgc2FtZSBhcyB0aGUgdGFibGVzIGluIHRoZSBzcGVjLCBi dXQgSSB0aGluayB0aGVyZSdzCj4gPiBzb21ldGhpbmcgZmlzaHkgYWJvdXQgdGhlIEFVWCB0cmFz bmZlciBzaXplIGNhbGN1bGF0aW9ucyBpbiB0aGUgc3BlYy4KPiA+IEFsc28gdGhlIHdheSB0aGUg aTJjIHRyYW5zZmVyIGxlbmd0aCBpcyBjYWxjdWxhdGVkIGlzIHNvbWV3aGF0IG9wZW4gdG8KPiA+ IGludGVycHJldGF0aW9uLgo+ID4gCj4gPiBOb3RlIHRoYXQgY3VycmVudGx5IHdlIGFzc3VtZSAx MCBrSHogc3BlZWQgZm9yIHRoZSBpMmMgYnVzLiBTb21lIHJlYWwKPiA+IHdvcmxkIGRldmljZXMg KGVnLiBzb21lIEFwcGxlIERQLT5WR0EgZG9uZ2xlKSBmYWlscyB3aXRoIGxlc3MgdGhhbiAxNgo+ ID4gcmV0cmllcywgYW5kIHRoYXQgd291bGQgY29ycmVzcG9uZCB0byBzb21ldGhpbmcgY2xvc2Ug dG8gMjAga0h6LCBzbyB3ZQo+ID4gYXNzdW1lIDEwIGtIeiB0byBiZSBvbiB0aGUgc2FmZSBzaWRl LiBJZGVhbGx5IHdlIHNob3VsZCBxdWVyeS9zZXQgdGhlCj4gPiBpMmMgYnVzIHNwZWVkIHZpYSBE UENEIGJ1dCBmb3Igbm93IHRoaXMgc2hvdWxkIGF0IGxlYWFzdCByZW1vdmUgdGhlCj4gPiByZWdy ZXNzaW9uIGZyb20gdGhlIDEtPjE2IGJ5dGUgdHJhc25mZXIgc2l6ZSBjaGFuZ2UuIEFuZCBvZiBj b3Vyc2UgaWYKPiA+IHRoZSBzaW5rIGNvbXBsZXRlcyB0aGUgdHJhbnNmZXIgcXVpY2tlciB0aGlz IHNob3VsZG4ndCBzbG93IHRoaW5ncyBkb3duCj4gPiBzaW5jZSB3ZSBkb24ndCBjaGFuZ2UgdGhl IGludGVydmFsIGJldHdlZW4gcmV0cmllcy4KPiA+IAo+ID4gRml4ZXMgYSByZWdyZXNzaW9uIHdp dGggc29tZSBEUCBkb25nbGVzIGZyb206Cj4gPiBjb21taXQgMWQwMDJmYTcyMDczOGJjZDBiZGRi OTE3OGU5ZWEwNzczMjg4ZTFkZAo+ID4gQXV0aG9yOiBTaW1vbiBGYXJuc3dvcnRoIDxzaW1vbi5m YXJuc3dvcnRoQG9uZWxhbi5jby51az4KPiA+IERhdGU6ICAgVHVlIEZlYiAxMCAxODozODowOCAy MDE1ICswMDAwCj4gPiAKPiA+ICAgICBkcm0vZHA6IFVzZSBsYXJnZSB0cmFuc2FjdGlvbnMgZm9y IEkyQyBvdmVyIEFVWAo+ID4gCj4gPiBDYzogU2ltb24gRmFybnN3b3J0aCA8c2ltb24uZmFybnN3 b3J0aEBvbmVsYW4uY29tPgo+ID4gQ2M6IG1vb3NvdGNAZ21haWwuY29tCj4gPiBCdWd6aWxsYTog aHR0cHM6Ly9idWdzLmZyZWVkZXNrdG9wLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTE0NTEKPiA+IFNp Z25lZC1vZmYtYnk6IFZpbGxlIFN5cmrDpGzDpCA8dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5j b20+Cj4gPiAtLS0KPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2RwX2hlbHBlci5jIHwgNjQgKysr KysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrLS0KPiA+ICAxIGZpbGUgY2hhbmdl ZCwgNjIgaW5zZXJ0aW9ucygrKSwgMiBkZWxldGlvbnMoLSkKPiA+IAo+ID4gZGlmZiAtLWdpdCBh L2RyaXZlcnMvZ3B1L2RybS9kcm1fZHBfaGVscGVyLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2Rw X2hlbHBlci5jCj4gPiBpbmRleCA4MGEwMmE0Li4yYjZiN2Y5IDEwMDY0NAo+ID4gLS0tIGEvZHJp dmVycy9ncHUvZHJtL2RybV9kcF9oZWxwZXIuYwo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2Ry bV9kcF9oZWxwZXIuYwo+ID4gQEAgLTQyMiw2ICs0MjIsNjEgQEAgc3RhdGljIHUzMiBkcm1fZHBf aTJjX2Z1bmN0aW9uYWxpdHkoc3RydWN0IGkyY19hZGFwdGVyICphZGFwdGVyKQo+ID4gIAkgICAg ICAgSTJDX0ZVTkNfMTBCSVRfQUREUjsKPiA+ICB9Cj4gPiAgCj4gPiArI2RlZmluZSBBVVhfUFJF Q0hBUkdFX0xFTiAxNiAvKiAxMCB0byAxNiAqLwo+IAo+IEknZCBjaGFuZ2UgdGhpcyB0byAxMCAt IHNlZSBiZWxvdyBmb3IgcmVhc29uaW5nLgo+IAo+ID4gKyNkZWZpbmUgQVVYX1NZTkNfTEVOICgx NiArIDQpIC8qIHByZWFtYmxlICsgQVVYX1NZTkNfRU5EICovCj4gPiArI2RlZmluZSBBVVhfU1RP UF9MRU4gNAo+ID4gKyNkZWZpbmUgQVVYX0NNRF9MRU4gNAo+ID4gKyNkZWZpbmUgQVVYX0FERFJF U1NfTEVOIDIwCj4gPiArI2RlZmluZSBBVVhfUkVQTFlfUEFEX0xFTiA0Cj4gPiArI2RlZmluZSBB VVhfTEVOR1RIX0xFTiA4Cj4gPiArCj4gPiArI2RlZmluZSBBVVhfUkVTUE9OU0VfVElNRU9VVCAz MDAKPiA+ICsKPiA+ICtzdGF0aWMgaW50IGRybV9kcF9hdXhfcmVxX2xlbihjb25zdCBzdHJ1Y3Qg ZHJtX2RwX2F1eF9tc2cgKm1zZykKPiA+ICt7Cj4gPiArCWludCBsZW4gPSBBVVhfUFJFQ0hBUkdF X0xFTiArIEFVWF9TWU5DX0xFTiArIEFVWF9TVE9QX0xFTiArCj4gPiArCQlBVVhfQ01EX0xFTiAr IEFVWF9BRERSRVNTX0xFTiArIEFVWF9MRU5HVEhfTEVOOwo+ID4gKwo+ID4gKwlpZiAoKG1zZy0+ cmVxdWVzdCAmIERQX0FVWF9JMkNfUkVBRCkgPT0gMCkKPiA+ICsJCWxlbiArPSBtc2ctPnNpemUg KiA4Owo+ID4gKwo+ID4gKwlyZXR1cm4gbGVuOwo+ID4gK30KPiA+ICsKPiA+ICtzdGF0aWMgaW50 IGRybV9kcF9hdXhfcmVwbHlfbGVuKGNvbnN0IHN0cnVjdCBkcm1fZHBfYXV4X21zZyAqbXNnKQo+ ID4gK3sKPiA+ICsJaW50IGxlbiA9IEFVWF9QUkVDSEFSR0VfTEVOICsgQVVYX1NZTkNfTEVOICsg QVVYX1NUT1BfTEVOICsKPiA+ICsJCUFVWF9DTURfTEVOICsgQVVYX1JFUExZX1BBRF9MRU47Cj4g PiArCj4gPiArCWlmIChtc2ctPnJlcXVlc3QgJiBEUF9BVVhfSTJDX1JFQUQpCj4gPiArCQlsZW4g Kz0gbXNnLT5zaXplICogODsKPiA+ICsJZWxzZQo+ID4gKwkJbGVuICs9IDg7IC8qIDAgb3IgMSBk YXRhIGJ5dGVzIGZvciB3cml0ZSByZXBseSAqLwo+ID4gKwo+ID4gKwlyZXR1cm4gbGVuOwo+ID4g K30KPiA+ICsKPiA+ICsjZGVmaW5lIEkyQ19TVEFSVF9MRU4gMQo+ID4gKyNkZWZpbmUgSTJDX1NU T1BfTEVOIDEKPiA+ICsjZGVmaW5lIEkyQ19BRERSX0xFTiA5IC8qIEFERFJFU1MgKyBSL1cgKyBB Q0svTkFDSyAqLwo+ID4gKyNkZWZpbmUgSTJDX0RBVEFfTEVOIDkgLyogREFUQSArIEFDSy9OQUNL ICovCj4gPiArCj4gPiArc3RhdGljIGludCBkcm1fZHBfaTJjX21zZ19sZW4oY29uc3Qgc3RydWN0 IGRybV9kcF9hdXhfbXNnICptc2csCj4gPiArCQkJICAgICAgaW50IGkyY19zcGVlZF9raHopCj4g PiArewo+ID4gKwlyZXR1cm4gKEkyQ19TVEFSVF9MRU4gKyBJMkNfQUREUl9MRU4gKyBtc2ctPnNp emUgKiBJMkNfREFUQV9MRU4gKwo+ID4gKwkJSTJDX1NUT1BfTEVOKSAqIDEwMDAgLyBpMmNfc3Bl ZWRfa2h6Owo+ID4gK30KPiA+ICsKPiAKPiBUaGlzIGZ1bmN0aW9uIGNhdWdodCBtZSBvdXQgYXQg Zmlyc3QgLSBiZWNhdXNlIHlvdSdyZSBub3QgdHJhY2tpbmcgc3RhdGUKPiBiZXR3ZWVuIHR3byBB VVggdHJhbnNhY3Rpb25zLCB5b3UgYXNzdW1lIHRoYXQgZXZlcnkgRFAgQVVYIHRyYW5zYWN0aW9u IGlzCj4gbWFwcGVkIHRvIGEgc2luZ2xlIEkyQyB0cmFuc2FjdGlvbi4gVGhpcyB0aGVuIGdldHMg eW91IHRoZSB3b3JzdCBjYXNlIHRpbWluZ3MKPiBmb3IgdGhlIEkyQyB0cmFuc2FjdGlvbi4KClll YWggSSBkaWQgY29uc2lkZXIgbWFraW5nIGl0IG1vcmUgYWNjdXJhdGUgYnV0IHRoZW4gdGhvdWdo dCBpdCdzIGVhc2llcgp0byBqdXN0IGFzc3VtZSB3b3JzdCBjYXNlIGJlaGF2aW91ci4KCj4gCj4g PiArc3RhdGljIGludCBkcm1fZHBfaTJjX3JldHJ5X2NvdW50KGNvbnN0IHN0cnVjdCBkcm1fZHBf YXV4X21zZyAqbXNnLAo+ID4gKwkJCSAgICAgIGludCBpMmNfc3BlZWRfa2h6KQo+ID4gK3sKPiA+ ICsJaW50IGF1eF9sZW4gPSBkcm1fZHBfYXV4X3JlcV9sZW4obXNnKSArIGRybV9kcF9hdXhfcmVw bHlfbGVuKG1zZykgKyBBVVhfUkVTUE9OU0VfVElNRU9VVDsKPiA+ICsJaW50IGkyY19sZW4gPSBk cm1fZHBfaTJjX21zZ19sZW4obXNnLCBpMmNfc3BlZWRfa2h6KSAtIEFVWF9SRVNQT05TRV9USU1F T1VUOwo+ID4gKwo+ID4gKwlyZXR1cm4gRElWX1JPVU5EX1VQKGkyY19sZW4sIGF1eF9sZW4pOwo+ ID4gK30KPiA+ICsKPiAKPiBUaGUgbG9naWMgaGVyZSBsb29rcyB3cm9uZyB0byBtZSAtIGF1eF9s ZW4gaXMgdGhlIHdvcnN0IGNhc2UgdGltZSBmb3IgYQo+IHNpbmdsZSBBVVggdHJhbnNhY3Rpb24s IHdoaWxlIEknZCBleHBlY3QgaXQgdG8gYmUgdGhlIGJlc3QgY2FzZSB0aW1lIChhcyBpdCdzCj4g dGhlIGRpdmlzb3IpLiBTaW1pbGFybHksIEknZCBleHBlY3QgaTJjX2xlbiB0byBiZSB0aGUgd29y c3QgY2FzZSB0aW1lIGZvciB0aGUKPiBJMkMgdHJhbnNhY3Rpb24sIGJ1dCB0aGVuIHlvdSBzdWJ0 cmFjdCBhIHJlc3BvbnNlIHRpbWVvdXQgZnJvbSBpdC4KPiAKPiBJJ2QgY2hhbmdlIEFVWF9QUkVD SEFSR0VfTEVOIHRvIDEwLCBzbyB0aGF0IGRybV9kcF9hdXhfcmVxX2xlbiBhbmQKPiBkcm1fZHBf YXV4X3JlcGx5X2xlbiByZXR1cm4gYmVzdCBjYXNlIHRpbWVzIGZvciB0aGUgQVVYIHRyYW5zYWN0 aW9uLiBUaGVuCj4gdGhpcyBmdW5jdGlvbiBiZWNvbWVzOgo+IAo+IHN0YXRpYyBpbnQgZHJtX2Rw X2kyY19yZXRyeV9jb3VudChjb25zdCBzdHJ1Y3QgZHJtX2RwX2F1eF9tc2cgKm1zZywKPiAJCQkg ICAgICBpbnQgaTJjX3NwZWVkX2toeikKPiB7Cj4gCWludCBhdXhfbGVuID0gZHJtX2RwX2F1eF9y ZXFfbGVuKG1zZykgKyBkcm1fZHBfYXV4X3JlcGx5X2xlbihtc2cpOwo+IAlpbnQgaTJjX2xlbiA9 IGRybV9kcF9pMmNfbXNnX2xlbihtc2csIGkyY19zcGVlZF9raHopOwo+IAo+IAlyZXR1cm4gRElW X1JPVU5EX1VQKGkyY19sZW4gKyBBVVhfUkVTUE9OU0VfVElNRU9VVCwgYXV4X2xlbik7Cj4gfQo+ IAo+IEluIG90aGVyIHdvcmRzLCByZXRyeSB1bnRpbCB0aGUgYmVzdCBjYXNlIHRpbWluZ3MgZm9y IHRoZSBBVVggdHJhbnNhY3Rpb25zCj4gZXhjZWVkIHRoZSB3b3JzdCBjYXNlIHRpbWluZ3MgZm9y IHRoZSBJMkMgdHJhbnNhY3Rpb24gcGx1cyB0aGUgQVVYIHJlc3BvbnNlCj4gdGltZW91dC4KClll YWggdGhhdCB3b3VsZCBtYWtlIHNlbnNlLiBJdCBkaWQgY3Jvc3MgbXkgbWluZCwgYnV0IEkga2Vw dCB0aGUKY2FsY3VsYXRpb24gYXMgdGhlIHNwZWMgaGFkIGl0IChtb3JlIG9yIGxlc3MpLiBCdXQg aXQncyBoYXJkIHRvIGFyZ3VlCndpdGggdGhlIGxvZ2ljIG9mIGNvbXBhcmluZyB0aGUgd29yc3Qg Y2FzZSBpMmMgdG8gdGhlIGJlc3QgY2FzZSBhdXgsIHNvCkknbGwgcmVzcGluZyB0aGUgcGF0Y2gg d2l0aCB0aGF0IGNoYW5nZS4gSSdsbCBhbHNvIG5lZWQgdG8gcmVtb3ZlIHRoZQonbGVuICs9IDgn IGZyb20gdGhlIGF1eCB3cml0ZSByZXBseSBlc3RpbWF0ZS4KCj4gCj4gPiAgLyoKPiA+ICAgKiBU cmFuc2ZlciBhIHNpbmdsZSBJMkMtb3Zlci1BVVggbWVzc2FnZSBhbmQgaGFuZGxlIHZhcmlvdXMg ZXJyb3IgY29uZGl0aW9ucywKPiA+ICAgKiByZXRyeWluZyB0aGUgdHJhbnNhY3Rpb24gYXMgYXBw cm9wcmlhdGUuICBJdCBpcyBhc3N1bWVkIHRoYXQgdGhlCj4gPiBAQCAtNDM0LDEzICs0ODksMTgg QEAgc3RhdGljIGludCBkcm1fZHBfaTJjX2RvX21zZyhzdHJ1Y3QgZHJtX2RwX2F1eCAqYXV4LCBz dHJ1Y3QgZHJtX2RwX2F1eF9tc2cgKm1zZykKPiA+ICB7Cj4gPiAgCXVuc2lnbmVkIGludCByZXRy eSwgZGVmZXJfaTJjOwo+ID4gIAlpbnQgcmV0Owo+ID4gLQo+ID4gIAkvKgo+ID4gIAkgKiBEUDEu MiBzZWN0aW9ucyAyLjcuNy4xLjUuNi4xIGFuZCAyLjcuNy4xLjYuNi4xOiBBIERQIFNvdXJjZSBk ZXZpY2UKPiA+ICAJICogaXMgcmVxdWlyZWQgdG8gcmV0cnkgYXQgbGVhc3Qgc2V2ZW4gdGltZXMg dXBvbiByZWNlaXZpbmcgQVVYX0RFRkVSCj4gPiAgCSAqIGJlZm9yZSBnaXZpbmcgdXAgdGhlIEFV WCB0cmFuc2FjdGlvbi4KPiA+ICsJICoKPiA+ICsJICogV2UgYWxzbyB0cnkgdG8gYWNjb3VudCBm b3IgdGhlIGkyYyBidXMgc3BlZWQuCj4gPiArCSAqIEZJWE1FIGN1cnJlbnRseSBhc3N1bWVzIDEw IGtIeiBhcyBzb21lIHJlYWwgd29ybGQgZGV2aWNlcyBzZWVtCj4gPiArCSAqIHRvIHJlcXVpcmUg aXQuIFdlIHNob3VsZCBxdWVyeS9zZXQgdGhlIHNwZWVkIHZpYSBEUENEIGlmIHN1cHBvcnRlZC4K PiA+ICAJICovCj4gPiAtCWZvciAocmV0cnkgPSAwLCBkZWZlcl9pMmMgPSAwOyByZXRyeSA8ICg3 ICsgZGVmZXJfaTJjKTsgcmV0cnkrKykgewo+ID4gKwlpbnQgbWF4X3JldHJpZXMgPSBtYXgoNywg ZHJtX2RwX2kyY19yZXRyeV9jb3VudChtc2csIDEwKSk7Cj4gPiArCj4gPiArCWZvciAocmV0cnkg PSAwLCBkZWZlcl9pMmMgPSAwOyByZXRyeSA8IChtYXhfcmV0cmllcyArIGRlZmVyX2kyYyk7IHJl dHJ5KyspIHsKPiA+ICAJCW11dGV4X2xvY2soJmF1eC0+aHdfbXV0ZXgpOwo+ID4gIAkJcmV0ID0g YXV4LT50cmFuc2ZlcihhdXgsIG1zZyk7Cj4gPiAgCQltdXRleF91bmxvY2soJmF1eC0+aHdfbXV0 ZXgpOwo+ID4gCj4gCj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwo+IFRoaXMgZW1haWwgaGFzIGJlZW4gc2Nhbm5l ZCBieSB0aGUgU3ltYW50ZWMgRW1haWwgU2VjdXJpdHkuY2xvdWQgc2VydmljZS4KPiBGb3IgbW9y ZSBpbmZvcm1hdGlvbiBwbGVhc2UgdmlzaXQgaHR0cDovL3d3dy5zeW1hbnRlY2Nsb3VkLmNvbQo+ IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxp c3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==