Detect iMX RT variant to support more than 1060

[tannewt]

Detailed description

This uses a word of the ROM to distinguish different chips. Once distinguished, we can use the correct FLEXSPI base address. Specifically, the 1011 has the FLEXSPI at a different address. Tested on the 1011, 1015, 1020, 1040, 1050 and 1060 EVK boards.

This change also checks the length of the SFDP basic parameters table to ensure it is long enough to include page size. The 1011 and 1015 EVKs use the AT25SF128A which has an old SFDP table that is only 9 DWORDS long. Page size is in DWORD 11.

The 1176 is detected but the flash SFDP doesn't work still. So, it doesn't fully work.

Your checklist for this pull request

• I've read the Code of Conduct
• I've read the guidelines for contributing to this repository
• It builds for hardware native (make PROBE_HOST=native)
• It builds as BMDA (make PROBE_HOST=hosted)
• I've tested it to the best of my ability
• My commit messages provide a useful short description of what the commits do

[dragonmux]

Please update your commit message to conform to the contribution guidelines and split the SPI and SFDP changes out into their own commits. They are not directly related to the i.MXRT support improvements and so don't belong in the same commit.

[dragonmux]

Thank you for the contribution! This is an initial review and complements the items given in the lint pass from CI. This is looking good, but could do with a little more work before we can merge it.

[dragonmux]

Thank you for splitting the contribution into commits. We have now given the code a more proper review, see below. It looks like a lot of items, for which apologies, but most of them should be fairly trivial to address and are quite mechanical in nature.

We look forward to testing this on our i.MXRT1060 board and getting this merged soon!

[dragonmux]

There's a small additional change we'd like to request based on some research @mubes has been doing - it turns out that allowing the Cortex-M probe routine to poke for Kinetis parts before iMXRT parts puts certain iMXRT parts in a bad state when we arrive, so please apply this patch in a new commit as part of this PR:

diff --git a/src/target/cortexm.c b/src/target/cortexm.c
index 6d80f4f8..28bdc5d9 100644
--- a/src/target/cortexm.c
+++ b/src/target/cortexm.c
@@ -686,8 +686,8 @@ bool cortexm_probe(adiv5_access_port_s *ap)
 
 	switch (t->designer_code) {
 	case JEP106_MANUFACTURER_FREESCALE:
-		PROBE(kinetis_probe);
 		PROBE(imxrt_probe);
+		PROBE(kinetis_probe);
 		break;
 	case JEP106_MANUFACTURER_GIGADEVICE:
 		PROBE(gd32f1_probe);

[dragonmux]

There are only two items we saw doing a further review. With these taken care of we think this is ready to be merged. Please rebase this on main (git rebase -i main), which will eliminate that merge commit, and then squish fixes such as the spi.c print removal into the original commit for that - we are happy to provide help with this if you would like.

Thank you for your hard work with this and persevering!

[tannewt]

we are happy to provide help with this if you would like.

Please feel free to finish this. I've lost the context for it (not working with imx currently.)

[dragonmux]

Thank you - we'll get on that in the next day or two then.

We have now touched the files, needs Esden's review.

[esden]

LGTM