[linux-sunxi] Re: [PATCH v3 3/3] ARM: dts: sun4i: gemei-g9: Enable sun4i audio codec support
Maxime Ripard
maxime.ripard at free-electrons.com
Thu Nov 19 08:09:23 PST 2015
Hi,
On Thu, Nov 12, 2015 at 08:53:19PM +0200, Priit Laes wrote:
> On Mon, 2015-11-09 at 11:59 +0800, Chen-Yu Tsai wrote:
> > On Sat, Nov 7, 2015 at 1:54 AM, Priit Laes <plaes at plaes.org> wrote:
> > > Gemei G9 has internal speakers and headphone jack. Audio switching
> > > from internal speakers to headphones is automatically handled by
> > > extra FT2012Q audio amplifier chip that works out of the box.
> >
> > Nice that it works out of the box. The FEX file does mention:
> >
> > audio_pa_ctrl = port:PH15<1><default><default><0>
>
> Nice catch.
>
> Setting it low mutes audio, and setting it back high unmutes.
Then you just volunteered yourself to fix the FIXME in the driver ;)
> > So either it is floating or pulled up by default? Since it works
> > now I don't see any reason to block it. On the other hand once
> > that binding is introduced it would be nice to add it for power
> > management reasons.
>
> Should I just add comment about it or do something like this:
>
> &codec {
> status = "okay";
> /*
> * TODO: Add codec_ext_pwr_pin to turn off external audio AMP
> &pio {
> codec_ext_pwr_pin: codec_ext_pwr_pin at 0 {
> allwinner,pins = "PH15";
> allwinner,function = "gpio_out";
> allwinner,drive = <SUN4I_PINCTRL_10_MA>;
> allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
> }
> }
> */
> }
More like
&pio {
codec_ext_pwr_pin: codec_ext_pwr_pin at 0 {
allwinner,pins = "PH15";
allwinner,function = "gpio_out";
allwinner,drive = <SUN4I_PINCTRL_10_MA>;
allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
};
};
&codec {
/* This pin is used to turn off the GPIO amp pin */
pinctrl-names = "default";
pinctrl-0 <&codec_ext_pwr_pin>;
status = "okay";
};
Of course, that's a temporary measure, and you should actually use
that GPIO to enable / disable the Amplifier when you are playing
sound. It shouldn't be that hard with ASoC.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151119/0b76ebe7/attachment.sig>
More information about the linux-arm-kernel
mailing list