Messages in this thread | | | Date | Thu, 1 Aug 2002 00:00:03 +0100 | From | Russell King <> | Subject | Re: [PATCH] console part 2. |
| |
On Wed, Jul 31, 2002 at 03:27:57PM -0700, James Simmons wrote: > Here is the second patch. It has many fixes and alot of major changes > internally.
A quick read through reveals:
- printk("mdacon: MDA card not detected.\n"); + printk("KERN_WARNING mdacon: MDA card not detected.\n");
KERN_WARNING and friends should be outside the quotes.
Secondly, the absolutely gigantic "switch (vc_state) {" stuff with extra layers of switch statements below it in decvte.c - I find this rather disgusting to read. I bet the resulting asm is also disgusting. Isn't there a cleaner solution to this? (I've been carrying around since 2.2 patches to the console layer to split this up mainly because some older versions of ARM gcc choked on it. I'm not certain about current versions though.)
Also, something that should probably be fixed one day, but I wouldn't call it a show stopper:
-#define SIZE(x) (sizeof(x)/sizeof((x)[0])) +#define SIZE(x) (sizeof(x)/sizeof((x)[0]))
We have ARRAY_SIZE(x) in linux/kernel.h which does this already.
-- Russell King (rmk@arm.linux.org.uk) The developer of ARM Linux http://www.arm.linux.org.uk/personal/aboutme.html
- 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/
| |