On Thu, Nov 22, 2018 at 06:23:18PM +0800, Chen-Yu Tsai wrote: > On the Allwinner A64 SoC, the audio codec has a built-in headphone > amplifier. This amplifier has a power supply separate from the rest of > the analog audio circuitry. > > Add a regulator supply property to handle this. > > Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxx> > --- > Documentation/devicetree/bindings/sound/sun50i-codec-analog.txt | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/sun50i-codec-analog.txt b/Documentation/devicetree/bindings/sound/sun50i-codec-analog.txt > index 4f8ad0e04d20..056a098495cc 100644 > --- a/Documentation/devicetree/bindings/sound/sun50i-codec-analog.txt > +++ b/Documentation/devicetree/bindings/sound/sun50i-codec-analog.txt > @@ -4,9 +4,11 @@ Required properties: > - compatible: must be one of the following compatibles: > - "allwinner,sun50i-a64-codec-analog" > - reg: must contain the registers location and length > +- cpvdd-supply: Regulator supply for the headphone amplifier I guess we should make this optional, since even if it's missing, it's going to create a dummy regulator. and that allows to have backward compatibility. For the rest of the series, Acked-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxx> Thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
Attachment:
signature.asc
Description: PGP signature