Skip to content

Conversation

@jimmyzhe
Copy link
Contributor

Some boards build fail with tests/drivers/build_all/eeprom because the label name eeprom already exists in their devicetree (adp_xc7k/ae350, bytesensi_l, same54_xpro), conflicting with the test case's overlay.

Rename the eeprom node label in the test case as the noted in tests/drivers/build_all/eeprom/app.overlay.

 * Names in this file should be chosen in a way that won't conflict
 * with real-world devicetree nodes, to allow these tests to run on
 * (and be extended to test) real hardware.

Renamed the label of 'ti_tmp116_eeprom@0' node because 'eeprom' label name
already exists in some boards, such as adp_xc7k/ae350, bytesensi_l,
same54_xpro.

Signed-off-by: Jimmy Zheng <[email protected]>
@zephyrbot zephyrbot added area: EEPROM size: XS A PR changing only a single line of code labels Nov 14, 2024
@dkalowsk dkalowsk merged commit d267402 into zephyrproject-rtos:main Nov 14, 2024
22 checks passed
@jimmyzhe jimmyzhe deleted the fix_test_eeprom_label branch November 14, 2024 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: EEPROM size: XS A PR changing only a single line of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants