diff options
author | Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> | 2012-05-25 02:56:19 -0400 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-06-03 08:06:39 -0400 |
commit | b1226dc59d55ecde7fc9a338d8cb2a313821fac0 (patch) | |
tree | 4e1cd7a3036d8e6d1e3f0427b70b38d2bcd583f7 /sound/soc | |
parent | 5514efdfe0384576ef38c66b1672b6826696fbf3 (diff) |
ASoC: fsi: use PIO handler if DMA handler was invalid
PIO handler is not good performance, but works on all platform.
So, switch to PIO handler if DMA handler was invalid case.
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc')
-rw-r--r-- | sound/soc/sh/fsi.c | 29 |
1 files changed, 20 insertions, 9 deletions
diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c index fcaa6b8abb0c..53486ff9c2af 100644 --- a/sound/soc/sh/fsi.c +++ b/sound/soc/sh/fsi.c | |||
@@ -247,7 +247,7 @@ struct fsi_priv { | |||
247 | struct fsi_stream_handler { | 247 | struct fsi_stream_handler { |
248 | int (*init)(struct fsi_priv *fsi, struct fsi_stream *io); | 248 | int (*init)(struct fsi_priv *fsi, struct fsi_stream *io); |
249 | int (*quit)(struct fsi_priv *fsi, struct fsi_stream *io); | 249 | int (*quit)(struct fsi_priv *fsi, struct fsi_stream *io); |
250 | int (*probe)(struct fsi_priv *fsi, struct fsi_stream *io); | 250 | int (*probe)(struct fsi_priv *fsi, struct fsi_stream *io, struct device *dev); |
251 | int (*transfer)(struct fsi_priv *fsi, struct fsi_stream *io); | 251 | int (*transfer)(struct fsi_priv *fsi, struct fsi_stream *io); |
252 | int (*remove)(struct fsi_priv *fsi, struct fsi_stream *io); | 252 | int (*remove)(struct fsi_priv *fsi, struct fsi_stream *io); |
253 | void (*start_stop)(struct fsi_priv *fsi, struct fsi_stream *io, | 253 | void (*start_stop)(struct fsi_priv *fsi, struct fsi_stream *io, |
@@ -571,16 +571,16 @@ static int fsi_stream_transfer(struct fsi_stream *io) | |||
571 | #define fsi_stream_stop(fsi, io)\ | 571 | #define fsi_stream_stop(fsi, io)\ |
572 | fsi_stream_handler_call(io, start_stop, fsi, io, 0) | 572 | fsi_stream_handler_call(io, start_stop, fsi, io, 0) |
573 | 573 | ||
574 | static int fsi_stream_probe(struct fsi_priv *fsi) | 574 | static int fsi_stream_probe(struct fsi_priv *fsi, struct device *dev) |
575 | { | 575 | { |
576 | struct fsi_stream *io; | 576 | struct fsi_stream *io; |
577 | int ret1, ret2; | 577 | int ret1, ret2; |
578 | 578 | ||
579 | io = &fsi->playback; | 579 | io = &fsi->playback; |
580 | ret1 = fsi_stream_handler_call(io, probe, fsi, io); | 580 | ret1 = fsi_stream_handler_call(io, probe, fsi, io, dev); |
581 | 581 | ||
582 | io = &fsi->capture; | 582 | io = &fsi->capture; |
583 | ret2 = fsi_stream_handler_call(io, probe, fsi, io); | 583 | ret2 = fsi_stream_handler_call(io, probe, fsi, io, dev); |
584 | 584 | ||
585 | if (ret1 < 0) | 585 | if (ret1 < 0) |
586 | return ret1; | 586 | return ret1; |
@@ -1173,7 +1173,7 @@ static void fsi_dma_push_start_stop(struct fsi_priv *fsi, struct fsi_stream *io, | |||
1173 | fsi_master_mask_set(master, CLK_RST, clk, (enable) ? clk : 0); | 1173 | fsi_master_mask_set(master, CLK_RST, clk, (enable) ? clk : 0); |
1174 | } | 1174 | } |
1175 | 1175 | ||
1176 | static int fsi_dma_probe(struct fsi_priv *fsi, struct fsi_stream *io) | 1176 | static int fsi_dma_probe(struct fsi_priv *fsi, struct fsi_stream *io, struct device *dev) |
1177 | { | 1177 | { |
1178 | dma_cap_mask_t mask; | 1178 | dma_cap_mask_t mask; |
1179 | 1179 | ||
@@ -1181,8 +1181,19 @@ static int fsi_dma_probe(struct fsi_priv *fsi, struct fsi_stream *io) | |||
1181 | dma_cap_set(DMA_SLAVE, mask); | 1181 | dma_cap_set(DMA_SLAVE, mask); |
1182 | 1182 | ||
1183 | io->chan = dma_request_channel(mask, fsi_dma_filter, &io->slave); | 1183 | io->chan = dma_request_channel(mask, fsi_dma_filter, &io->slave); |
1184 | if (!io->chan) | 1184 | if (!io->chan) { |
1185 | return -EIO; | 1185 | |
1186 | /* switch to PIO handler */ | ||
1187 | if (fsi_stream_is_play(fsi, io)) | ||
1188 | fsi->playback.handler = &fsi_pio_push_handler; | ||
1189 | else | ||
1190 | fsi->capture.handler = &fsi_pio_pop_handler; | ||
1191 | |||
1192 | dev_info(dev, "switch handler (dma => pio)\n"); | ||
1193 | |||
1194 | /* probe again */ | ||
1195 | return fsi_stream_probe(fsi, dev); | ||
1196 | } | ||
1186 | 1197 | ||
1187 | tasklet_init(&io->tasklet, fsi_dma_do_tasklet, (unsigned long)io); | 1198 | tasklet_init(&io->tasklet, fsi_dma_do_tasklet, (unsigned long)io); |
1188 | 1199 | ||
@@ -1672,7 +1683,7 @@ static int fsi_probe(struct platform_device *pdev) | |||
1672 | master->fsia.master = master; | 1683 | master->fsia.master = master; |
1673 | master->fsia.info = &info->port_a; | 1684 | master->fsia.info = &info->port_a; |
1674 | fsi_handler_init(&master->fsia); | 1685 | fsi_handler_init(&master->fsia); |
1675 | ret = fsi_stream_probe(&master->fsia); | 1686 | ret = fsi_stream_probe(&master->fsia, &pdev->dev); |
1676 | if (ret < 0) { | 1687 | if (ret < 0) { |
1677 | dev_err(&pdev->dev, "FSIA stream probe failed\n"); | 1688 | dev_err(&pdev->dev, "FSIA stream probe failed\n"); |
1678 | goto exit_iounmap; | 1689 | goto exit_iounmap; |
@@ -1683,7 +1694,7 @@ static int fsi_probe(struct platform_device *pdev) | |||
1683 | master->fsib.master = master; | 1694 | master->fsib.master = master; |
1684 | master->fsib.info = &info->port_b; | 1695 | master->fsib.info = &info->port_b; |
1685 | fsi_handler_init(&master->fsib); | 1696 | fsi_handler_init(&master->fsib); |
1686 | ret = fsi_stream_probe(&master->fsib); | 1697 | ret = fsi_stream_probe(&master->fsib, &pdev->dev); |
1687 | if (ret < 0) { | 1698 | if (ret < 0) { |
1688 | dev_err(&pdev->dev, "FSIB stream probe failed\n"); | 1699 | dev_err(&pdev->dev, "FSIB stream probe failed\n"); |
1689 | goto exit_fsia; | 1700 | goto exit_fsia; |