Re: [PATCHv2] parport: add parallel port support for SGI O2
From: Arnaud Giersch
Date: Tue Jan 24 2006 - 18:31:53 EST
Mardi 24 janvier 2006, vers 00:38:57 (+0100), j'ai écrit:
> I will correct the style.
[...]
> I will add a comment to explain it in the code.
[...]
> I did not realized that. I will try to correct the problem.
An updated version is available at
http://arnaud.giersch.free.fr/parport_ip32/parport_ip32-0.6.patch.gz
Since all changes are non-functional ones, I did not want to flood the
lists again with the full patch.
Changes are:
* Address comments from Andrew Morton <akpm_at_osdl.org>:
- Style: avoid single line "if" and "case" statements.
- Turn some printk() into pr_debug1(), so that they only exist
in debugging mode.
- Add a comment about "volatile".
- Add a comment about readsb() and writesb().
* Only define DEBUG_PARPORT_IP32 if it is not already defined.
* Remove the CVS $Id$ line.
Arnaud
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/