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

From: John Watts
Date: Fri Aug 11 2023 - 10:26:49 EST


On Fri, Aug 11, 2023 at 02:59:35PM +0100, Mark Brown wrote:
> 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.

Hi there, the pin is tristate, Z, 0 and 1. How would that be represented?

John.