Re: [PATCH] sal cleanup

From: Matthew Wilcox <willy_at_debian.org>
Date: 2004-03-03 04:57:01
On Tue, Mar 02, 2004 at 09:42:33AM -0800, David Mosberger wrote:
> >>>>> On Tue, 2 Mar 2004 15:56:01 +0000, Matthew Wilcox <willy@debian.org> said:
> 
>   Matthew> I don't have an objection to doing that, but how about
>   Matthew> making it Just Work for this particular case?  Something
>   Matthew> like ..
> 
> Well, that would be more convenient for me, but I don't know how many
> other "strange" SAL version numbers are out there in the wild.  It's
> can be handy to have an override option when dealing with buggy
> firmware (fortunately, quite rare with production systems, but it's
> all to common with prototypes).

Yeah.  That's why I moved it into a separate function so we can add
other bad version numbers.  If we were really keen, we could even add a
check for the oem and product IDs matching, or a data table of known-bad
BIOSes but let's keep it simple until we have to make it complex.

> Just a caveat:
> 
>   Matthew> SAL_VERSION_CODE(49, 29)
> 
> shouldn't these be in hex?

Nope, that's the beauty of it:

#define SAL_VERSION_CODE(major, minor) ((BIN2BCD(major) << 8) | BIN2BCD(minor))

BIN2BCD is defined in linux/bcd.h as ((((val)/10)<<4) + (val)%10)

So you get to specify things in decimal.

-- 
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Tue Mar 2 12:58:25 2004

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:24 EST