[BUG] FlashID is read incorrectly in a2_boot/main.c

[janhoogerbrugge]

Is there an existing core-v-mcu bug for this?

• I have searched the existing bug issues

Bug Description

I see:

udma_flash_readid(lFlashID);

I assume that it should be:

udma_flash_readid((uint32_t *) &lFlashID);

[MikeOpenHWGroup]

Hi @janhoogerbrugge, thanks for your interest in CORE-V-MCU. What file is this code in? Is it part of this repo or the CORE-V-MCU-CLI-TEST repo?

[janhoogerbrugge]

You can find it here: https://github.com/openhwgroup/core-v-mcu/blob/master/a2_boot/main.c

[janhoogerbrugge]

It should be: udma_flash_readid((uint32_t) &lFlashID);

[MikeOpenHWGroup]

Hi @janhoogerbrugge, yes you are correct this is a bug and your fix is valid. As it turns out, you've highlighted a deeper issue in a2_boot in which it needs a flash to be present. This suites our application, but not the general case where a flash may not be present. We are investigating a proper fix.