Re: Linux v2.6.18-rc4, snd-aoa link error

[Johannes Berg <johannes@sipsolutions.net>]

Takashi Iwai wrote:
>> --- aoa/Kconfig~	2006-08-08 16:02:30.000000000 -0400
>> +++ aoa/Kconfig	2006-08-08 16:03:26.000000000 -0400
>> @@ -5,6 +5,7 @@
>>  	tristate "Apple Onboard Audio driver"
>>  	depends on SND
>>  	select SND_PCM
>> +	select I2C
>>  	---help---
>>  	This option enables the new driver for the various
>>  	Apple Onboard Audio components.
>>     
>
> Maybe better to put it in aoa/codecs/Kconfig.
> The requirement of i2c is not in the core part of aoa driver but in
> the codec support codes.
>   
Yes, each codec except for toonie should select I2C. Looks like I missed 
that, sorry.

However, I was recently told that the expected behaviour with I2C (in a 
thread discussing the motion sensor) that I2C should be depended upon, 
not selected. I very much disagree (*), however, and would like to see a 
select here.

Unfortunately, I started new work in Nürnberg, don't have any internet 
at home right now and am not supposed to bring my own laptop to work. 
Would someone else create the required patch?
> Should I2C_POWERMANC be selected, too?
It makes sense since the aoa driver is specific to machines that have 
the i2c powermac bus, but I'm ambivalent about it. Though, does the G5 
require I2C_POWERMAC? I'm not completely sure about that.

johannes

(*) I don't think it makes sense to bother someone configuring the 
kernel with the some concept called I2C. The user should be able to just 
select the sound driver necessary.

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642