lkml.org 
[lkml]   [2005]   [Jul]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subject2.6.13-rc2-mm1: some speakup nitpicks
On Thu, Jul 07, 2005 at 04:00:37AM -0700, Andrew Morton wrote:
>...
> Changes since 2.6.13-rc1-mm1:
>...
> +gregkh-driver-speakup-docs.patch
> +gregkh-driver-speakup-core.patch
>
> driver-core updates
>...

These aren't driver-core updates, these are new drivers.

It seems I missed when this was sent for review to linus-kernel.

Some random nitpicks:

- SPEAKUP_DEFAULT shouldn't be asked if SPEAKUP=n
- "make namespacecheck" shows tons of needlessly global code
- the static variable special_handler is EXPORT_SYMBOL'ed
- #define MIN should be removed
- the file cvsversion.h only for keeping a CVS date is a bit
overkill
- spk_con_module.h is not exactly how we use header files in the kernel
- many of the #ifdef MODULE's point to things that could be done better
(especially the #include "mod_code.c"'s)
- the things in synthlist.h could be done less ugly
- speakupconf is a userspace script that belongs under Documentation/
- dtload.c is not kernel code, and should therefore not be in that
directory
- the code should follow Documentation/CodingStyle better
(no spaces between the braces and function arguments)
- building speakup_keyhelp.c modular even in a kernel that doesn't
support modules is silly
- #include <linux/...> belongs before #include <asm/...>

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
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: 2005-07-09 04:11    [W:0.999 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site