[Jaunty, armel] PATCH and RFC

[Amit Kucheria]

On Thu, Apr 30, 2009 at 03:13:09PM +0200, Stefan Bader wrote:
> Alright, this time from the usual suspect and with the patch updated to 
> reflect the discussion with Amit.
>
> Summary: the rtl8187se driver is not needed because versatile is only 
> used in Qemu and the risk for side effects by enabling 
> CONFIG_WIRELESS_EXT is higher.
>
> So dropping the patch for removing the defines as it is not strictly 
> necessary and adapted the configs to just disable the driver from 
> versatile.
>

ACK.

> From 6e9d0d0ca614ad41d7dcb3d151e24f42dedb4222 Mon Sep 17 00:00:00 2001
> From: Stefan Bader <stefan.bader at canonical.com>
> Date: Thu, 30 Apr 2009 09:42:38 +0200
> Subject: [PATCH] UBUNTU: SAUCE: Make rtl8187se depend on WIRELESS_EXT
> 
> Bug: #366144
> BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/366144
> 
> Seen to fail to build when this is not set.
> 
> Signed-off-by: Stefan Bader <stefan.bader at canonical.com>
> ---
>  drivers/staging/rtl8187se/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/staging/rtl8187se/Kconfig b/drivers/staging/rtl8187se/Kconfig
> index 79c225a..236e427 100644
> --- a/drivers/staging/rtl8187se/Kconfig
> +++ b/drivers/staging/rtl8187se/Kconfig
> @@ -1,5 +1,6 @@
>  config RTL8187SE
>  	tristate "RealTek RTL8187SE Wireless LAN NIC driver"
>  	depends on PCI
> +	depends on WIRELESS_EXT
>  	default N
>  	---help---
> -- 
> 1.6.0.4
> 

> From 8990cf65da24ffb225384a4e286e7d4f6e0c71b0 Mon Sep 17 00:00:00 2001
> From: Stefan Bader <stefan.bader at canonical.com>
> Date: Thu, 30 Apr 2009 15:06:35 +0200
> Subject: [PATCH] UBUNTU: Disable CONFIG_RTL8187SE for armel.versatile
> 
> Bug: #366144
> BugLink: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/366144
> 
> This disables CONFIG_RTL8187SE for armel.versatile as that has not defined
> CONFIG_WIRELESS_EXT.
> 
> Signed-off-by: Stefan Bader <stefan.bader at canonical.com>
> ---
>  debian/config/armel/config.versatile |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/debian/config/armel/config.versatile b/debian/config/armel/config.versatile
> index 7c56c96..daddc67 100644
> --- a/debian/config/armel/config.versatile
> +++ b/debian/config/armel/config.versatile
> @@ -620,7 +620,6 @@ CONFIG_RTC_DRV_PL031=y
>  CONFIG_RTC_HCTOSYS=y
>  CONFIG_RTC_HCTOSYS_DEVICE="rtc0"
>  # CONFIG_RTC_INTF_DEV_UIE_EMUL is not set
> -CONFIG_RTL8187SE=m
>  # CONFIG_RT_MUTEX_TESTER is not set
>  # CONFIG_SATA_AHCI is not set
>  # CONFIG_SATA_PMP is not set
> -- 
> 1.6.0.4
> 


-- 
----------------------------------------------------------------------
Amit Kucheria, Kernel Engineer || amit at canonical.com
----------------------------------------------------------------------