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

From: J . A . Magallon (jamagallon@able.es)
Date: Thu Oct 11 2001 - 08:40:39 EST

Next message: jkniiv@hushmail.com: "Re: RE: Dilemma: Replace Escalade with Adaptec 2400A or Promise Super trak66?"
Previous message: Stefan Smietanowski: "Re: Re: [PATCH] Re: Lost Partition"
In reply to: Thomas Hood: "[PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Next in thread: Thomas Hood: "Re: [PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Reply: Thomas Hood: "Re: [PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

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/

Next message: jkniiv@hushmail.com: "Re: RE: Dilemma: Replace Escalade with Adaptec 2400A or Promise Super trak66?"
Previous message: Stefan Smietanowski: "Re: Re: [PATCH] Re: Lost Partition"
In reply to: Thomas Hood: "[PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Next in thread: Thomas Hood: "Re: [PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Reply: Thomas Hood: "Re: [PATCH] 2.4.10-ac11 parport_pc.c bugfix"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Mon Oct 15 2001 - 21:00:39 EST