Re: [patch] NCR5380.c fix

From: James Bottomley (James.Bottomley@steeleye.com)
Date: Thu May 15 2003 - 17:52:06 EST

Next message: Fred Garvin: "Re: [PATCH] Fix vesafb with large memory, this time properly"
Previous message: Adam Mercer: "[PATCH] Fix vesafb with large memory, this time properly"
In reply to: Andries.Brouwer@cwi.nl: "[patch] NCR5380.c fix"
Next in thread: Andries.Brouwer@cwi.nl: "Re: [patch] NCR5380.c fix"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Thu, 2003-05-15 at 17:45, Andries.Brouwer@cwi.nl wrote:
> - else if (cmd->SCp.Status != GOOD)
> + else if (status_byte(cmd->SCp.Status) != GOOD)

Well...if we're doing it this way, any reason not to use the newly
minted SAM_STAT_GOOD and SAM_STAT_CHECK_CONDITION?

James

-
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: Fred Garvin: "Re: [PATCH] Fix vesafb with large memory, this time properly"
Previous message: Adam Mercer: "[PATCH] Fix vesafb with large memory, this time properly"
In reply to: Andries.Brouwer@cwi.nl: "[patch] NCR5380.c fix"
Next in thread: Andries.Brouwer@cwi.nl: "Re: [patch] NCR5380.c fix"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Thu May 15 2003 - 22:00:58 EST