[Meego-kernel] [PATCH] ar6003 : Remove AR6002_REV43 flag in Makefile

[Samuel Ortiz]

Hi Samuel,

On Thu, Jul 07, 2011 at 08:26:07AM +0000, Chang, Samuel wrote:
> Thanks Samuel.Ortiz.
> 
> Please check the V2 patch
> 
> Subject: Remove AR6002_REV43 flag in Makefile
> 
> This flag is for AR6003 hw 2.1.1 firmware only.
> Confirm with RF/FW engineer, Driver set this flag will cause
> driver to read wrong data by an extra shift.
> 
> Signed-off-by: Samuel Chang <samuelc at qca.qualcomm.com>
Acked-by: Samuel Ortiz <sameo at linux.intel.com>

This fixes BOO #10011 and BOO #10219.
Kristen, could you please apply this one ?

Cheers,
Samuel.


> 
> diff -ruN kernel-2.6.37.6-24.3_cfg80211_fix_patch_1/drivers/staging/ar6003/Makefile kernel-2.6.37.6-24.3_Remove_AR6002_REV43_patch_1.2/drivers/staging/ar6003/Makefile
> --- kernel-2.6.37.6-24.3_cfg80211_fix_patch_1/drivers/staging/ar6003/Makefile	2011-07-03 12:29:58.000000000 +0800
> +++ kernel-2.6.37.6-24.3_Remove_AR6002_REV43_patch_1.2/drivers/staging/ar6003/Makefile	2011-07-07 16:14:44.506760435 +0800
> @@ -117,7 +117,6 @@
>  
>  ifeq ($(CONFIG_AR6003_2_1_1),y)
>     EXTRA_CFLAGS += -DAR6003
> -   EXTRA_CFLAGS += -DAR6002_REV43
>  endif
>  
>  ifeq ($(CONFIG_AR6003_ENABLE_COEXISTENCE),y)
> 
> ________________________________________
> 寄件者: Samuel Ortiz [sameo at linux.intel.com]
> 寄件日期: 2011年7月7日 上午 03:05
> 收件者: Chang, Samuel
> 副本: meego-kernel at lists.meego.com; kristen.c.accardi at intel.com; Wu, Ricky; Huang, Maggie; Yang, Jack
> 主旨: Re: [Meego-kernel] [PATCH] ar6003 : Remove AR6002_REV43 flag in Makefile
> 
> Hi Samuel,
> 
> On Wed, Jul 06, 2011 at 06:44:24PM +0000, Chang, Samuel wrote:
> > Hi Samuel,
> >
> > This patch should be related to below BOO #
> >
> > Bug 10219
> > Bug 10011
> Fair enough. Please remove the unrelated chunk from your patch and I'll ACK
> it.
> 
> Cheers,
> Samuel.
> 
> --
> Intel Open Source Technology Centre
> http://oss.intel.com/

> 
> Subject: Remove AR6002_REV43 flag in Makefile
> 
> This flag is for AR6003 hw 2.1.1 firmware only.
> Confirm with RF/FW engineer, Driver set this flag will cause
> driver to read wrong data by an extra shift.
> 
> Signed-off-by: Samuel Chang <samuelc at qca.qualcomm.com>
> 
> 
> diff -ruN kernel-2.6.37.6-24.3_cfg80211_fix_patch_1/drivers/staging/ar6003/Makefile kernel-2.6.37.6-24.3_Remove_AR6002_REV43_patch_1.2/drivers/staging/ar6003/Makefile
> --- kernel-2.6.37.6-24.3_cfg80211_fix_patch_1/drivers/staging/ar6003/Makefile	2011-07-03 12:29:58.000000000 +0800
> +++ kernel-2.6.37.6-24.3_Remove_AR6002_REV43_patch_1.2/drivers/staging/ar6003/Makefile	2011-07-07 16:14:44.506760435 +0800
> @@ -117,7 +117,6 @@
>  
>  ifeq ($(CONFIG_AR6003_2_1_1),y)
>     EXTRA_CFLAGS += -DAR6003
> -   EXTRA_CFLAGS += -DAR6002_REV43
>  endif
>  
>  ifeq ($(CONFIG_AR6003_ENABLE_COEXISTENCE),y)


-- 
Intel Open Source Technology Centre
http://oss.intel.com/