x86: Move common PCH code into a common place
authorSimon Glass <sjg@chromium.org>
Wed, 16 Mar 2016 13:44:36 +0000 (07:44 -0600)
committerBin Meng <bmeng.cn@gmail.com>
Thu, 17 Mar 2016 02:27:27 +0000 (10:27 +0800)
The SATA indexed register write functions are common to several Intel PCHs.
Move this into a common location.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
arch/x86/cpu/intel_common/Makefile
arch/x86/cpu/intel_common/pch.c [new file with mode: 0644]
arch/x86/cpu/ivybridge/cpu.c
arch/x86/cpu/ivybridge/sata.c
arch/x86/include/asm/arch-ivybridge/pch.h
arch/x86/include/asm/pch_common.h [new file with mode: 0644]

index 1918ca30a845d1501ec7cae5a95246b7ffe17fa6..74b097a6c145cee3ba0b33e2b3dc049067eb2a1c 100644 (file)
@@ -11,4 +11,5 @@ obj-$(CONFIG_HAVE_MRC) += me_status.o
 ifndef CONFIG_TARGET_EFI
 obj-y += microcode.o
 endif
+obj-y += pch.o
 obj-$(CONFIG_HAVE_MRC) += report_platform.o
diff --git a/arch/x86/cpu/intel_common/pch.c b/arch/x86/cpu/intel_common/pch.c
new file mode 100644 (file)
index 0000000..1f05b44
--- /dev/null
@@ -0,0 +1,25 @@
+/*
+ * Copyright (c) 2016 Google, Inc
+ *
+ * SPDX-License-Identifier:    GPL-2.0
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <asm/pch_common.h>
+
+u32 pch_common_sir_read(struct udevice *dev, int idx)
+{
+       u32 data;
+
+       dm_pci_write_config32(dev, SATA_SIRI, idx);
+       dm_pci_read_config32(dev, SATA_SIRD, &data);
+
+       return data;
+}
+
+void pch_common_sir_write(struct udevice *dev, int idx, u32 value)
+{
+       dm_pci_write_config32(dev, SATA_SIRI, idx);
+       dm_pci_write_config32(dev, SATA_SIRD, value);
+}
index bb720ba938aabf119e5a80fdd9692b8ccc338085..0f9390517ff92b0a6afb3983fffcaf54af52e60f 100644 (file)
@@ -21,6 +21,7 @@
 #include <asm/intel_regs.h>
 #include <asm/io.h>
 #include <asm/lapic.h>
+#include <asm/lpc_common.h>
 #include <asm/microcode.h>
 #include <asm/msr.h>
 #include <asm/mtrr.h>
index da6455b3bd86674a03665e05f2f436838e26bb24..c3d1057c291d5b9aaf022b275b520e00c673b7cd 100644 (file)
@@ -9,28 +9,13 @@
 #include <dm.h>
 #include <fdtdec.h>
 #include <asm/io.h>
+#include <asm/pch_common.h>
 #include <asm/pci.h>
 #include <asm/arch/pch.h>
 #include <asm/arch/bd82x6x.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
-static inline u32 sir_read(struct udevice *dev, int idx)
-{
-       u32 data;
-
-       dm_pci_write_config32(dev, SATA_SIRI, idx);
-       dm_pci_read_config32(dev, SATA_SIRD, &data);
-
-       return data;
-}
-
-static inline void sir_write(struct udevice *dev, int idx, u32 value)
-{
-       dm_pci_write_config32(dev, SATA_SIRI, idx);
-       dm_pci_write_config32(dev, SATA_SIRD, value);
-}
-
 static void common_sata_init(struct udevice *dev, unsigned int port_map)
 {
        u32 reg32;
@@ -177,27 +162,27 @@ static void bd82x6x_sata_init(struct udevice *dev, struct udevice *pch)
                pch_iobp_update(pch, SATA_IOBP_SP1G3IR, 0, port_tx);
 
        /* Additional Programming Requirements */
-       sir_write(dev, 0x04, 0x00001600);
-       sir_write(dev, 0x28, 0xa0000033);
-       reg32 = sir_read(dev, 0x54);
+       pch_common_sir_write(dev, 0x04, 0x00001600);
+       pch_common_sir_write(dev, 0x28, 0xa0000033);
+       reg32 = pch_common_sir_read(dev, 0x54);
        reg32 &= 0xff000000;
        reg32 |= 0x5555aa;
-       sir_write(dev, 0x54, reg32);
-       sir_write(dev, 0x64, 0xcccc8484);
-       reg32 = sir_read(dev, 0x68);
+       pch_common_sir_write(dev, 0x54, reg32);
+       pch_common_sir_write(dev, 0x64, 0xcccc8484);
+       reg32 = pch_common_sir_read(dev, 0x68);
        reg32 &= 0xffff0000;
        reg32 |= 0xcccc;
-       sir_write(dev, 0x68, reg32);
-       reg32 = sir_read(dev, 0x78);
+       pch_common_sir_write(dev, 0x68, reg32);
+       reg32 = pch_common_sir_read(dev, 0x78);
        reg32 &= 0x0000ffff;
        reg32 |= 0x88880000;
-       sir_write(dev, 0x78, reg32);
-       sir_write(dev, 0x84, 0x001c7000);
-       sir_write(dev, 0x88, 0x88338822);
-       sir_write(dev, 0xa0, 0x001c7000);
-       sir_write(dev, 0xc4, 0x0c0c0c0c);
-       sir_write(dev, 0xc8, 0x0c0c0c0c);
-       sir_write(dev, 0xd4, 0x10000000);
+       pch_common_sir_write(dev, 0x78, reg32);
+       pch_common_sir_write(dev, 0x84, 0x001c7000);
+       pch_common_sir_write(dev, 0x88, 0x88338822);
+       pch_common_sir_write(dev, 0xa0, 0x001c7000);
+       pch_common_sir_write(dev, 0xc4, 0x0c0c0c0c);
+       pch_common_sir_write(dev, 0xc8, 0x0c0c0c0c);
+       pch_common_sir_write(dev, 0xd4, 0x10000000);
 
        pch_iobp_update(pch, 0xea004001, 0x3fffffff, 0xc0000000);
        pch_iobp_update(pch, 0xea00408a, 0xfffffcff, 0x00000100);
index e72ff2afc93f708ca12a5a09856927abb437a257..4725250d9184fa4cafc3f823bd125727c0cbe8c4 100644 (file)
@@ -69,8 +69,6 @@
 #define RTC_POWER_FAILED       (1 << 1)
 #define SLEEP_AFTER_POWER_FAIL (1 << 0)
 
-#define PMBASE                 0x40
-#define ACPI_CNTL              0x44
 #define BIOS_CNTL              0xDC
 #define GPIO_BASE              0x48 /* LPC GPIO Base Address Register */
 #define GPIO_CNTL              0x4C /* LPC GPIO Control Register */
 #define GPIO_CNTL              0x4C /* LPC GPIO Control Register */
 #define GPIO_ROUT              0xb8
 
-#define LPC_IO_DEC             0x80 /* IO Decode Ranges Register */
-#define  COMB_DEC_RANGE                (1 << 4)  /* 0x2f8-0x2ff (COM2) */
-#define  COMA_DEC_RANGE                (0 << 0)  /* 0x3f8-0x3ff (COM1) */
-#define LPC_EN                 0x82 /* LPC IF Enables Register */
-#define  CNF2_LPC_EN           (1 << 13) /* 0x4e/0x4f */
-#define  CNF1_LPC_EN           (1 << 12) /* 0x2e/0x2f */
-#define  MC_LPC_EN             (1 << 11) /* 0x62/0x66 */
-#define  KBC_LPC_EN            (1 << 10) /* 0x60/0x64 */
-#define  GAMEH_LPC_EN          (1 << 9)  /* 0x208/0x20f */
-#define  GAMEL_LPC_EN          (1 << 8)  /* 0x200/0x207 */
-#define  FDD_LPC_EN            (1 << 3)  /* LPC_IO_DEC[12] */
-#define  LPT_LPC_EN            (1 << 2)  /* LPC_IO_DEC[9:8] */
-#define  COMB_LPC_EN           (1 << 1)  /* LPC_IO_DEC[6:4] */
-#define  COMA_LPC_EN           (1 << 0)  /* LPC_IO_DEC[3:2] */
-#define LPC_GEN1_DEC           0x84 /* LPC IF Generic Decode Range 1 */
-#define LPC_GEN2_DEC           0x88 /* LPC IF Generic Decode Range 2 */
-#define LPC_GEN3_DEC           0x8c /* LPC IF Generic Decode Range 3 */
-#define LPC_GEN4_DEC           0x90 /* LPC IF Generic Decode Range 4 */
-#define LPC_GENX_DEC(x)                (0x84 + 4 * (x))
-#define  GEN_DEC_RANGE_256B    0xfc0000  /* 256 Bytes */
-#define  GEN_DEC_RANGE_128B    0x7c0000  /* 128 Bytes */
-#define  GEN_DEC_RANGE_64B     0x3c0000  /* 64 Bytes */
-#define  GEN_DEC_RANGE_32B     0x1c0000  /* 32 Bytes */
-#define  GEN_DEC_RANGE_16B     0x0c0000  /* 16 Bytes */
-#define  GEN_DEC_RANGE_8B      0x040000  /* 8 Bytes */
-#define  GEN_DEC_RANGE_4B      0x000000  /* 4 Bytes */
-#define  GEN_DEC_RANGE_EN      (1 << 0)  /* Range Enable */
-
 /* PCI Configuration Space (D31:F1): IDE */
 #define PCH_IDE_DEV            PCI_BDF(0, 0x1f, 1)
 #define PCH_SATA_DEV           PCI_BDF(0, 0x1f, 2)
 #define PCH_SATA2_DEV          PCI_BDF(0, 0x1f, 5)
 
-#define INTR_LN                        0x3c
-#define IDE_TIM_PRI            0x40    /* IDE timings, primary */
-#define   IDE_DECODE_ENABLE    (1 << 15)
-#define   IDE_SITRE            (1 << 14)
-#define   IDE_ISP_5_CLOCKS     (0 << 12)
-#define   IDE_ISP_4_CLOCKS     (1 << 12)
-#define   IDE_ISP_3_CLOCKS     (2 << 12)
-#define   IDE_RCT_4_CLOCKS     (0 <<  8)
-#define   IDE_RCT_3_CLOCKS     (1 <<  8)
-#define   IDE_RCT_2_CLOCKS     (2 <<  8)
-#define   IDE_RCT_1_CLOCKS     (3 <<  8)
-#define   IDE_DTE1             (1 <<  7)
-#define   IDE_PPE1             (1 <<  6)
-#define   IDE_IE1              (1 <<  5)
-#define   IDE_TIME1            (1 <<  4)
-#define   IDE_DTE0             (1 <<  3)
-#define   IDE_PPE0             (1 <<  2)
-#define   IDE_IE0              (1 <<  1)
-#define   IDE_TIME0            (1 <<  0)
-#define IDE_TIM_SEC            0x42    /* IDE timings, secondary */
-
 #define IDE_SDMA_CNT           0x48    /* Synchronous DMA control */
 #define   IDE_SSDE1            (1 <<  3)
 #define   IDE_SSDE0            (1 <<  2)
                (((d) << DIR_IDR) | ((c) << DIR_ICR) | \
                        ((b) << DIR_IBR) | ((a) << DIR_IAR))
 
-#define RC             0x3400  /* 32bit */
 #define HPTC           0x3404  /* 32bit */
-#define GCS            0x3410  /* 32bit */
 #define BUC            0x3414  /* 32bit */
 #define PCH_DISABLE_GBE                (1 << 5)
 #define FD             0x3418  /* 32bit */
diff --git a/arch/x86/include/asm/pch_common.h b/arch/x86/include/asm/pch_common.h
new file mode 100644 (file)
index 0000000..924ccc4
--- /dev/null
@@ -0,0 +1,56 @@
+/*
+ * Copyright (c) 2016 Google, Inc
+ *
+ * SPDX-License-Identifier:    GPL-2.0
+ */
+
+#ifndef __asm_pch_common_h
+#define __asm_pch_common_h
+
+/* Common Intel SATA registers */
+#define SATA_SIRI              0xa0 /* SATA Indexed Register Index */
+#define SATA_SIRD              0xa4 /* SATA Indexed Register Data */
+#define SATA_SP                        0xd0 /* Scratchpad */
+
+#define INTR_LN                        0x3c
+#define IDE_TIM_PRI            0x40    /* IDE timings, primary */
+#define   IDE_DECODE_ENABLE    (1 << 15)
+#define   IDE_SITRE            (1 << 14)
+#define   IDE_ISP_5_CLOCKS     (0 << 12)
+#define   IDE_ISP_4_CLOCKS     (1 << 12)
+#define   IDE_ISP_3_CLOCKS     (2 << 12)
+#define   IDE_RCT_4_CLOCKS     (0 <<  8)
+#define   IDE_RCT_3_CLOCKS     (1 <<  8)
+#define   IDE_RCT_2_CLOCKS     (2 <<  8)
+#define   IDE_RCT_1_CLOCKS     (3 <<  8)
+#define   IDE_DTE1             (1 <<  7)
+#define   IDE_PPE1             (1 <<  6)
+#define   IDE_IE1              (1 <<  5)
+#define   IDE_TIME1            (1 <<  4)
+#define   IDE_DTE0             (1 <<  3)
+#define   IDE_PPE0             (1 <<  2)
+#define   IDE_IE0              (1 <<  1)
+#define   IDE_TIME0            (1 <<  0)
+#define IDE_TIM_SEC            0x42    /* IDE timings, secondary */
+
+#define SERIRQ_CNTL            0x64
+
+/**
+ * pch_common_sir_read() - Read from a SATA indexed register
+ *
+ * @dev:       SATA device
+ * @idx:       Register index to read
+ * @return value read from register
+ */
+u32 pch_common_sir_read(struct udevice *dev, int idx);
+
+/**
+ * pch_common_sir_write() - Write to a SATA indexed register
+ *
+ * @dev:       SATA device
+ * @idx:       Register index to write
+ * @value:     Value to write
+ */
+void pch_common_sir_write(struct udevice *dev, int idx, u32 value);
+
+#endif