Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755919AbbLQKGQ (ORCPT ); Thu, 17 Dec 2015 05:06:16 -0500 Received: from gloria.sntech.de ([95.129.55.99]:47465 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755837AbbLQKGL (ORCPT ); Thu, 17 Dec 2015 05:06:11 -0500 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Chris Zhong Cc: linux-rockchip@lists.infradead.org, mark.yao@rock-chips.com, treding@nvidia.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Kumar Gala , Ian Campbell , Rob Herring , David Airlie , Pawel Moll , Mark Rutland , Russell King , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v6 0/6] Add mipi dsi support for rk3288 Date: Thu, 17 Dec 2015 10:29:53 +0100 Message-ID: <3523259.ZRgH1Shd9Y@diego> User-Agent: KMail/4.14.10 (Linux/4.2.0-1-amd64; KDE/4.14.13; x86_64; ; ) In-Reply-To: <1450260616-21160-1-git-send-email-zyw@rock-chips.com> References: <1450260616-21160-1-git-send-email-zyw@rock-chips.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 Content-Length: 2816 Lines: 72 Hi Chris, Am Mittwoch, 16. Dezember 2015, 18:10:10 schrieb Chris Zhong: > The rk3288 MIPI DSI is a Synopsys DesignWare MIPI DSI host controller > IP. This series adds support for a Synopsys DesignWare MIPI DSI host > controller DRM driver. > > The MIPI DSI feature is tested on rk3288 evb board, backport them to > chrome os kernel chrome_v3.14, and it can display normally. > > This patchset is base on the patchset from Ying.liu@freescale.com. > > > According to the suggestion from Thierry, I have get rid of the bridge, > and register the encoder & connecter in drm/rockchip/dw-mipi-dsi.c. from my point it looks good (although I cannot really speak on the drm-related internals). But I do think we'll be doing atomic modesetting conversion first (last version posted from Mark today), so you should probably make sure your dsi driver still works after that conversion and make changes if needed. Heiko > > > Changes in v6: > - update the document, since the bridge device has been deleted. > - Do not use bridge driver (Thierry Reding) > - Optimization the phy init sequence > > Changes in v5: > - modify the clk name to SCLK_MIPIDSI_24M > - add a blank line befor lcd_en > > Changes in v4: > - use clk_round_rate to check the clock rate in vop_crtc_mode_fixup > > Changes in v3: > - move dw_mipi_dsi_rockchip.txt to bindings/display/rockchip/ > > Chris Zhong (5): > drm/rockchip: return a true clock rate to adjusted_mode > Documentation: dt-bindings: Add bindings for rk3288 DW MIPI DSI driver > drm: rockchip: Support Synopsys DW MIPI DSI > ARM: dts: rockchip: add rk3288 mipi_dsi nodes > ARM: dts: rockchip: add support mipi panel tv080wum-nl0 > > Liu Ying (1): > drm/dsi: Add a helper to get bits per pixel of MIPI DSI pixel format > > .../display/rockchip/dw_mipi_dsi_rockchip.txt | 60 + > arch/arm/boot/dts/rk3288-evb.dtsi | 20 +- > arch/arm/boot/dts/rk3288.dtsi | 39 + > drivers/gpu/drm/rockchip/Kconfig | 10 + > drivers/gpu/drm/rockchip/Makefile | 1 + > drivers/gpu/drm/rockchip/dw-mipi-dsi.c | 1200 > ++++++++++++++++++++ drivers/gpu/drm/rockchip/rockchip_drm_vop.c | > 8 + > include/drm/drm_mipi_dsi.h | 14 + > 8 files changed, 1351 insertions(+), 1 deletion(-) > create mode 100644 > Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt > create mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi.c -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/