On Sun, Jun 29, 2003 at 05:36:36PM -0700, ilya@theIlya.com wrote:
> This is the patch that includes most of things that I have in IP32-specific
> parts of my tree.
> 2. Propper memory detection pathc by Keith.
Please don't apply this. It's a mostly nice piece of code with one
horrible hack, and we're better off for now staying with
CONFIG_ARC_MEMORY until we decide how to best support >256MB of
memory.
The CRIME error handling part is golden however.
> 3. Some other minor fixlets.
> void ip32_irq1(struct pt_regs *regs)
> Index: arch/mips/sgi-ip32/ip32-setup.c
>
> +#ifdef CONFIG_FB_SGIO2
> +#include "../../../drivers/video/sgio2fb.h"
> +void *sgio2fb_mem;
> +#endif
You can't reference this because that driver doesn't exist yet. And
including that here is kind of ugly anyway.
> -#ifdef CONFIG_SERIAL_CONSOLE
> +#ifdef CONFIG_SERIAL_MACE_SGIO2
> +#warning O2MACECONSOLE compiled in
> + o2serial_console_init();
> +#endif
Debugging snippet leaked in.
> -}
> -
> -int __init page_is_ram (unsigned long pagenr)
> -{
> - /* XXX: to do? */
> - return 1;
> }
Yes, this appears to be unused.
> Index: include/asm-mips64/ip32/crime.h
> ===================================================================
> RCS file: /home/cvs/linux/include/asm-mips64/ip32/crime.h,v
...
> -static inline u64 crime_read_64 (unsigned long __offset) {
> - return *((volatile u64 *) (CRIME_BASE + __offset));
> -}
> -static inline void crime_write_64 (unsigned long __offset, u64 __val) {
> - *((volatile u64 *) (CRIME_BASE + __offset)) = __val;
> -}
> +#define crime_read_64(__offset) __in64(CRIME_BASE+(__offset))
> +#define crime_write_64(__offset,__val)
> __out64(__val,CRIME_BASE+(__offset))
You can't do this yet because I just sent Ralf the patch to add those
functions, and they're going to be named __raw_readq and __raw_writeq
as well. I'll take care of this as soon as that patch is approved.
> --- /dev/null Sun Jul 17 16:46:18 1994
> +++ include/asm-mips/io64.h Sun Jun 15 10:35:18 2003
There will be no io64.h.
--
Keith M Wesolowski <wesolows@foobazco.org> http://foobazco.org/~wesolows
------(( Project Foobazco Coordinator and Network Administrator ))------
"May Buddha bless all stubborn people!"
-- Uliassutai Karakorum Blake
|