[ECOS] CS8900 Ethernet Device Driver in EDB7xxx

[honda@itri.org.tw]

>>
>> Hello,
>>      In the subroutine edb7xxx_cs8900_init of file if_edb7xxx.c, the
code
>> under the comment // Fetch hardware address is shown below:
>>
>>           #if defined(CYGPKG_REDBOOT) && \
>>               defined(CYGSEM_REDBOOT_FLASH_CONFIG) && \
>>               !defined(CYGSEM_ARM_EDB7XXX_SET_ESA)
>>               flash_get_config("edb7xxx_esa", enaddr, CONFIG_ESA);
>>           #else
>>               for (i = 0;  i < ETHER_ADDR_LEN;  i += 2) {
>>                   unsigned short esa_reg = get_reg(PP_IA+i);
>>                   enaddr[i] = esa_reg & 0xFF;
>>                   enaddr[i+1] = esa_reg >> 8;
>>               }
>>           #endif
>>
>>      If I define CYGSEM_ARM_EDB7XXX_SET_ESA only and not CYGPKG_REDBOOT
and
>> CYGSEM_REDBOOT_FLASH_CONFIG,, the address enaddr will be filled by
get_reg
>> (...), so it won't be the value I set in the ConfigTool, is that right?

>Yes, it must be a bug. That #else should be a:

>#elif !defined(CYGSEM_ARM_EDB7XXX_SET_ESA)

>Jifl

Yes, It did work. I've tried it last night.
Otherwise, the ESA would be FF:FF:FF:FF:FF:FF, it cannot get any response.
I think that the DHCP may fail for that.

Kuang-Yang Chuang

OES / ITRI
J220, Bldg. 78, 195-8, Sec. 4, Chung-Hsing Rd.
Chutung, Hsinchu 310, Taiwan, ROC

TEL: 886-3-5917546
FAX: 886-3-5917531