[U-Boot] [PATCH 01/11 V12] EXYNOS5: Create a common board file
Minkyu Kang
mk7.kang at samsung.com
Mon Dec 23 09:41:19 CET 2013
Dear Rajeshwari S Shinde,
On 20/12/13 18:34, Rajeshwari S Shinde wrote:
> Create a common board.c file for all functions which are common across
> all EXYNOS5 platforms.
>
> exynos_init function is provided for platform specific code.
>
> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s at samsung.com>
> Acked-by: Simon Glass <sjg at chromium.org>
> ---
> Changes in V2:
> - None
> Changes in V3:
> - None
> Changes in V4:
> - Added check for the compilation of MAX77686 pmic.
> Changes in V5:
> - Moved board_eth_init and board_mmc_init in case of
> device tree support
> Changes in V6:
> - None.
> Changes in V7:
> - None
> Changes in V8:
> - None
> Changes in V9:
> - Reabsed on latest code.
> Changes in V10:
> - Corrected coding style nits.
> Changes in V11:
> -Rebased on latest u-boot-samsung tree
> Chnages in V12:
> - None
> arch/arm/include/asm/arch-exynos/board.h | 17 ++
> board/samsung/common/Makefile | 4 +
> board/samsung/common/board.c | 408 +++++++++++++++++++++++++++++++
> board/samsung/smdk5250/exynos5-dt.c | 352 +-------------------------
> board/samsung/smdk5250/smdk5250.c | 182 +-------------
> include/configs/exynos5250-dt.h | 2 +
> 6 files changed, 436 insertions(+), 529 deletions(-)
> create mode 100644 arch/arm/include/asm/arch-exynos/board.h
> create mode 100644 board/samsung/common/board.c
Sorry but patch failed.
u-boot-samsung (master)$ patch --dry-run -p1 < U-Boot-01-11-V12-EXYNOS5-Create-a-common-board-file.patch
patching file arch/arm/include/asm/arch-exynos/board.h
patching file board/samsung/common/Makefile
patching file board/samsung/common/board.c
patching file board/samsung/smdk5250/exynos5-dt.c
Hunk #2 FAILED at 37.
Hunk #3 succeeded at 307 (offset -2 lines).
Hunk #4 succeeded at 330 (offset -2 lines).
1 out of 4 hunks FAILED -- saving rejects to file board/samsung/smdk5250/exynos5-dt.c.rej
patching file board/samsung/smdk5250/smdk5250.c
patching file include/configs/exynos5250-dt.h
Piotr's patch seems affected your patch.
commit 2d8f1e27695a8a9a3ed863a510be58284b6b411c
Author: Piotr Wilczek <p.wilczek at samsung.com>
Date: Wed Nov 20 10:43:49 2013 +0100
driver:i2c:s3c24x0: adapt driver to new i2c
>
> diff --git a/arch/arm/include/asm/arch-exynos/board.h b/arch/arm/include/asm/arch-exynos/board.h
> new file mode 100644
> index 0000000..243fb12
> --- /dev/null
> +++ b/arch/arm/include/asm/arch-exynos/board.h
> @@ -0,0 +1,17 @@
> +/*
> + * (C) Copyright 2013 Samsung Electronics
> + * Rajeshwari Shinde <rajeshwari.s at samsung.com>
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
> +
> +#ifndef _EXYNOS_BOARD_H
> +#define _EXYNOS_BOARD_H
> +
> +/*
> + * Exynos baord specific changes for
> + * board_init
> + */
> +int exynos_init(void);
> +
> +#endif /* EXYNOS_BOARD_H */
> diff --git a/board/samsung/common/Makefile b/board/samsung/common/Makefile
> index 501d974..22bd6b1 100644
> --- a/board/samsung/common/Makefile
> +++ b/board/samsung/common/Makefile
> @@ -8,3 +8,7 @@
> obj-$(CONFIG_SOFT_I2C_MULTI_BUS) += multi_i2c.o
> obj-$(CONFIG_THOR_FUNCTION) += thor.o
> obj-$(CONFIG_CMD_USB_MASS_STORAGE) += ums.o
> +
> +ifndef CONFIG_SPL_BUILD
> +obj-$(CONFIG_BOARD_COMMON) += board.o
> +endif
> diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
> new file mode 100644
> index 0000000..95df346
> --- /dev/null
> +++ b/board/samsung/common/board.c
> @@ -0,0 +1,408 @@
> +/*
> + * (C) Copyright 2013 SAMSUNG Electronics
> + * Rajeshwari Shinde <rajeshwari.s at samsung.com>
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
> +
> +#include <common.h>
> +#include <cros_ec.h>
> +#include <errno.h>
> +#include <fdtdec.h>
> +#include <spi.h>
> +#include <tmu.h>
> +#include <netdev.h>
> +#include <asm/io.h>
> +#include <asm/arch/board.h>
> +#include <asm/arch/cpu.h>
> +#include <asm/arch/dwmmc.h>
> +#include <asm/arch/gpio.h>
> +#include <asm/arch/mmc.h>
> +#include <asm/arch/pinmux.h>
> +#include <asm/arch/power.h>
> +#include <power/pmic.h>
> +#include <asm/arch/sromc.h>
> +#include <power/max77686_pmic.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +struct local_info {
> + struct cros_ec_dev *cros_ec_dev; /* Pointer to cros_ec device */
> + int cros_ec_err; /* Error for cros_ec, 0 if ok */
> +};
> +
> +static struct local_info local;
> +
> +#if defined CONFIG_EXYNOS_TMU
> +/* Boot Time Thermal Analysis for SoC temperature threshold breach */
> +static void boot_temp_check(void)
> +{
> + int temp;
> +
> + switch (tmu_monitor(&temp)) {
> + case TMU_STATUS_NORMAL:
> + break;
> + case TMU_STATUS_TRIPPED:
> + /* Status TRIPPED ans WARNING means corresponding threshold breach */
It's a minor issue.
Could please fix indentations?
> + puts("EXYNOS_TMU: TRIPPING! Device power going down ...\n");
> + set_ps_hold_ctrl();
> + hang();
> + break;
> + case TMU_STATUS_WARNING:
> + puts("EXYNOS_TMU: WARNING! Temperature very high\n");
> + break;
> + case TMU_STATUS_INIT:
> + /*
> + * TMU_STATUS_INIT means something is wrong with temperature sensing
> + * and TMU status was changed back from NORMAL to INIT.
> + */
ditto.
> + puts("EXYNOS_TMU: WARNING! Temperature sensing not done\n");
> + break;
> + default:
> + debug("EXYNOS_TMU: Unknown TMU state\n");
> + }
> +}
> +#endif
> +
> +int board_init(void)
> +{
> + gd->bd->bi_boot_params = (PHYS_SDRAM_1 + 0x100UL);
> +#if defined CONFIG_EXYNOS_TMU
> + if (tmu_init(gd->fdt_blob) != TMU_STATUS_NORMAL) {
> + debug("%s: Failed to init TMU\n", __func__);
> + return -1;
> + }
> + boot_temp_check();
> +#endif
> +
> +#ifdef CONFIG_EXYNOS_SPI
> + spi_init();
> +#endif
> + return exynos_init();
> +}
> +
> +int dram_init(void)
> +{
> + int i;
> + u32 addr;
> +
> + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
> + addr = CONFIG_SYS_SDRAM_BASE + (i * SDRAM_BANK_SIZE);
> + gd->ram_size += get_ram_size((long *)addr, SDRAM_BANK_SIZE);
> + }
> + return 0;
> +}
> +
> +void dram_init_banksize(void)
> +{
> + int i;
> + u32 addr, size;
> +
> + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
> + addr = CONFIG_SYS_SDRAM_BASE + (i * SDRAM_BANK_SIZE);
> + size = get_ram_size((long *)addr, SDRAM_BANK_SIZE);
> +
> + gd->bd->bi_dram[i].start = addr;
> + gd->bd->bi_dram[i].size = size;
> + }
> +}
> +
> +static int board_uart_init(void)
> +{
> + int err, uart_id, ret = 0;
> +
> + for (uart_id = PERIPH_ID_UART0; uart_id <= PERIPH_ID_UART3; uart_id++) {
> + err = exynos_pinmux_config(uart_id, PINMUX_FLAG_NONE);
> + if (err) {
> + debug("UART%d not configured\n",
> + (uart_id - PERIPH_ID_UART0));
> + ret |= err;
> + }
> + }
> + return ret;
> +}
> +
> +#ifdef CONFIG_BOARD_EARLY_INIT_F
> +int board_early_init_f(void)
> +{
> + int err;
> +
> + err = board_uart_init();
> + if (err) {
> + debug("UART init failed\n");
> + return err;
> + }
> +
> +#ifdef CONFIG_SYS_I2C_INIT_BOARD
> + board_i2c_init(gd->fdt_blob);
> +#endif
> +
> + return err;
> +}
> +#endif
> +
> +struct cros_ec_dev *board_get_cros_ec_dev(void)
> +{
> + return local.cros_ec_dev;
> +}
> +
> +static int board_init_cros_ec_devices(const void *blob)
> +{
> + local.cros_ec_err = cros_ec_init(blob, &local.cros_ec_dev);
> + if (local.cros_ec_err)
> + return -1; /* Will report in board_late_init() */
> +
> + return 0;
> +}
> +
> +#if defined(CONFIG_POWER)
> +#ifdef CONFIG_POWER_MAX77686
> +static int pmic_reg_update(struct pmic *p, int reg, uint regval)
> +{
> + u32 val;
> + int ret = 0;
> +
> + ret = pmic_reg_read(p, reg, &val);
> + if (ret) {
> + debug("%s: PMIC %d register read failed\n", __func__, reg);
> + return -1;
> + }
> + val |= regval;
> + ret = pmic_reg_write(p, reg, val);
> + if (ret) {
> + debug("%s: PMIC %d register write failed\n", __func__, reg);
> + return -1;
> + }
> + return 0;
> +}
> +
> +static int max77686_init(void)
> +{
> + struct pmic *p;
> +
> + if (pmic_init(I2C_PMIC))
> + return -1;
> +
> + p = pmic_get("MAX77686_PMIC");
> + if (!p)
> + return -ENODEV;
> +
> + if (pmic_probe(p))
> + return -1;
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_32KHZ, MAX77686_32KHCP_EN))
> + return -1;
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_BBAT,
> + MAX77686_BBCHOSTEN | MAX77686_BBCVS_3_5V))
> + return -1;
> +
> + /* VDD_MIF */
> + if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK1OUT,
> + MAX77686_BUCK1OUT_1V)) {
> + debug("%s: PMIC %d register write failed\n", __func__,
> + MAX77686_REG_PMIC_BUCK1OUT);
> + return -1;
> + }
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK1CRTL,
> + MAX77686_BUCK1CTRL_EN))
> + return -1;
> +
> + /* VDD_ARM */
> + if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK2DVS1,
> + MAX77686_BUCK2DVS1_1_3V)) {
> + debug("%s: PMIC %d register write failed\n", __func__,
> + MAX77686_REG_PMIC_BUCK2DVS1);
> + return -1;
> + }
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK2CTRL1,
> + MAX77686_BUCK2CTRL_ON))
> + return -1;
> +
> + /* VDD_INT */
> + if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK3DVS1,
> + MAX77686_BUCK3DVS1_1_0125V)) {
> + debug("%s: PMIC %d register write failed\n", __func__,
> + MAX77686_REG_PMIC_BUCK3DVS1);
> + return -1;
> + }
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK3CTRL,
> + MAX77686_BUCK3CTRL_ON))
> + return -1;
> +
> + /* VDD_G3D */
> + if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK4DVS1,
> + MAX77686_BUCK4DVS1_1_2V)) {
> + debug("%s: PMIC %d register write failed\n", __func__,
> + MAX77686_REG_PMIC_BUCK4DVS1);
> + return -1;
> + }
> +
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK4CTRL1,
> + MAX77686_BUCK3CTRL_ON))
> + return -1;
> +
> + /* VDD_LDO2 */
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO2CTRL1,
> + MAX77686_LD02CTRL1_1_5V | EN_LDO))
> + return -1;
> +
> + /* VDD_LDO3 */
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO3CTRL1,
> + MAX77686_LD03CTRL1_1_8V | EN_LDO))
> + return -1;
> +
> + /* VDD_LDO5 */
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO5CTRL1,
> + MAX77686_LD05CTRL1_1_8V | EN_LDO))
> + return -1;
> +
> + /* VDD_LDO10 */
> + if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO10CTRL1,
> + MAX77686_LD10CTRL1_1_8V | EN_LDO))
> + return -1;
> +
> + return 0;
> +}
> +#endif
> +
> +int power_init_board(void)
> +{
> + int ret = 0;
> +
> + set_ps_hold_ctrl();
> +
> + i2c_init(CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
> +
> +#ifdef CONFIG_POWER_MAX77686
> + ret = max77686_init();
> +#endif
> +
> + return ret;
> +}
> +#endif
> +
> +#ifdef CONFIG_OF_CONTROL
> +static int decode_sromc(const void *blob, struct fdt_sromc *config)
> +{
> + int err;
> + int node;
> +
> + node = fdtdec_next_compatible(blob, 0, COMPAT_SAMSUNG_EXYNOS5_SROMC);
> + if (node < 0) {
> + debug("Could not find SROMC node\n");
> + return node;
> + }
> +
> + config->bank = fdtdec_get_int(blob, node, "bank", 0);
> + config->width = fdtdec_get_int(blob, node, "width", 2);
> +
> + err = fdtdec_get_int_array(blob, node, "srom-timing", config->timing,
> + FDT_SROM_TIMING_COUNT);
> + if (err < 0) {
> + debug("Could not decode SROMC configuration Error: %s\n",
> + fdt_strerror(err));
> + return -FDT_ERR_NOTFOUND;
> + }
> + return 0;
> +}
> +
> +int board_eth_init(bd_t *bis)
> +{
> +#ifdef CONFIG_SMC911X
> + u32 smc_bw_conf, smc_bc_conf;
> + struct fdt_sromc config;
> + fdt_addr_t base_addr;
> + int node;
> +
> + node = decode_sromc(gd->fdt_blob, &config);
> + if (node < 0) {
> + debug("%s: Could not find sromc configuration\n", __func__);
> + return 0;
> + }
> + node = fdtdec_next_compatible(gd->fdt_blob, node, COMPAT_SMSC_LAN9215);
> + if (node < 0) {
> + debug("%s: Could not find lan9215 configuration\n", __func__);
> + return 0;
> + }
> +
> + /* We now have a node, so any problems from now on are errors */
> + base_addr = fdtdec_get_addr(gd->fdt_blob, node, "reg");
> + if (base_addr == FDT_ADDR_T_NONE) {
> + debug("%s: Could not find lan9215 address\n", __func__);
> + return -1;
> + }
> +
> + /* Ethernet needs data bus width of 16 bits */
> + if (config.width != 2) {
> + debug("%s: Unsupported bus width %d\n", __func__,
> + config.width);
> + return -1;
> + }
> + smc_bw_conf = SROMC_DATA16_WIDTH(config.bank)
> + | SROMC_BYTE_ENABLE(config.bank);
> +
> + smc_bc_conf = SROMC_BC_TACS(config.timing[FDT_SROM_TACS]) |
> + SROMC_BC_TCOS(config.timing[FDT_SROM_TCOS]) |
> + SROMC_BC_TACC(config.timing[FDT_SROM_TACC]) |
> + SROMC_BC_TCOH(config.timing[FDT_SROM_TCOH]) |
> + SROMC_BC_TAH(config.timing[FDT_SROM_TAH]) |
> + SROMC_BC_TACP(config.timing[FDT_SROM_TACP]) |
> + SROMC_BC_PMC(config.timing[FDT_SROM_PMC]);
> +
> + /* Select and configure the SROMC bank */
> + exynos_pinmux_config(PERIPH_ID_SROMC, config.bank);
> + s5p_config_sromc(config.bank, smc_bw_conf, smc_bc_conf);
> + return smc911x_initialize(0, base_addr);
> +#endif
> + return 0;
> +}
> +
> +#ifdef CONFIG_GENERIC_MMC
> +int board_mmc_init(bd_t *bis)
> +{
> + int ret;
> +
> + /* dwmmc initializattion for available channels */
> + ret = exynos_dwmmc_init(gd->fdt_blob);
> + if (ret)
> + debug("dwmmc init failed\n");
> +
> + return ret;
> +}
> +#endif
> +#endif
> +
> +#ifdef CONFIG_BOARD_LATE_INIT
> +int board_late_init(void)
> +{
> + stdio_print_current_devices();
> +
> + if (local.cros_ec_err) {
> + /* Force console on */
> + gd->flags &= ~GD_FLG_SILENT;
> +
> + printf("cros-ec communications failure %d\n",
> + local.cros_ec_err);
> + puts("\nPlease reset with Power+Refresh\n\n");
> + panic("Cannot init cros-ec device");
> + return -1;
> + }
> + return 0;
> +}
> +#endif
> +
> +int arch_early_init_r(void)
> +{
> +#ifdef CONFIG_CROS_EC
> + if (board_init_cros_ec_devices(gd->fdt_blob)) {
> + printf("%s: Failed to init EC\n", __func__);
> + return 0;
> + }
> +#endif
> +
> + return 0;
> +}
Thanks,
Minkyu Kang.
More information about the U-Boot
mailing list