From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754309AbaIXKbl (ORCPT ); Wed, 24 Sep 2014 06:31:41 -0400 Received: from regular3.263xmail.com ([211.157.153.3]:58211 "EHLO regular3.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754105AbaIXKbf (ORCPT ); Wed, 24 Sep 2014 06:31:35 -0400 X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 X-RL-SENDER: jeff.chen@rock-chips.com X-FST-TO: olof@lixom.net X-SENDER-IP: 58.22.7.114 X-LOGIN-NAME: jeff.chen@rock-chips.com X-UNIQUE-TAG: X-ATTACHMENT-NUM: 0 X-DNS-TYPE: 0 Message-ID: <54229D49.4070008@rock-chips.com> Date: Wed, 24 Sep 2014 18:30:33 +0800 From: jeff chen User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.1.1 MIME-Version: 1.0 To: Rob Clark CC: Mark Rutland , =?UTF-8?B?SGVpa28gU3TDvGJuZXI=?= , linux-doc@vger.kernel.org, Kever Yang , "dri-devel@lists.freedesktop.org" , Linux Kernel Mailing List , Jianqun Xu , Chris Zhong , linux-api@vger.kernel.org, David Airlie , Boris BREZILLON , simon xue , linux-rockchip@lists.infradead.org, kfx@rock-chips.com, Grant Likely , =?UTF-8?B?546L5pmT6IW+?= , Tao Huang , "devicetree@vger.kernel.org" , Pawel Moll , Ian Campbell , dbehr@chromium.org, yxj@rock-chips.com, Eddie Cai , Rob Herring , John Stultz , Daniel Kurtz , =?UTF-8?B?U3TDqXBoYW5lIE1hcmNoZXNpbg==?= , Mark yao , Rom Lemarchand , xw@rock-chips.com, Greg Kroah-Hartman , Randy Dunlap , Douglas Anderson , Kumar Gala , Olof Johansson Subject: Re: [PATCH v4 5/5] drm/rockchip: Add support for Rockchip Soc EDP References: <1411382820-1615-1-git-send-email-mark.yao@rock-chips.com> <1411383728-2075-1-git-send-email-mark.yao@rock-chips.com> <5421338B.3020302@rock-chips.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2014/9/24 7:35, Rob Clark wrote: > On Tue, Sep 23, 2014 at 9:56 AM, Rob Clark wrote: >> On Tue, Sep 23, 2014 at 4:47 AM, cym wrote: >>> On Tuesday, September 23, 2014 03:20 AM, Rob Clark wrote: >>>> On Mon, Sep 22, 2014 at 7:02 AM, Mark yao wrote: >>>>> This adds support for Rockchip soc edp found on rk3288 >>>>> >>>>> Signed-off-by: Mark Yao >>>>> Signed-off-by: Jeff Chen >>>>> --- >>>>> Changes in v2: >>>>> - fix code sytle >>>>> - use some define from drm_dp_helper.h >>>>> - use panel-simple driver for primary display. >>>>> - remove unnecessary clock clk_24m_parent. >>>>> >>>>> Changes in v3: None >>>>> >>>>> Changes in v4: None >>>>> >>>>> drivers/gpu/drm/rockchip/Kconfig | 9 + >>>>> drivers/gpu/drm/rockchip/Makefile | 2 + >>>>> drivers/gpu/drm/rockchip/rockchip_edp_core.c | 853 ++++++++++++++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_core.h | 309 +++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_reg.c | 1202 >>>>> ++++++++++++++++++++++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_reg.h | 345 ++++++++ >>>>> 6 files changed, 2720 insertions(+) >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_core.h >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_reg.c >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_reg.h >>>>> >>>>> diff --git a/drivers/gpu/drm/rockchip/Kconfig >>>>> b/drivers/gpu/drm/rockchip/Kconfig >>>>> index 7146c80..04b1f8c 100644 >>>>> --- a/drivers/gpu/drm/rockchip/Kconfig >>>>> +++ b/drivers/gpu/drm/rockchip/Kconfig >>>>> @@ -17,3 +17,12 @@ config DRM_ROCKCHIP >>>>> management to userspace. This driver does not provides >>>>> 2D or 3D acceleration; acceleration is performed by other >>>>> IP found on the SoC. >>>>> + >>>>> +config ROCKCHIP_EDP >>>>> + bool "Rockchip edp support" >>>>> + depends on DRM_ROCKCHIP >>>>> + help >>>>> + Choose this option if you have a Rockchip eDP. >>>>> + Rockchip rk3288 SoC has eDP TX Controller can be used. >>>>> + If you have an Embedded DisplayPort Panel, say Y to enable its >>>>> + driver. >>>>> diff --git a/drivers/gpu/drm/rockchip/Makefile >>>>> b/drivers/gpu/drm/rockchip/Makefile >>>>> index 6e6d468..a0fc3a1 100644 >>>>> --- a/drivers/gpu/drm/rockchip/Makefile >>>>> +++ b/drivers/gpu/drm/rockchip/Makefile >>>>> @@ -7,4 +7,6 @@ ccflags-y := -Iinclude/drm -Idrivers/gpu/drm/rockchip >>>>> rockchipdrm-y := rockchip_drm_drv.o rockchip_drm_fb.o >>>>> rockchip_drm_fbdev.o \ >>>>> rockchip_drm_gem.o rockchip_drm_vop.o >>>>> >>>>> +rockchipdrm-$(CONFIG_ROCKCHIP_EDP) += rockchip_edp_core.o >>>>> rockchip_edp_reg.o >>>>> + >>>>> obj-$(CONFIG_DRM_ROCKCHIP) += rockchipdrm.o >>>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> b/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> new file mode 100644 >>>>> index 0000000..5450d1fa >>>>> --- /dev/null >>>>> +++ b/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> @@ -0,0 +1,853 @@ >>>>> +/* >>>>> +* Copyright (C) Fuzhou Rockchip Electronics Co.Ltd >>>>> +* Author: >>>>> +* Andy yan >>>>> +* Jeff chen >>>>> +* >>>>> +* based on exynos_dp_core.c >>>>> +* >>>> hmm, did you look at all at drm_dp_helpers? The exynos code probably >>>> pre-dates the helpers, so might not be the best example to work off >>>> of.. >>>> >>>> If there is actually a valid reason not to use the dp-helpers, then >>>> you should mention the reasons, at least in the commit msg if not the >>>> code >>>> >>>> BR, >>>> -R >>> Thanks Rob,Because RK3288 eDP controller IP design is similar to exynos.They >>> from same IP vendors but have some difference. >>> So we choosed exynos_dp as example to work off of.exynos_dp only used some >>> defines from drm_dp_helper.h like DPCD. >>> >> >> Hmm, it sounds like it perhaps should be refactored out into a >> drm_bridge so more of it can be shared between rockchip and exynos. >> >> Either way, it should be using the drm_dp_helpers.. That "the code I >> copied did it wrong" isn't a terribly good reason for new drivers to >> do it wrong. >> >> So NAK for the eDP part until you use the helpers. > and btw, if it wasn't clear, go ahead and at least repost the core > part of the driver.. the first patch just needed a few small tweaks to > get my r-b even if it takes longer to sort out something sane for the > DP part.. > > BR, > -R thanks,I will modify the core part of the driver. BR, -Jeff > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > > From mboxrd@z Thu Jan 1 00:00:00 1970 From: jeff chen Subject: Re: [PATCH v4 5/5] drm/rockchip: Add support for Rockchip Soc EDP Date: Wed, 24 Sep 2014 18:30:33 +0800 Message-ID: <54229D49.4070008@rock-chips.com> References: <1411382820-1615-1-git-send-email-mark.yao@rock-chips.com> <1411383728-2075-1-git-send-email-mark.yao@rock-chips.com> <5421338B.3020302@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Rob Clark Cc: Mark Rutland , =?UTF-8?B?SGVpa28gU3TDvGJuZXI=?= , linux-doc@vger.kernel.org, Kever Yang , "dri-devel@lists.freedesktop.org" , Linux Kernel Mailing List , Jianqun Xu , Chris Zhong , linux-api@vger.kernel.org, David Airlie , Boris BREZILLON , simon xue , linux-rockchip@lists.infradead.org, kfx@rock-chips.com, Grant Likely , =?UTF-8?B?546L5pmT6IW+?= , Tao Huang , "devicetree@vger.kernel.org" , Pawel Moll , Ian Campbell , dbehr@chromium.org, yxj@rock-chips.com, Eddie Cai List-Id: devicetree@vger.kernel.org On 2014/9/24 7:35, Rob Clark wrote: > On Tue, Sep 23, 2014 at 9:56 AM, Rob Clark wrote: >> On Tue, Sep 23, 2014 at 4:47 AM, cym wrote: >>> On Tuesday, September 23, 2014 03:20 AM, Rob Clark wrote: >>>> On Mon, Sep 22, 2014 at 7:02 AM, Mark yao wrote: >>>>> This adds support for Rockchip soc edp found on rk3288 >>>>> >>>>> Signed-off-by: Mark Yao >>>>> Signed-off-by: Jeff Chen >>>>> --- >>>>> Changes in v2: >>>>> - fix code sytle >>>>> - use some define from drm_dp_helper.h >>>>> - use panel-simple driver for primary display. >>>>> - remove unnecessary clock clk_24m_parent. >>>>> >>>>> Changes in v3: None >>>>> >>>>> Changes in v4: None >>>>> >>>>> drivers/gpu/drm/rockchip/Kconfig | 9 + >>>>> drivers/gpu/drm/rockchip/Makefile | 2 + >>>>> drivers/gpu/drm/rockchip/rockchip_edp_core.c | 853 ++++++++++++++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_core.h | 309 +++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_reg.c | 1202 >>>>> ++++++++++++++++++++++++++ >>>>> drivers/gpu/drm/rockchip/rockchip_edp_reg.h | 345 ++++++++ >>>>> 6 files changed, 2720 insertions(+) >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_core.h >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_reg.c >>>>> create mode 100644 drivers/gpu/drm/rockchip/rockchip_edp_reg.h >>>>> >>>>> diff --git a/drivers/gpu/drm/rockchip/Kconfig >>>>> b/drivers/gpu/drm/rockchip/Kconfig >>>>> index 7146c80..04b1f8c 100644 >>>>> --- a/drivers/gpu/drm/rockchip/Kconfig >>>>> +++ b/drivers/gpu/drm/rockchip/Kconfig >>>>> @@ -17,3 +17,12 @@ config DRM_ROCKCHIP >>>>> management to userspace. This driver does not provides >>>>> 2D or 3D acceleration; acceleration is performed by other >>>>> IP found on the SoC. >>>>> + >>>>> +config ROCKCHIP_EDP >>>>> + bool "Rockchip edp support" >>>>> + depends on DRM_ROCKCHIP >>>>> + help >>>>> + Choose this option if you have a Rockchip eDP. >>>>> + Rockchip rk3288 SoC has eDP TX Controller can be used. >>>>> + If you have an Embedded DisplayPort Panel, say Y to enable its >>>>> + driver. >>>>> diff --git a/drivers/gpu/drm/rockchip/Makefile >>>>> b/drivers/gpu/drm/rockchip/Makefile >>>>> index 6e6d468..a0fc3a1 100644 >>>>> --- a/drivers/gpu/drm/rockchip/Makefile >>>>> +++ b/drivers/gpu/drm/rockchip/Makefile >>>>> @@ -7,4 +7,6 @@ ccflags-y := -Iinclude/drm -Idrivers/gpu/drm/rockchip >>>>> rockchipdrm-y := rockchip_drm_drv.o rockchip_drm_fb.o >>>>> rockchip_drm_fbdev.o \ >>>>> rockchip_drm_gem.o rockchip_drm_vop.o >>>>> >>>>> +rockchipdrm-$(CONFIG_ROCKCHIP_EDP) += rockchip_edp_core.o >>>>> rockchip_edp_reg.o >>>>> + >>>>> obj-$(CONFIG_DRM_ROCKCHIP) += rockchipdrm.o >>>>> diff --git a/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> b/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> new file mode 100644 >>>>> index 0000000..5450d1fa >>>>> --- /dev/null >>>>> +++ b/drivers/gpu/drm/rockchip/rockchip_edp_core.c >>>>> @@ -0,0 +1,853 @@ >>>>> +/* >>>>> +* Copyright (C) Fuzhou Rockchip Electronics Co.Ltd >>>>> +* Author: >>>>> +* Andy yan >>>>> +* Jeff chen >>>>> +* >>>>> +* based on exynos_dp_core.c >>>>> +* >>>> hmm, did you look at all at drm_dp_helpers? The exynos code probably >>>> pre-dates the helpers, so might not be the best example to work off >>>> of.. >>>> >>>> If there is actually a valid reason not to use the dp-helpers, then >>>> you should mention the reasons, at least in the commit msg if not the >>>> code >>>> >>>> BR, >>>> -R >>> Thanks Rob,Because RK3288 eDP controller IP design is similar to exynos.They >>> from same IP vendors but have some difference. >>> So we choosed exynos_dp as example to work off of.exynos_dp only used some >>> defines from drm_dp_helper.h like DPCD. >>> >> >> Hmm, it sounds like it perhaps should be refactored out into a >> drm_bridge so more of it can be shared between rockchip and exynos. >> >> Either way, it should be using the drm_dp_helpers.. That "the code I >> copied did it wrong" isn't a terribly good reason for new drivers to >> do it wrong. >> >> So NAK for the eDP part until you use the helpers. > and btw, if it wasn't clear, go ahead and at least repost the core > part of the driver.. the first patch just needed a few small tweaks to > get my r-b even if it takes longer to sort out something sane for the > DP part.. > > BR, > -R thanks,I will modify the core part of the driver. BR, -Jeff > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip > > >