Re: [PATCH] sal cleanup

From: David Mosberger <davidm_at_napali.hpl.hp.com>
Date: 2004-02-28 05:52:46
>>>>> On Thu, 26 Feb 2004 22:04:56 +0000, Matthew Wilcox <willy@debian.org> 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.
Why?

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

	--david
-
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 Fri Feb 27 14:04:33 2004

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