[U-Boot] [PATCH 2/4 V3] EXYNOS5: Add gpio pin numbering feature

Simon Glass sjg at chromium.org
Thu Feb 21 21:53:17 CET 2013


Hi Rajeshwari,

On Mon, Feb 18, 2013 at 4:10 AM, Rajeshwari Birje
<rajeshwari.birje at gmail.com> wrote:
> Hi Simon,
>
> Thank you for comments.
>
> On Sun, Feb 17, 2013 at 11:51 AM, Simon Glass <sjg at chromium.org> wrote:
>> Hi Rajeshwari,
>>
>> On Thu, Feb 7, 2013 at 4:00 AM, Rajeshwari Shinde
>> <rajeshwari.s at samsung.com> wrote:
>>> This patch adds support for gpio pin numbering support on EXYNOS5
>>> pinmux.
>>>
>>> Signed-off-by: Leela Krishna Amudala <l.krishna at samsung.com>
>>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s at samsung.com>
>>
>> If we are going to have GPIO numbering it needs to be continuous. The
>> scheme in the Chrome OS tree does this.
> -Okay
>>
>> It's great to have gpio_cfg_pin() but I can't see the actual definition.
> The definition for same is in "S5P: GPIO: Add generic pin numbering API's"
> patch of same patch set.
>>
>> Also please carefully check that the GPIO numbers are all correct - it
>> doesn't look right to me.
> I had verified for each value. Will recheck the same.
>>
>> Also for the benefit of people using FDT who still don't have symbols
>> and who don't yet have proper GPIO support in U-Boot (phandle to bank
>> + number within bank, as in the kernel), can you please annotate the
>> first enum value of each GPIO group (A, B, C) with its hex value? That
>> will also provide a check that things are correct.
> So you want me to assign each bank GPIO group with its Base adress in hex?

No, just add a comment with the GPIO number that each bank happens to
fall on. See the Chromium tree for what we did there in the exynos5
gpio.h file. It just helps to follow what is going on.

>>
> One more doubt as per Minkyu comment to support the same for EXYNOS4
> and avoid error do we have to append EXYNOS5_ at the start of enum
> values? Or anyother solution you would suggest?

I don't think it is needed. GPIO_... should be enough. After all we
are only building for one or the other - we will not at present
include both header files.

Regards,
Simon

>> Regards,
>> Simon
>>
>>> ---
>>> Changes in V2:
>>>         - none.
>>> Changes in V3:
>>>         - none.
>>>  arch/arm/cpu/armv7/exynos/pinmux.c      |  148 +++++--------
>>>  arch/arm/include/asm/arch-exynos/gpio.h |  360 ++++++++++++++++++++++++++++++-
>>>  2 files changed, 413 insertions(+), 95 deletions(-)
>>>
>>> diff --git a/arch/arm/cpu/armv7/exynos/pinmux.c b/arch/arm/cpu/armv7/exynos/pinmux.c
>>> index bd499b4..c79d58e 100644
>>> --- a/arch/arm/cpu/armv7/exynos/pinmux.c
>>> +++ b/arch/arm/cpu/armv7/exynos/pinmux.c
>>> @@ -29,89 +29,77 @@
>>>
>>>  static void exynos5_uart_config(int peripheral)
>>>  {
>>> -       struct exynos5_gpio_part1 *gpio1 =
>>> -               (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>> -       struct s5p_gpio_bank *bank;
>>>         int i, start, count;
>>>
>>>         switch (peripheral) {
>>>         case PERIPH_ID_UART0:
>>> -               bank = &gpio1->a0;
>>> -               start = 0;
>>> +               start = GPIO_A00;
>>>                 count = 4;
>>>                 break;
>>>         case PERIPH_ID_UART1:
>>> -               bank = &gpio1->d0;
>>> -               start = 0;
>>> +               start = GPIO_D00;
>>>                 count = 4;
>>>                 break;
>>>         case PERIPH_ID_UART2:
>>> -               bank = &gpio1->a1;
>>> -               start = 0;
>>> +               start = GPIO_A10;
>>>                 count = 4;
>>>                 break;
>>>         case PERIPH_ID_UART3:
>>> -               bank = &gpio1->a1;
>>> -               start = 4;
>>> +               start = GPIO_A14;
>>>                 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));
>>> +               gpio_set_pull(i, GPIO_PULL_NONE);
>>> +               gpio_cfg_pin(i, GPIO_FUNC(0x2));
>>>         }
>>>  }
>>>
>>>  static int exynos5_mmc_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 = 0, gpio_func = 0;
>>> +       int i, start, start_ext, gpio_func = 0;
>>>
>>>         switch (peripheral) {
>>>         case PERIPH_ID_SDMMC0:
>>> -               bank = &gpio1->c0;
>>> -               bank_ext = &gpio1->c1;
>>> -               start = 0;
>>> +               start = GPIO_C00;
>>> +               start_ext = GPIO_C10;
>>>                 gpio_func = GPIO_FUNC(0x2);
>>>                 break;
>>>         case PERIPH_ID_SDMMC1:
>>> -               bank = &gpio1->c2;
>>> -               bank_ext = NULL;
>>> +               start = GPIO_C20;
>>> +               start_ext = 0;
>>>                 break;
>>>         case PERIPH_ID_SDMMC2:
>>> -               bank = &gpio1->c3;
>>> -               bank_ext = &gpio1->c4;
>>> -               start = 3;
>>> +               start = GPIO_C30;
>>> +               start_ext = GPIO_C43;
>>>                 gpio_func = GPIO_FUNC(0x3);
>>>                 break;
>>>         case PERIPH_ID_SDMMC3:
>>> -               bank = &gpio1->c4;
>>> -               bank_ext = NULL;
>>> +               start = GPIO_C40;
>>> +               start_ext = 0;
>>>                 break;
>>>         }
>>> -       if ((flags & PINMUX_FLAG_8BIT_MODE) && !bank_ext) {
>>> +       if ((flags & PINMUX_FLAG_8BIT_MODE) && !start_ext) {
>>>                 debug("SDMMC device %d does not support 8bit mode",
>>>                                 peripheral);
>>>                 return -1;
>>>         }
>>>         if (flags & PINMUX_FLAG_8BIT_MODE) {
>>> -               for (i = start; i <= (start + 3); i++) {
>>> -                       s5p_gpio_cfg_pin(bank_ext, i, gpio_func);
>>> -                       s5p_gpio_set_pull(bank_ext, i, GPIO_PULL_UP);
>>> -                       s5p_gpio_set_drv(bank_ext, i, GPIO_DRV_4X);
>>> +               for (i = start_ext; i <= (start_ext + 3); i++) {
>>> +                       gpio_cfg_pin(i, gpio_func);
>>> +                       gpio_set_pull(i, GPIO_PULL_UP);
>>> +                       gpio_set_drv(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);
>>> +               gpio_cfg_pin(start + i, GPIO_FUNC(0x2));
>>> +               gpio_set_pull(start + i, GPIO_PULL_NONE);
>>> +               gpio_set_drv(start + 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);
>>> +               gpio_cfg_pin(start + i, GPIO_FUNC(0x2));
>>> +               gpio_set_pull(start + i, GPIO_PULL_UP);
>>> +               gpio_set_drv(start + i, GPIO_DRV_4X);
>>>         }
>>>
>>>         return 0;
>>> @@ -119,8 +107,6 @@ static int exynos5_mmc_config(int peripheral, int flags)
>>>
>>>  static void exynos5_sromc_config(int flags)
>>>  {
>>> -       struct exynos5_gpio_part1 *gpio1 =
>>> -               (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>>         int i;
>>>
>>>         /*
>>> @@ -138,13 +124,13 @@ static void exynos5_sromc_config(int flags)
>>>          * GPY1[2]      SROM_WAIT(2)
>>>          * GPY1[3]      EBI_DATA_RDn(2)
>>>          */
>>> -       s5p_gpio_cfg_pin(&gpio1->y0, (flags & PINMUX_FLAG_BANK),
>>> +       gpio_cfg_pin(GPIO_Y00 + (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));
>>> +       gpio_cfg_pin(GPIO_Y04, GPIO_FUNC(2));
>>> +       gpio_cfg_pin(GPIO_Y05, GPIO_FUNC(2));
>>>
>>>         for (i = 0; i < 4; i++)
>>> -               s5p_gpio_cfg_pin(&gpio1->y1, i, GPIO_FUNC(2));
>>> +               gpio_cfg_pin(GPIO_Y10 + i, GPIO_FUNC(2));
>>>
>>>         /*
>>>          * EBI: 8 Addrss Lines
>>> @@ -179,55 +165,52 @@ static void exynos5_sromc_config(int flags)
>>>          * 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);
>>> +               gpio_cfg_pin(GPIO_Y30 + i, GPIO_FUNC(2));
>>> +               gpio_set_pull(GPIO_Y30 + i, GPIO_PULL_UP);
>>>
>>> -               s5p_gpio_cfg_pin(&gpio1->y5, i, GPIO_FUNC(2));
>>> -               s5p_gpio_set_pull(&gpio1->y5, i, GPIO_PULL_UP);
>>> +               gpio_cfg_pin(GPIO_Y50 + i, GPIO_FUNC(2));
>>> +               gpio_set_pull(GPIO_Y50 + i, GPIO_PULL_UP);
>>>
>>> -               s5p_gpio_cfg_pin(&gpio1->y6, i, GPIO_FUNC(2));
>>> -               s5p_gpio_set_pull(&gpio1->y6, i, GPIO_PULL_UP);
>>> +               gpio_cfg_pin(GPIO_Y60 + i, GPIO_FUNC(2));
>>> +               gpio_set_pull(GPIO_Y60 + i, GPIO_PULL_UP);
>>>         }
>>>  }
>>>
>>>  static void exynos5_i2c_config(int peripheral, int flags)
>>>  {
>>>
>>> -       struct exynos5_gpio_part1 *gpio1 =
>>> -               (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>> -
>>>         switch (peripheral) {
>>>         case PERIPH_ID_I2C0:
>>> -               s5p_gpio_cfg_pin(&gpio1->b3, 0, GPIO_FUNC(0x2));
>>> -               s5p_gpio_cfg_pin(&gpio1->b3, 1, GPIO_FUNC(0x2));
>>> +               gpio_cfg_pin(GPIO_B30, GPIO_FUNC(0x2));
>>> +               gpio_cfg_pin(GPIO_B31, GPIO_FUNC(0x2));
>>>                 break;
>>>         case PERIPH_ID_I2C1:
>>> -               s5p_gpio_cfg_pin(&gpio1->b3, 2, GPIO_FUNC(0x2));
>>> -               s5p_gpio_cfg_pin(&gpio1->b3, 3, GPIO_FUNC(0x2));
>>> +               gpio_cfg_pin(GPIO_B32, GPIO_FUNC(0x2));
>>> +               gpio_cfg_pin(GPIO_B33, GPIO_FUNC(0x2));
>>>                 break;
>>>         case PERIPH_ID_I2C2:
>>> -               s5p_gpio_cfg_pin(&gpio1->a0, 6, GPIO_FUNC(0x3));
>>> -               s5p_gpio_cfg_pin(&gpio1->a0, 7, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A06, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A07, GPIO_FUNC(0x3));
>>>                 break;
>>>         case PERIPH_ID_I2C3:
>>> -               s5p_gpio_cfg_pin(&gpio1->a1, 2, GPIO_FUNC(0x3));
>>> -               s5p_gpio_cfg_pin(&gpio1->a1, 3, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A12, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A13, GPIO_FUNC(0x3));
>>>                 break;
>>>         case PERIPH_ID_I2C4:
>>> -               s5p_gpio_cfg_pin(&gpio1->a2, 0, GPIO_FUNC(0x3));
>>> -               s5p_gpio_cfg_pin(&gpio1->a2, 1, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A20, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A21, GPIO_FUNC(0x3));
>>>                 break;
>>>         case PERIPH_ID_I2C5:
>>> -               s5p_gpio_cfg_pin(&gpio1->a2, 2, GPIO_FUNC(0x3));
>>> -               s5p_gpio_cfg_pin(&gpio1->a2, 3, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A22, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_A23, GPIO_FUNC(0x3));
>>>                 break;
>>>         case PERIPH_ID_I2C6:
>>> -               s5p_gpio_cfg_pin(&gpio1->b1, 3, GPIO_FUNC(0x4));
>>> -               s5p_gpio_cfg_pin(&gpio1->b1, 4, GPIO_FUNC(0x4));
>>> +               gpio_cfg_pin(GPIO_B13, GPIO_FUNC(0x4));
>>> +               gpio_cfg_pin(GPIO_B14, GPIO_FUNC(0x4));
>>>                 break;
>>>         case PERIPH_ID_I2C7:
>>> -               s5p_gpio_cfg_pin(&gpio1->b2, 2, GPIO_FUNC(0x3));
>>> -               s5p_gpio_cfg_pin(&gpio1->b2, 3, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_B22, GPIO_FUNC(0x3));
>>> +               gpio_cfg_pin(GPIO_B23, GPIO_FUNC(0x3));
>>>                 break;
>>>         }
>>>  }
>>> @@ -235,53 +218,42 @@ static void exynos5_i2c_config(int peripheral, int flags)
>>>  static void exynos5_i2s_config(int peripheral)
>>>  {
>>>         int i;
>>> -       struct exynos5_gpio_part1 *gpio1 =
>>> -               (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>>
>>>         for (i = 0; i < 5; i++)
>>> -               s5p_gpio_cfg_pin(&gpio1->b0, i, GPIO_FUNC(0x02));
>>> +               gpio_cfg_pin(GPIO_B00+i, GPIO_FUNC(0x02));
>>>  }
>>>
>>>  void exynos5_spi_config(int peripheral)
>>>  {
>>>         int cfg = 0, pin = 0, i;
>>> -       struct s5p_gpio_bank *bank = NULL;
>>> -       struct exynos5_gpio_part1 *gpio1 =
>>> -               (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>> -       struct exynos5_gpio_part2 *gpio2 =
>>> -               (struct exynos5_gpio_part2 *) samsung_get_base_gpio_part2();
>>>
>>>         switch (peripheral) {
>>>         case PERIPH_ID_SPI0:
>>> -               bank = &gpio1->a2;
>>>                 cfg = GPIO_FUNC(0x2);
>>> -               pin = 0;
>>> +               pin = GPIO_A20;
>>>                 break;
>>>         case PERIPH_ID_SPI1:
>>> -               bank = &gpio1->a2;
>>>                 cfg = GPIO_FUNC(0x2);
>>> -               pin = 4;
>>> +               pin = GPIO_A24;
>>>                 break;
>>>         case PERIPH_ID_SPI2:
>>> -               bank = &gpio1->b1;
>>>                 cfg = GPIO_FUNC(0x5);
>>> -               pin = 1;
>>> +               pin = GPIO_B11;
>>>                 break;
>>>         case PERIPH_ID_SPI3:
>>> -               bank = &gpio2->f1;
>>>                 cfg = GPIO_FUNC(0x2);
>>> -               pin = 0;
>>> +               pin = GPIO_F10;
>>>                 break;
>>>         case PERIPH_ID_SPI4:
>>>                 for (i = 0; i < 2; i++) {
>>> -                       s5p_gpio_cfg_pin(&gpio2->f0, i + 2, GPIO_FUNC(0x4));
>>> -                       s5p_gpio_cfg_pin(&gpio2->e0, i + 4, GPIO_FUNC(0x4));
>>> +                       gpio_cfg_pin(GPIO_F02 + i, GPIO_FUNC(0x4));
>>> +                       gpio_cfg_pin(GPIO_E04 + i, GPIO_FUNC(0x4));
>>>                 }
>>>                 break;
>>>         }
>>>         if (peripheral != PERIPH_ID_SPI4) {
>>>                 for (i = pin; i < pin + 4; i++)
>>> -                       s5p_gpio_cfg_pin(bank, i, cfg);
>>> +                       gpio_cfg_pin(i, cfg);
>>>         }
>>>  }
>>>
>>> diff --git a/arch/arm/include/asm/arch-exynos/gpio.h b/arch/arm/include/asm/arch-exynos/gpio.h
>>> index cfe1024..af882dd 100644
>>> --- a/arch/arm/include/asm/arch-exynos/gpio.h
>>> +++ b/arch/arm/include/asm/arch-exynos/gpio.h
>>> @@ -272,15 +272,355 @@ void s5p_gpio_set_rate(struct s5p_gpio_bank *bank, int gpio, int mode);
>>>             - EXYNOS5_GPIO_PART3_BASE) / sizeof(struct s5p_gpio_bank)) \
>>>           * GPIO_PER_BANK) + pin) + EXYNOS5_GPIO_PART2_MAX)
>>>
>>> +/* A list of valid GPIO numbers for the asm-generic/gpio.h interface */
>>> +enum exynos5_gpio_pin {
>>> +       /* GPIO_PART1_STARTS */
>>> +       GPIO_A00,
>>> +       GPIO_A01,
>>> +       GPIO_A02,
>>> +       GPIO_A03,
>>> +       GPIO_A04,
>>> +       GPIO_A05,
>>> +       GPIO_A06,
>>> +       GPIO_A07,
>>> +       GPIO_A10,
>>> +       GPIO_A11,
>>> +       GPIO_A12,
>>> +       GPIO_A13,
>>> +       GPIO_A14,
>>> +       GPIO_A15,
>>> +       GPIO_A16,
>>> +       GPIO_A17,
>>> +       GPIO_A20,
>>> +       GPIO_A21,
>>> +       GPIO_A22,
>>> +       GPIO_A23,
>>> +       GPIO_A24,
>>> +       GPIO_A25,
>>> +       GPIO_A26,
>>> +       GPIO_A27,
>>> +       GPIO_B00,
>>> +       GPIO_B01,
>>> +       GPIO_B02,
>>> +       GPIO_B03,
>>> +       GPIO_B04,
>>> +       GPIO_B05,
>>> +       GPIO_B06,
>>> +       GPIO_B07,
>>> +       GPIO_B10,
>>> +       GPIO_B11,
>>> +       GPIO_B12,
>>> +       GPIO_B13,
>>> +       GPIO_B14,
>>> +       GPIO_B15,
>>> +       GPIO_B16,
>>> +       GPIO_B17,
>>> +       GPIO_B20,
>>> +       GPIO_B21,
>>> +       GPIO_B22,
>>> +       GPIO_B23,
>>> +       GPIO_B24,
>>> +       GPIO_B25,
>>> +       GPIO_B26,
>>> +       GPIO_B27,
>>> +       GPIO_B30,
>>> +       GPIO_B31,
>>> +       GPIO_B32,
>>> +       GPIO_B33,
>>> +       GPIO_B34,
>>> +       GPIO_B35,
>>> +       GPIO_B36,
>>> +       GPIO_B37,
>>> +       GPIO_C00,
>>> +       GPIO_C01,
>>> +       GPIO_C02,
>>> +       GPIO_C03,
>>> +       GPIO_C04,
>>> +       GPIO_C05,
>>> +       GPIO_C06,
>>> +       GPIO_C07,
>>> +       GPIO_C10,
>>> +       GPIO_C11,
>>> +       GPIO_C12,
>>> +       GPIO_C13,
>>> +       GPIO_C14,
>>> +       GPIO_C15,
>>> +       GPIO_C16,
>>> +       GPIO_C17,
>>> +       GPIO_C20,
>>> +       GPIO_C21,
>>> +       GPIO_C22,
>>> +       GPIO_C23,
>>> +       GPIO_C24,
>>> +       GPIO_C25,
>>> +       GPIO_C26,
>>> +       GPIO_C27,
>>> +       GPIO_C30,
>>> +       GPIO_C31,
>>> +       GPIO_C32,
>>> +       GPIO_C33,
>>> +       GPIO_C34,
>>> +       GPIO_C35,
>>> +       GPIO_C36,
>>> +       GPIO_C37,
>>> +       GPIO_D00,
>>> +       GPIO_D01,
>>> +       GPIO_D02,
>>> +       GPIO_D03,
>>> +       GPIO_D04,
>>> +       GPIO_D05,
>>> +       GPIO_D06,
>>> +       GPIO_D07,
>>> +       GPIO_D10,
>>> +       GPIO_D11,
>>> +       GPIO_D12,
>>> +       GPIO_D13,
>>> +       GPIO_D14,
>>> +       GPIO_D15,
>>> +       GPIO_D16,
>>> +       GPIO_D17,
>>> +       GPIO_Y00,
>>> +       GPIO_Y01,
>>> +       GPIO_Y02,
>>> +       GPIO_Y03,
>>> +       GPIO_Y04,
>>> +       GPIO_Y05,
>>> +       GPIO_Y06,
>>> +       GPIO_Y07,
>>> +       GPIO_Y10,
>>> +       GPIO_Y11,
>>> +       GPIO_Y12,
>>> +       GPIO_Y13,
>>> +       GPIO_Y14,
>>> +       GPIO_Y15,
>>> +       GPIO_Y16,
>>> +       GPIO_Y17,
>>> +       GPIO_Y20,
>>> +       GPIO_Y21,
>>> +       GPIO_Y22,
>>> +       GPIO_Y23,
>>> +       GPIO_Y24,
>>> +       GPIO_Y25,
>>> +       GPIO_Y26,
>>> +       GPIO_Y27,
>>> +       GPIO_Y30,
>>> +       GPIO_Y31,
>>> +       GPIO_Y32,
>>> +       GPIO_Y33,
>>> +       GPIO_Y34,
>>> +       GPIO_Y35,
>>> +       GPIO_Y36,
>>> +       GPIO_Y37,
>>> +       GPIO_Y40,
>>> +       GPIO_Y41,
>>> +       GPIO_Y42,
>>> +       GPIO_Y43,
>>> +       GPIO_Y44,
>>> +       GPIO_Y45,
>>> +       GPIO_Y46,
>>> +       GPIO_Y47,
>>> +       GPIO_Y50,
>>> +       GPIO_Y51,
>>> +       GPIO_Y52,
>>> +       GPIO_Y53,
>>> +       GPIO_Y54,
>>> +       GPIO_Y55,
>>> +       GPIO_Y56,
>>> +       GPIO_Y57,
>>> +       GPIO_Y60,
>>> +       GPIO_Y61,
>>> +       GPIO_Y62,
>>> +       GPIO_Y63,
>>> +       GPIO_Y64,
>>> +       GPIO_Y65,
>>> +       GPIO_Y66,
>>> +       GPIO_Y67,
>>> +       RES = GPIO_Y67 + (0x3 * GPIO_PER_BANK),
>>
>> What is happening here?
>>
>> Have you checked that the values are correct? To me it looks like
>> GPIO_C40 will be set to RES + 1, but it should be set to GPIO_Y67 + 1
>> I think.
> As per the User manual there is a gap between these base adresses
> hence had to add a difference variable.

Yes, but that is a chip feature. It is irrelevant to the API you
present. Your GPIO numbering in U-Boot should start at 0 and header up
to n-1, where n is the number of available GPIOs. There should be no
gaps - they serve no perpose to those using your code, and actually
make things harder.

Again this is implemented in the Chromium tree.

Thanks for your efforts on this.

Regards,
Simon


>>
>>> +       GPIO_C40,
>>> +       GPIO_C41,
>>> +       GPIO_C42,
>>> +       GPIO_C43,
>>> +       GPIO_C44,
>>> +       GPIO_C45,
>>> +       GPIO_C46,
>>> +       GPIO_C47,
>>> +       RES1 = GPIO_C47 + (0x48 * GPIO_PER_BANK),
>>> +       GPIO_X00,
>>> +       GPIO_X01,
>>> +       GPIO_X02,
>>> +       GPIO_X03,
>>> +       GPIO_X04,
>>> +       GPIO_X05,
>>> +       GPIO_X06,
>>> +       GPIO_X07,
>>> +       GPIO_X10,
>>> +       GPIO_X11,
>>> +       GPIO_X12,
>>> +       GPIO_X13,
>>> +       GPIO_X14,
>>> +       GPIO_X15,
>>> +       GPIO_X16,
>>> +       GPIO_X17,
>>> +       GPIO_X20,
>>> +       GPIO_X21,
>>> +       GPIO_X22,
>>> +       GPIO_X23,
>>> +       GPIO_X24,
>>> +       GPIO_X25,
>>> +       GPIO_X26,
>>> +       GPIO_X27,
>>> +       GPIO_X30,
>>> +       GPIO_X31,
>>> +       GPIO_X32,
>>> +       GPIO_X33,
>>> +       GPIO_X34,
>>> +       GPIO_X35,
>>> +       GPIO_X36,
>>> +       GPIO_X37,
>>> +
>>> +       /* GPIO_PART2_STARTS */
>>> +       GPIO_PART1_MAX,
>>> +       GPIO_E00 = GPIO_PART1_MAX,
>>> +       GPIO_E01,
>>> +       GPIO_E02,
>>> +       GPIO_E03,
>>> +       GPIO_E04,
>>> +       GPIO_E05,
>>> +       GPIO_E06,
>>> +       GPIO_E07,
>>> +       GPIO_E10,
>>> +       GPIO_E11,
>>> +       GPIO_E12,
>>> +       GPIO_E13,
>>> +       GPIO_E14,
>>> +       GPIO_E15,
>>> +       GPIO_E16,
>>> +       GPIO_E17,
>>> +       GPIO_F00,
>>> +       GPIO_F01,
>>> +       GPIO_F02,
>>> +       GPIO_F03,
>>> +       GPIO_F04,
>>> +       GPIO_F05,
>>> +       GPIO_F06,
>>> +       GPIO_F07,
>>> +       GPIO_F10,
>>> +       GPIO_F11,
>>> +       GPIO_F12,
>>> +       GPIO_F13,
>>> +       GPIO_F14,
>>> +       GPIO_F15,
>>> +       GPIO_F16,
>>> +       GPIO_F17,
>>> +       GPIO_G00,
>>> +       GPIO_G01,
>>> +       GPIO_G02,
>>> +       GPIO_G03,
>>> +       GPIO_G04,
>>> +       GPIO_G05,
>>> +       GPIO_G06,
>>> +       GPIO_G07,
>>> +       GPIO_G10,
>>> +       GPIO_G11,
>>> +       GPIO_G12,
>>> +       GPIO_G13,
>>> +       GPIO_G14,
>>> +       GPIO_G15,
>>> +       GPIO_G16,
>>> +       GPIO_G17,
>>> +       GPIO_G20,
>>> +       GPIO_G21,
>>> +       GPIO_G22,
>>> +       GPIO_G23,
>>> +       GPIO_G24,
>>> +       GPIO_G25,
>>> +       GPIO_G26,
>>> +       GPIO_G27,
>>> +       GPIO_H00,
>>> +       GPIO_H01,
>>> +       GPIO_H02,
>>> +       GPIO_H03,
>>> +       GPIO_H04,
>>> +       GPIO_H05,
>>> +       GPIO_H06,
>>> +       GPIO_H07,
>>> +       GPIO_H10,
>>> +       GPIO_H11,
>>> +       GPIO_H12,
>>> +       GPIO_H13,
>>> +       GPIO_H14,
>>> +       GPIO_H15,
>>> +       GPIO_H16,
>>> +       GPIO_H17,
>>> +
>>> +       /* GPIO_PART3_STARTS */
>>> +       GPIO_PART2_MAX,
>>> +       GPIO_V00 = GPIO_PART2_MAX,
>>> +       GPIO_V01,
>>> +       GPIO_V02,
>>> +       GPIO_V03,
>>> +       GPIO_V04,
>>> +       GPIO_V05,
>>> +       GPIO_V06,
>>> +       GPIO_V07,
>>> +       GPIO_V10,
>>> +       GPIO_V11,
>>> +       GPIO_V12,
>>> +       GPIO_V13,
>>> +       GPIO_V14,
>>> +       GPIO_V15,
>>> +       GPIO_V16,
>>> +       GPIO_V17,
>>> +       RES3 = GPIO_V17 + GPIO_PER_BANK,
>>> +       GPIO_V20,
>>> +       GPIO_V21,
>>> +       GPIO_V22,
>>> +       GPIO_V23,
>>> +       GPIO_V24,
>>> +       GPIO_V25,
>>> +       GPIO_V26,
>>> +       GPIO_V27,
>>> +       GPIO_V30,
>>> +       GPIO_V31,
>>> +       GPIO_V32,
>>> +       GPIO_V33,
>>> +       GPIO_V34,
>>> +       GPIO_V35,
>>> +       GPIO_V36,
>>> +       GPIO_V37,
>>> +       RES4 = GPIO_V37 + GPIO_PER_BANK,
>>> +       GPIO_V40,
>>> +       GPIO_V41,
>>> +       GPIO_V42,
>>> +       GPIO_V43,
>>> +       GPIO_V44,
>>> +       GPIO_V45,
>>> +       GPIO_V46,
>>> +       GPIO_V47,
>>> +
>>> +       /* GPIO_PART4_STARTS */
>>> +       GPIO_PART3_MAX,
>>> +       GPIO_Z0 = GPIO_PART3_MAX,
>>> +       GPIO_Z1,
>>> +       GPIO_Z2,
>>> +       GPIO_Z3,
>>> +       GPIO_Z4,
>>> +       GPIO_Z5,
>>> +       GPIO_Z6,
>>> +       GPIO_MAX_PORT
>>> +};
>>> +
>>>  static inline unsigned int s5p_gpio_base(int nr)
>>>  {
>>>         if (cpu_is_exynos5()) {
>>> -               if (nr < EXYNOS5_GPIO_PART1_MAX)
>>> +               if (nr < GPIO_PART1_MAX)
>>>                         return EXYNOS5_GPIO_PART1_BASE;
>>> -               else if (nr < EXYNOS5_GPIO_PART2_MAX)
>>> +               else if (nr < GPIO_PART2_MAX)
>>>                         return EXYNOS5_GPIO_PART2_BASE;
>>> -               else
>>> +               else if (nr < GPIO_PART3_MAX)
>>>                         return EXYNOS5_GPIO_PART3_BASE;
>>> +               else
>>> +                       return EXYNOS5_GPIO_PART4_BASE;
>>>
>>>         } else if (cpu_is_exynos4()) {
>>>                 if (nr < EXYNOS4_GPIO_PART1_MAX)
>>> @@ -295,12 +635,14 @@ static inline unsigned int s5p_gpio_base(int nr)
>>>  static inline unsigned int s5p_gpio_part_max(int nr)
>>>  {
>>>         if (cpu_is_exynos5()) {
>>> -               if (nr < EXYNOS5_GPIO_PART1_MAX)
>>> +               if (nr < GPIO_PART1_MAX)
>>>                         return 0;
>>> -               else if (nr < EXYNOS5_GPIO_PART2_MAX)
>>> -                       return EXYNOS5_GPIO_PART1_MAX;
>>> +               else if (nr < GPIO_PART2_MAX)
>>> +                       return GPIO_PART1_MAX;
>>> +               else if (nr < GPIO_PART3_MAX)
>>> +                       return GPIO_PART2_MAX;
>>>                 else
>>> -                       return EXYNOS5_GPIO_PART2_MAX;
>>> +                       return GPIO_PART3_MAX;
>>>
>>>         } else if (cpu_is_exynos4()) {
>>>                 if (nr < EXYNOS4_GPIO_PART1_MAX)
>>> @@ -311,6 +653,10 @@ static inline unsigned int s5p_gpio_part_max(int nr)
>>>
>>>         return 0;
>>>  }
>>> +
>>> +void gpio_cfg_pin(int gpio, int cfg);
>>> +void gpio_set_pull(int gpio, int mode);
>>> +void gpio_set_drv(int gpio, int mode);
>>
>> You set these in the header file but provide no definition, nor any
>> enums/defines for the cfg and mode parameters.
>>
>>>  #endif
>>>
>>>  /* Pin configurations */
>>> --
>>> 1.7.4.4
>>>
>>
>> Regards,
>> Simon
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>
>
>
> --
> Regards,
> Rajeshwari Shinde


More information about the U-Boot mailing list