From: Ricardo Neri <rneri@dextratech.com>
To: Ricardo Neri <rneri@dextratech.com>
Cc: tomi.valkeinen@ti.com, broonie@opensource.wolfsonmicro.com,
lrg@ti.com, tony@atomide.com, peter.ujfalusi@ti.com,
dzuniga@dextratech.com, linux-omap@vger.kernel.org,
alsa-devel@alsa-project.org
Subject: Re: [PATCH v3 0/5] ARM: OMAP4+: HDMI: Update platform devices for audio
Date: Thu, 03 Jan 2013 11:33:12 -0600 [thread overview]
Message-ID: <50E5C0D8.9@dextratech.com> (raw)
In-Reply-To: <1355794243-32345-1-git-send-email-rneri@dextratech.com>
Hi Mark, Tomi,
Just wondering if you had a chance to take a look at these patches.
Thanks!
Ricardo
On 12/17/2012 07:30 PM, Ricardo Neri wrote:
> Hi Mark, Tomi, Liam, Tony,
>
> This set aims to be the version 3 of my previous submission[1] and aims to
> address the comments that Mark and Tomi kindly provided on such submission.
>
> The creation of the platform device for the HDMI audio interface from within the
> OMAPDSS HDMI driver that was previously submitted[2] is resubmitted to be
> complemented with code to relocate to arch/arm/mach-omap2/display.c the
> creation of the platform devices for the HDMI ASoC codec and card drivers. This
> series does not break the HDMI audio functionality in any patch.
>
> Also, the names of the platform devices are changed to give them more logical
> and more descriptive names.
>
> As the commit
>
> commit 14840b9a83c6a56629db2ba0ec247503e975f143
> Author: Ricardo Neri <ricardo.n...@ti.com>
> Date: Tue Nov 6 00:19:17 2012 -0600
>
> OMAPDSS: HDMI: Create platform device for audio support
>
> is reverted in Tomi's git://gitorious.org/linux-omap-dss2/linux.git master
> branch, this series applies cleanly.
>
> Changes from v1:
> *Put in a single series all the patches related to platform device updates.
> *Now HDMI audio works correctly in every patch.
> *Remove reference to the TPD12S015 HDMI companion chip as the ASoC drivers
> are not aware of this and other chips could be used in the future.
>
> Changes from v2:
> *Split in two patches the renaming and the relocation of the platform devices.
> *Create a separate patch to pass only the address offset of the DMA data port
> to the audio interface platform device.
> *Keep the name hdmi-audio-codec to not refer to explicitly to OMAP. The codec
> can be made generic in a different patch series submitted to alsa-devel.
>
> BR,
>
> Ricardo
>
>
> [1]. http://www.mail-archive.com/linux-omap@vger.kernel.org/msg80785.html
> [2]. http://www.mail-archive.com/linux-omap@vger.kernel.org/msg79795.html
>
> Ricardo Neri (5):
> ARM: OMAP4+: HDMI: Rename platform devices for ASoC drivers
> ARM: OMAP4: Assign IDs to DSS HDMI devices
> ARM4: OMAP4+: HDMI: Relocate devices for audio codec and card
> ARM: OMAP4+: HDMI: Relocate the device for audio interface
> ARM: OMAP4+: HDMI: Refine the DMA port resource for audio
>
> arch/arm/mach-omap2/board-4430sdp.c | 9 ++---
> arch/arm/mach-omap2/board-omap4panda.c | 9 ++---
> arch/arm/mach-omap2/devices.c | 31 ----------------
> arch/arm/mach-omap2/display.c | 31 ++++++++++++++++
> drivers/video/omap2/dss/hdmi.c | 62 ++++++++++++++++++++++++++++++++
> sound/soc/omap/omap-hdmi-card.c | 4 +--
> sound/soc/omap/omap-hdmi.c | 5 ++-
> sound/soc/omap/omap-hdmi.h | 2 --
> 8 files changed, 103 insertions(+), 50 deletions(-)
>
> --
> 1.7.10.4
>
next prev parent reply other threads:[~2013-01-03 17:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-18 1:30 [PATCH v3 0/5] ARM: OMAP4+: HDMI: Update platform devices for audio Ricardo Neri
2012-12-18 1:30 ` [PATCH v3 1/5] ARM: OMAP4+: HDMI: Rename platform devices for ASoC drivers Ricardo Neri
2012-12-18 1:30 ` [PATCH v3 2/5] ARM: OMAP4: Assign IDs to DSS HDMI devices Ricardo Neri
2012-12-18 1:30 ` [PATCH v3 3/5] ARM4: OMAP4+: HDMI: Relocate devices for audio codec and card Ricardo Neri
2012-12-18 1:30 ` [PATCH v3 4/5] ARM: OMAP4+: HDMI: Relocate the device for audio interface Ricardo Neri
2012-12-18 1:30 ` [PATCH v3 5/5] ARM: OMAP4+: HDMI: Refine the DMA port resource for audio Ricardo Neri
2013-01-03 17:33 ` Ricardo Neri [this message]
2013-01-04 10:42 ` [PATCH v3 0/5] ARM: OMAP4+: HDMI: Update platform devices " Mark Brown
2013-01-04 23:19 ` Ricardo Neri
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=50E5C0D8.9@dextratech.com \
--to=rneri@dextratech.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=dzuniga@dextratech.com \
--cc=linux-omap@vger.kernel.org \
--cc=lrg@ti.com \
--cc=peter.ujfalusi@ti.com \
--cc=tomi.valkeinen@ti.com \
--cc=tony@atomide.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.