Re: [PATCH 1/2] ASoC: codecs: aw88261: fix changing sample rate and bit width

[Mark Brown]

On Mon, Apr 20, 2026 at 04:39:58PM -0300, Val Packett wrote:
> The aw88261 driver only worked with 32-bit 48kHz streams so far due to
> the lack of a proper PLL initialization sequence. Fix by selecting all
> the necessary PLL settings based on what was passed to us in the
> hw_params DAI callback. This replaces the strange downstream routine
> that tries two divider modes in sequence.

> +static int aw88261_dev_configure_syspll(struct aw88261 *aw88261)
>  {


> +	switch (aw88261->sample_rate) {
> +	case 8000:
> +	case 16000:
> +	case 32000:
> +	case 44100:
> +	case 48000:
> +	case 96000:
> +	case 192000:

...

> +	default:
> +		dev_err(aw_dev->dev, "unsupported sample rate %d\n",
> +			aw88261->sample_rate);
> +		return -EINVAL;
>  	}

The driver advertises more rats than this, including 11.025kHz, 12kHz
and multiples thereof.  It's unlikely anything would actually *use* them
but it'd be better to update the constraints.

Attachment: signature.asc
Description: PGP signature