lkml.org 
[lkml]   [2006]   [Jun]   [22]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateThu, 22 Jun 2006 00:11:04 -0700
FromAndrew Morton <>
SubjectRe: [PATCH] cardbus: revert IO window limit
On Thu, 22 Jun 2006 09:48:05 +0300 (EEST)
Pekka J Enberg <penberg@cs.Helsinki.FI> wrote:

> From: Pekka Enberg <penberg@cs.helsinki.fi>
> 
> 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 <alesan@manoweb.com>
> Cc: Linus Torvalds <torvalds@osdl.org>
> Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
> ---
> 
> 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 majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-06-22 09:13    [from the cache]
©2003-2008