[RFC PATCH 0/4] exynos-drm-hdmi driver to CDF complaint display driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This patch set is a proposal to change Exynos Drm Hdmi driver to a CDF
complaint panel driver. This migration serves 2 purposes. One is to eliminate
duplication due to v4l and drm hdmi drivers. Second is to add support for Hdmi
audio ALSA codec which is not possible with drm/v4l hdmi driver (specially for
exynos as hdmi audio and hdmi core registers are intermixed.).

This patch series is based on the Second RFC of CDF from Laurent Pinchart.

[PATCH 1/4] video: display: add event handling, set mode and hdmi ops to cdf
core

This patch adds
1) Event Notification to CDF Core:
	Adds simple event notification mechanism supports multiple
	subscribers. This is used for hot-plug notification to the clients
	of hdmi display i.e. exynos-drm and alsa-codec. CDF Core maintains
	multiple subscriber list. When entity reports a event Core will
	route it to all of them. Un-superscription is not implemented which
	can be done if notification callback is Null.

2) set_mode to generic ops:
	It is meaningful for a panel like hdmi which supports multiple
	resolutions.

3) Provision for platform specific interfaces through void *private in display
entity:

	It has added void *private to display entity which can be used to
	expose interfaces which are very much specific to a particular platform.

	In exynos, hpd is connected to the soc via gpio bus. During initial
	hdmi poweron, hpd interrupt is not raised as there is no change in the
	gpio status. This is solved by providing a platform specific interface
	which is queried by the drm to get the hpd state. This interface may
	not be required by all platforms.

4) hdmi ops:
	get_edid: to query raw EDID data and length from the panel.
	check_mode: To check if a given mode is supported by exynos HDMI IP
			"AND" Connected HDMI Sink (tv/monitor).
	init_audio: Configure hdmi audio registers for Audio interface type
	(i2s/ spdif), SF, Audio Channels, BPS.
	set_audiostate: enable disable audio.

[PATCH 2/4] drm/edid: temporarily exposing generic edid-read interface from drm

It exposes generic interface from drm_edid.c to get the edid data and length
by any display entity. Once I get clear idea about edid handling in CDF, I need
to revert these temporary changes.

[PATCH 3/4] drm/exynos: moved drm hdmi driver to cdf framework

This patch implements exynos_hdmi_cdf.c which is a glue component between
exynos DRM and hdmi cdf panel. It is a platform driver register through
exynos_drm_drv.c. Exynos_hdmi.c is modified to register hdmi as display panel.
exynos_hdmi_cdf.c registers for exynos hdmi display entity and if successful,
proceeds for mode setting.

[PATCH 4/4] alsa/soc: add hdmi audio codec based on cdf

It registers hdmi-audio codec to the ALSA framework. This is the second client
to the hdmi panel. Once notified by the CDF Core it proceeds towards audio
setting and audio control. It also subscribes for hpd notification to implement
hpd related audio requirements.

Tested for:
1) hdmi mode negotiation.
2) Hpd Scenarios for hdmi and hdmi-audio.
3) hdmi mode switching.
4) hdmi-audio static audio configuration enable/ disable.

Pending:
1) Registering hdmi-audio sound card.
2) Moving exynos_hdmi to driver/video/display

Rahul Sharma (4):
  video: display: add event handling, set mode and hdmi ops to cdf core
  drm/edid: temporarily exposing generic edid-read interface from drm
  drm/exynos: moved drm hdmi driver to cdf framework
  alsa/soc: add hdmi audio codec based on cdf

 drivers/gpu/drm/drm_edid.c               |  88 ++++++
 drivers/gpu/drm/exynos/Kconfig           |   6 +
 drivers/gpu/drm/exynos/Makefile          |   1 +
 drivers/gpu/drm/exynos/exynos_drm_drv.c  |  24 ++
 drivers/gpu/drm/exynos/exynos_drm_drv.h  |   1 +
 drivers/gpu/drm/exynos/exynos_hdmi.c     | 446 ++++++++++++++++---------------
 drivers/gpu/drm/exynos/exynos_hdmi_cdf.c | 370 +++++++++++++++++++++++++
 drivers/video/display/display-core.c     |  85 ++++++
 include/video/display.h                  | 111 +++++++-
 include/video/exynos_hdmi.h              |  25 ++
 sound/soc/codecs/Kconfig                 |   4 +
 sound/soc/codecs/Makefile                |   2 +
 sound/soc/codecs/exynos_hdmi_audio.c     | 307 +++++++++++++++++++++
 13 files changed, 1253 insertions(+), 217 deletions(-)
 create mode 100644 drivers/gpu/drm/exynos/exynos_hdmi_cdf.c
 mode change 100755 => 100644 drivers/video/display/display-core.c
 mode change 100755 => 100644 include/video/display.h
 create mode 100644 include/video/exynos_hdmi.h
 create mode 100644 sound/soc/codecs/exynos_hdmi_audio.c

-- 
1.8.0

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux