From: Mark Brown <broonie@kernel.org>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Jean-Francois Moine <moinejf@free.fr>,
alsa-devel@alsa-project.org, Dave Airlie <airlied@gmail.com>,
dri-devel@lists.freedesktop.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, Rob Clark <robdclark@gmail.com>
Subject: Re: [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID
Date: Mon, 27 Jan 2014 20:54:37 +0000 [thread overview]
Message-ID: <20140127205437.GC11841@sirena.org.uk> (raw)
In-Reply-To: <20140127204915.GC15937@n2100.arm.linux.org.uk>
[-- Attachment #1: Type: text/plain, Size: 563 bytes --]
On Mon, Jan 27, 2014 at 08:49:15PM +0000, Russell King - ARM Linux wrote:
> On Mon, Jan 27, 2014 at 08:44:41PM +0000, Mark Brown wrote:
> > Can this parsing code be factored out - it (or large parts of it) should
> > be usable by other HDMI devices shouldn't it?
> Yes, preferably as a generic ALSA helper rather than an ASoC helper -
> I don't see any need for this to be ASoC specific (I have a pure ALSA
> driver which has very similar code in it.)
Indeed, definitely ALSA generic - ideally we could factor a lot of the
integration with the video side out.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: broonie@kernel.org (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID
Date: Mon, 27 Jan 2014 20:54:37 +0000 [thread overview]
Message-ID: <20140127205437.GC11841@sirena.org.uk> (raw)
In-Reply-To: <20140127204915.GC15937@n2100.arm.linux.org.uk>
On Mon, Jan 27, 2014 at 08:49:15PM +0000, Russell King - ARM Linux wrote:
> On Mon, Jan 27, 2014 at 08:44:41PM +0000, Mark Brown wrote:
> > Can this parsing code be factored out - it (or large parts of it) should
> > be usable by other HDMI devices shouldn't it?
> Yes, preferably as a generic ALSA helper rather than an ASoC helper -
> I don't see any need for this to be ASoC specific (I have a pure ALSA
> driver which has very similar code in it.)
Indeed, definitely ALSA generic - ideally we could factor a lot of the
integration with the video side out.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140127/2d20285d/attachment.sig>
next prev parent reply other threads:[~2014-01-27 20:54 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-27 9:04 [PATCH 0/4] add a TDA998x CODEC Jean-Francois Moine
2014-01-27 9:04 ` Jean-Francois Moine
2014-01-27 9:04 ` Jean-Francois Moine
2014-01-26 18:02 ` [PATCH 1/4] drm/i2c: tda998x: add a function for dynamic audio input switch Jean-Francois Moine
2014-01-26 18:02 ` Jean-Francois Moine
2014-01-26 18:02 ` Jean-Francois Moine
2014-01-26 18:45 ` [PATCH 2/4] ASoC: tda998x: add a codec driver for TDA998x Jean-Francois Moine
2014-01-26 18:45 ` Jean-Francois Moine
2014-01-26 18:45 ` Jean-Francois Moine
2014-01-27 20:42 ` Mark Brown
2014-01-27 20:42 ` Mark Brown
2014-01-27 8:34 ` [PATCH 3/4] ASoC: tda998x: add DT documentation Jean-Francois Moine
2014-01-27 8:34 ` Jean-Francois Moine
2014-01-27 8:34 ` Jean-Francois Moine
2014-01-27 20:43 ` Mark Brown
2014-01-27 20:43 ` Mark Brown
[not found] ` <20140127204302.GY11841-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-01-27 20:45 ` Russell King - ARM Linux
2014-01-27 20:45 ` Russell King - ARM Linux
2014-01-27 20:45 ` Russell King - ARM Linux
2014-01-27 20:56 ` Mark Brown
2014-01-27 20:56 ` Mark Brown
2014-01-27 8:48 ` [PATCH 4/4] ASoC: tda998x: adjust the audio hw parameters from EDID Jean-Francois Moine
2014-01-27 8:48 ` Jean-Francois Moine
2014-01-27 8:48 ` Jean-Francois Moine
2014-01-27 20:44 ` Mark Brown
2014-01-27 20:44 ` Mark Brown
2014-01-27 20:49 ` Russell King - ARM Linux
2014-01-27 20:49 ` Russell King - ARM Linux
2014-01-27 20:54 ` Mark Brown [this message]
2014-01-27 20:54 ` Mark Brown
2014-01-28 9:23 ` [alsa-devel] " Takashi Iwai
2014-01-28 9:23 ` Takashi Iwai
2014-01-28 9:23 ` Takashi Iwai
2014-01-28 11:00 ` Mark Brown
2014-01-28 11:00 ` [alsa-devel] " Mark Brown
2014-01-28 11:00 ` Mark Brown
2014-01-28 11:12 ` Takashi Iwai
2014-01-28 11:12 ` Takashi Iwai
2014-01-28 11:12 ` Takashi Iwai
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=20140127205437.GC11841@sirena.org.uk \
--to=broonie@kernel.org \
--cc=airlied@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=moinejf@free.fr \
--cc=robdclark@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 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.