[PATCH 4/4] configs: at91: sama7g5ek: enable CONFIG_PHANDLE_CHECK_SEQ

Eugen Hristev eugen.hristev at microchip.com
Tue Jan 4 19:43:59 CET 2022


CONFIG_PHANDLE_CHECK_SEQ will allow different sequence number for nodes
that have the same name, but they are different.
In sama7g5ek case, there are multiple 'i2c at 600' nodes which are child
nodes of different parent 'flexcom' nodes.
These are different i2c busses even if the node is the same, and have to be
differentiated.
Without this config, the sequence number 0 is reused for two i2c busses, and
this is something that we have to avoid:

Looking for 'i2c' at 4704, name i2c at 600
   - serial0, /ahb/apb/serial at e1824200
   - i2c0, /ahb/apb/flexcom at e181c000/i2c at 600
Found seq 0
i2c_post_bind: i2c at 600, seq=0
Looking for 'i2c' at 6236, name i2c at 600
   - serial0, /ahb/apb/serial at e1824200
   - i2c0, /ahb/apb/flexcom at e181c000/i2c at 600
Found seq 0
i2c_post_bind: i2c at 600, seq=0

After this patch:

Looking for 'i2c' at 4704, name i2c at 600
   - serial0, /ahb/apb/serial at e1824200
   - i2c0, /ahb/apb/flexcom at e181c000/i2c at 600
   - i2c1, /ahb/apb/flexcom at e2818000/i2c at 600
Found seq 1

Before the patch:
=> i2c bus
Bus 0:  i2c at 600
Bus 0:  i2c at 600  (active 0)
   52: eeprom at 52, offset len 1, flags 0
   53: eeprom at 53, offset len 1, flags 0
=>

After the patch:
=> i2c bus
Bus 0:  i2c at 600
Bus 1:  i2c at 600  (active 1)
   52: eeprom at 52, offset len 1, flags 0
   53: eeprom at 53, offset len 1, flags 0

Signed-off-by: Eugen Hristev <eugen.hristev at microchip.com>
---
 configs/sama7g5ek_mmc1_defconfig | 1 +
 configs/sama7g5ek_mmc_defconfig  | 1 +
 2 files changed, 2 insertions(+)

diff --git a/configs/sama7g5ek_mmc1_defconfig b/configs/sama7g5ek_mmc1_defconfig
index e297489900..15a5c54e75 100644
--- a/configs/sama7g5ek_mmc1_defconfig
+++ b/configs/sama7g5ek_mmc1_defconfig
@@ -72,3 +72,4 @@ CONFIG_TIMER=y
 CONFIG_MCHP_PIT64B_TIMER=y
 CONFIG_OF_LIBFDT_OVERLAY=y
 # CONFIG_EFI_LOADER_HII is not set
+CONFIG_PHANDLE_CHECK_SEQ=y
diff --git a/configs/sama7g5ek_mmc_defconfig b/configs/sama7g5ek_mmc_defconfig
index 4f68a46456..7abd5c8a38 100644
--- a/configs/sama7g5ek_mmc_defconfig
+++ b/configs/sama7g5ek_mmc_defconfig
@@ -72,3 +72,4 @@ CONFIG_TIMER=y
 CONFIG_MCHP_PIT64B_TIMER=y
 CONFIG_OF_LIBFDT_OVERLAY=y
 # CONFIG_EFI_LOADER_HII is not set
+CONFIG_PHANDLE_CHECK_SEQ=y
-- 
2.25.1



More information about the U-Boot mailing list