Re: [PATCH] cardbus: revert IO window limit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 22 Jun 2006 09:48:05 +0300 (EEST)
Pekka J Enberg <[email protected]> wrote:

> From: Pekka Enberg <[email protected]>
> 
> This patch reverts commit 4196c3af25d98204216a5d6c37ad2cb303a1f2bf "cardbus:
> limit IO windows to 256 bytes" which breaks Alessio Sangalli's machine boot
> when APM support is enabled. See http://lkml.org/lkml/2006/6/16/33 for
> description of the problem.
> 
> Cc: Alessio Sangalli <[email protected]>
> Cc: Linus Torvalds <[email protected]>
> Signed-off-by: Pekka Enberg <[email protected]>
> ---
> 
> diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c
> index 28ce3a7..657be94 100644
> --- a/drivers/pci/setup-bus.c
> +++ b/drivers/pci/setup-bus.c
> @@ -40,7 +40,7 @@ #define ROUND_UP(x, a)		(((x) + (a) - 1)
>   * FIXME: IO should be max 256 bytes.  However, since we may
>   * have a P2P bridge below a cardbus bridge, we need 4K.
>   */
> -#define CARDBUS_IO_SIZE		(256)
> +#define CARDBUS_IO_SIZE		(4*1024)
>  #define CARDBUS_MEM_SIZE	(32*1024*1024)
>  
>  static void __devinit

There is something bad happening in there.  Presumably, this patch will
break the ThinkPad 600x series machines again though.

It'd be nice if this was related to
http://bugzilla.kernel.org/show_bug.cgi?id=6725, but I guess not.

Didn't all this stuff work in 2.4?
-
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/

[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]
  Powered by Linux