All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.