浏览代码

powerpc: Refactor board check for PCI quirks on FSL boards with uli1575

Refactor the check to determine if the quirk is applicable to the boards
into one inline function so we only have to change one place to add more
boards that the quirks might be applicable to.

Also removed a warning related to unused temp variable.

Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
Kumar Gala 16 年之前
父节点
当前提交
9bf8b2741e
共有 1 个文件被更改,包括 12 次插入11 次删除
  1. 12 11
      arch/powerpc/platforms/fsl_uli1575.c

+ 12 - 11
arch/powerpc/platforms/fsl_uli1575.c

@@ -51,13 +51,19 @@ u8 uli_pirq_to_irq[8] = {
 	ULI_8259_NONE,		/* PIRQH */
 	ULI_8259_NONE,		/* PIRQH */
 };
 };
 
 
+static inline bool is_quirk_valid(void)
+{
+	return (machine_is(mpc86xx_hpcn) ||
+		machine_is(mpc8544_ds) ||
+		machine_is(mpc8572_ds));
+}
+
 /* Bridge */
 /* Bridge */
 static void __devinit early_uli5249(struct pci_dev *dev)
 static void __devinit early_uli5249(struct pci_dev *dev)
 {
 {
 	unsigned char temp;
 	unsigned char temp;
 
 
-	if (!machine_is(mpc86xx_hpcn) && !machine_is(mpc8544_ds) &&
-			!machine_is(mpc8572_ds))
+	if (!is_quirk_valid())
 		return;
 		return;
 
 
 	pci_write_config_word(dev, PCI_COMMAND, PCI_COMMAND_IO |
 	pci_write_config_word(dev, PCI_COMMAND, PCI_COMMAND_IO |
@@ -80,8 +86,7 @@ static void __devinit quirk_uli1575(struct pci_dev *dev)
 {
 {
 	int i;
 	int i;
 
 
-	if (!machine_is(mpc86xx_hpcn) && !machine_is(mpc8544_ds) &&
-			!machine_is(mpc8572_ds))
+	if (!is_quirk_valid())
 		return;
 		return;
 
 
 	/*
 	/*
@@ -149,8 +154,7 @@ static void __devinit quirk_final_uli1575(struct pci_dev *dev)
 	 * IRQ 14: Edge
 	 * IRQ 14: Edge
 	 * IRQ 15: Edge
 	 * IRQ 15: Edge
 	 */
 	 */
-	if (!machine_is(mpc86xx_hpcn) && !machine_is(mpc8544_ds) &&
-			!machine_is(mpc8572_ds))
+	if (!is_quirk_valid())
 		return;
 		return;
 
 
 	outb(0xfa, 0x4d0);
 	outb(0xfa, 0x4d0);
@@ -176,8 +180,7 @@ static void __devinit quirk_uli5288(struct pci_dev *dev)
 	unsigned char c;
 	unsigned char c;
 	unsigned int d;
 	unsigned int d;
 
 
-	if (!machine_is(mpc86xx_hpcn) && !machine_is(mpc8544_ds) &&
-			!machine_is(mpc8572_ds))
+	if (!is_quirk_valid())
 		return;
 		return;
 
 
 	/* read/write lock */
 	/* read/write lock */
@@ -201,8 +204,7 @@ static void __devinit quirk_uli5229(struct pci_dev *dev)
 {
 {
 	unsigned short temp;
 	unsigned short temp;
 
 
-	if (!machine_is(mpc86xx_hpcn) && !machine_is(mpc8544_ds) &&
-			!machine_is(mpc8572_ds))
+	if (!is_quirk_valid())
 		return;
 		return;
 
 
 	pci_write_config_word(dev, PCI_COMMAND, PCI_COMMAND_INTX_DISABLE |
 	pci_write_config_word(dev, PCI_COMMAND, PCI_COMMAND_INTX_DISABLE |
@@ -270,7 +272,6 @@ static void __devinit hpcd_quirk_uli1575(struct pci_dev *dev)
 static void __devinit hpcd_quirk_uli5288(struct pci_dev *dev)
 static void __devinit hpcd_quirk_uli5288(struct pci_dev *dev)
 {
 {
 	unsigned char c;
 	unsigned char c;
-	unsigned short temp;
 
 
 	if (!machine_is(mpc86xx_hpcd))
 	if (!machine_is(mpc86xx_hpcd))
 		return;
 		return;