[alsa-devel] hda_analyzer vs alc668 asus NJ750k

[Takashi Iwai]

At Tue, 29 Jul 2014 23:09:02 +1000 (EST),
Patrick Shirkey wrote:
> 
> Hi,
> 
> I get some errors running hda_analyzer with this alc668 chipset.
> 
> Traceback (most recent call last):
>   File "/dev/shm/hda-analyzer/hda_analyzer.py", line 546, in <module>
>     sys.exit(main(sys.argv))
>   File "/dev/shm/hda-analyzer/hda_analyzer.py", line 523, in main
>     if read_nodes(sys.argv[1:]) == 0:
>   File "/dev/shm/hda-analyzer/hda_analyzer.py", line 89, in read_nodes
>     read_nodes2(c.card, i)
>   File "/dev/shm/hda-analyzer/hda_analyzer.py", line 60, in read_nodes2
>     c = HDACodec(card, codec)
>   File "/dev/shm/hda-analyzer/hda_codec.py", line 1041, in __init__
>     self.parse_proc()
>   File "/dev/shm/hda-analyzer/hda_codec.py", line 1140, in parse_proc
>     self.proc_codec = HDACodecProc(self.card, self.device, file)
>   File "/dev/shm/hda-analyzer/hda_proc.py", line 479, in __init__
>     self.parse(proc_file)
>   File "/dev/shm/hda-analyzer/hda_proc.py", line 677, in parse
>     node.add_digital(line[11:])
>   File "/dev/shm/hda-analyzer/hda_proc.py", line 296, in add_digital
>     self.wrongfile('unknown dig1 bit %s' % repr(b))
>   File "/dev/shm/hda-analyzer/hda_proc.py", line 143, in wrongfile
>     raise ValueError, "wrong proc file format (%s)" % msg
> ValueError: wrong proc file format (unknown dig1 bit 'KAE')
> 
> 
> If I disable the check for add_digital I get this error:
> 
> ValueError: wrong proc file format (  Digital: Enabled KAE)
> 
> - Is it a problem to add a new DIG1_BIT to hda_proc.py:277
> 
>  def add_digital(self, line):
>     bits = {
>       'Enabled': DIG1_BITS['ENABLE'],
>       'Validity': DIG1_BITS['VALIDITY'],
>       'ValidityCfg': DIG1_BITS['VALIDITYCFG'],
>       'Preemphasis': DIG1_BITS['EMPHASIS'],
>       'Copyright': DIG1_BITS['COPYRIGHT'], # old buggy format
>       'Non-Copyright': DIG1_BITS['COPYRIGHT'],
>       'Non-Audio': DIG1_BITS['NONAUDIO'],
>       'Pro': DIG1_BITS['PROFESSIONAL'],
>       'GenLevel': DIG1_BITS['LEVEL'],
>       'KAE': DIG1_BITS['KAE'],
>     }
> 
> hda_codec.py:224
> 
>  DIG1_BITS = {
>   'ENABLE': 0,
>   'VALIDITY': 1,
>   'VALIDITYCFG': 2,
>   'EMPHASIS': 3,
>   'COPYRIGHT': 4,
>   'NONAUDIO': 5,
>   'PROFESSIONAL': 6,
>   'LEVEL': 7,
>   'KAE': 8
> }

KAE belongs to DIG3, so shouldn't be there.
I updated the git repo now.

While we are at it: it would make sense to move this from alsa to
alsa-tools repo, IMO.  Jaroslav, any objection?

The only question is the installation to the system, though.


Takashi