Re: [PATCH] sal cleanup

From: David Mosberger <>
Date: 2004-02-28 05:52:46
>>>>> On Thu, 26 Feb 2004 22:04:56 +0000, Matthew Wilcox <> said:

  Matthew> On Thu, Feb 26, 2004 at 01:55:34PM -0800, David Mosberger
  Matthew> wrote:
  >> How about printing "none" or something like that instead?

  Matthew> Sure.

  Matthew> This patch reorganises sal.c and adds two new functions:

  Matthew>  - Split the body of the switch statement in
  Matthew> ia64_sal_init() into separate functions.  - Delete min/max
  Matthew> since they were only assigned to, never used.  - Print the
  Matthew> version of the SAL implementation as well as the spec
  Matthew> version it supports.  - Stop printing the pal_proc and
  Matthew> sal_proc entry addresses.  - Print "none" if there are no
  Matthew> platform features.  - Remember the sal revision and version
  Matthew> and add two functions to compare them.

The patch also changes the arguments for ia64_sal_pci_config_read()
and ia64_sal_pci_config_write(), without changing any callers.

Also, how about putting a colon after "SAL Platform features"?

To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to
More majordomo info at
Received on Fri Feb 27 14:04:33 2004

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