From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v2 3/5] drm/i915: Split broadwell_load_luts() into smaller functions Date: Thu, 26 Jan 2017 15:05:26 +0200 Message-ID: <20170126130526.GD31595@intel.com> References: <1485429865-10687-1-git-send-email-ander.conselvan.de.oliveira@intel.com> <1485429865-10687-4-git-send-email-ander.conselvan.de.oliveira@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2D27C6E267 for ; Thu, 26 Jan 2017 13:05:30 +0000 (UTC) Content-Disposition: inline In-Reply-To: <1485429865-10687-4-git-send-email-ander.conselvan.de.oliveira@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ander Conselvan de Oliveira Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBKYW4gMjYsIDIwMTcgYXQgMDE6MjQ6MjNQTSArMDIwMCwgQW5kZXIgQ29uc2VsdmFu IGRlIE9saXZlaXJhIHdyb3RlOgo+IFNwbGl0IHRoZSBsb2dpYyBmb3IgcHJvZ2FtbWluZyBlYWNo IExVVCBvdXQgb2YgYnJvYWR3ZWxsX2xvYWRfbHV0cygpLCBzbwo+IHdlIGNhbiByZXVzZSBwYXJ0 IG9mIGl0IGZvciBnZW1pbmlsYWtlLgo+IAo+IENjOiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5 cmphbGFAbGludXguaW50ZWwuY29tPgo+IFNpZ25lZC1vZmYtYnk6IEFuZGVyIENvbnNlbHZhbiBk ZSBPbGl2ZWlyYSA8YW5kZXIuY29uc2VsdmFuLmRlLm9saXZlaXJhQGludGVsLmNvbT4KClJldmll d2VkLWJ5OiBWaWxsZSBTeXJqw6Rsw6QgPHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwuY29tPgoK PiAtLS0KPiAgZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9yZWcuaCAgICB8ICAxICsKPiAgZHJp dmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfY29sb3IuYyB8IDQzICsrKysrKysrKysrKysrKysrKysr KysrKysrKystLS0tLS0tLS0tCj4gIDIgZmlsZXMgY2hhbmdlZCwgMzMgaW5zZXJ0aW9ucygrKSwg MTEgZGVsZXRpb25zKC0pCj4gCj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5 MTVfcmVnLmggYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X3JlZy5oCj4gaW5kZXggOTk0NzM1 NC4uMDZiYmU1NSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pOTE1X3JlZy5o Cj4gKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9yZWcuaAo+IEBAIC04MTcxLDYgKzgx NzEsNyBAQCBlbnVtIHsKPiAgI2RlZmluZSAgIFBBTF9QUkVDXzEwXzEyX0JJVAkJKDAgPDwgMzEp Cj4gICNkZWZpbmUgICBQQUxfUFJFQ19TUExJVF9NT0RFCQkoMSA8PCAzMSkKPiAgI2RlZmluZSAg IFBBTF9QUkVDX0FVVE9fSU5DUkVNRU5UCSgxIDw8IDE1KQo+ICsjZGVmaW5lICAgUEFMX1BSRUNf SU5ERVhfVkFMVUVfTUFTSwkoMHgzZmYgPDwgMCkKPiAgI2RlZmluZSBfUEFMX1BSRUNfREFUQV9B CTB4NEE0MDQKPiAgI2RlZmluZSBfUEFMX1BSRUNfREFUQV9CCTB4NEFDMDQKPiAgI2RlZmluZSBf UEFMX1BSRUNfREFUQV9DCTB4NEI0MDQKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJtL2k5 MTUvaW50ZWxfY29sb3IuYyBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2NvbG9yLmMKPiBp bmRleCAzNDk1MmQwLi44MmEzYmM5IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1 L2ludGVsX2NvbG9yLmMKPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9jb2xvci5j Cj4gQEAgLTM0MCwyMCArMzQwLDEyIEBAIHN0YXRpYyB2b2lkIGhhc3dlbGxfbG9hZF9sdXRzKHN0 cnVjdCBkcm1fY3J0Y19zdGF0ZSAqY3J0Y19zdGF0ZSkKPiAgCQloc3dfZW5hYmxlX2lwcyhpbnRl bF9jcnRjKTsKPiAgfQo+ICAKPiAtLyogTG9hZHMgdGhlIHBhbGV0dGUvZ2FtbWEgdW5pdCBmb3Ig dGhlIENSVEMgb24gQnJvYWR3ZWxsKy4gKi8KPiAtc3RhdGljIHZvaWQgYnJvYWR3ZWxsX2xvYWRf bHV0cyhzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKnN0YXRlKQo+ICtzdGF0aWMgdm9pZCBiZHdfbG9h ZF9kZWdhbW1hX2x1dChzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKnN0YXRlKQo+ICB7Cj4gLQlzdHJ1 Y3QgZHJtX2NydGMgKmNydGMgPSBzdGF0ZS0+Y3J0YzsKPiAtCXN0cnVjdCBkcm1faTkxNV9wcml2 YXRlICpkZXZfcHJpdiA9IHRvX2k5MTUoY3J0Yy0+ZGV2KTsKPiAtCXN0cnVjdCBpbnRlbF9jcnRj X3N0YXRlICppbnRlbF9zdGF0ZSA9IHRvX2ludGVsX2NydGNfc3RhdGUoc3RhdGUpOwo+IC0JZW51 bSBwaXBlIHBpcGUgPSB0b19pbnRlbF9jcnRjKGNydGMpLT5waXBlOwo+ICsJc3RydWN0IGRybV9p OTE1X3ByaXZhdGUgKmRldl9wcml2ID0gdG9faTkxNShzdGF0ZS0+Y3J0Yy0+ZGV2KTsKPiArCWVu dW0gcGlwZSBwaXBlID0gdG9faW50ZWxfY3J0YyhzdGF0ZS0+Y3J0YyktPnBpcGU7Cj4gIAl1aW50 MzJfdCBpLCBsdXRfc2l6ZSA9IElOVEVMX0lORk8oZGV2X3ByaXYpLT5jb2xvci5kZWdhbW1hX2x1 dF9zaXplOwo+ICAKPiAtCWlmIChjcnRjX3N0YXRlX2lzX2xlZ2FjeShzdGF0ZSkpIHsKPiAtCQlo YXN3ZWxsX2xvYWRfbHV0cyhzdGF0ZSk7Cj4gLQkJcmV0dXJuOwo+IC0JfQo+IC0KPiAgCUk5MTVf V1JJVEUoUFJFQ19QQUxfSU5ERVgocGlwZSksCj4gIAkJICAgUEFMX1BSRUNfU1BMSVRfTU9ERSB8 IFBBTF9QUkVDX0FVVE9fSU5DUkVNRU5UKTsKPiAgCj4gQEAgLTM3Nyw2ICszNjksMTggQEAgc3Rh dGljIHZvaWQgYnJvYWR3ZWxsX2xvYWRfbHV0cyhzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKnN0YXRl KQo+ICAJCQkJICAgKHYgPDwgMjApIHwgKHYgPDwgMTApIHwgdik7Cj4gIAkJfQo+ICAJfQo+ICt9 Cj4gKwo+ICtzdGF0aWMgdm9pZCBiZHdfbG9hZF9nYW1tYV9sdXQoc3RydWN0IGRybV9jcnRjX3N0 YXRlICpzdGF0ZSwgdTMyIG9mZnNldCkKPiArewo+ICsJc3RydWN0IGRybV9pOTE1X3ByaXZhdGUg KmRldl9wcml2ID0gdG9faTkxNShzdGF0ZS0+Y3J0Yy0+ZGV2KTsKPiArCWVudW0gcGlwZSBwaXBl ID0gdG9faW50ZWxfY3J0YyhzdGF0ZS0+Y3J0YyktPnBpcGU7Cj4gKwl1aW50MzJfdCBpLCBsdXRf c2l6ZSA9IElOVEVMX0lORk8oZGV2X3ByaXYpLT5jb2xvci5nYW1tYV9sdXRfc2l6ZTsKPiArCj4g KwlXQVJOX09OKG9mZnNldCAmIH5QQUxfUFJFQ19JTkRFWF9WQUxVRV9NQVNLKTsKPiArCj4gKwlJ OTE1X1dSSVRFKFBSRUNfUEFMX0lOREVYKHBpcGUpLAo+ICsJCSAgIFBBTF9QUkVDX1NQTElUX01P REUgfCBQQUxfUFJFQ19BVVRPX0lOQ1JFTUVOVCB8IG9mZnNldCk7Cj4gIAo+ICAJaWYgKHN0YXRl LT5nYW1tYV9sdXQpIHsKPiAgCQlzdHJ1Y3QgZHJtX2NvbG9yX2x1dCAqbHV0ID0KPiBAQCAtNDEw LDYgKzQxNCwyMyBAQCBzdGF0aWMgdm9pZCBicm9hZHdlbGxfbG9hZF9sdXRzKHN0cnVjdCBkcm1f Y3J0Y19zdGF0ZSAqc3RhdGUpCj4gIAkJSTkxNV9XUklURShQUkVDX1BBTF9HQ19NQVgocGlwZSwg MSksICgxIDw8IDE2KSAtIDEpOwo+ICAJCUk5MTVfV1JJVEUoUFJFQ19QQUxfR0NfTUFYKHBpcGUs IDIpLCAoMSA8PCAxNikgLSAxKTsKPiAgCX0KPiArfQo+ICsKPiArLyogTG9hZHMgdGhlIHBhbGV0 dGUvZ2FtbWEgdW5pdCBmb3IgdGhlIENSVEMgb24gQnJvYWR3ZWxsKy4gKi8KPiArc3RhdGljIHZv aWQgYnJvYWR3ZWxsX2xvYWRfbHV0cyhzdHJ1Y3QgZHJtX2NydGNfc3RhdGUgKnN0YXRlKQo+ICt7 Cj4gKwlzdHJ1Y3QgZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYgPSB0b19pOTE1KHN0YXRlLT5j cnRjLT5kZXYpOwo+ICsJc3RydWN0IGludGVsX2NydGNfc3RhdGUgKmludGVsX3N0YXRlID0gdG9f aW50ZWxfY3J0Y19zdGF0ZShzdGF0ZSk7Cj4gKwllbnVtIHBpcGUgcGlwZSA9IHRvX2ludGVsX2Ny dGMoc3RhdGUtPmNydGMpLT5waXBlOwo+ICsKPiArCWlmIChjcnRjX3N0YXRlX2lzX2xlZ2FjeShz dGF0ZSkpIHsKPiArCQloYXN3ZWxsX2xvYWRfbHV0cyhzdGF0ZSk7Cj4gKwkJcmV0dXJuOwo+ICsJ fQo+ICsKPiArCWJkd19sb2FkX2RlZ2FtbWFfbHV0KHN0YXRlKTsKPiArCWJkd19sb2FkX2dhbW1h X2x1dChzdGF0ZSwKPiArCQkJICAgSU5URUxfSU5GTyhkZXZfcHJpdiktPmNvbG9yLmRlZ2FtbWFf bHV0X3NpemUpOwo+ICAKPiAgCWludGVsX3N0YXRlLT5nYW1tYV9tb2RlID0gR0FNTUFfTU9ERV9N T0RFX1NQTElUOwo+ICAJSTkxNV9XUklURShHQU1NQV9NT0RFKHBpcGUpLCBHQU1NQV9NT0RFX01P REVfU1BMSVQpOwo+IC0tIAo+IDIuNS41CgotLSAKVmlsbGUgU3lyasOkbMOkCkludGVsIE9UQwpf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZngg bWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK