[PATCH] [Lucid] UBUNTU: SAUCE: Make CONFIG_{OMNIBOOK, AVERATEC_5100P, PACKARDBELL_E5} depend on X86

[Andy Whitcroft]

On Wed, Jan 27, 2010 at 10:02:24PM +0800, Eric Miao wrote:
> commit a9244d2e2bb71094fe64b0063ba35963bfc6d656
> Author: Eric Miao <eric.y.miao at gmail.com>
> Date:   Wed Jan 27 21:50:51 2010 +0800
> 
>     UBUNTU: SAUCE: Make
> CONFIG_{OMNIBOOK,AVERATEC_5100P,PACKARDBELL_E5} depend on X86
> 
>     There options are really X86 specific, and the source won't compile on
>     ARM platforms. Make them depend on X86 and less confusing when syncing
>     config options to ARM.
> 
>     Signed-off-by: Eric Miao <eric.miao at canonical.com>
> 
> diff --git a/ubuntu/omnibook/Kconfig b/ubuntu/omnibook/Kconfig
> index 370b379..acad37a 100644
> --- a/ubuntu/omnibook/Kconfig
> +++ b/ubuntu/omnibook/Kconfig
> @@ -1,4 +1,5 @@
>  config OMNIBOOK
>  	tristate "Kernel module for HP and Toshiba laptops"
> +	depends on X86
>  	default m
>  	---help---
> diff --git a/ubuntu/rfkill/Kconfig b/ubuntu/rfkill/Kconfig
> index ff6d8a1..efb62c2 100644
> --- a/ubuntu/rfkill/Kconfig
> +++ b/ubuntu/rfkill/Kconfig
> @@ -1,7 +1,9 @@
>  config AVERATEC_5100P
>  	tristate "Software kill switch for Averatec 5100P"
> +	depends on X86
>  	default m
> 
>  config PACKARDBELL_E5
>  	tristate "Software kill switch for Packard Bell EasyNote E5"
> +	depends on X86
>  	default m

This looks reasonable to me assuming they are really only available on
X86.  Manjo the omnibook one was yours, I believe its X86 only can you
confirm?  The other two also sound so.  Based on that I think its a
reasonable patch.

Manjo?

-apw