From: Yakir Yang <ykk@rock-chips.com>
To: Jingoo Han <jingoohan1@gmail.com>, 'David Airlie' <airlied@linux.ie>
Cc: 'Krzysztof Kozlowski' <k.kozlowski@samsung.com>,
"'open list:ARM/Rockchip SoC...'"
<linux-rockchip@lists.infradead.org>,
'Doug Anderson' <dianders@chromium.org>,
'dri-devel' <dri-devel@lists.freedesktop.org>,
'Javier Martinez Canillas' <javier@osg.samsung.com>,
'Rob Herring' <robh+dt@kernel.org>,
'Caesar Wang' <wxt@rock-chips.com>
Subject: Re: [GIT PULL v2]: drm: bridge: add analogix_dp driver
Date: Wed, 13 Apr 2016 14:56:09 +0800 [thread overview]
Message-ID: <570DED89.2080503@rock-chips.com> (raw)
In-Reply-To: <00a801d19546$bd0f7050$372e50f0$@com>
On 04/13/2016 01:38 PM, Jingoo Han wrote:
> On Tuesday, April 05, 2016 11:46 AM, Yakir Yang wrote:
>> Hi David,
>>
>> This v2 pull request just fixed the module compiled failed problem which
>> pointed by Guenter,
>> detail relate to [1]
>>
>> Thanks,
>> - Yakir
>>
>> [1]: https://lkml.org/lkml/2016/3/30/762
>>
>>
>>
>> The following changes since commit 4604202ca8d2a5e33b4bca0812b5d92975ed1bd8:
>>
>> Merge branch 'drm-next-4.6' of
>> git://people.freedesktop.org/~agd5f/linux into drm-next (2016-03-25
>> 16:02:06 +1000)
>>
>> are available in the git repository at:
>>
>>
>> git@github.com:yakir-Yang/linux.git drm-next-analogix-dp-v2
>>
>> for you to fetch changes up to 7b4b7a8db439dca4342a0424b3d99d878a4e5a12:
> Hi Yakir Yang,
>
> I checked your git repository.
> I give my Ack for the following patches.
>
> Heiko Stuebner
> drm/exynos: dp: rename implementation specific driver part
> drm: bridge: analogix/dp: rename register constants
>
> Yakir Yang
> drm: bridge: analogix/dp: split exynos dp driver to bridge directory
> drm: bridge: analogix/dp: fix some obvious code style
> drm: bridge: analogix/dp: remove duplicate configuration of link rate and link count
> drm: bridge: analogix/dp: dynamic parse sync_pol & interlace & dynamic_range
> ARM: dts: exynos/dp: remove some properties that deprecated by analogix_dp driver
> drm: bridge: analogix/dp: add max link rate and lane count limit for RK3288
> drm: bridge: analogix/dp: try force hpd after plug in lookup failed
Thanks, Jingoo, and glad to say that David have merged those patches ;)
commit d00b39c17573ece6f5fb1385314877d29f540db8
Merge: 85bd5ac 7b4b7a8
Author: Dave Airlie <airlied@redhat.com>
Date: Wed Apr 6 09:57:33 2016 +1000
Merge branch 'drm-next-analogix-dp-v2' of
github.com:yakir-Yang/linux into drm-next
- Yakir
> Thank you.
>
> Best regards,
> Jingoo Han
>
>> drm: bridge: analogix/dp: Fix the possible dead lock in bridge
>> disable time (2016-04-05 10:13:10 +0800)
>>
>> ----------------------------------------------------------------
>> Heiko Stuebner (2):
>> drm/exynos: dp: rename implementation specific driver part
>> drm: bridge: analogix/dp: rename register constants
>>
>> Yakir Yang (15):
>> drm: bridge: analogix/dp: split exynos dp driver to bridge directory
>> drm: bridge: analogix/dp: fix some obvious code style
>> drm: bridge: analogix/dp: remove duplicate configuration of link
>> rate and link count
>> drm: bridge: analogix/dp: dynamic parse sync_pol & interlace &
>> dynamic_range
>> dt-bindings: add document for analogix display port driver
>> ARM: dts: exynos/dp: remove some properties that deprecated by
>> analogix_dp driver
>> drm: rockchip: dp: add rockchip platform dp driver
>> dt-bindings: add document for rockchip variant of analogix_dp
>> drm: bridge: analogix/dp: add some rk3288 special registers setting
>> drm: bridge: analogix/dp: add max link rate and lane count limit
>> for RK3288
>> drm: bridge: analogix/dp: try force hpd after plug in lookup failed
>> drm: bridge: analogix/dp: move hpd detect to connector detect
>> function
>> drm: bridge: analogix/dp: add edid modes parse in get_modes method
>> drm: bridge: analogix/dp: add panel prepare/unprepare in
>> suspend/resume time
>> drm: bridge: analogix/dp: Fix the possible dead lock in bridge
>> disable time
>>
>>
>>
>>
>>
>> On 03/29/2016 11:20 AM, Yakir Yang wrote:
>>> Hi David,
>>>
>>> This pull request want to land the analogix_dp driver into drm/bridge
>>> directory,
>>> which reused the Exynos DP code, and add Rockchip DP support. And those
>>> patches have been:
>>> - Acked-by: Inki Dae <inki.dae@samsung.com>
>>> - Acked-by: Mark Yao <ykk@rock-chips.com>
>>> - Acked-by: Rob Herring <robh@kernel.org>
>>> - Acked-by: Jingoo Han <jingoohan1@gmail.com>
>>> - Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>> - Tested-by: Caesar Wang <wxt@rock-chips.com>
>>> - Tested-by: Douglas Anderson <dianders@chromium.org>
>>> - Tested-by: Heiko Stuebner <heiko@sntech.de>
>>> - Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>>
>>> Just like you mentioned at email thread [0], I just send the pull
>>> request directly,
>>> not via another maintainer layer, wish you could land them.
>>>
>>> [0]: https://lkml.org/lkml/2016/3/22/846
>>>
>>> Thanks
>>>
>>>
>>> The following changes since commit
>>> 4604202ca8d2a5e33b4bca0812b5d92975ed1bd8:
>>>
>>> Merge branch 'drm-next-4.6' of
>>> git://people.freedesktop.org/~agd5f/linux into drm-next (2016-03-25
>>> 16:02:06 +1000)
>>>
>>> are available in the git repository at:
>>>
>>>
>>> git@github.com:yakir-Yang/linux.git drm-next-analogix-dp
>>>
>>> for you to fetch changes up to 5a6387f2b878e3ae23b22626d9a7c72a760b55fc:
>>>
>>> drm: bridge: analogix/dp: Fix the possible dead lock in bridge
>>> disable time (2016-03-29 11:09:45 +0800)
>>>
>>> ----------------------------------------------------------------
>>> Heiko Stuebner (2):
>>> drm/exynos: dp: rename implementation specific driver part
>>> drm: bridge: analogix/dp: rename register constants
>>>
>>> Yakir Yang (15):
>>> drm: bridge: analogix/dp: split exynos dp driver to bridge
>>> directory
>>> drm: bridge: analogix/dp: fix some obvious code style
>>> drm: bridge: analogix/dp: remove duplicate configuration of link
>>> rate and link count
>>> drm: bridge: analogix/dp: dynamic parse sync_pol & interlace &
>>> dynamic_range
>>> dt-bindings: add document for analogix display port driver
>>> ARM: dts: exynos/dp: remove some properties that deprecated by
>>> analogix_dp driver
>>> drm: rockchip: dp: add rockchip platform dp driver
>>> dt-bindings: add document for rockchip variant of analogix_dp
>>> drm: bridge: analogix/dp: add some rk3288 special registers setting
>>> drm: bridge: analogix/dp: add max link rate and lane count limit
>>> for RK3288
>>> drm: bridge: analogix/dp: try force hpd after plug in lookup failed
>>> drm: bridge: analogix/dp: move hpd detect to connector detect
>>> function
>>> drm: bridge: analogix/dp: add edid modes parse in get_modes method
>>> drm: bridge: analogix/dp: add panel prepare/unprepare in
>>> suspend/resume time
>>> drm: bridge: analogix/dp: Fix the possible dead lock in bridge
>>> disable time
>>>
>>> Documentation/devicetree/bindings/display/bridge/analogix_dp.txt |
>>> 52 ++++
>>> Documentation/devicetree/bindings/display/exynos/exynos_dp.txt | 93
>>> ++-----
>>> Documentation/devicetree/bindings/display/rockchip/analogix_dp-rockchip.txt
>>> | 92 +++++++
>>> arch/arm/boot/dts/exynos5250-arndale.dts | 2 -
>>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 2 -
>>> arch/arm/boot/dts/exynos5250-snow-common.dtsi | 4 +-
>>> arch/arm/boot/dts/exynos5250-spring.dts | 4 +-
>>> arch/arm/boot/dts/exynos5420-peach-pit.dts | 4 +-
>>> arch/arm/boot/dts/exynos5420-smdk5420.dts | 2 -
>>> arch/arm/boot/dts/exynos5800-peach-pi.dts | 2 -
>>> drivers/gpu/drm/bridge/Kconfig | 2 +
>>> drivers/gpu/drm/bridge/Makefile | 1 +
>>> drivers/gpu/drm/bridge/analogix/Kconfig | 3 +
>>> drivers/gpu/drm/bridge/analogix/Makefile | 1 +
>>> drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 1430
>>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>> drivers/gpu/drm/bridge/analogix/analogix_dp_core.h | 281
>>> ++++++++++++++++++++
>>> drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c | 1320
>>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>> drivers/gpu/drm/{exynos/exynos_dp_reg.h =>
>>> bridge/analogix/analogix_dp_reg.h} | 270 ++++++++++---------
>>> drivers/gpu/drm/exynos/Kconfig | 3 +-
>>> drivers/gpu/drm/exynos/Makefile | 2 +-
>>> drivers/gpu/drm/exynos/exynos_dp.c | 314 ++++++++++++++++++++++
>>> drivers/gpu/drm/exynos/exynos_dp_core.c | 1499
>>> ----------------------------------------------------------------------------------------------------
>> ---
>>> drivers/gpu/drm/exynos/exynos_dp_core.h | 282 --------------------
>>> drivers/gpu/drm/exynos/exynos_dp_reg.c | 1263
>>> --------------------------------------------------------------------------------------
>>> drivers/gpu/drm/rockchip/Kconfig | 9 +
>>> drivers/gpu/drm/rockchip/Makefile | 1 +
>>> drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 384
>>> +++++++++++++++++++++++++++
>>> include/drm/bridge/analogix_dp.h | 41 +++
>>> 28 files changed, 4105 insertions(+), 3258 deletions(-)
>>> create mode 100644
>>> Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
>>> create mode 100644
>>> Documentation/devicetree/bindings/display/rockchip/analogix_dp-rockchip.txt
>>> create mode 100644 drivers/gpu/drm/bridge/analogix/Kconfig
>>> create mode 100644 drivers/gpu/drm/bridge/analogix/Makefile
>>> create mode 100644 drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
>>> create mode 100644 drivers/gpu/drm/bridge/analogix/analogix_dp_core.h
>>> create mode 100644 drivers/gpu/drm/bridge/analogix/analogix_dp_reg.c
>>> rename drivers/gpu/drm/{exynos/exynos_dp_reg.h =>
>>> bridge/analogix/analogix_dp_reg.h} (62%)
>>> create mode 100644 drivers/gpu/drm/exynos/exynos_dp.c
>>> delete mode 100644 drivers/gpu/drm/exynos/exynos_dp_core.c
>>> delete mode 100644 drivers/gpu/drm/exynos/exynos_dp_core.h
>>> delete mode 100644 drivers/gpu/drm/exynos/exynos_dp_reg.c
>>> create mode 100644 drivers/gpu/drm/rockchip/analogix_dp-rockchip.c
>>> create mode 100644 include/drm/bridge/analogix_dp.h
>>>
>
>
>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
prev parent reply other threads:[~2016-04-13 6:56 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-29 3:20 [GIT PULL]: drm: bridge: add analogix_dp driver Yakir Yang
2016-04-05 2:45 ` [GIT PULL v2]: " Yakir Yang
2016-04-13 5:38 ` Jingoo Han
2016-04-13 6:56 ` Yakir Yang [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=570DED89.2080503@rock-chips.com \
--to=ykk@rock-chips.com \
--cc=airlied@linux.ie \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=javier@osg.samsung.com \
--cc=jingoohan1@gmail.com \
--cc=k.kozlowski@samsung.com \
--cc=linux-rockchip@lists.infradead.org \
--cc=robh+dt@kernel.org \
--cc=wxt@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).