All of lore.kernel.org
 help / color / mirror / Atom feed
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 14:43:28 +0300	[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
>


WARNING: multiple messages have this Message-ID (diff)
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
>


  parent reply	other threads:[~2014-08-29 11:43 UTC|newest]

Thread overview: 19+ 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-25 19:04 ` Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 01/15] OMAPDSS: hdmi.h: Add HDMI_AUDIO_LAYOUT_6CH enum value Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 02/15] OMAPDSS: hdmi: Remove most of OMAP[45]_DSS_HDMI_AUDIO ifdefs Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 03/15] OMAPDSS: hdmi4_core: Remove unused hdmi4_audio_get_dma_port() Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 04/15] OMAPDSS: hdmi_wp: Add function for getting audio dma address Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 05/15] OMAPDSS: hdmi: Make hdmi structure public Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 06/15] OMAPDSS: hdmi: Add exported functions for storing HDMI audio data Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 07/15] OMAPDSS: hdmi: Make hdmi_mode_has_audio() more user friedly Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 08/15] ASoC: omap-hdmi-audio: Add OMAP HDMI audio support library Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 09/15] OMAPDSS: Kconfig: Implement options for OMAP4 and OMAP5 HDMI audio support Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 10/15] OMAPDSS: hdmi4: Register HDMI audio with omap_hdmi_audio_register() Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 11/15] OMAPDSS: hdmi5: " Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 12/15] ASoC: omap: Remove obsolete HDMI audio code and Kconfig options Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 13/15] OMAPDSS: hdmi4: Remove callbacks for an external ASoC DAI driver Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 14/15] OMAPDSS: hdmi5: " Jyri Sarha
2014-08-25 19:04 ` [PATCH v4 15/15] OMAPDSS: Remove all references to obsolete HDMI audio callbacks Jyri Sarha
2014-08-29 11:43 ` Jyri Sarha [this message]
2014-08-29 11:43   ` [PATCH v4 00/15] Rework OMAP4+ HDMI audio support Jyri Sarha

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 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.