From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751996AbbKZHEL (ORCPT ); Thu, 26 Nov 2015 02:04:11 -0500 Received: from regular1.263xmail.com ([211.150.99.131]:36234 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750930AbbKZHEJ (ORCPT ); Thu, 26 Nov 2015 02:04:09 -0500 X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-ADDR-CHECKED: 0 X-RL-SENDER: zyw@rock-chips.com X-FST-TO: linux-kernel@vger.kernel.org X-SENDER-IP: 58.22.7.114 X-LOGIN-NAME: zyw@rock-chips.com X-UNIQUE-TAG: <2a44aa406ac8fca840932fef4432c4e8> X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Subject: Re: [PATCH v4 06/13] drm/bridge: Add Synopsys DesignWare MIPI DSI host controller driver To: Thierry Reding References: <1448007339-10966-1-git-send-email-zyw@rock-chips.com> <1448007339-10966-7-git-send-email-zyw@rock-chips.com> <20151120160706.GE3300@ulmo.nvidia.com> Cc: heiko@sntech.de, linux-rockchip@lists.infradead.org, mark.yao@rock-chips.com, emil.l.velikov@gmail.com, airlied@linux.ie, ajaykumar.rs@samsung.com, rmk+kernel@arm.linux.org.uk, dri-devel@lists.freedesktop.org, Inki Dae , Takashi Iwai , Vincent Palatin , Andy Yan , Rahul Sharma , Sean Paul , Andrew Bresticker , linux-kernel@vger.kernel.org From: Chris Zhong Message-ID: <5656AEDA.4080607@rock-chips.com> Date: Thu, 26 Nov 2015 15:03:54 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20151120160706.GE3300@ulmo.nvidia.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Thierry Thanks for your feedback. On 11/21/2015 12:07 AM, Thierry Reding wrote: > On Fri, Nov 20, 2015 at 04:15:32PM +0800, Chris Zhong wrote: >> add Synopsys DesignWare MIPI DSI host controller driver support. >> >> Signed-off-by: Chris Zhong >> --- >> >> Changes in v4: >> eliminate some warnning >> >> Changes in v3: None >> Changes in v2: None >> >> drivers/gpu/drm/bridge/Kconfig | 11 + >> drivers/gpu/drm/bridge/Makefile | 1 + >> drivers/gpu/drm/bridge/dw_mipi_dsi.c | 1056 ++++++++++++++++++++++++++++++++++ >> include/drm/bridge/dw_mipi_dsi.h | 27 + >> 4 files changed, 1095 insertions(+) >> create mode 100644 drivers/gpu/drm/bridge/dw_mipi_dsi.c >> create mode 100644 include/drm/bridge/dw_mipi_dsi.h >> >> >> +struct dw_mipi_dsi { >> + struct mipi_dsi_host dsi_host; >> + struct drm_connector connector; >> + struct drm_encoder *encoder; > struct drm_bridge already has a pointer to an encoder, can't you reuse > that instead? > >> + struct drm_bridge *bridge; > Typically you'd embed the bridge into the driver structure. I'm almost done with all the modifications according to your comments. Remaining this point I can not understand, the dsi is a pointer of drm_bridge, hence I can not get the bridge by container_of. Although, the "bridge->driver_private = dsi" have done in dw_mipi_dsi_register. > >> + struct drm_panel *panel; >> + struct device *dev; >> + >> + void __iomem *base; >> + >> + struct clk *pllref_clk; >> + struct clk *cfg_clk; >> + struct clk *pclk; >> + >> + unsigned int lane_mbps; /* per lane */ >> + u32 channel; >> + u32 lanes; >> + u32 format; >> + u16 input_div; >> + u16 feedback_div; >> + struct drm_display_mode *mode; >> + >> + const struct dw_mipi_dsi_plat_data *pdata; >> + >> + bool enabled; >> +}; >> + >> >> >> Thierry From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Zhong Subject: Re: [PATCH v4 06/13] drm/bridge: Add Synopsys DesignWare MIPI DSI host controller driver Date: Thu, 26 Nov 2015 15:03:54 +0800 Message-ID: <5656AEDA.4080607@rock-chips.com> References: <1448007339-10966-1-git-send-email-zyw@rock-chips.com> <1448007339-10966-7-git-send-email-zyw@rock-chips.com> <20151120160706.GE3300@ulmo.nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20151120160706.GE3300@ulmo.nvidia.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Thierry Reding Cc: Vincent Palatin , Andrew Bresticker , emil.l.velikov@gmail.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, Andy Yan , rmk+kernel@arm.linux.org.uk, Rahul Sharma , ajaykumar.rs@samsung.com List-Id: linux-rockchip.vger.kernel.org SGkgVGhpZXJyeQoKVGhhbmtzIGZvciB5b3VyIGZlZWRiYWNrLgoKCk9uIDExLzIxLzIwMTUgMTI6 MDcgQU0sIFRoaWVycnkgUmVkaW5nIHdyb3RlOgo+IE9uIEZyaSwgTm92IDIwLCAyMDE1IGF0IDA0 OjE1OjMyUE0gKzA4MDAsIENocmlzIFpob25nIHdyb3RlOgo+PiBhZGQgU3lub3BzeXMgRGVzaWdu V2FyZSBNSVBJIERTSSBob3N0IGNvbnRyb2xsZXIgZHJpdmVyIHN1cHBvcnQuCj4+Cj4+IFNpZ25l ZC1vZmYtYnk6IENocmlzIFpob25nIDx6eXdAcm9jay1jaGlwcy5jb20+Cj4+IC0tLQo+Pgo+PiBD aGFuZ2VzIGluIHY0Ogo+PiBlbGltaW5hdGUgc29tZSB3YXJubmluZwo+Pgo+PiBDaGFuZ2VzIGlu IHYzOiBOb25lCj4+IENoYW5nZXMgaW4gdjI6IE5vbmUKPj4KPj4gICBkcml2ZXJzL2dwdS9kcm0v YnJpZGdlL0tjb25maWcgICAgICAgfCAgIDExICsKPj4gICBkcml2ZXJzL2dwdS9kcm0vYnJpZGdl L01ha2VmaWxlICAgICAgfCAgICAxICsKPj4gICBkcml2ZXJzL2dwdS9kcm0vYnJpZGdlL2R3X21p cGlfZHNpLmMgfCAxMDU2ICsrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysKPj4gICBp bmNsdWRlL2RybS9icmlkZ2UvZHdfbWlwaV9kc2kuaCAgICAgfCAgIDI3ICsKPj4gICA0IGZpbGVz IGNoYW5nZWQsIDEwOTUgaW5zZXJ0aW9ucygrKQo+PiAgIGNyZWF0ZSBtb2RlIDEwMDY0NCBkcml2 ZXJzL2dwdS9kcm0vYnJpZGdlL2R3X21pcGlfZHNpLmMKPj4gICBjcmVhdGUgbW9kZSAxMDA2NDQg aW5jbHVkZS9kcm0vYnJpZGdlL2R3X21pcGlfZHNpLmgKPj4KPj4KPj4gK3N0cnVjdCBkd19taXBp X2RzaSB7Cj4+ICsJc3RydWN0IG1pcGlfZHNpX2hvc3QgZHNpX2hvc3Q7Cj4+ICsJc3RydWN0IGRy bV9jb25uZWN0b3IgY29ubmVjdG9yOwo+PiArCXN0cnVjdCBkcm1fZW5jb2RlciAqZW5jb2RlcjsK PiBzdHJ1Y3QgZHJtX2JyaWRnZSBhbHJlYWR5IGhhcyBhIHBvaW50ZXIgdG8gYW4gZW5jb2Rlciwg Y2FuJ3QgeW91IHJldXNlCj4gdGhhdCBpbnN0ZWFkPwo+Cj4+ICsJc3RydWN0IGRybV9icmlkZ2Ug KmJyaWRnZTsKPiBUeXBpY2FsbHkgeW91J2QgZW1iZWQgdGhlIGJyaWRnZSBpbnRvIHRoZSBkcml2 ZXIgc3RydWN0dXJlLgpJJ20gYWxtb3N0IGRvbmUgd2l0aCBhbGwgdGhlIG1vZGlmaWNhdGlvbnMg YWNjb3JkaW5nIHRvIHlvdXIgY29tbWVudHMuClJlbWFpbmluZyB0aGlzIHBvaW50IEkgY2FuIG5v dCB1bmRlcnN0YW5kLCB0aGUgZHNpIGlzIGEgcG9pbnRlciBvZiAKZHJtX2JyaWRnZSwKaGVuY2Ug SSBjYW4gbm90IGdldCB0aGUgYnJpZGdlIGJ5IGNvbnRhaW5lcl9vZi4KQWx0aG91Z2gsIHRoZSAi YnJpZGdlLT5kcml2ZXJfcHJpdmF0ZSA9IGRzaSIgaGF2ZSBkb25lIGluIApkd19taXBpX2RzaV9y ZWdpc3Rlci4KCj4KPj4gKwlzdHJ1Y3QgZHJtX3BhbmVsICpwYW5lbDsKPj4gKwlzdHJ1Y3QgZGV2 aWNlICpkZXY7Cj4+ICsKPj4gKwl2b2lkIF9faW9tZW0gKmJhc2U7Cj4+ICsKPj4gKwlzdHJ1Y3Qg Y2xrICpwbGxyZWZfY2xrOwo+PiArCXN0cnVjdCBjbGsgKmNmZ19jbGs7Cj4+ICsJc3RydWN0IGNs ayAqcGNsazsKPj4gKwo+PiArCXVuc2lnbmVkIGludCBsYW5lX21icHM7IC8qIHBlciBsYW5lICov Cj4+ICsJdTMyIGNoYW5uZWw7Cj4+ICsJdTMyIGxhbmVzOwo+PiArCXUzMiBmb3JtYXQ7Cj4+ICsJ dTE2IGlucHV0X2RpdjsKPj4gKwl1MTYgZmVlZGJhY2tfZGl2Owo+PiArCXN0cnVjdCBkcm1fZGlz cGxheV9tb2RlICptb2RlOwo+PiArCj4+ICsJY29uc3Qgc3RydWN0IGR3X21pcGlfZHNpX3BsYXRf ZGF0YSAqcGRhdGE7Cj4+ICsKPj4gKwlib29sIGVuYWJsZWQ7Cj4+ICt9Owo+PiArCj4+Cj4+Cj4+ IFRoaWVycnkKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK aHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=