Linux/AM4378: Creating custom board. what changes to u-boot do i need if i don't include the EEPROM?

"Yes, if you remove TI_I2C_BOARD_DETECT it will NOT work."

so i NEED that line of code? or can i exclude that line of code and just comment out some function calls in board.c? does board detect happen no matter what?

i see your "patches" but what i am hearing is that if i base my board off of a ti board (evm, like everyone says to around here) i either need to modify the core code of u-boot/boards/ti/common.c or include an EEPROM.

also, since you keep pointing me to posts where you directed people to use your patches that means there is no wiki for how to take u-boot and modify it completely for a custom board.

i looked at your patch in board_detect.c, do you see anything wrong with the following change to your patch:

rc = ti_i2c_eeprom_get(bus_addr, dev_addr, TI_EEPROM_HEADER_MAGIC,
sizeof(am_ep), (uint8_t *)&am_ep);
if (rc)
{
char tempMAC[TI_EEPROM_HDR_NO_OF_MAC_ADDR][TI_EEPROM_HDR_ETH_ALEN] = { {0, 0x1B, 0xC4, 0x03, 0xD0, 0x8F},
{0, 0x1B, 0xC4, 0x03, 0xD0, 0x8F},
{0, 0x1B, 0xC4, 0x03, 0xD0, 0x8F} } ;

strlcpy(ep->name, "am437x_UTI", TI_EEPROM_HDR_NAME_LEN + 1);
strlcpy(ep->version, "v1.0", TI_EEPROM_HDR_REV_LEN + 1);
strlcpy(ep->serial, "434ENG00002", TI_EEPROM_HDR_SERIAL_LEN + 1);
//strlcpy(ep->config, config stuff here, TI_EEPROM_HDR_CONFIG_LEN + 1);
memcpy(ep->mac_addr, tempMAC,
TI_EEPROM_HDR_NO_OF_MAC_ADDR * TI_EEPROM_HDR_ETH_ALEN); //TI_EEPROM_HDR_NO_OF_MAC_ADDR = 3, TI_EEPROM_HDR_ETH_ALEN = 6
}
else
{
strlcpy(ep->name, am_ep.name, TI_EEPROM_HDR_NAME_LEN + 1);
/* BeagleBone Green '1' eeprom, board_rev: 0x1a 0x00 0x00 0x00 */
if (am_ep.version[0] == 0x1a && am_ep.version[1] == 0x00 &&
am_ep.version[2] == 0x00 && am_ep.version[3] == 0x00)
strlcpy(ep->version, "BBG1", TI_EEPROM_HDR_REV_LEN + 1);
else
strlcpy(ep->version, am_ep.version, TI_EEPROM_HDR_REV_LEN + 1);

strlcpy(ep->serial, am_ep.serial, TI_EEPROM_HDR_SERIAL_LEN + 1);
strlcpy(ep->config, am_ep.config, TI_EEPROM_HDR_CONFIG_LEN + 1);
memcpy(ep->mac_addr, am_ep.mac_addr,
TI_EEPROM_HDR_NO_OF_MAC_ADDR * TI_EEPROM_HDR_ETH_ALEN);
}

ep->header = am_ep.header;

ti_eeprom_string_cleanup(ep->name);
ti_eeprom_string_cleanup(ep->version);
ti_eeprom_string_cleanup(ep->serial);
ti_eeprom_string_cleanup(ep->config);

in fact, since i am completely new to this whole process (first linux board for us) is there someone i can send all my code changes to for u-boot to determine if it will likely work/made mistakes/forgot something?

so last question (i think):  is there someone i can send all my code changes to for u-boot to determine if it will likely work/made mistakes/forgot something?

You will probably need to do the code review and testing in your company or by yourself. We do not provide continuous integration services. If you are done with this thread please mark it as resolved by clicking on "This resolved my issue" button and create a new thread if you have more MAC related doubts I will assign it to network engineer.

cobsonchael said:

i asked: so as u-boot exists, without modification to the code that isn't my custom board, u-boot will only function if this eeprom exists, correct? based on your answer i would say it is impossible to make it work changing only my board.c file

you answered: It depends which board.c file you mean but correct.

cobsonchael said:

then i asked: so if i do not include CONFIG_TI_I2C_BOARD_DETECT in that Kconfig file it looks like it won't try to do a board detect, right? so that would fix my issue and i could just modify board.c to not use any "board detected" variables/routine calls to set up my hardware, right?

you answered: Yes. Later wherever your see the board_is_evm() function in <Processor SDK>/board-support/u-boot-<version>/board/ti/am43xx/board.c and <Processor SDK>/board-support/u-boot-<version>/board/ti/am43xx/mux.c file you should make that part of the code taken into account

cobsonchael said:

so which is it, do i need to change board_detect.c to account for board detect because just changing board.c isn't enough OR changing my board files is enough and i just need to cut out enough conditional code to make it happen?

cobsonchael said:

you also did not answer this: so it seems, based on what i've found, that you guys (TI) provides a unique MAC address for the unit and we only need to load our own MAC if we want to. is that accurate?

Hello cobsonchael,

Since you are continuing the question in new thread, please close this one by clicking on "This resolved my issue" button.

Thanks,
Kemal