From: Jyri Sarha <jsarha@ti.com>
To: alsa-devel@alsa-project.org, linux-fbdev@vger.kernel.org,
linux-omap@vger.kernel.org
Cc: peter.ujfalusi@ti.com, broonie@kernel.org,
liam.r.girdwood@linux.intel.com, tomi.valkeinen@ti.com,
detheridge@ti.com
Subject: Re: [PATCH v4 00/15] Rework OMAP4+ HDMI audio support
Date: Fri, 29 Aug 2014 11:43:28 +0000 [thread overview]
Message-ID: <54006760.2010404@ti.com> (raw)
In-Reply-To: <cover.1408993242.git.jsarha@ti.com>
After discussing with Tomi we decided to turn the omap-hdmi-audio ASoC
library into a platform device. So do not waste too much time in
reviewing these patches. I'll mail a new series soon.
Best regards,
Jyri
On 08/25/2014 10:04 PM, Jyri Sarha wrote:
> The patches are based on:
> git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git for-next
>
> The base, the patches, and couple of additional not-to-be-merged
> omap2plus_defconfig patches can be found here:
>
> git://git.ti.com/~jyrisarha/ti-linux-kernel/jyrisarhas-audio-video-linux-feature-tree.git omap-hdmi-audio
>
> Changes since v3:
> - Move ASoC parts into library module under sound/soc/omap
> - Come up with API for the library
> - Some cleaning up
>
> The patch set fixes OMAP4+ HDMI audio. The structure of the
> implementation looks a bit different than before. Instead of creating
> a driver specific API for a separate ASoC component driver to connect
> to, these patches implement a single audio library module under
> sound/soc/omap for ASoC side implementation. The library exports
> omap_hdmi_audio_register() and omap_hdmi_audio_unregister()
> functions. With the functions OMAPDSS HDMI implementation registers
> and unregisters all ASoC components needed for OMAP HDMI audio.
>
> The library implements cpu-dai component using the callbacks provided
> by OMAPDSS. Omap-pcm is registered for platform component, dummy
> hdmi-audio-codec is registered for codec component, and
> asoc-simple-card is registered for machine driver.
>
> Big part of the HDMI audio code is still unchanged and there is a need
> for a cleanup there. Also there is still probably something wrong with
> speaker mapping of multi-channel streams. I will get back to cleaning
> up these issues later.
>
> Best regards,
> Jyri
>
> Jyri Sarha (15):
> OMAPDSS: hdmi.h: Add HDMI_AUDIO_LAYOUT_6CH enum value
> OMAPDSS: hdmi: Remove most of OMAP[45]_DSS_HDMI_AUDIO ifdefs
> OMAPDSS: hdmi4_core: Remove unused hdmi4_audio_get_dma_port()
> OMAPDSS: hdmi_wp: Add function for getting audio dma address
> OMAPDSS: hdmi: Make hdmi structure public
> OMAPDSS: hdmi: Add exported functions for storing HDMI audio data
> OMAPDSS: hdmi: Make hdmi_mode_has_audio() more user friedly
> ASoC: omap-hdmi-audio: Add OMAP HDMI audio support library
> OMAPDSS: Kconfig: Implement options for OMAP4 and OMAP5 HDMI audio
> support
> OMAPDSS: hdmi4: Register HDMI audio with omap_hdmi_audio_register()
> OMAPDSS: hdmi5: Register HDMI audio with omap_hdmi_audio_register()
> ASoC: omap: Remove obsolete HDMI audio code and Kconfig options
> OMAPDSS: hdmi4: Remove callbacks for an external ASoC DAI driver
> OMAPDSS: hdmi5: Remove callbacks for an external ASoC DAI driver
> OMAPDSS: Remove all references to obsolete HDMI audio callbacks
>
> .../fbdev/omap2/displays-new/connector-hdmi.c | 99 ------
> .../fbdev/omap2/displays-new/encoder-tpd12s015.c | 56 ---
> drivers/video/fbdev/omap2/dss/Kconfig | 28 +-
> drivers/video/fbdev/omap2/dss/hdmi.h | 35 +-
> drivers/video/fbdev/omap2/dss/hdmi4.c | 233 ++++++-------
> drivers/video/fbdev/omap2/dss/hdmi4_core.c | 14 -
> drivers/video/fbdev/omap2/dss/hdmi4_core.h | 4 -
> drivers/video/fbdev/omap2/dss/hdmi5.c | 216 +++++-------
> drivers/video/fbdev/omap2/dss/hdmi5_core.c | 6 -
> drivers/video/fbdev/omap2/dss/hdmi5_core.h | 2 -
> drivers/video/fbdev/omap2/dss/hdmi_common.c | 18 +-
> drivers/video/fbdev/omap2/dss/hdmi_wp.c | 8 +-
> include/sound/omap-hdmi-audio.h | 50 +++
> include/video/omapdss.h | 34 +-
> sound/soc/omap/Kconfig | 15 +-
> sound/soc/omap/Makefile | 6 +-
> sound/soc/omap/omap-hdmi-audio.c | 357 +++++++++++++++++++
> sound/soc/omap/omap-hdmi-card.c | 87 -----
> sound/soc/omap/omap-hdmi.c | 364 --------------------
> sound/soc/omap/omap-hdmi.h | 38 --
> 20 files changed, 676 insertions(+), 994 deletions(-)
> create mode 100644 include/sound/omap-hdmi-audio.h
> create mode 100644 sound/soc/omap/omap-hdmi-audio.c
> delete mode 100644 sound/soc/omap/omap-hdmi-card.c
> delete mode 100644 sound/soc/omap/omap-hdmi.c
> delete mode 100644 sound/soc/omap/omap-hdmi.h
>
prev parent reply other threads:[~2014-08-29 11:43 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-25 19:04 [PATCH v4 00/15] Rework OMAP4+ HDMI audio support Jyri Sarha
2014-08-29 11:43 ` Jyri Sarha [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=54006760.2010404@ti.com \
--to=jsarha@ti.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=detheridge@ti.com \
--cc=liam.r.girdwood@linux.intel.com \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=peter.ujfalusi@ti.com \
--cc=tomi.valkeinen@ti.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).