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

Jeff Kirsher
2014-10-15T18:54:18Z

[-- Attachment #1: Type: text/plain, Size: 1107 bytes --]

On Wed, 2014-10-15 at 15:24 -0300, Fabio Estevam wrote:
> On Wed, Oct 15, 2014 at 2:42 PM, Jeff Kirsher
> <jeffrey.t.kirsher@intel.com> wrote:
>
> > I like the idea, but won't this exclude other 32 bit systems like MIPS
> > and I am sure there might be others. I could assume that Beceem WIMAX
> > was intended for x86 arch's that are 32 bit only, which may be a safe
> > bet. The only thing we know for sure is that is broken on 64bit.
>
> What about this?

That works for me. I am putting together a v2 patch now, since you came
up with the change, I will put you as the author Fabio, ok?

>
> diff --git a/drivers/staging/bcm/Kconfig b/drivers/staging/bcm/Kconfig
> index 8acf4b2..fa5a3a4 100644
> --- a/drivers/staging/bcm/Kconfig
> +++ b/drivers/staging/bcm/Kconfig
> @@ -1,6 +1,6 @@
> config BCM_WIMAX
> tristate "Beceem BCS200/BCS220-3 and BCSM250 wimax support"
> - depends on USB && NET
> + depends on USB && NET && !64BIT
> help
> This is an experimental driver for the Beceem WIMAX chipset used
> by Sprint 4G.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]