Hi,
On Thu, 29 Jun 2023 at 14:04, <suny...@nj.iscas.ac.cn> wrote:
>
> Hi,
> We noticed that in the code implementation board/atmel/sama7g5ek/sama7g5ek.c,
> "#if (IS_ENABLED(CONFIG_I2C_EEPROM))" is located in the condition "#if
> (IS_ENABLED(CONFIG_MISC_INIT_R))".
>
> Consider that the CONFIG_I2C_EEPROM option requires access to EEPROM memory
> after normal initial operations are complete.
>
> Do you think it is necessary to add a dependency on MISC_INIT_R
> in the I2C_EEPROM option definition of drivers/misc/Kconfig file?
That is a board-specific file, so that board should perhaps always
enable include MISC_INIT_R. But other boards may not need this.
You could add 'select MISC_INIT_R' to board/atmel/sama7g5ek/Kconfig
perhaps, e.g.
config BOARD_SPECIFIC_OPTIONS # dummy
def_bool y
select MISC_INIT_R
Regards,
Simon
