* Question about SuperH HDMI sound
@ 2010-07-09 0:19 Kuninori Morimoto
2010-07-09 1:52 ` Mark Brown
0 siblings, 1 reply; 4+ messages in thread
From: Kuninori Morimoto @ 2010-07-09 0:19 UTC (permalink / raw)
To: Mark Brown; +Cc: Linux-ALSA, Guennadi
Dear Mark
Cc: Guennadi
In these days, Guennadi's "SuperH HDMI support patches"
which support HDMI video output were posted to ML.
And I created HDMI sound support for it in my local environment.
It works and I would like to post this patch.
Thus, I would like to ask you about HDMI "sound" support file position.
HDMI "video" support file is under "drivers/video/sh_mobile_hdmi.c"
Should I add sound support in it ? or under "sound/soc/codecs" ?
Best regards
--
Kuninori Morimoto
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Question about SuperH HDMI sound
2010-07-09 0:19 Question about SuperH HDMI sound Kuninori Morimoto
@ 2010-07-09 1:52 ` Mark Brown
2010-07-09 7:31 ` Kuninori Morimoto
2010-07-09 7:31 ` Kuninori Morimoto
0 siblings, 2 replies; 4+ messages in thread
From: Mark Brown @ 2010-07-09 1:52 UTC (permalink / raw)
To: Kuninori Morimoto; +Cc: Linux-ALSA, Guennadi
On Fri, Jul 09, 2010 at 09:19:24AM +0900, Kuninori Morimoto wrote:
> HDMI "video" support file is under "drivers/video/sh_mobile_hdmi.c"
> Should I add sound support in it ? or under "sound/soc/codecs" ?
One or the other :)
On PCs there is support for HDMI audio within the HDA subsystem so it
would seem reasonable to add the support in ASoC if the hardware shares
anything with the rest of the audio hardware in the SoC. It may also
make sense to provide a driver which presents both video and audio
interfaces, depending on how the hardware is structured.
If the hardware doesn't share anything with the other audio hardware
then it might be as easy to just implement outside ASoC as a regular
ALSA driver since HDMI "CODECs" are usually the TVs or other remote
devices that are plugged in to the system rather than a component of the
system itself so the CODEC/CPU split of ASoC isn't such a clear benefit.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Question about SuperH HDMI sound
2010-07-09 1:52 ` Mark Brown
@ 2010-07-09 7:31 ` Kuninori Morimoto
2010-07-09 7:31 ` Kuninori Morimoto
1 sibling, 0 replies; 4+ messages in thread
From: Kuninori Morimoto @ 2010-07-09 7:31 UTC (permalink / raw)
To: Mark Brown; +Cc: Linux-ALSA, Guennadi
Dear Mark
Thank you for response
> > HDMI "video" support file is under "drivers/video/sh_mobile_hdmi.c"
> > Should I add sound support in it ? or under "sound/soc/codecs" ?
(snip)
> make sense to provide a driver which presents both video and audio
> interfaces, depending on how the hardware is structured.
I use above idea. Thanks.
I send this patch after SH HDMI video support patch was applied.
Best regards
--
Kuninori Morimoto
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Question about SuperH HDMI sound
2010-07-09 1:52 ` Mark Brown
2010-07-09 7:31 ` Kuninori Morimoto
@ 2010-07-09 7:31 ` Kuninori Morimoto
1 sibling, 0 replies; 4+ messages in thread
From: Kuninori Morimoto @ 2010-07-09 7:31 UTC (permalink / raw)
To: Mark Brown; +Cc: Linux-ALSA, Guennadi
Dear Mark
Thank you for response
> > HDMI "video" support file is under "drivers/video/sh_mobile_hdmi.c"
> > Should I add sound support in it ? or under "sound/soc/codecs" ?
(snip)
> make sense to provide a driver which presents both video and audio
> interfaces, depending on how the hardware is structured.
I use above idea. Thanks.
I send this patch after SH HDMI video support patch was applied.
Best regards
--
Kuninori Morimoto
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-07-09 7:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-09 0:19 Question about SuperH HDMI sound Kuninori Morimoto
2010-07-09 1:52 ` Mark Brown
2010-07-09 7:31 ` Kuninori Morimoto
2010-07-09 7:31 ` Kuninori Morimoto
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.