From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?ISO-8859-1?Q?St=FCbner?= Subject: Re: [PATCH v10 0/17] Add Analogix Core Display Port Driver Date: Tue, 15 Dec 2015 01:06:30 +0100 Message-ID: <4659407.pYb0RRCQQX@diego> References: <1449470239-30667-1-git-send-email-ykk@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <1449470239-30667-1-git-send-email-ykk@rock-chips.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Yakir Yang Cc: Inki Dae , Mark Yao , Jingoo Han , Thierry Reding , Krzysztof Kozlowski , Rob Herring , Andrzej Hajda , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , Russell King , emil.l.velikov@gmail.com, Gustavo Padovan , Kishon Vijay Abraham I , ajaynumb@gmail.com, javier@osg.samsung.com, Andy Yan , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hi Yakir, Am Montag, 7. Dezember 2015, 14:37:19 schrieb Yakir Yang: > The Samsung Exynos eDP controller and Rockchip RK3288 eDP controller > share the same IP, so a lot of parts can be re-used. I split the common > code into bridge directory, then rk3288 and exynos only need to keep > some platform code. Cause I can't find the exact IP name of exynos dp > controller, so I decide to name dp core driver with "analogix" which I > find in rk3288 eDP TRM so it looks like the hotplug works nicely now. I was able to test it sucessfully on both a Jerry and a Minnie device without needing to force hotplug :-) . As I needed to adapt some patches when applying the lastest ones, I think it would be good for a full send of the latest version as v11. When going over the patches before sending, please also fix the indentation issues in analogix_dp_core.h - both newly added elements to analogix_dp_device use spaces between type and name, where the rest uses tabs. [This should of course be fixed in the patches adding these lines :-) ] Heiko