Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1362925

Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver

Show all headers | View raw

Hello,

On 03/18/2016 07:53 PM, Doug Anderson wrote:
> Hi,
> 
> On Thu, Mar 17, 2016 at 11:41 PM, Caesar Wang <caesar.upstream@gmail.com> wrote:
>> Hi all,
>>
>> I pick this series up and test on C101PA chromebook, after Heiko update the
>> 3 patches.
>>
>> [v14.1,09/17] drm: rockchip: dp: add rockchip platform dp driver
>> [v14.1,04/17] drm: bridge: analogix/dp: fix some obvious code style
>> [v14.1,01/17] drm: bridge: analogix/dp: split exynos dp driver to bridge
>> directory
>>
>> C101PA chromebook:
>> http://www.amazon.co.uk/10-1-Inch-Chromebook-Rockchip-Integrated-Graphics/dp/B0107N7WT6/ref=lp_758129031_1_4?s=computers&ie=UTF8&qid=1458282912&sr=1-4
>>
>> So, verified for the rockchip eDp contrller on my github.
>> https://github.com/Caesar-github/rockchip/tree/veyron/next-stable-chromeos
> 
> I did similar to Caesar but I tested on veyron-jerry instead of
> veyron-mickey.  Like Caesar, I picked these together Heiko's v14.1
> patches to resolve merge conflicts.  I also picked into a slightly
> different tree (mine was a 4.4-based tree with backports, including
> Heiko's latest eDP device tree patches).
>

I also tested this series on an Exynos5800 Peach Pi Chromebook, using
Heiko's branch [0] that is based on today's mainline (968f3e374fa).

I've seen no regressions with these patches, DP is working fine and
also comes up after a S2R and DPMS on/off using the exynos-drm sysfs.

NOTE: I had to cherry-pick a SPI core fix [1] in order to make the Pi
to boot. Not related to this series of course but I just mention it in
case someone else wants to do the same test on this machine.
 
> Tough I haven't reviewed the code and am not a graphics expert, I'd
> very much love to see this landed, presumably right after the merge
> window closes.  Yakir's first patch
> <https://patchwork.kernel.org/patch/6959211/> is nearly 1 year old and
> it really seems like if there were any major objections left they'd
> have been stated by now.  v14 appears to be about 1 month old with no
> major comments.
>

Same here, this is the second time I tested this series (first time was
v6 on October 25 [2]) and I think that has been out there for too long.
 
> Tested-by: Douglas Anderson <dianders@chromium.org>
> 

Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>

> 
> -Doug
> 

[0]: git://github.com/mmind/linux-rockchip.git tmp/analogix-dp-clean
[1]: http://comments.gmane.org/gmane.linux.kernel.spi.devel/23563
[2]: http://lists.infradead.org/pipermail/linux-rockchip/2015-October/004884.html

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver Caesar Wang <caesar.upstream@gmail.com> - 2016-03-18 07:50 +0100
  Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver Doug Anderson <dianders@chromium.org> - 2016-03-19 00:00 +0100
    Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver Javier Martinez Canillas <javier@osg.samsung.com> - 2016-03-22 20:30 +0100
      Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver] Heiko Stübner <heiko@sntech.de> - 2016-03-22 23:20 +0100
        Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Inki Dae <inki.dae@samsung.com> - 2016-03-22 23:50 +0100
          Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix Core Display Port Driver] Heiko Stübner <heiko@sntech.de> - 2016-03-23 00:00 +0100
            Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Inki Dae <inki.dae@samsung.com> - 2016-03-23 00:10 +0100
              Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-03-23 00:40 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Inki Dae <inki.dae@samsung.com> - 2016-03-23 01:00 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-03-23 01:10 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Dave Airlie <airlied@gmail.com> - 2016-03-23 01:50 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Mark yao <mark.yao@rock-chips.com> - 2016-03-23 02:10 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Thierry Reding <treding@nvidia.com> - 2016-03-23 16:50 +0100
                Re: Who is going to merge it [Was: Re: [PATCH v14 0/17] Add Analogix  Core Display Port Driver] Yakir Yang <ykk@rock-chips.com> - 2016-03-24 12:20 +0100

csiph-web