Re: [PATCH v2 1/2] ASoC: cs47l15: Add codec driver for Cirrus Logic CS47L15

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

 



On Thu, Jul 25, 2019 at 01:31:51PM +0100, Charles Keepax wrote:

>  	select SND_SOC_CS47L24 if MFD_CS47L24
> +	select SND_SOC_CS47L15 if MFD_CS47L15
>  	select SND_SOC_CS47L35 if MFD_CS47L35

The Makefile and Kconfig sorting is a bit off.  Please send a followup
fixing this.

> +static irqreturn_t cs47l15_adsp2_irq(int irq, void *data)
> +{
> +	struct cs47l15 *cs47l15 = data;
> +	struct madera_priv *priv = &cs47l15->core;
> +	struct madera *madera = priv->madera;
> +	int ret;
> +
> +	ret = wm_adsp_compr_handle_irq(&priv->adsp[0]);
> +	if (ret == -ENODEV) {
> +		dev_err(madera->dev, "Spurious compressed data IRQ\n");
> +		return IRQ_NONE;
> +	}
> +
> +	return IRQ_HANDLED;
> +}
> +
> +static irqreturn_t cs47l15_dsp_bus_error(int irq, void *data)
> +{
> +	struct wm_adsp *dsp = (struct wm_adsp *)data;
> +
> +	return wm_adsp2_bus_error(dsp);
> +}

Noticeable difference in pattern between these two interrupt helpers
(and makes me wonder why we need the wrappers at all).

Attachment: signature.asc
Description: PGP signature

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

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

  Powered by Linux