[FR][P8] support multiple SPI flash for one platform

[ildar]

While Colme P8 is one platform, it happens to have different SPI flash components.
The fixed SPIFLASH_MANUFACTURER looks excessive. Could that be flexible or ignored?

[ildar]

I hope I did everything right. So

1. apply patch to apache-mynewt-core
   0001-hw-drivers-flash-spiflash-add-JEDEC_MFC_XTX-and-XT25.patch.gz (would be nice to upstream it)
2. add this to hw/bsp/nrf52/syscfg.yml:

    SPIFLASH_XT25F32B:      1   # PineTime standard chip

3. having XT25F32B added to the number of standard chips, define "custom" chip as it always was (only one allowed!). E.g. as @tt1pjm suggested, SPIFLASH_MANUFACTURER: 0x20

@tt1pjm, could you please read the SPI flash chip model you have? I guess it should be Micron chip.

[tt1pjm]

Hey @ildar,
When I stepped through the mcuboot code to get the SPIFLASH_MANUFACTURER I did notice that a lookup was performed against the SPIFlash to be able to compare with the value provided. I think there was an assertion failure if no match and no further progress on booting/boot looping. It seems to be a redundant check as you suggest. If we're able to determine the correct jedec manufacturer we should be able to use it and continue to boot but would require changes to mcuboot.
On my SPIFlash I can see the text "XMC QH32BWIG" which I believe corresponds to the XM25QH32B model.

[ildar]

1. I got datasheet of your chip and confirmed that jedec manufacturer is 0x20, just as you wrote in tt1pjm/pinetime-rust-mynewt@3fb35bc 2. As I wrote, it's possible to support several chips in one mcuboot. You just need to add it to the list of well-known chips and use it in syscfg.yml, just as I did in my patch: #6 (comment) 3. would you confirm the bootloader works with your P8 as well as with mine? attaching the hex. [mynewt.elf.hex.gz](https://github.com/JF002/pinetime-mcuboot-bootloader/files/6137280/mynewt.elf.hex.gz)

[tt1pjm]

I flashed your bootloader above using OpenOCD. On restart all worked as expected and booted last loaded firmware. I've also tested a couple of OTA installs of firmware which also work as expected.

[ildar]

very nice! thanks a lot for testing. I plan to make OTA packages, would you be able to test?

[tt1pjm]

Happy to help. I'll be taking a closer look at the newer bootloader/reloader compatibility with the P8 so keep posting your progress.

[ildar]

apache/mynewt-core#2552 is now upstreamed.

[ildar]

@tt1pjm , you could add your P8's flash chip to mynewt-core, as I did for PineTime: ildar/mynewt-core@6555463

[tt1pjm]

Flash chip for P8 now merged into mynewt-core:
apache/mynewt-core#2582

[StarGate01]

Multiple flash chips are allowed via #10 , see https://github.com/InfiniTimeOrg/pinetime-mcuboot-bootloader/pull/10/files#diff-1bcea6a127a33f80b40cfb7ecb5f8de407aaaaca084179d9e3d32904e9c40bdeR64 . P25Q32H support was upstreamed some time ago: apache/mynewt-core#2798 .