Re: [PATCH] IP27: misc fixes

[Ralf Baechle <ralf@linux-mips.org>]

On Thu, Apr 17, 2008 at 10:07:42PM +0200, Thomas Bogendoerfer wrote:

> @@ -100,6 +103,13 @@ int __cpuinit bridge_probe(nasid_t nasid, int widget_id, 
> int masterwid)
>        */
>       bridge->b_wid_control |= BRIDGE_CTRL_IO_SWAP |
>                                BRIDGE_CTRL_MEM_SWAP;
> +#ifdef CONFIG_PAGE_SIZE_4KB
> +     bridge->b_wid_control &= ~BRIDGE_CTRL_PAGE_SIZE;
> +#elif defined(CONFIG_PAGE_SIZE_16KB)
> +     bridge->b_wid_control |= BRIDGE_CTRL_PAGE_SIZE;
> +#else
> +#error Fixme for page size other than 4kB and 16kB
> +#endif

This blows up with 64k pages or larger (if anybody is mad enough ...)  The
reason for this setting is so the BRIDGE ASIC knows about the pagesize
and can avoid prefetching over a page boundary.  That is a too small value
is always safe.  I applied your patch with this small fix on top.

Thanks,

  Ralf