Re: [PATCH 2/4] ASoC: pxa-ssp: PXA2xx/3xx SSP ports start from 1, not 0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 3, 2009 at 5:47 PM, Mark Brown <broonie@xxxxxxxxxxxxx> wrote:
> On Tue, Feb 03, 2009 at 05:19:41PM +0100, Philipp Zabel wrote:
>> Signed-off-by: Philipp Zabel <philipp.zabel@xxxxxxxxx>
>
> Meh, all the display names use the documentation names and the id is
> used for indexing into arrays (which may well be why you're not getting
> any audio out...).  I'd be more inclined to leave this as-is but if not
> it needs to adjust all the array usages.

Meh, indeed. Jumping between pxa2xx-ssp and pxa-ssp must have confused
me. A lot.
The only thing wrong with pxa-ssp is that it requests the wrong SSP port, then:

diff --git a/sound/soc/pxa/pxa-ssp.c b/sound/soc/pxa/pxa-ssp.c
index 73cb6b4..7f7c23e 100644
--- a/sound/soc/pxa/pxa-ssp.c
+++ b/sound/soc/pxa/pxa-ssp.c
@@ -751,7 +751,7 @@ static int pxa_ssp_probe(struct platform_device *pdev,
        if (!priv)
                return -ENOMEM;

-       priv->dev.ssp = ssp_request(dai->id, "SoC audio");
+       priv->dev.ssp = ssp_request(dai->id + 1, "SoC audio");
        if (priv->dev.ssp == NULL) {
                ret = -ENODEV;
                goto err_priv;

But this isn't right either. Because if we request the correct SSP
port here, the subsequent request of the same SSP port in ssp_init
(called from ops->startup) fails.
How to fix?

regards
Philipp
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux