On Tue, 13 Jan 2009 09:52:14 +0200
"ext [email protected]" <[email protected]> wrote:

> From: Eero Nurkkala <[email protected]>
> 
> ASoC has an annoying bug letting either L or R channel to be
> played on L channel. In other words, L and R channels can
> switch at random. This provides McBSP funtionality that may
> be used to fix this feature.
> 
I can confirm this. Long lasting problem unfortunately not solved yet.
This is visible on 2420 as well so this patch is not enough.

http://marc.info/?l=linux-omap&m=120643770810683&w=2

...
> +void omap_mcbsp_xmit_enable(unsigned int id, u8 enable)
> +{
> +     struct omap_mcbsp *mcbsp;
> +     void __iomem *io_base;
> +     u16 w;
> +
> +     if (!(cpu_is_omap2430() || cpu_is_omap34xx()))
> +             return;
> +
> +     if (!omap_mcbsp_check_valid_id(id)) {
> +             printk(KERN_ERR "%s: Invalid id (%d)\n", __func__, id + 1);
> +             return;
> +     }
> +
> +     mcbsp = id_to_mcbsp_ptr(id);
> +     io_base = mcbsp->io_base;
> +
> +     w = OMAP_MCBSP_READ(io_base, XCCR);
> +
> +     if (enable)
> +             OMAP_MCBSP_WRITE(io_base, XCCR, w & ~(XDISABLE));

This doesn't fix problem on 2420. I assume when it works on 2420, the
same fix will fix the 34xx as well. I fear the mcbsp initialization
procedure in omap_mcbsp_start is not correct somewhat for 2420-34xx.


Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to