Re: [PATCH i-g-t] lib/igt_eld: Fix 88.2kHz/176.4kHz sample rate parsing

["Sharma, Swati2" <swati2.sharma@intel.com>]

Hi Chaitanya

On 01-09-2026 05:03 pm, Chaitanya Kumar Borah wrote:
> parse_sad_rate() matched the rounded labels 88000/176000, but ALSA's
> eld proc file reports the exact rates 88200/176400, so those never
> matched and were silently dropped even when present in the ELD:
>
>    $ cat /proc/asound/card0/eld#2.4
>    [...]
>    sad_count               1
>    sad0_rates              [0x1ee0] 32000 44100 48000 88200 96000 176400 192000
>
> Match the exact rates ALSA reports instead of the rounded ones.

Patch LGTM

Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

>
> Cc: Simon Ser <contact@emersion.fr>
> Cc: Swati Sharma <swati2.sharma@intel.com>
> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
> ---
>   lib/igt_eld.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/igt_eld.c b/lib/igt_eld.c
> index 4a6f867ab..896ab0e6b 100644
> --- a/lib/igt_eld.c
> +++ b/lib/igt_eld.c
> @@ -76,11 +76,11 @@ static enum cea_sad_sampling_rate parse_sad_rate(const char *value)
>   		return CEA_SAD_SAMPLING_RATE_44KHZ;
>   	case 48000:
>   		return CEA_SAD_SAMPLING_RATE_48KHZ;
> -	case 88000:
> +	case 88200:
>   		return CEA_SAD_SAMPLING_RATE_88KHZ;
>   	case 96000:
>   		return CEA_SAD_SAMPLING_RATE_96KHZ;
> -	case 176000:
> +	case 176400:
>   		return CEA_SAD_SAMPLING_RATE_176KHZ;
>   	case 192000:
>   		return CEA_SAD_SAMPLING_RATE_192KHZ;