From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up Date: Tue, 18 Jul 2017 16:39:43 +0300 Message-ID: <1549201.DAvZbLDVYF@avalon> References: <1500378232-21424-1-git-send-email-philippe.cornu@st.com> <1500378232-21424-3-git-send-email-philippe.cornu@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [IPv6:2001:4b98:dc2:45:216:3eff:febb:480d]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7C8CA6E07A for ; Tue, 18 Jul 2017 13:39:36 +0000 (UTC) In-Reply-To: <1500378232-21424-3-git-send-email-philippe.cornu@st.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Philippe CORNU Cc: Maxime Coquelin , Alexandre Torgue , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Yannick Fertre , Fabien Dessenne , Mickael Reulier , Vincent Abriou , Gabriel Fernandez , Ludovic Barre List-Id: dri-devel@lists.freedesktop.org SGkgUGhpbGlwcGUsCgpUaGFuayB5b3UgZm9yIHRoZSBwYXRjaC4KCk9uIFR1ZXNkYXkgMTggSnVs IDIwMTcgMTM6NDM6NTIgUGhpbGlwcGUgQ09STlUgd3JvdGU6Cj4gVGhpcyBwYXRjaCBjbGVhbnMg dXAgdGhlIFN5bm9wc3lzIG1pcGkgZHNpIHJlZ2lzdGVyIGxpc3Q6Cj4gLSByZW1vdmUgdW51c2Vk IHJlZ2lzdGVycwoKSXMgdGhlIGRvY3VtZW50YXRpb24gZm9yIHRoZSBEU0kgdHJhbnNtaXR0ZXIg Y29yZSBwdWJsaWMgPyBJZiBub3QsIGl0IGNvdWxkIGJlIAp1c2VmdWwgdG8ga2VlcCB1bnVzZWQg cmVnaXN0ZXJzIGZvciByZWZlcmVuY2UuIFRoYXQncyBuaW5lIGxpbmVzIG9ubHkuCgo+IC0gcmVu YW1lIHJlZ2lzdGVycyBhY2NvcmRpbmcgdG8gdGhlIFN5bm9wc3lzIGRvY3VtZW50YXRpb24KPiAg ICgxLjMwICYgMS4zMSkKPiAtIGZpeCB0eXBvcwo+IC0gcmUtb3JkZXIgcmVnaXN0ZXJzIGZvciBh IGJldHRlciBjb2hlcmVuY3kKCkZvciB0aGUgcmVzdCwKClJldmlld2VkLWJ5OiBMYXVyZW50IFBp bmNoYXJ0IDxsYXVyZW50LnBpbmNoYXJ0QGlkZWFzb25ib2FyZC5jb20+Cgo+IFNpZ25lZC1vZmYt Ynk6IFBoaWxpcHBlIENPUk5VIDxwaGlsaXBwZS5jb3JudUBzdC5jb20+Cj4gLS0tCj4gIGRyaXZl cnMvZ3B1L2RybS9icmlkZ2Uvc3lub3BzeXMvZHctbWlwaS1kc2kuYyB8IDc3ICsrKysrKysrKy0t LS0tLS0tLS0tLS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgMzMgaW5zZXJ0aW9ucygrKSwgNDQgZGVs ZXRpb25zKC0pCgotLSAKUmVnYXJkcywKCkxhdXJlbnQgUGluY2hhcnQKCl9fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QK ZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9w Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752022AbdGRNji (ORCPT ); Tue, 18 Jul 2017 09:39:38 -0400 Received: from galahad.ideasonboard.com ([185.26.127.97]:57720 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751534AbdGRNjg (ORCPT ); Tue, 18 Jul 2017 09:39:36 -0400 From: Laurent Pinchart To: Philippe CORNU Cc: Archit Taneja , Andrzej Hajda , David Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Yannick Fertre , Benjamin Gaignard , Vincent Abriou , Alexandre Torgue , Maxime Coquelin , Gabriel Fernandez , Ludovic Barre , Fabien Dessenne , Mickael Reulier Subject: Re: [PATCH v1 2/2] drm/bridge/synopsys: dsi: Register list clean up Date: Tue, 18 Jul 2017 16:39:43 +0300 Message-ID: <1549201.DAvZbLDVYF@avalon> User-Agent: KMail/4.14.10 (Linux/4.9.34-gentoo; KDE/4.14.32; x86_64; ; ) In-Reply-To: <1500378232-21424-3-git-send-email-philippe.cornu@st.com> References: <1500378232-21424-1-git-send-email-philippe.cornu@st.com> <1500378232-21424-3-git-send-email-philippe.cornu@st.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Philippe, Thank you for the patch. On Tuesday 18 Jul 2017 13:43:52 Philippe CORNU wrote: > This patch cleans up the Synopsys mipi dsi register list: > - remove unused registers Is the documentation for the DSI transmitter core public ? If not, it could be useful to keep unused registers for reference. That's nine lines only. > - rename registers according to the Synopsys documentation > (1.30 & 1.31) > - fix typos > - re-order registers for a better coherency For the rest, Reviewed-by: Laurent Pinchart > Signed-off-by: Philippe CORNU > --- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 77 +++++++++--------------- > 1 file changed, 33 insertions(+), 44 deletions(-) -- Regards, Laurent Pinchart