From 015c64335d13b9e213fa93127d621c574b17018e Mon Sep 17 00:00:00 2001 From: Aamir Bohra Date: Thu, 6 Apr 2017 11:15:18 +0530 Subject: soc/intel/skylake: Use LPSS common library Use lpss common library to program reset and clock register for lpss modules. Change-Id: I198feba7c6f6d033ab77ed25a5bd9ea99411a1e4 Signed-off-by: Aamir Bohra Reviewed-on: https://review.coreboot.org/19153 Tested-by: build bot (Jenkins) Reviewed-by: Subrata Banik Reviewed-by: Aaron Durbin --- src/soc/intel/skylake/Kconfig | 1 + src/soc/intel/skylake/bootblock/i2c.c | 10 ++-------- src/soc/intel/skylake/bootblock/uart.c | 19 ++++++++----------- src/soc/intel/skylake/include/soc/serialio.h | 11 ----------- 4 files changed, 11 insertions(+), 30 deletions(-) diff --git a/src/soc/intel/skylake/Kconfig b/src/soc/intel/skylake/Kconfig index e22f1e37a4..e597d6238a 100644 --- a/src/soc/intel/skylake/Kconfig +++ b/src/soc/intel/skylake/Kconfig @@ -51,6 +51,7 @@ config CPU_SPECIFIC_OPTIONS select SOC_INTEL_COMMON_ACPI_WAKE_SOURCE select SOC_INTEL_COMMON_BLOCK select SOC_INTEL_COMMON_BLOCK_GSPI + select SOC_INTEL_COMMON_BLOCK_LPSS select SOC_INTEL_COMMON_BLOCK_PCR select SOC_INTEL_COMMON_BLOCK_RTC select SOC_INTEL_COMMON_BLOCK_SA diff --git a/src/soc/intel/skylake/bootblock/i2c.c b/src/soc/intel/skylake/bootblock/i2c.c index 11d145c7bf..59cb5c8bc2 100644 --- a/src/soc/intel/skylake/bootblock/i2c.c +++ b/src/soc/intel/skylake/bootblock/i2c.c @@ -18,11 +18,11 @@ #include #include #include +#include #include #include #include #include -#include #include "chip.h" uintptr_t lpss_i2c_base_address(unsigned int bus) @@ -49,8 +49,6 @@ static void i2c_early_init_bus(unsigned int bus) pci_devfn_t dev; int devfn; uintptr_t base; - uint32_t value; - void *reg; /* Find the PCI device for this bus controller */ devfn = i2c_bus_to_devfn(bus); @@ -77,11 +75,7 @@ static void i2c_early_init_bus(unsigned int bus) PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER); /* Take device out of reset */ - reg = (void *)(base + SIO_REG_PPR_RESETS); - value = read32(reg); - value |= SIO_REG_PPR_RESETS_FUNC | SIO_REG_PPR_RESETS_APB | - SIO_REG_PPR_RESETS_IDMA; - write32(reg, value); + lpss_reset_release(base); /* Initialize the controller */ lpss_i2c_init(bus, &config->i2c[bus]); diff --git a/src/soc/intel/skylake/bootblock/uart.c b/src/soc/intel/skylake/bootblock/uart.c index d51d179577..cf06359eae 100644 --- a/src/soc/intel/skylake/bootblock/uart.c +++ b/src/soc/intel/skylake/bootblock/uart.c @@ -18,12 +18,12 @@ #include #include #include +#include #include #include #include #include #include -#include #include /* Serial IO UART controller legacy mode */ @@ -32,6 +32,10 @@ #define PCR_SIO_PCH_LEGACY_UART1 (1 << 1) #define PCR_SIO_PCH_LEGACY_UART2 (1 << 2) +/* Clock divider parameters for 115200 baud rate */ +#define CLOCK_DIV_M_VAL 0x30 +#define CLOCK_DIV_N_VAL 0xc35 + /* UART2 pad configuration. Support RXD and TXD for now. */ static const struct pad_config uart2_pads[] = { /* UART2_RXD */ PAD_CFG_NF(GPP_C20, NONE, DEEP, NF1), @@ -42,7 +46,7 @@ void pch_uart_init(void) { device_t dev = PCH_DEV_UART2; u32 tmp; - u8 *base = (void *)uart_platform_base(CONFIG_UART_FOR_CONSOLE); + uintptr_t base = uart_platform_base(CONFIG_UART_FOR_CONSOLE); /* Set configured UART2 base address */ pci_write_config32(dev, PCI_BASE_ADDRESS_0, (u32)base); @@ -53,21 +57,14 @@ void pch_uart_init(void) pci_write_config32(dev, PCI_COMMAND, tmp); /* Take UART2 out of reset */ - tmp = read32(base + SIO_REG_PPR_RESETS); - tmp |= SIO_REG_PPR_RESETS_FUNC | SIO_REG_PPR_RESETS_APB | - SIO_REG_PPR_RESETS_IDMA; - write32(base + SIO_REG_PPR_RESETS, tmp); + lpss_reset_release(base); /* * Set M and N divisor inputs and enable clock. * Main reference frequency to UART is: * 120MHz * M / N = 120MHz * 48 / 3125 = 1843200Hz */ - tmp = read32(base + SIO_REG_PPR_CLOCK); - tmp |= SIO_REG_PPR_CLOCK_EN | SIO_REG_PPR_CLOCK_UPDATE | - (SIO_REG_PPR_CLOCK_N_DIV << 16) | - (SIO_REG_PPR_CLOCK_M_DIV << 1); - write32(base + SIO_REG_PPR_CLOCK, tmp); + lpss_clk_update(base, CLOCK_DIV_M_VAL, CLOCK_DIV_N_VAL); /* Put UART2 in byte access mode for 16550 compatibility */ if (!IS_ENABLED(CONFIG_DRIVERS_UART_8250MEM_32)) diff --git a/src/soc/intel/skylake/include/soc/serialio.h b/src/soc/intel/skylake/include/soc/serialio.h index d66bf8083e..a07f29c9b0 100644 --- a/src/soc/intel/skylake/include/soc/serialio.h +++ b/src/soc/intel/skylake/include/soc/serialio.h @@ -17,17 +17,6 @@ #ifndef _SERIALIO_H_ #define _SERIALIO_H_ -#define SIO_REG_PPR_CLOCK 0x200 -#define SIO_REG_PPR_CLOCK_EN (1 << 0) -#define SIO_REG_PPR_CLOCK_UPDATE (1 << 31) -#define SIO_REG_PPR_CLOCK_N_DIV 0xc35 -#define SIO_REG_PPR_CLOCK_M_DIV 0x30 - -#define SIO_REG_PPR_RESETS 0x204 -#define SIO_REG_PPR_RESETS_FUNC (1 << 0) -#define SIO_REG_PPR_RESETS_APB (1 << 1) -#define SIO_REG_PPR_RESETS_IDMA (1 << 2) - typedef enum { PchSerialIoDisabled, PchSerialIoAcpi, -- cgit v1.2.3