[U-Boot] [PATCH 1/2 V2] EXYNOS5: PINMUX: Added default pinumx settings
Minkyu Kang
promsoft at gmail.com
Thu May 31 11:57:39 CEST 2012
Dear Rajeshwari Shinde,
On 23 May 2012 17:54, Rajeshwari Shinde <rajeshwari.s at samsung.com> wrote:
> This patch performs the pinmux configuration in a common file.
> As of now only EXYNOS5 pinmux for SDMMC, UART and Ethernet is
> supported.
>
> Signed-off-by: Abhilash Kesavan <a.kesavan at samsung.com>
> Signed-off-by: Che-Liang Chiou <clchiou at chromium.org>
> Signed-off-by: Rajeshwari Shinde <rajeshwari.s at samsung.com>
> ---
> changes in V2:
> - Adding pinmux.c to Makefile moved to this patch.
> - exynos5_pinmux_config made static.
> arch/arm/cpu/armv7/exynos/Makefile | 2 +-
> arch/arm/cpu/armv7/exynos/pinmux.c | 189 +++++++++++++++++++++++++++++
> arch/arm/include/asm/arch-exynos/pinmux.h | 77 ++++++++++++
> 3 files changed, 267 insertions(+), 1 deletions(-)
> create mode 100644 arch/arm/cpu/armv7/exynos/pinmux.c
> create mode 100644 arch/arm/include/asm/arch-exynos/pinmux.h
>
> diff --git a/arch/arm/cpu/armv7/exynos/Makefile b/arch/arm/cpu/armv7/exynos/Makefile
> index 90ec2bd..9119961 100644
> --- a/arch/arm/cpu/armv7/exynos/Makefile
> +++ b/arch/arm/cpu/armv7/exynos/Makefile
> @@ -22,7 +22,7 @@ include $(TOPDIR)/config.mk
>
> LIB = $(obj)lib$(SOC).o
>
> -COBJS += clock.o power.o soc.o system.o
> +COBJS += clock.o power.o soc.o system.o pinmux.o
>
> SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> OBJS := $(addprefix $(obj),$(COBJS) $(SOBJS))
> diff --git a/arch/arm/cpu/armv7/exynos/pinmux.c b/arch/arm/cpu/armv7/exynos/pinmux.c
> new file mode 100644
> index 0000000..c6392b2
> --- /dev/null
> +++ b/arch/arm/cpu/armv7/exynos/pinmux.c
> @@ -0,0 +1,189 @@
> +/*
> + * Copyright (c) 2012 Samsung Electronics.
> + * Abhilash Kesavan <a.kesavan at samsung.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +
> +#include <common.h>
> +#include <asm/arch/cpu.h>
cpu.h is already included.
> +#include <asm/arch/gpio.h>
> +#include <asm/arch/pinmux.h>
> +#include <asm/arch/sromc.h>
> +
> +static int exynos5_pinmux_config(int peripheral, int flags)
> +{
> + struct exynos5_gpio_part1 *gpio1 =
> + (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
> + struct s5p_gpio_bank *bank, *bank_ext;
> + int i, start, count;
> +
> + switch (peripheral) {
> + case PERIPH_ID_UART0:
> + case PERIPH_ID_UART1:
> + case PERIPH_ID_UART2:
> + case PERIPH_ID_UART3:
> + switch (peripheral) {
> + case PERIPH_ID_UART0:
> + bank = &gpio1->a0;
> + start = 0; count = 4;
Please don't put multiple statement on a single line. fix it globally.
> + break;
> + case PERIPH_ID_UART1:
> + bank = &gpio1->a0;
> + start = 4; count = 4;
> + break;
> + case PERIPH_ID_UART2:
> + bank = &gpio1->a1;
> + start = 0; count = 4;
> + break;
> + case PERIPH_ID_UART3:
> + bank = &gpio1->a1;
> + start = 4; count = 2;
> + break;
> + }
> + for (i = start; i < start + count; i++) {
> + s5p_gpio_set_pull(bank, i, GPIO_PULL_NONE);
> + s5p_gpio_cfg_pin(bank, i, GPIO_FUNC(0x2));
> + }
> + break;
Looks confused.
Why don't you make function for each peripherals?
e.g: exynos5_uart_config, exynos5_mmc_config.
> + case PERIPH_ID_SDMMC0:
> + case PERIPH_ID_SDMMC1:
> + case PERIPH_ID_SDMMC2:
> + case PERIPH_ID_SDMMC3:
> + switch (peripheral) {
> + case PERIPH_ID_SDMMC0:
> + bank = &gpio1->c0; bank_ext = &gpio1->c1;
> + break;
> + case PERIPH_ID_SDMMC1:
> + bank = &gpio1->c1; bank_ext = NULL;
> + break;
> + case PERIPH_ID_SDMMC2:
> + bank = &gpio1->c2; bank_ext = &gpio1->c3;
> + break;
> + case PERIPH_ID_SDMMC3:
> + bank = &gpio1->c3; bank_ext = NULL;
> + break;
> + }
> + if ((flags & PINMUX_FLAG_8BIT_MODE) && !bank_ext) {
> + debug("SDMMC device %d does not support 8bit mode",
> + peripheral);
> + return -1;
> + }
> + if (flags & PINMUX_FLAG_8BIT_MODE) {
> + for (i = 3; i <= 6; i++) {
> + s5p_gpio_cfg_pin(bank_ext, i, GPIO_FUNC(0x3));
> + s5p_gpio_set_pull(bank_ext, i, GPIO_PULL_UP);
> + s5p_gpio_set_drv(bank_ext, i, GPIO_DRV_4X);
> + }
> + }
> + for (i = 0; i < 2; i++) {
> + s5p_gpio_cfg_pin(bank, i, GPIO_FUNC(0x2));
> + s5p_gpio_set_pull(bank, i, GPIO_PULL_NONE);
> + s5p_gpio_set_drv(bank, i, GPIO_DRV_4X);
> + }
> + for (i = 3; i <= 6; i++) {
> + s5p_gpio_cfg_pin(bank, i, GPIO_FUNC(0x2));
> + s5p_gpio_set_pull(bank, i, GPIO_PULL_UP);
> + s5p_gpio_set_drv(bank, i, GPIO_DRV_4X);
> + }
> + break;
> + case PERIPH_ID_SROMC:
> + /*
> + * SROM:CS1 and EBI
> + *
> + * GPY0[0] SROM_CSn[0]
> + * GPY0[1] SROM_CSn[1](2)
> + * GPY0[2] SROM_CSn[2]
> + * GPY0[3] SROM_CSn[3]
> + * GPY0[4] EBI_OEn(2)
> + * GPY0[5] EBI_EEn(2)
> + *
> + * GPY1[0] EBI_BEn[0](2)
> + * GPY1[1] EBI_BEn[1](2)
> + * GPY1[2] SROM_WAIT(2)
> + * GPY1[3] EBI_DATA_RDn(2)
> + */
> + s5p_gpio_cfg_pin(&gpio1->y0, (flags & PINMUX_FLAG_BANK),
> + GPIO_FUNC(2));
> + s5p_gpio_cfg_pin(&gpio1->y0, 4, GPIO_FUNC(2));
> + s5p_gpio_cfg_pin(&gpio1->y0, 5, GPIO_FUNC(2));
> +
> + for (i = 0; i < 4; i++)
> + s5p_gpio_cfg_pin(&gpio1->y1, i, GPIO_FUNC(2));
> +
> + /*
> + * EBI: 8 Addrss Lines
> + *
> + * GPY3[0] EBI_ADDR[0](2)
> + * GPY3[1] EBI_ADDR[1](2)
> + * GPY3[2] EBI_ADDR[2](2)
> + * GPY3[3] EBI_ADDR[3](2)
> + * GPY3[4] EBI_ADDR[4](2)
> + * GPY3[5] EBI_ADDR[5](2)
> + * GPY3[6] EBI_ADDR[6](2)
> + * GPY3[7] EBI_ADDR[7](2)
> + *
> + * EBI: 16 Data Lines
> + *
> + * GPY5[0] EBI_DATA[0](2)
> + * GPY5[1] EBI_DATA[1](2)
> + * GPY5[2] EBI_DATA[2](2)
> + * GPY5[3] EBI_DATA[3](2)
> + * GPY5[4] EBI_DATA[4](2)
> + * GPY5[5] EBI_DATA[5](2)
> + * GPY5[6] EBI_DATA[6](2)
> + * GPY5[7] EBI_DATA[7](2)
> + *
> + * GPY6[0] EBI_DATA[8](2)
> + * GPY6[1] EBI_DATA[9](2)
> + * GPY6[2] EBI_DATA[10](2)
> + * GPY6[3] EBI_DATA[11](2)
> + * GPY6[4] EBI_DATA[12](2)
> + * GPY6[5] EBI_DATA[13](2)
> + * GPY6[6] EBI_DATA[14](2)
> + * GPY6[7] EBI_DATA[15](2)
> + */
> + for (i = 0; i < 8; i++) {
> + s5p_gpio_cfg_pin(&gpio1->y3, i, GPIO_FUNC(2));
> + s5p_gpio_set_pull(&gpio1->y3, i, GPIO_PULL_UP);
> +
> + s5p_gpio_cfg_pin(&gpio1->y5, i, GPIO_FUNC(2));
> + s5p_gpio_set_pull(&gpio1->y5, i, GPIO_PULL_UP);
> +
> + s5p_gpio_cfg_pin(&gpio1->y6, i, GPIO_FUNC(2));
> + s5p_gpio_set_pull(&gpio1->y6, i, GPIO_PULL_UP);
> + }
> + break;
> + default:
> + debug("%s: invalid peripheral %d", __func__, peripheral);
> + return -1;
> + }
> +
> + return 0;
> +}
> +
> +int exynos_pinmux_config(int peripheral, int flags)
> +{
> + if (cpu_is_exynos5())
> + return exynos5_pinmux_config(peripheral, flags);
> + else{
need space between else and {
Well, I think you don't need this else statement.
> + debug("pinmux functionality not supported\n");
> + return -1;
> + }
> +}
Thanks.
Minkyu Kang.
--
from. prom.
www.promsoft.net
More information about the U-Boot
mailing list