[PATCH 0/8] Improve style of OM files
Andy Green
andy at openmoko.com
Fri Oct 17 10:57:07 CEST 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Somebody in the thread at some point said:
|> PS: No offense, Nelson. The time you are investing in this is
|> definitely appeciated... let's just get your time focussed on
|> something more meaningful -- it's the best way to learn, really. Hope
|> the above list can be of some inspiration to you...
|
| It's OK. My initial goal was to be comfortable with Stgit and I like
it now.
Ah great, it will definitely pay off. Here is another script I use
(when I remember anyway) called /usr/local/bin/stgcheckpatch here
#!/bin/sh
PATCH=$1
if [ -z "$1" ] ; then
~ stg refresh
~ PATCH=`stg top`
~ echo "*** checking top patch $PATCH"
fi
stg export $PATCH && \
./scripts/checkpatch.pl patches-`stg branch`/$PATCH
with no argument it checks the top patch on the stack for kernel style
using ./scripts/checkpatch.pl, or you can have it check a named patch on
the stack.
| As I've said before I'm getting ready for next month. Now I think I'd
better
| wait until then.
OK... I think code cleaning is actually a good way to go through the
codebase getting the idea of how it hangs together, but actually the
kernel style takes a while to sink into the fingers, so ironically code
cleaning for style is not perfect task to start with.
Some of Jonas' ideas sound good, maybe the drivers/mmc/host/s3cmci.c
would make sense.
- -Andy
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org
iEYEARECAAYFAkj4U10ACgkQOjLpvpq7dMrEqQCfTTM6SXxaaBjj4yXIrHMOeOfv
SIUAn3kSgV0jD1m9HrYLcZn0C2A+L4FH
=TNuq
-----END PGP SIGNATURE-----
More information about the openmoko-kernel
mailing list