Hi Jean, The scx200/CS5535/etc only exist with/on the Geode processor, which is x86. So, I think the best option is to make it depend on X86. Thank, Ben > -----Original Message----- > From: Jean Delvare [mailto:khali at linux-fr.org] > Sent: Friday, January 20, 2006 2:06 PM > To: Gardner, Ben > Cc: LM Sensors > Subject: scx200_acb compilation breakage > > Hi Ben, > > Andrew Morton is reporting scx200_acb compilation breakage on non-x86 > due to the use of asm/msr.h, which only i386 and x86_64 have. This is > caused by your patch adding CS5535 support. > > Now, how do we fix the problem? > > First solution is to make the driver depend on X86. But are > there other > architectures where this driver may be used? I'd guess not. > > Second solution is to surround the CS5535 specific parts with #ifdef > X86 so that the driver can still be compiled on other architectures. > > Opinions? > > Thanks, > -- > Jean Delvare > >
