Re: [PATCH 3/3] ASoC: dt-bindings: wlf,wm8782: Add max-rate property

From: Mark Brown
Date: Fri Aug 11 2023 - 10:01:50 EST


On Fri, Aug 11, 2023 at 08:49:30AM +1000, John Watts wrote:

> + - max-rate : max supported audio rate configured by FSAMPEN pin, defaults to 48000

This seems like a cumbersome and error prone way to configure this - why
not just have the binding specify the state of the pin? That's just a
boolean high/low whereas this requires getting the rate right and then
having the driver validate it.

Attachment: signature.asc
Description: PGP signature