[U-Boot] [PATCH] SMDK5250: Add ethernet support

Chander Kashyap chander.kashyap at linaro.org
Thu Feb 9 11:15:41 UTC 2012


Hi Simon,

On 24 January 2012 21:49, Simon Glass <sjg at chromium.org> wrote:
> Hi Chander,
>
> On Tue, Jan 24, 2012 at 5:03 AM, Chander Kashyap
> <chander.kashyap at linaro.org> wrote:
>> This patch enable support for SMC911X based ethernet device.
>>
>> Signed-off-by: Chander Kashyap <chander.kashyap at linaro.org>
>> ---
>>  board/samsung/smdk5250/smdk5250.c |   89 +++++++++++++++++++++++++++++++++++++
>>  include/configs/smdk5250.h        |   13 +++++-
>>  2 files changed, 100 insertions(+), 2 deletions(-)
>>
>> diff --git a/board/samsung/smdk5250/smdk5250.c b/board/samsung/smdk5250/smdk5250.c
>> index 2684620..a1b1a48 100644
>> --- a/board/samsung/smdk5250/smdk5250.c
>> +++ b/board/samsung/smdk5250/smdk5250.c
>> @@ -31,10 +31,95 @@
>>  DECLARE_GLOBAL_DATA_PTR;
>>  struct exynos5_gpio_part1 *gpio1;
>>
>> +static void smc9115_pre_init(void)
>
> Is there a common place where this function could go, so that other
> boards based on this SOC could use it also? Perhaps a
> board/samsun/exynos5 common directory?
As of now there is no common directory. Perhaps if more boards with
same gpio configuration are coming the n might be possible.
>
>> +{
>> +       u32 smc_bw_conf, smc_bc_conf;
>> +       int i;
>> +
>> +       /*
>> +        * 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, CONFIG_ENV_SROM_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_DATA110](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);
>> +       }
>> +
>> +       /* Ethernet needs data bus width of 16 bits */
>> +       smc_bw_conf = SROMC_DATA16_WIDTH(CONFIG_ENV_SROM_BANK)
>> +                       | SROMC_BYTE_ENABLE(CONFIG_ENV_SROM_BANK);
>> +
>> +       smc_bc_conf = SROMC_BC_TACS(0x01) | SROMC_BC_TCOS(0x01)
>> +                       | SROMC_BC_TACC(0x06) | SROMC_BC_TCOH(0x01)
>> +                       | SROMC_BC_TAH(0x0C)  | SROMC_BC_TACP(0x09)
>> +                       | SROMC_BC_PMC(0x01);
>> +
>> +       /* Select and configure the SROMC bank */
>> +       s5p_config_sromc(CONFIG_ENV_SROM_BANK, smc_bw_conf, smc_bc_conf);
>> +}
>> +
>>  int board_init(void)
>>  {
>>        gpio1 = (struct exynos5_gpio_part1 *) samsung_get_base_gpio_part1();
>>
>> +       smc9115_pre_init();
>
> Could this be done once later, in board_eth_init(), to avoid init of a
> device which might not be used?
Yes sure. I will move it before smc911x_initialize in  board_eth_init().
>
>> +
>>        gd->bd->bi_boot_params = (PHYS_SDRAM_1 + 0x100UL);
>>        return 0;
>>  }
>> @@ -79,6 +164,10 @@ void dram_init_banksize(void)
>>        gd->bd->bi_dram[7].size = get_ram_size((long *)PHYS_SDRAM_8, \
>>                                                        PHYS_SDRAM_8_SIZE);
>>  }
>> +int board_eth_init(bd_t *bis)
>> +{
>> +       return smc911x_initialize(0, CONFIG_SMC911X_BASE);
>> +}
>>
>>  #ifdef CONFIG_DISPLAY_BOARDINFO
>>  int checkboard(void)
>> diff --git a/include/configs/smdk5250.h b/include/configs/smdk5250.h
>> index 6f2ef4c..4ad4bd2 100644
>> --- a/include/configs/smdk5250.h
>> +++ b/include/configs/smdk5250.h
>> @@ -98,8 +98,7 @@
>>  #define CONFIG_CMD_MMC
>>  #define CONFIG_CMD_EXT2
>>  #define CONFIG_CMD_FAT
>> -#undef CONFIG_CMD_NET
>> -#undef CONFIG_CMD_NFS
>> +#define CONFIG_CMD_NET
>>
>>  #define CONFIG_BOOTDELAY               3
>>  #define CONFIG_ZERO_BOOTDELAY_CHECK
>> @@ -196,6 +195,16 @@
>>  #define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR - 0x1000000)
>>  #define MMC_MAX_CHANNEL                5
>>
>> +/* Ethernet Controllor Driver */
>> +#ifdef CONFIG_CMD_NET
>> +#define CONFIG_NET_MULTI
>> +#define CONFIG_SMC911X
>> +#define CONFIG_SMC911X_BASE            0x5000000
>> +#define CONFIG_SMC911X_16_BIT
>> +#define CONFIG_ENV_SROM_BANK           1
>> +#define CONFIG_ETHADDR                 "00:40:5c:26:0a:5b"
>
> This should come from the ethaddr environment variable I think.
Yes true. I will remove this.
>
>> +#endif /*CONFIG_CMD_NET*/
>> +
>>  /* Enable devicetree support */
>>  #define CONFIG_OF_LIBFDT
>>
>> --
>> 1.7.5.4
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>
> Regards,
> Simon



-- 
with warm regards,
Chander Kashyap



More information about the linaro-dev mailing list