Re: [PATCH] 2.4.10-ac11 parport_pc.c bugfix

J . A . Magallon (jamagallon@able.es)
Thu, 11 Oct 2001 15:40:39 +0200


On 20011011 Thomas Hood wrote:
> } else {
>- if ( dev->irq_resource[0].start == -1 ) {
>+ if ( dev->irq_resource[0].start == (unsigned long)-1 ) {
^^^^^^^^^ ^
Uh ?

Perhaps I miss some black magic in kernel programming, but could not this
be written much cleaner like

>+ if ( dev->irq_resource[0].start == ~0U ) {

-- 
J.A. Magallon                           #  Let the source be with you...        
mailto:jamagallon@able.es
Mandrake Linux release 8.2 (Cooker) for i586
Linux werewolf 2.4.10-ac11-beo #2 SMP Thu Oct 11 02:41:04 CEST 2001 i686
-
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/