Add patch to remove prefixes from register names

[jeandudey]

This is just a proof of concept, remove prefixes from register names on each peripheral.

For example on ADC when there is a list of registers named ADC_<name> it will remove the ADC_ part leaving <name>.

I've thought of using the peripheral name as a guess but on the ATDF file I'm converting the FLEXCOM peripherals have registers named FLEX_<> so it wasn't possible, so I opted to find the longest matching common prefix on the registers name that ended with an underscore.

This could be under a command line flag if it causes problems on other ATDF files, for example it can also mistakenly remove prefixes that should probably not be removed (e.g.: a set of registers that are CH_CR, CH_SR, etc.), haven't found something like that though.

[Rahix]

In general, I personally think it is best to keep the original names as much as possible so they can be easily referenced in the datasheet. But I don't know what MCUs you're dealing with so the situation might be different for you.

This could be under a command line flag

I think this is the best approach. This way I can keep it disabled for our builds but people like you can opt-in to this behavior where needed.

[jeandudey]

See 17cf656. Added a --patch <patch-name> command line flag, by default the existing patches are used if none specified, otherwise only the specified will be used, --patch can be specified multiple times.

[Rahix]

Thanks! Now I got around to looking at the actual patch implementation in more detail as well. I have two more comments but overall I am fine with the change.

One more thing: What ATDF files are you testing against? If I can download them as well, I can incorporate them into my own testing in the future. This will help to make sure your files also don't get regressed by other changes.

It would be especially interesting when (if) I get around to adding a proper regression test-suite here...

[jeandudey]

What ATDF files are you testing against? If I can download them as well,

SAMA5D2's ATDF files:

• https://github.com/Microchip-MPLAB-Harmony/dev_packs/tree/master/Microchip/SAMA5D2_DFP/1.7.91/atdf
• https://packs.download.microchip.com/#collapse-Microchip-SAMA5D2-DFP-pdsc (a more recent version than the one in GitHub)
  • From https://packs.download.microchip.com/

[Rahix]

Thanks!