[parisc-linux-cvs] DIFF arch/parisc/kernel/inventory.c

Grant Grundler grundler@cup.hp.com
Thu, 22 Feb 2001 16:32:12 -0800


Corrected comments which were wrong and removed un-needed PDC PAT call.
Thanks to jsm for pointing those out.

grant

Index: arch/parisc/kernel/inventory.c
===================================================================
RCS file: /home/cvs/parisc/linux/arch/parisc/kernel/inventory.c,v
retrieving revision 1.25
diff -u -p -r1.25 inventory.c
--- inventory.c	2001/02/03 19:36:57	1.25
+++ inventory.c	2001/02/23 00:25:36
@@ -130,32 +130,21 @@ static int do_pat_inventory(void)
 	ulong cell_num;
 	ulong pcell_loc;
 
+	/*
+	** Note:  Prelude (and it's successors: Lclass, A400/500) only
+	**        implement PDC_PAT_CELL sub-options 0 and 2.
+	*/
 	pdc_pat = (pdc_pat_cell_get_number(&pdc_result) == PDC_OK);
 	if (!pdc_pat) {
 		return 0;
 	}
 
-	cell_num = pdc_result[0];	/* Cell number call was made */
+	cell_num = pdc_result[0];	/* Cell number call was made from */
+	pcell_loc = pdc_result[1];	/* Physical location of this cell */
 
-	/* As of PDC PAT ARS 2.5, ret[1] is NOT architected! */
-	pcell_loc = pdc_result[1];	/* Physical location of the cell */
-
 #ifdef DEBUG_PAT
 	printk("CELL_GET_NUMBER: 0x%lx 0x%lx\n", cell_num, pcell_loc);
 #endif
-
-	status = pdc_pat_cell_num_to_loc(&pdc_result, cell_num);
-	if (status == PDC_BAD_OPTION) {
-		/* Prelude (and it's successors: Lclass, A400/500) only
-		** implement PDC_PAT_CELL sub-options 0 and 2.
-		** "Home cook'n is best anyhow!"
-		*/
-	} else if (PDC_OK == status) {
-		/* so far only Halfdome supports this */
-		pcell_loc = pdc_result[0];
-	} else {
-		panic("WTF? CELL_GET_NUMBER give me invalid cell number?");
-	}
 
 	while (PDC_RET_OK == pat_query_module(pcell_loc, mod_index)) {
 		mod_index++;