mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
To: Kukjin Kim <kgene@kernel.org>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Tomasz Figa <tomasz.figa@gmail.com>,
	Mike Turquette <mturquette@linaro.org>,
	Rahul Sharma <rahul.sharma@samsung.com>,
	Arun Kumar K <arun.kk@samsung.com>,
	Doug Anderson <dianders@chromium.org>,
	Olof Johansson <olof@lixom.net>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Tobias Jakobi <liquid.acid@gmx.net>,
	Andrzej Hajda <a.hajda@samsung.com>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Kevin Hilman <khilman@linaro.org>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	linux-samsung-soc@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	Javier Martinez Canillas <javier.martinez@collabora.co.uk>
Subject: [PATCH 0/2] Add HDMI support for Exynos5420 platform
Date: Tue, 20 Jan 2015 11:35:25 +0100	[thread overview]
Message-ID: <1421750127-22536-1-git-send-email-javier.martinez@collabora.co.uk> (raw)

Hello,

This series adds HDMI support for Exynos5420/5422/5800 machines by adding a
node for the DISP1 power domain which is used by the HDMI and MIXER modules.

It also adds IDs for the attached devices parent and input clocks that are
used by the Exynos power domain driver to re-parent the devices input clocks
when the DISP1 power domain is powered off and on.

Thanks a lot to Joonyoung Shim and Krzysztof Kozlowski for point me in the
right direction.

The series is based on Marek Szyprowski's patch
'ARM: DTS: Exynos: convert to generic power domain bindings'
(http://www.spinics.net/lists/linux-samsung-soc/msg40584.html)

The patches were tested on an Exynos5420 Peach Pit and Exynos5800 Peach Pi
Chromebooks and an Exynos5422 Odroid XU3.

NOTE: There is an issue noticed with these patches and is that if the kconfig
CONFIG_DRM_EXYNOS_DP symbol is disabled and only CONFIG_DRM_EXYNOS_HDMI is
enabled, then disabling the framebuffer and enabling it again with:

# echo 0 > /sys/devices/platform/exynos-drm/graphics/fb0/blank                                                           
# echo 1 > /sys/devices/platform/exynos-drm/graphics/fb0/blank

makes the system to crash due the exynos_mixer driver needing clocks enabled
by the exynos_hdmi driver that are gated on hdmi_poweroff().

The same issue has been reported on other Exynos platforms so it's orthogonal
to this series and needs to be fixed for all platforms.

Javier Martinez Canillas (2):
  clk: exynos5420: Add IDs for clocks used in DISP1 power domain
  ARM: dts: Add DISP1 power domain for exynos5420

 arch/arm/boot/dts/exynos5420.dtsi      | 17 +++++++++++++++++
 drivers/clk/samsung/clk-exynos5420.c   | 23 ++++++++++++-----------
 include/dt-bindings/clock/exynos5420.h |  6 ++++++
 3 files changed, 35 insertions(+), 11 deletions(-)

Patches 1 and 2 depend on each other so they have to either be merged through
the same tree or patch 1 has to be merged through clock tree and create a topic
branch to be pulled by the linux-samsung tree.

Best regards,
Javier

             reply	other threads:[~2015-01-20 10:35 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-20 10:35 Javier Martinez Canillas [this message]
2015-01-20 10:35 ` [PATCH 1/2] clk: exynos5420: Add IDs for clocks used in DISP1 power domain Javier Martinez Canillas
2015-01-20 14:04   ` Sylwester Nawrocki
2015-01-20 17:54     ` Mike Turquette
2015-01-21 10:38       ` Javier Martinez Canillas
2015-01-23  6:51         ` Kukjin Kim
2015-01-23  8:48           ` Javier Martinez Canillas
2015-01-20 10:35 ` [PATCH 2/2] ARM: dts: Add DISP1 power domain for exynos5420 Javier Martinez Canillas

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=1421750127-22536-1-git-send-email-javier.martinez@collabora.co.uk \
    --to=javier.martinez@collabora.co.uk \
    --cc=a.hajda@samsung.com \
    --cc=arun.kk@samsung.com \
    --cc=dianders@chromium.org \
    --cc=jy0922.shim@samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=khilman@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=liquid.acid@gmx.net \
    --cc=m.szyprowski@samsung.com \
    --cc=mturquette@linaro.org \
    --cc=olof@lixom.net \
    --cc=rahul.sharma@samsung.com \
    --cc=s.nawrocki@samsung.com \
    --cc=tomasz.figa@gmail.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

all inboxes | Powered by JetHome®