Re: Fwd: [PATCH] sound working on the macbook pro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Sun, 18 Feb 2007 17:24:37 +0000,
=?UTF-8?Q?Rafael_Esp=C3=ADndola?= wrote:
> 
> ---------- Forwarded message ----------
> From: Rafael Espíndola <rafael.espindola@xxxxxxxxx>
> Date: Feb 18, 2007 5:06 PM
> Subject: [PATCH] sound working on the macbook pro
> To: alsa-devel@xxxxxxxxxxxxxxxx
> 
> 
> The problem with the pin config of the macbook pro is that it has no
> line outs. Because of this, snd_hda_parse_pin_def_config tries to hack
> the speaker into being a line out. This hack makes the subsequent
> setup fail.

How (and what) does it fail exactly?


> The attached patch does the followig:
> 1) install the correct pin config (extracted from macos)
> 2) removes the hack from snd_hda_parse_pin_def_config
> 3) stac92xx_parse_auto_config no longuer aborts if line_outs == 0
> 
> item 2 might cause some problems for parts of the system that expect
> this hack. IMHO, we should try to fix these problems and remove the
> hack. It is very confusing to see an Speaker appearing as a line out!

I'm sure that this would break many other devices.  So, the patch is
unacceptable as it is, unfortunately.

The line_outs are actually primary outputs, not exactly "line-out".
You can find pretty many codes that refer to this as the primary
output.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux