[alsa-devel] idt blue jack patch

Wu Fengguang fengguang.wu at intel.com
Mon Feb 23 13:58:20 CET 2009


On Mon, Feb 23, 2009 at 12:21:18PM +0200, Takashi Iwai wrote:
> At Mon, 23 Feb 2009 17:53:51 +0800,
> Wu Fengguang wrote:
> > 
> > On Mon, Feb 23, 2009 at 10:54:36AM +0200, Takashi Iwai wrote:
> > > At Sat, 21 Feb 2009 17:28:21 +0100,
> > > I wrote:
> > > > > > Regarding the default pin configurations.
> > > > > > I made a series of patches to add the interface to change the default
> > > > > > pincfg values dynamically via hwdep sysfs.  The patches are found in
> > > > > > test/hda-pincfg branch of sound git tree
> > > > > >     git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
> > > > > > 
> > > > > > and included in the very latest alsa-driver-unstable snapshot
> > > > > >     ftp://ftp.kernel.org/pub/linux/kernel/people/tiwai/snapshot/alsa-driver-unstable-snapshot.tar.gz
> > > > > > 
> > > > > > Basically, you can get / set the default pincfg values on the fly, and
> > > > > > trigger the codec reconfiguration after that.
> > > > > > 
> > > > > > The init_pin_configs file shows the initial default pincfgs set by BIOS.
> > > > > > 
> > > > > > 	# cat /sys/class/sound/hwC0D0/init_pin_configs
> > > > > > 	0x14 0x9993013f
> > > > > > 	0x15 0x01014c10
> > > > > > 	0x16 0x99030120
> > > > > > 	0x18 0x01a19c30
> > > > > > 	0x19 0x02a11c31
> > > > > > 	0x1a 0x01813c32
> > > > > > 	0x1b 0x02211c1f
> > > > > > 	0x1c 0x99330133
> > > > > > 	0x1d 0x411111f0
> > > > > > 	0x1e 0x411111f0
> > > > > > 	0x1f 0x411111f0
> > > > > > 
> > > > > > Then, you can override the default value by writing to
> > > > > > override_pin_configs sysfs file.
> > > > > > 
> > > > > > 	# cat /sys/class/sound/hwC0D0/override_pin_configs
> > > > > > 	# echo 0x1d 0x02a11c30 > /sys/class/sound/hwC0D0/override_pin_configs
> > > > > > 	# cat /sys/class/sound/hwC0D0/override_pin_configs
> > > > > > 	0x1d 0x02a11c30
> > > > > > 
> > > > > > Note that this value won't appear in init_pin_configs.
> > > > > > 
> > > > > > Also, there is another file, cur_pin_configs.  This contains the
> > > > > > pincfgs that are set by the driver explicitly.  For example,
> > > > > > the static pincfg table in patch_sigmatel.c appears here.
> > > > > > 
> > > > > > Then you can trigger the codec reconfiguration by
> > > > > > 
> > > > > > 	# echo 1 > /sys/class/sound/hwC0D0/reconfig
> > > > > > 
> > > > > > This will re-setup the driver based on the given pin configs.
> > > > > 
> > > > > Cool!  This interface will greatly ease the developer-user interactions.
> > > > > 
> > > > > Without knowing this feature, I had to write a patch for Paulo,
> > > > > and compile/verify it before sending it to him. Paulo also have to
> > > > > pull the kernel source, apply the patch, compile and run it...
> > > > > 
> > > > > Now you turned that tedious process into two simple echo commands :-)
> > > > > 
> > > > > > Unfortunately, in the current scenario, it can't override the static
> > > > > > pincfg table in the driver like in patch_sigmatel.c.  It's just for
> > > > > > fixing and BIOS setup.
> > > > > 
> > > > > Unfortunately patch_sigmatel has a long list of static pincfgs, i.e.:
> > > > > 
> > > > >         static unsigned int ref927x_pin_configs[14] = {
> > > > >                 0x02214020, 0x02a19080, 0x0181304e, 0x01014010,
> > > > >                 0x01a19040, 0x01011012, 0x01016011, 0x0101201f,
> > > > >                 0x183301f0, 0x18a001f0, 0x18a001f0, 0x01442070,
> > > > >                 0x01c42190, 0x40000100,
> > > > >         };
> > > > > 
> > > > > This effectively prohibits itself from being dynamically reconfigured
> > > > > via the override_pin_configs interface... 
> > > > 
> > > > Right.  Considering the implementation again, I think it must be
> > > > relatively easy to fix that.  Just changing the order of the evaluation
> > > > of pincfg, and add the check not to overwrite the value.
> > > > 
> > > > I'll work on it in the next week.
> > > 
> > > Fixed now on sound-unstable tree, together with renaming of override_pin
> > > and cur_pin to user_pin and driver_pin.
> > > 
> > > It'd be appreciated if someone can test it with a device with
> > > Sigmatel/IDT (as I have no such hardware)...
> > 
> > Hi Takashi,
> > 
> > Here are the good behaviors:
> > 
> > - the driver_pin_configs matches the numbers defined in source code
> > - the init_pin_configs/user_pin_configs will be exactly restored between module reloading
> > - setting user_pin_configs and doing reconfig did changed the 0x0c pin from output to input.
> > 
> > However,
> > 
> > - after reconfig, Surround, Center and LFE channels are muted and have to be
> >   turned on again in alsamixer.
> 
> That's a feature.  You can restore via "alsactl restore"

Ah OK, I'd take it as a workaround ;-)

> > - if reconfig while playing audio, it will trigger a panic:
> 
> Ah, that's bad.  I know this can be a problem (and thus is marked as
> EXPERIMENTAL :)
> 
> We need some mutex and/or flag to disable the reconfig during the
> critical operation.

It won't be a trivial work. The reset/rebuild is a long and complex process.
For the moment I'd suggest to simply fail reconfig if there are active streams,
so as to let this useful feature go upstream ASAP.

Thanks,
Fengguang


More information about the Alsa-devel mailing list