| Message ID | 20230913171552.92252-4-contact@jookia.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ASoC: wm8782: Allow higher audio rates | expand |
On Thu, Sep 14, 2023 at 03:15:52AM +1000, John Watts wrote: > The WM8782 can safely support rates higher than 48kHz by changing the > value of the FSAMPEN pin. > > Allow specifying the FSAMPEN pin value in the device tree. > > Signed-off-by: John Watts <contact@jookia.org> > --- > Documentation/devicetree/bindings/sound/wm8782.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/wm8782.txt b/Documentation/devicetree/bindings/sound/wm8782.txt > index 256cdec6ec4d..d217a616e103 100644 > --- a/Documentation/devicetree/bindings/sound/wm8782.txt > +++ b/Documentation/devicetree/bindings/sound/wm8782.txt > @@ -8,10 +8,15 @@ Required properties: > - Vdda-supply : phandle to a regulator for the analog power supply (2.7V - 5.5V) > - Vdd-supply : phandle to a regulator for the digital power supply (2.7V - 3.6V) > > +Optional properties: > + > + - wlf,fsampen : FSAMPEN pin value, 0 for low, 1 for high, 2 for disconnected What's the default if the property is not present? > + > Example: > > wm8782: stereo-adc { > compatible = "wlf,wm8782"; > Vdda-supply = <&vdda_supply>; > Vdd-supply = <&vdd_supply>; > + wlf,fsampen = <2>; /* 192KHz */ > }; > -- > 2.42.0 >
On Thu, Sep 14, 2023 at 09:52:34AM -0500, Rob Herring wrote:
> What's the default if the property is not present?
0. Should I specify it here?
John.
On Fri, Sep 15, 2023 at 03:39:49PM +1000, John Watts wrote: > On Thu, Sep 14, 2023 at 09:52:34AM -0500, Rob Herring wrote: > > What's the default if the property is not present? > > 0. Should I specify it here? Yes.
diff --git a/Documentation/devicetree/bindings/sound/wm8782.txt b/Documentation/devicetree/bindings/sound/wm8782.txt index 256cdec6ec4d..d217a616e103 100644 --- a/Documentation/devicetree/bindings/sound/wm8782.txt +++ b/Documentation/devicetree/bindings/sound/wm8782.txt @@ -8,10 +8,15 @@ Required properties: - Vdda-supply : phandle to a regulator for the analog power supply (2.7V - 5.5V) - Vdd-supply : phandle to a regulator for the digital power supply (2.7V - 3.6V) +Optional properties: + + - wlf,fsampen : FSAMPEN pin value, 0 for low, 1 for high, 2 for disconnected + Example: wm8782: stereo-adc { compatible = "wlf,wm8782"; Vdda-supply = <&vdda_supply>; Vdd-supply = <&vdd_supply>; + wlf,fsampen = <2>; /* 192KHz */ };
The WM8782 can safely support rates higher than 48kHz by changing the value of the FSAMPEN pin. Allow specifying the FSAMPEN pin value in the device tree. Signed-off-by: John Watts <contact@jookia.org> --- Documentation/devicetree/bindings/sound/wm8782.txt | 5 +++++ 1 file changed, 5 insertions(+)