From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yakir Yang Subject: Re: [PATCH v5 0/17] Add Analogix Core Display Port Driver Date: Wed, 07 Oct 2015 19:05:15 +0800 Message-ID: <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> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1028182125==" Return-path: In-Reply-To: <5614E553.6060108@osg.samsung.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Javier Martinez Canillas , 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 Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Russell King , Pawel Moll , Ian Campbell , linux-kernel@vger.kernel.org, emil.l.velikov@gmail.com, dianders@chromium.org, Kishon Vijay Abraham I , linux-rockchip@lists.infradead.org, Kukjin Kim , dri-devel@lists.freedesktop.org, Kumar Gala , ajaynumb@gmail.com, robherring2@gmail.com, Andy Yan , Gustavo Padovan , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org This is a multi-part message in MIME format. --===============1028182125== Content-Type: multipart/alternative; boundary="------------040407000308030704080101" This is a multi-part message in MIME format. --------------040407000308030704080101 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit 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. 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. > 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 Thanks, - Yakir >> Thanks, >> - Yakir >> > Best regards, --------------040407000308030704080101 Content-Type: text/html; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Hi Javier,

On 10/07/2015 05:26 PM, Javier Martine= z 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 [ht=
tps://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.

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.
=A0 
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_H=
ELPER
drivers/gpu/drm/Kconfig:28: symbol DRM_KMS_HELPER is selected by DRM_ANAL=
OGIX_DP
drivers/gpu/drm/bridge/analogix/Kconfig:1: symbol DRM_ANALOGIX_DP is sele=
cted by DRM_EXYNOS_DP
drivers/gpu/drm/exynos/Kconfig:57: symbol DRM_EXYNOS_DP depends on DRM_EX=
YNOS_FIMD
drivers/gpu/drm/exynos/Kconfig:19: symbol DRM_EXYNOS_FIMD depends on FB_S=
3C
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 @@
=A0config DRM_ANALOGIX_DP
=A0=A0=A0=A0=A0=A0=A0 tristate
=A0=A0=A0=A0=A0=A0=A0 depends on DRM
-=A0=A0=A0=A0=A0=A0 select DRM_KMS_HELPER


Thanks,
- Yakir

      

        
Thanks,
- Yakir



      

      
Best regards,


    

    

  


--------------040407000308030704080101--


--===============1028182125==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline

X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs
IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0
cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK

--===============1028182125==--