Re: [PATCH] allow vesafb to build when no CONFIG_MTRR

> +#ifdef CONFIG_MTRR
>  	if (mtrr) {
> -		int temp_size = size_total;
> +		unsigned int temp_size = size_total;
>  		/* Find the largest power-of-two */
>  		while (temp_size & (temp_size - 1))
>                  	temp_size &= (temp_size - 1);
> @@ -396,6 +401,7 @@ static int __init vesafb_probe(struct de
>  			temp_size >>= 1;
>  		}
>  	}
> +#endif

I'd just do that to avoid cluttering up the source with
#ifdef's, otherwise it looks ok to me ;)

  Gerd

-- 
-mm seems unusually stable at present.
	-- akpm about 2.6.12-rc3-mm3
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] allow vesafb to build when no CONFIG_MTRR
    • From: "Jan Beulich" <[email protected]>

• Prev by Date: Re: [patch 11/12] drivers/block/umem.c: Use the DMA_{64, 32}BIT_MASK constants
• Next by Date: [PATCH] Pointer cast warnings in scripts/
• Previous by thread: [PATCH] allow vesafb to build when no CONFIG_MTRR
• Next by thread: Re: [PATCH] allow vesafb to build when no CONFIG_MTRR
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]