From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark yao Subject: Re: [PATCH v7 3/5] drm: rockchip: Support Synopsys DW MIPI DSI Date: Wed, 06 Jan 2016 14:30:19 +0800 Message-ID: <568CB47B.1010205@rock-chips.com> References: <1452053038-32098-1-git-send-email-zyw@rock-chips.com> <1452053038-32098-4-git-send-email-zyw@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1452053038-32098-4-git-send-email-zyw@rock-chips.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Chris Zhong , heiko@sntech.de, linux-rockchip@lists.infradead.org, treding@nvidia.com Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org List-Id: linux-rockchip.vger.kernel.org SGkgQ2hyaXMKCkxvb2tzIGdvb2QgZm9yIG1lIGV4Y2VwdCBzb21lIHRpbnkgaXNzdWVzLgoKCk9u IDIwMTblubQwMeaciDA25pelIDEyOjAzLCBDaHJpcyBaaG9uZyB3cm90ZToKPiArc3RydWN0IGR3 X21pcGlfZHNpIHsKPiArCXN0cnVjdCBkcm1fZW5jb2RlciBlbmNvZGVyOwo+ICsJc3RydWN0IGRy bV9jb25uZWN0b3IgY29ubmVjdG9yOwo+ICsJc3RydWN0IG1pcGlfZHNpX2hvc3QgZHNpX2hvc3Q7 Cj4gKwlzdHJ1Y3QgZHJtX3BhbmVsICpwYW5lbDsKPiArCXN0cnVjdCBkZXZpY2UgKmRldjsKPiAr CXN0cnVjdCByZWdtYXAgKmdyZl9yZWdtYXA7Cj4gKwl2b2lkIF9faW9tZW0gKmJhc2U7Cj4gKwo+ ICsJc3RydWN0IGNsayAqcGxscmVmX2NsazsKPiArCXN0cnVjdCBjbGsgKnBjbGs7Cj4gKwo+ICsJ dW5zaWduZWQgaW50IGxhbmVfbWJwczsgLyogcGVyIGxhbmUgKi8KPiArCXUzMiBjaGFubmVsOwo+ ICsJdTMyIGxhbmVzOwo+ICsJdTMyIGZvcm1hdDsKPiArCXUxNiBpbnB1dF9kaXY7Cj4gKwl1MTYg ZmVlZGJhY2tfZGl2Owo+ICsJc3RydWN0IGRybV9kaXNwbGF5X21vZGUgKm1vZGU7Cj4gKwo+ICsJ Y29uc3Qgc3RydWN0IGR3X21pcGlfZHNpX3BsYXRfZGF0YSAqcGRhdGE7Cj4gKwoKcmVtb3ZlIHRo aXMgYmxhbmsgbGluZS4KCj4gK307Cj4gKwoKWy4uLl0KCj4gK3N0YXRpYyB2b2lkIGR3X21pcGlf ZHNpX2VuY29kZXJfbW9kZV9zZXQoc3RydWN0IGRybV9lbmNvZGVyICplbmNvZGVyLAo+ICsJCQkJ CXN0cnVjdCBkcm1fZGlzcGxheV9tb2RlICptb2RlLAo+ICsJCQkJCXN0cnVjdCBkcm1fZGlzcGxh eV9tb2RlICphZGp1c3RlZF9tb2RlKQo+ICt7Cj4gKwlzdHJ1Y3QgZHdfbWlwaV9kc2kgKmRzaSA9 IGVuY29kZXJfdG9fZHNpKGVuY29kZXIpOwo+ICsJaW50IHJldDsKPiArCj4gKwlkc2ktPm1vZGUg PSBhZGp1c3RlZF9tb2RlOwo+ICsKPiArCXJldCA9IGR3X21pcGlfZHNpX2dldF9sYW5lX2Jwcyhk c2kpOwo+ICsJaWYgKHJldCA8IDApCj4gKwkJcmV0dXJuOwo+ICsKPiArCWlmIChjbGtfcHJlcGFy ZV9lbmFibGUoZHNpLT5wY2xrKSkgewo+ICsJCWRldl9lcnIoZHNpLT5kZXYsICIlczogRmFpbGVk IHRvIGVuYWJsZSBwY2xrXG4iLCBfX2Z1bmNfXyk7Cj4gKwkJcmV0dXJuOwo+ICsJfQo+ICsKPiAr CWR3X21pcGlfZHNpX2luaXQoZHNpKTsKPiArCWR3X21pcGlfZHNpX2RwaV9jb25maWcoZHNpLCBt b2RlKTsKPiArCWR3X21pcGlfZHNpX3BhY2tldF9oYW5kbGVyX2NvbmZpZyhkc2kpOwo+ICsJZHdf bWlwaV9kc2lfdmlkZW9fbW9kZV9jb25maWcoZHNpKTsKPiArCWR3X21pcGlfZHNpX3ZpZGVvX3Bh Y2tldF9jb25maWcoZHNpLCBtb2RlKTsKPiArCWR3X21pcGlfZHNpX2NvbW1hbmRfbW9kZV9jb25m aWcoZHNpKTsKPiArCWR3X21pcGlfZHNpX2xpbmVfdGltZXJfY29uZmlnKGRzaSk7Cj4gKwlkd19t aXBpX2RzaV92ZXJ0aWNhbF90aW1pbmdfY29uZmlnKGRzaSk7Cj4gKwlkd19taXBpX2RzaV9kcGh5 X3RpbWluZ19jb25maWcoZHNpKTsKPiArCWR3X21pcGlfZHNpX2RwaHlfaW50ZXJmYWNlX2NvbmZp Zyhkc2kpOwo+ICsJZHdfbWlwaV9kc2lfY2xlYXJfZXJyKGRzaSk7Cj4gKwlpZiAoZHJtX3BhbmVs X3ByZXBhcmUoZHNpLT5wYW5lbCkpIHsKPiArCQlkZXZfZXJyKGRzaS0+ZGV2LCAiZmFpbGVkIHRv IHByZXBhcmUgcGFuZWxcbiIpOwoKRG9uJ3QgZm9yZ2V0IGRvaW5nIGNsa19kaXNhYmxlX3VucHJl cGFyZSB3aGVuIHJ1biBpbnRvIGVycm9yIHBhdGguCgo+ICsJCXJldHVybjsKPiArCX0KPiArCWNs a19kaXNhYmxlX3VucHJlcGFyZShkc2ktPnBjbGspOwo+ICt9Cj4gKwoKCi0tIArvvK1hcmsgWWFv CgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRl dmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9s aXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.yao@rock-chips.com (Mark yao) Date: Wed, 06 Jan 2016 14:30:19 +0800 Subject: [PATCH v7 3/5] drm: rockchip: Support Synopsys DW MIPI DSI In-Reply-To: <1452053038-32098-4-git-send-email-zyw@rock-chips.com> References: <1452053038-32098-1-git-send-email-zyw@rock-chips.com> <1452053038-32098-4-git-send-email-zyw@rock-chips.com> Message-ID: <568CB47B.1010205@rock-chips.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Chris Looks good for me except some tiny issues. On 2016?01?06? 12:03, Chris Zhong wrote: > +struct dw_mipi_dsi { > + struct drm_encoder encoder; > + struct drm_connector connector; > + struct mipi_dsi_host dsi_host; > + struct drm_panel *panel; > + struct device *dev; > + struct regmap *grf_regmap; > + void __iomem *base; > + > + struct clk *pllref_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; > + remove this blank line. > +}; > + [...] > +static void dw_mipi_dsi_encoder_mode_set(struct drm_encoder *encoder, > + struct drm_display_mode *mode, > + struct drm_display_mode *adjusted_mode) > +{ > + struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder); > + int ret; > + > + dsi->mode = adjusted_mode; > + > + ret = dw_mipi_dsi_get_lane_bps(dsi); > + if (ret < 0) > + return; > + > + if (clk_prepare_enable(dsi->pclk)) { > + dev_err(dsi->dev, "%s: Failed to enable pclk\n", __func__); > + return; > + } > + > + dw_mipi_dsi_init(dsi); > + dw_mipi_dsi_dpi_config(dsi, mode); > + dw_mipi_dsi_packet_handler_config(dsi); > + dw_mipi_dsi_video_mode_config(dsi); > + dw_mipi_dsi_video_packet_config(dsi, mode); > + dw_mipi_dsi_command_mode_config(dsi); > + dw_mipi_dsi_line_timer_config(dsi); > + dw_mipi_dsi_vertical_timing_config(dsi); > + dw_mipi_dsi_dphy_timing_config(dsi); > + dw_mipi_dsi_dphy_interface_config(dsi); > + dw_mipi_dsi_clear_err(dsi); > + if (drm_panel_prepare(dsi->panel)) { > + dev_err(dsi->dev, "failed to prepare panel\n"); Don't forget doing clk_disable_unprepare when run into error path. > + return; > + } > + clk_disable_unprepare(dsi->pclk); > +} > + -- ?ark Yao From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752133AbcAFGae (ORCPT ); Wed, 6 Jan 2016 01:30:34 -0500 Received: from regular1.263xmail.com ([211.150.99.135]:48857 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751341AbcAFGab (ORCPT ); Wed, 6 Jan 2016 01:30:31 -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: mark.yao@rock-chips.com X-FST-TO: linux-arm-kernel@lists.infradead.org X-SENDER-IP: 58.22.7.114 X-LOGIN-NAME: mark.yao@rock-chips.com X-UNIQUE-TAG: <92d2b46af2fdd8abafc4472baf670b50> X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Message-ID: <568CB47B.1010205@rock-chips.com> Date: Wed, 06 Jan 2016 14:30:19 +0800 From: Mark yao User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-Version: 1.0 To: Chris Zhong , heiko@sntech.de, linux-rockchip@lists.infradead.org, treding@nvidia.com CC: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, David Airlie , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v7 3/5] drm: rockchip: Support Synopsys DW MIPI DSI References: <1452053038-32098-1-git-send-email-zyw@rock-chips.com> <1452053038-32098-4-git-send-email-zyw@rock-chips.com> In-Reply-To: <1452053038-32098-4-git-send-email-zyw@rock-chips.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Chris Looks good for me except some tiny issues. On 2016年01月06日 12:03, Chris Zhong wrote: > +struct dw_mipi_dsi { > + struct drm_encoder encoder; > + struct drm_connector connector; > + struct mipi_dsi_host dsi_host; > + struct drm_panel *panel; > + struct device *dev; > + struct regmap *grf_regmap; > + void __iomem *base; > + > + struct clk *pllref_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; > + remove this blank line. > +}; > + [...] > +static void dw_mipi_dsi_encoder_mode_set(struct drm_encoder *encoder, > + struct drm_display_mode *mode, > + struct drm_display_mode *adjusted_mode) > +{ > + struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder); > + int ret; > + > + dsi->mode = adjusted_mode; > + > + ret = dw_mipi_dsi_get_lane_bps(dsi); > + if (ret < 0) > + return; > + > + if (clk_prepare_enable(dsi->pclk)) { > + dev_err(dsi->dev, "%s: Failed to enable pclk\n", __func__); > + return; > + } > + > + dw_mipi_dsi_init(dsi); > + dw_mipi_dsi_dpi_config(dsi, mode); > + dw_mipi_dsi_packet_handler_config(dsi); > + dw_mipi_dsi_video_mode_config(dsi); > + dw_mipi_dsi_video_packet_config(dsi, mode); > + dw_mipi_dsi_command_mode_config(dsi); > + dw_mipi_dsi_line_timer_config(dsi); > + dw_mipi_dsi_vertical_timing_config(dsi); > + dw_mipi_dsi_dphy_timing_config(dsi); > + dw_mipi_dsi_dphy_interface_config(dsi); > + dw_mipi_dsi_clear_err(dsi); > + if (drm_panel_prepare(dsi->panel)) { > + dev_err(dsi->dev, "failed to prepare panel\n"); Don't forget doing clk_disable_unprepare when run into error path. > + return; > + } > + clk_disable_unprepare(dsi->pclk); > +} > + -- Ｍark Yao