From mboxrd@z Thu Jan 1 00:00:00 1970 From: Prarit Bhargava Date: Wed, 18 May 2005 13:42:06 +0000 Subject: Re: [PATCH 4/6]: hotplug/ia64: SN Hotplug Driver - new SN PROM version Message-Id: <428B462E.1030109@sgi.com> MIME-Version: 1 Content-Type: multipart/mixed; boundary="------------050107080804010905000200" List-Id: References: <42835E91.2080403@sgi.com> In-Reply-To: <42835E91.2080403@sgi.com> To: linux-ia64@vger.kernel.org This is a multi-part message in MIME format. --------------050107080804010905000200 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Greg KH wrote: > On Thu, May 12, 2005 at 08:45:58PM -0400, Prarit Bhargava wrote: > >> >>- printk("SGI SAL version %x.%02x\n", major, minor); >>+ printk("SGI SAL version %x.%02x\n", version >> 8, version & 0x00FF); > > > No KERN_ level for this printk. The kernel janitors will come after you > with pitchforks... > > > --------------050107080804010905000200 Content-Type: text/plain; name="sn_version.patch-4" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="sn_version.patch-4" This patch is a rewrite of the code to check the PROM version. The current code has some deficiences in the way PROM comparisons were made. The minimum value of PROM that will boot has also been changed to 4.04. Signed-off-by: Prarit Bhargava --- commit e5a2c8250d8e984843afe28444144b1835620d1d tree f8014bc6a8330d22e2dbb0cac34d87e8a5352278 parent e294c434e2b07cd37bd92c4400ddc038bd1fb095 author Prarit Bhargava 1810557251 -0400 committer Prarit Bhargava 1810557251 -0400 Index: arch/ia64/sn/kernel/setup.c =================================================================== --- 0074994b061b33a65ec96ee3a8058c4025162add/arch/ia64/sn/kernel/setup.c (mode:100644 sha1:4fb44984afe61198a5f3c4c62795a6ced91eb265) +++ f8014bc6a8330d22e2dbb0cac34d87e8a5352278/arch/ia64/sn/kernel/setup.c (mode:100644 sha1:e75dc9ec351e2c8e37d6d8392d954c47e41287fa) @@ -268,7 +268,7 @@ { long status, ticks_per_sec, drift; int pxm; - int major = sn_sal_rev_major(), minor = sn_sal_rev_minor(); + u32 version = sn_sal_rev(); extern void sn_cpu_init(void); /* @@ -301,22 +301,22 @@ * support here so we don't have to listen to failed keyboard probe * messages. */ - if ((major < 2 || (major == 2 && minor <= 9)) && - acpi_kbd_controller_present) { + if (version <= 0x0209 && acpi_kbd_controller_present) { printk(KERN_INFO "Disabling legacy keyboard support as prom " "is too old and doesn't provide FADT\n"); acpi_kbd_controller_present = 0; } - printk("SGI SAL version %x.%02x\n", major, minor); + printk(KERN_NOTICE "SGI SAL version %x.%02x\n", version >> 8, + version & 0x00FF); /* * Confirm the SAL we're running on is recent enough... */ - if ((major < SN_SAL_MIN_MAJOR) || (major == SN_SAL_MIN_MAJOR && - minor < SN_SAL_MIN_MINOR)) { + if (version < SN_SAL_MIN_VERSION) { printk(KERN_ERR "This kernel needs SGI SAL version >= " - "%x.%02x\n", SN_SAL_MIN_MAJOR, SN_SAL_MIN_MINOR); + "%x.%02x\n", SN_SAL_MIN_VERSION >> 8, + SN_SAL_MIN_VERSION & 0x00FF); panic("PROM version too old\n"); } Index: arch/ia64/sn/pci/tioca_provider.c =================================================================== --- 0074994b061b33a65ec96ee3a8058c4025162add/arch/ia64/sn/pci/tioca_provider.c (mode:100644 sha1:8dae9eb45456d97525a6745a766f903eee2323d6) +++ f8014bc6a8330d22e2dbb0cac34d87e8a5352278/arch/ia64/sn/pci/tioca_provider.c (mode:100644 sha1:0452d7ba0e2a14a98df893e6bab5ebd3c7203ad6) @@ -589,8 +589,7 @@ /* sanity check prom rev */ - if (sn_sal_rev_major() < 4 || - (sn_sal_rev_major() == 4 && sn_sal_rev_minor() < 6)) { + if (sn_sal_rev() < 0x0406) { printk (KERN_ERR "%s: SGI prom rev 4.06 or greater required " "for tioca support\n", __FUNCTION__); Index: include/asm-ia64/sn/sn_sal.h =================================================================== --- 0074994b061b33a65ec96ee3a8058c4025162add/include/asm-ia64/sn/sn_sal.h (mode:100644 sha1:56d74ca76b5d0b9f4fc809c4f05c6152655b1311) +++ f8014bc6a8330d22e2dbb0cac34d87e8a5352278/include/asm-ia64/sn/sn_sal.h (mode:100644 sha1:6ad6a885f9aba3fa8be23f369224a037d4080407) @@ -125,43 +125,28 @@ #define SALRET_INVALID_ARG (-2) #define SALRET_ERROR (-3) - /** - * sn_sal_rev_major - get the major SGI SAL revision number - * - * The SGI PROM stores its version in sal_[ab]_rev_(major|minor). - * This routine simply extracts the major value from the - * @ia64_sal_systab structure constructed by ia64_sal_init(). - */ -static inline int -sn_sal_rev_major(void) + * sn_sal_revision - get the SGI SAL revision number + * + * The SGI PROM stores its version in the sal_[ab]_rev_(major|minor). + * This routine simply extracts the major and minor values and + * presents them in a u32 format. + * + * For example, version 4.05 would be represented at 0x0405. + */ +static inline u32 +sn_sal_rev(void) { struct ia64_sal_systab *systab = efi.sal_systab; - return (int)systab->sal_b_rev_major; -} - -/** - * sn_sal_rev_minor - get the minor SGI SAL revision number - * - * The SGI PROM stores its version in sal_[ab]_rev_(major|minor). - * This routine simply extracts the minor value from the - * @ia64_sal_systab structure constructed by ia64_sal_init(). - */ -static inline int -sn_sal_rev_minor(void) -{ - struct ia64_sal_systab *systab = efi.sal_systab; - - return (int)systab->sal_b_rev_minor; + return (u32)(systab->sal_b_rev_major << 8 | systab->sal_b_rev_minor); } /* * Specify the minimum PROM revsion required for this kernel. * Note that they're stored in hex format... */ -#define SN_SAL_MIN_MAJOR 0x4 /* SN2 kernels need at least PROM 4.0 */ -#define SN_SAL_MIN_MINOR 0x0 +#define SN_SAL_MIN_VERSION 0x0404 /* * Returns the master console nasid, if the call fails, return an illegal --------------050107080804010905000200--