Re: [alsa-devel] [PATCH 4/5 v2] ARM: shmobile: bockw: add R-Car sound support (PIO)
[ CC Mark Brown, alsa-devel ]
On Thu, Sep 05, 2013 at 02:57:51PM +0900, Simon Horman wrote:
On Mon, Aug 26, 2013 at 01:52:35AM -0700, Kuninori Morimoto wrote:
This patch enables R-Car sound, AK4643 (CN19) and AK4554 (CN20/CN21) codec chip on Bock-W.
But, it supports PIO transfer only at this point. User can check sound settings (Dip-switch/PFC etc) via this patch, but will get under/over flow error when playback/capture. Because PIO transfer via SSI will be interrupted "sampling rate" times per 1 second.
DMA transfer will be supported when HPB-DMAC was enabled on r8a7778.
You will notice strange ALSA sound card HW numbering on Bock-W board. This came from AK4554 strange format on playback/capture. The format on playback/capture is same on "normal" codec chip, but AK4554 was different. Because of that, AK4554 playback/capture are registered as a different sound card.
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com
v1 -> v2
Thanks, I have queued this up for v3.13.
There was a bit of diff-noise, please make sure I have applied it correctly. The patch I applied is below:
Unfortunately I didn't read your cover email carefully before queuing this up. I now see that it has dependencies that aren't present in v3.11-rc and thus I have temporarily dropped it.
It seems to me that the dependencies could be provided by the topic/car branch of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git.
Morimoto-san, is this correct? If so, Mark, would it be possible for you to provide topic/car or similar as stable branch for me to use as a base? My intention is to queue these up for v3.13.
On Thu, Sep 05, 2013 at 04:19:50PM +0900, Simon Horman wrote:
[ CC Mark Brown, alsa-devel ]
On Thu, Sep 05, 2013 at 02:57:51PM +0900, Simon Horman wrote:
On Mon, Aug 26, 2013 at 01:52:35AM -0700, Kuninori Morimoto wrote:
This patch enables R-Car sound, AK4643 (CN19) and AK4554 (CN20/CN21) codec chip on Bock-W.
But, it supports PIO transfer only at this point. User can check sound settings (Dip-switch/PFC etc) via this patch, but will get under/over flow error when playback/capture. Because PIO transfer via SSI will be interrupted "sampling rate" times per 1 second.
DMA transfer will be supported when HPB-DMAC was enabled on r8a7778.
You will notice strange ALSA sound card HW numbering on Bock-W board. This came from AK4554 strange format on playback/capture. The format on playback/capture is same on "normal" codec chip, but AK4554 was different. Because of that, AK4554 playback/capture are registered as a different sound card.
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com
v1 -> v2
Thanks, I have queued this up for v3.13.
There was a bit of diff-noise, please make sure I have applied it correctly. The patch I applied is below:
Unfortunately I didn't read your cover email carefully before queuing this up. I now see that it has dependencies that aren't present in v3.11-rc and thus I have temporarily dropped it.
It seems to me that the dependencies could be provided by the topic/car branch of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git.
Morimoto-san, is this correct? If so, Mark, would it be possible for you to provide topic/car or similar as stable branch for me to use as a base? My intention is to queue these up for v3.13.
Alternatively, if the dependencies are likely to appear in v3.12-rc1 or v3.12-rc2 then I can just wait until they are available there.
Hi Simon, Mark
Thank you for your hard work
This patch enables R-Car sound, AK4643 (CN19) and AK4554 (CN20/CN21) codec chip on Bock-W.
But, it supports PIO transfer only at this point. User can check sound settings (Dip-switch/PFC etc) via this patch, but will get under/over flow error when playback/capture. Because PIO transfer via SSI will be interrupted "sampling rate" times per 1 second.
DMA transfer will be supported when HPB-DMAC was enabled on r8a7778.
You will notice strange ALSA sound card HW numbering on Bock-W board. This came from AK4554 strange format on playback/capture. The format on playback/capture is same on "normal" codec chip, but AK4554 was different. Because of that, AK4554 playback/capture are registered as a different sound card.
Signed-off-by: Kuninori Morimoto kuninori.morimoto.gx@renesas.com
v1 -> v2
Thanks, I have queued this up for v3.13.
There was a bit of diff-noise, please make sure I have applied it correctly. The patch I applied is below:
Unfortunately I didn't read your cover email carefully before queuing this up. I now see that it has dependencies that aren't present in v3.11-rc and thus I have temporarily dropped it.
It seems to me that the dependencies could be provided by the topic/car branch of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git.
Yes, exactly.
But
- topic/car + topic/rcar
;P
Morimoto-san, is this correct? If so, Mark, would it be possible for you to provide topic/car or similar as stable branch for me to use as a base? My intention is to queue these up for v3.13.
Alternatively, if the dependencies are likely to appear in v3.12-rc1 or v3.12-rc2 then I can just wait until they are available there.
Can you help Mark ?
Best regards --- Kuninori Morimoto
On Thu, Sep 05, 2013 at 04:19:50PM +0900, Simon Horman wrote:
Morimoto-san, is this correct? If so, Mark, would it be possible for you to provide topic/car or similar as stable branch for me to use as a base? My intention is to queue these up for v3.13.
It's gone to Linus already so it's stable.
On Thu, Sep 05, 2013 at 02:53:01PM +0100, Mark Brown wrote:
On Thu, Sep 05, 2013 at 04:19:50PM +0900, Simon Horman wrote:
Morimoto-san, is this correct? If so, Mark, would it be possible for you to provide topic/car or similar as stable branch for me to use as a base? My intention is to queue these up for v3.13.
It's gone to Linus already so it's stable.
Thanks and sorry for missing that.
participants (3)
-
Kuninori Morimoto
-
Mark Brown
-
Simon Horman