linux-rockchip.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: "Jingoo Han" <jingoohan1@gmail.com>
To: 'Yakir Yang' <ykk@rock-chips.com>, 'David Airlie' <airlied@linux.ie>
Cc: 'Krzysztof Kozlowski' <k.kozlowski@samsung.com>,
	"'open list:ARM/Rockchip SoC...'"
	<linux-rockchip@lists.infradead.org>,
	'Jingoo Han' <jingoohan1@gmail.com>,
	'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:38:34 +0900	[thread overview]
Message-ID: <00a801d19546$bd0f7050$372e50f0$@com> (raw)
In-Reply-To: <570326D6.5040802@rock-chips.com>

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

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

  reply	other threads:[~2016-04-13  5:38 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 [this message]
2016-04-13  6:56     ` Yakir Yang

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='00a801d19546$bd0f7050$372e50f0$@com' \
    --to=jingoohan1@gmail.com \
    --cc=airlied@linux.ie \
    --cc=dianders@chromium.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=javier@osg.samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=robh+dt@kernel.org \
    --cc=wxt@rock-chips.com \
    --cc=ykk@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).