From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752269AbbJGLZq (ORCPT ); Wed, 7 Oct 2015 07:25:46 -0400 Received: from lists.s-osg.org ([54.187.51.154]:47627 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750952AbbJGLZn (ORCPT ); Wed, 7 Oct 2015 07:25:43 -0400 Subject: Re: [PATCH v5 0/17] Add Analogix Core Display Port Driver To: Yakir Yang , Inki Dae , Andrzej Hajda , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , Jingoo Han , Heiko Stuebner , Mark Yao , Thierry Reding , joe@perches.com, Krzysztof Kozlowski , Rob Herring References: <1441086371-24838-1-git-send-email-ykk@rock-chips.com> <1442906428-2609-1-git-send-email-ykk@rock-chips.com> <5614BADD.1070307@rock-chips.com> <5614DBE2.6080102@osg.samsung.com> <5614DFBC.8060105@rock-chips.com> <5614E553.6060108@osg.samsung.com> <5614FC6B.4080702@rock-chips.com> Cc: David Airlie , Russell King , djkurtz@chromium.org, dianders@chromium.org, Sean Paul , Kukjin Kim , Kumar Gala , emil.l.velikov@gmail.com, Ian Campbell , Gustavo Padovan , Kishon Vijay Abraham I , Pawel Moll , ajaynumb@gmail.com, robherring2@gmail.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 From: Javier Martinez Canillas X-Enigmail-Draft-Status: N1110 Message-ID: <56150128.2070309@osg.samsung.com> Date: Wed, 7 Oct 2015 13:25:28 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <5614FC6B.4080702@rock-chips.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Yakir, On 10/07/2015 01:05 PM, Yakir Yang wrote: > Hi Javier, > > On 10/07/2015 05:26 PM, Javier Martinez Canillas wrote: >> Hello Yakir, >> >> On 10/07/2015 11:02 AM, Yakir Yang wrote: >>> Hi Javier, >>> >>> On 10/07/2015 04:46 PM, Javier Martinez Canillas wrote: >>>> Hello Yakir, >>>> >>>> On 10/07/2015 08:25 AM, Yakir Yang wrote: >>>>> Hi all, >>>>> >>>>> Friendly ping..... :) >>>>> >>>>> >>>>> Best regards, >>>>> - Yakir >>>>> >>>>> >>>> Do you have a tree that I can use to test these patches? >>> Wow, thanks a lot, I do have a tree on github [https://github.com/yakir-Yang/linux/tree/analogix_dp], >>> crossing my finger, wish things works...... ;) >>> >> I tried your analogix_dp branch on an Exynos5800 Peach Pi Chromebook >> but the machine didn't boot. Unfortunately I need to do some soldering >> to have a serial console on this board so don't have a kernel boot log. >> >> I'll let you know if I can get more info about this issue. > > Whoops, sorry for the failed, much appreciated for your works. > > Besides, I thought maybe I can find a Peach Pit Chromebook in my side, > I remember that some of our guys have brought one, but previously I > thought that mainline kernel wouldn't run on Peach Pit directly. > Great, mainline works correctly on all Exynos based Chromebooks. > Maybe you can email me the method the run mainline kernel on Peach > Pit, so I can debug the analogix_dp driver at the same time, that would > be great. I wrote a little blog post explaining how to run mainline on these boards: http://blogs.s-osg.org/install-linux-mainline-kernel-distro-exynos-chromebooks/ That explains the simplest setup though so if you need a different one (i.e: chain loading a non verified u-boot) or if you have any questions, feel free to contact me in private and I can help you with the setup. >> Also, there is Kconfig recursive dependency that you may want to fix: >> >> $ make exynos_defconfig >> drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! >> drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HELPER >> drivers/gpu/drm/Kconfig:34: symbol DRM_KMS_FB_HELPER depends on DRM_KMS_HELPER >> drivers/gpu/drm/Kconfig:28: symbol DRM_KMS_HELPER is selected by DRM_ANALOGIX_DP >> drivers/gpu/drm/bridge/analogix/Kconfig:1: symbol DRM_ANALOGIX_DP is selected by DRM_EXYNOS_DP >> drivers/gpu/drm/exynos/Kconfig:57: symbol DRM_EXYNOS_DP depends on DRM_EXYNOS_FIMD >> drivers/gpu/drm/exynos/Kconfig:19: symbol DRM_EXYNOS_FIMD depends on FB_S3C >> drivers/video/fbdev/Kconfig:2023: symbol FB_S3C depends on FB >> > > Yeah, recursive dependency detected, guess I should remove the > "DRM_KMS_HELPER" from bridge analogix_dp Kconfig file, thanks > for your remind. > > --- a/drivers/gpu/drm/bridge/analogix/Kconfig > +++ b/drivers/gpu/drm/bridge/analogix/Kconfig > @@ -1,4 +1,3 @@ > config DRM_ANALOGIX_DP > tristate > depends on DRM > - select DRM_KMS_HELPER > > That fixes the recursive dependency issue indeed. Thanks. > Thanks, > - Yakir Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America From mboxrd@z Thu Jan 1 00:00:00 1970 From: javier@osg.samsung.com (Javier Martinez Canillas) Date: Wed, 7 Oct 2015 13:25:28 +0200 Subject: [PATCH v5 0/17] Add Analogix Core Display Port Driver In-Reply-To: <5614FC6B.4080702@rock-chips.com> References: <1441086371-24838-1-git-send-email-ykk@rock-chips.com> <1442906428-2609-1-git-send-email-ykk@rock-chips.com> <5614BADD.1070307@rock-chips.com> <5614DBE2.6080102@osg.samsung.com> <5614DFBC.8060105@rock-chips.com> <5614E553.6060108@osg.samsung.com> <5614FC6B.4080702@rock-chips.com> Message-ID: <56150128.2070309@osg.samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Yakir, On 10/07/2015 01:05 PM, Yakir Yang wrote: > Hi Javier, > > On 10/07/2015 05:26 PM, Javier Martinez Canillas wrote: >> Hello Yakir, >> >> On 10/07/2015 11:02 AM, Yakir Yang wrote: >>> Hi Javier, >>> >>> On 10/07/2015 04:46 PM, Javier Martinez Canillas wrote: >>>> Hello Yakir, >>>> >>>> On 10/07/2015 08:25 AM, Yakir Yang wrote: >>>>> Hi all, >>>>> >>>>> Friendly ping..... :) >>>>> >>>>> >>>>> Best regards, >>>>> - Yakir >>>>> >>>>> >>>> Do you have a tree that I can use to test these patches? >>> Wow, thanks a lot, I do have a tree on github [https://github.com/yakir-Yang/linux/tree/analogix_dp], >>> crossing my finger, wish things works...... ;) >>> >> I tried your analogix_dp branch on an Exynos5800 Peach Pi Chromebook >> but the machine didn't boot. Unfortunately I need to do some soldering >> to have a serial console on this board so don't have a kernel boot log. >> >> I'll let you know if I can get more info about this issue. > > Whoops, sorry for the failed, much appreciated for your works. > > Besides, I thought maybe I can find a Peach Pit Chromebook in my side, > I remember that some of our guys have brought one, but previously I > thought that mainline kernel wouldn't run on Peach Pit directly. > Great, mainline works correctly on all Exynos based Chromebooks. > Maybe you can email me the method the run mainline kernel on Peach > Pit, so I can debug the analogix_dp driver at the same time, that would > be great. I wrote a little blog post explaining how to run mainline on these boards: http://blogs.s-osg.org/install-linux-mainline-kernel-distro-exynos-chromebooks/ That explains the simplest setup though so if you need a different one (i.e: chain loading a non verified u-boot) or if you have any questions, feel free to contact me in private and I can help you with the setup. >> Also, there is Kconfig recursive dependency that you may want to fix: >> >> $ make exynos_defconfig >> drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! >> drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HELPER >> drivers/gpu/drm/Kconfig:34: symbol DRM_KMS_FB_HELPER depends on DRM_KMS_HELPER >> drivers/gpu/drm/Kconfig:28: symbol DRM_KMS_HELPER is selected by DRM_ANALOGIX_DP >> drivers/gpu/drm/bridge/analogix/Kconfig:1: symbol DRM_ANALOGIX_DP is selected by DRM_EXYNOS_DP >> drivers/gpu/drm/exynos/Kconfig:57: symbol DRM_EXYNOS_DP depends on DRM_EXYNOS_FIMD >> drivers/gpu/drm/exynos/Kconfig:19: symbol DRM_EXYNOS_FIMD depends on FB_S3C >> drivers/video/fbdev/Kconfig:2023: symbol FB_S3C depends on FB >> > > Yeah, recursive dependency detected, guess I should remove the > "DRM_KMS_HELPER" from bridge analogix_dp Kconfig file, thanks > for your remind. > > --- a/drivers/gpu/drm/bridge/analogix/Kconfig > +++ b/drivers/gpu/drm/bridge/analogix/Kconfig > @@ -1,4 +1,3 @@ > config DRM_ANALOGIX_DP > tristate > depends on DRM > - select DRM_KMS_HELPER > > That fixes the recursive dependency issue indeed. Thanks. > Thanks, > - Yakir Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America