Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Dan Carpenter
On Wed, Oct 15, 2014 at 01:26:07PM -0700, Jeff Kirsher wrote: On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Jeff Kirsher
On Thu, 2014-10-16 at 10:52 +0300, Dan Carpenter wrote: On Wed, Oct 15, 2014 at 01:26:07PM -0700, Jeff Kirsher wrote: On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Dan Carpenter
COMPILE_TEST is just so that you can view build warnings. We would say: depends on USB NET (!64BIT || COMPILE_TEST) of depends on USB NET (X86_32 || (X86 COMPILE_TEST)) If you don't have USB or NET, you're get build errors but they don't indicate bugs. If you build on X86_64 the

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-16 Thread Greg KH
On Wed, Oct 15, 2014 at 03:59:21PM -0700, Jeff Kirsher wrote: On Wed, 2014-10-15 at 12:01 -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In

[PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: drivers/staging/bcm/CmHost.c:1503:3: warning: cast to pointer from integer of different size

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Dan Carpenter
On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’:

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
On Wed, 2014-10-15 at 22:55 +0300, Dan Carpenter wrote: On Wed, Oct 15, 2014 at 12:01:41PM -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In

Re: [PATCH v2] bcm/CmHost.c: Fix noisy compile warnings

2014-10-15 Thread Jeff Kirsher
On Wed, 2014-10-15 at 12:01 -0700, Jeff Kirsher wrote: From: Fabio Estevam fabio.este...@freescale.com The Beceem WIMAX was generating compile warnings on 64bit machines, which were: drivers/staging/bcm/CmHost.c: In function ‘StoreCmControlResponseMessage’: