boards/kinetis-based: Model features in Kconfig

[leandrolanzieri]

Contribution description

This models the features provided by the kinetis CPUs and all the boards based on them:

• frdm-k22f
• frdm-k64f
• frdm-kw41z
• mulle
• openlabs-kw41z-mini
• openlabs-kw41z-mini-256kib
• pba-d-01-kw2x
• phynode-kw41z
• teensy31
• usb-kw41z

Also cleans up the Makefile.features of kinetis a bit.

Testing procedure

• Check the symbol organization in kinetis. As some features depend on the series I used intermediate symbols CPU_SERIES_ to group this.
• Features provided should not change for kinetis-based boards.
• tests/kconfig_features should pass for all boards.

Issues/PRs references

Part of #14148

[fjmolinas]

@leandrolanzieri do we actually need this new CPU_SERIES_ symbol, it seems that CPU_FAMILY is only used for openocd, could we replace CPU_SERIES_ by CPU_FAM_ and get rid of the current CPU_FAMILY variable altogether? It seems like the different openocd config are ultimately simply based on the core. We could have in boards/common/frdm/Makefile.include

ifeq ($(CPU_CORE), cortex-m0plus)
  OPENOCD_KINETIS_FAMILY = klx
else
  OPENOCD_KINETIS_FAMILY = kx
endif

OPENOCD_CONFIG ?= $(RIOTBOARD)/common/frdm/dist/openocd-$(OPENOCD_KINETIS_FAMILY).cfg

[leandrolanzieri]

I turned the CPU_SERIES_ symbols into CPU_FAM_. Now we are using the value of KINETIS_SERIES as CPU_FAM. Also renamed s9keaz128aclh because s9keaz128aclh48 does not exist apparently.

[leandrolanzieri]

I had to move the core selection to the models as I can't find a proper family classification that allows to imply that.

[fjmolinas]

I had to move the core selection to the models as I can't find a proper family classification that allows to imply that.

I looked into this as well, and kind find a matching category. The current one used in the makefiles is actually wrong, works by chance.

I turned the CPU_SERIES_ symbols into CPU_FAM_. Now we are using the value of KINETIS_SERIES as CPU_FAM. Also renamed s9keaz128aclh because s9keaz128aclh48 does not exist apparently.

How did you realize this?

[fjmolinas]

I had to move the core selection to the models as I can't find a proper family classification that allows to imply that.

I looked into this as well, and kind find a matching category. The current one used in the makefiles is actually wrong, works by chance.

I turned the CPU_SERIES_ symbols into CPU_FAM_. Now we are using the value of KINETIS_SERIES as CPU_FAM. Also renamed s9keaz128aclh because s9keaz128aclh48 does not exist apparently.

How did you realize this?

[leandrolanzieri]

How did you realize this?

When I was looking model by model for the core I could not find s9keaz128aclh48, just https://www.nxp.com/part/S9KEAZ128ACLH

[fjmolinas]

I flashed pba-d-01-kw2x and frdm-kw41z, both flashing correctly. That covers the openocd change.

2020-07-02 14:25:45,958 # Hello World!
2020-07-02 14:25:45,960 # You are running RIOT on a(n) frdm-kw41z board.
2020-07-02 14:25:45,961 # This board features a(n) kinetis MCU.

2020-07-02 14:25:06,582 # Hello World!
2020-07-02 14:25:06,586 # You are running RIOT on a(n) pba-d-01-kw2x board.
2020-07-02 14:25:06,589 # This board features a(n) kinetis MCU.

[fjmolinas]

I'm good with the symbols now, please squash @leandrolanzieri and trigger murdock!

[leandrolanzieri]

All green @fjmolinas !

[fjmolinas]

ACK!

[leandrolanzieri]

Thanks for the review and the suggestions!