toolkit/app-gen-toc.py: Remove updateDeviceConfig() function.

[dpgeorge]

It doesn't do anything except give an error if the config file contains certain entries. There's no harm having these entries there, so just skip the validation altogether.

[iabdalkader]

I was just trying to follow the original logic from Alif, which removed certain sub-sections from the miscellaneous section, and then wrote the file out again.

I didn't actually read what the original script did, so I missed that it removed those items. Could it be removing them because the SE can't handle them, or just to save space? If the SE can't handle them, it's probably best to leave them out somehow by removing them from the loaded config (if it's loaded at some point) or just leaving the error as it is. I've never actually tested running a config that includes these items so not sure if it works with them included.

[iabdalkader]

Apparently they're removed because the SE doesn't make any use of them,and they have no effect on the binary output, so it's probably fine to leave them or remove them if you like.

[dpgeorge]

Apparently they're removed because the SE doesn't make any use of them,and they have no effect on the binary output, so it's probably fine to leave them or remove them if you like.

OK so the options are:

1. Leave things as they are and continue to emit an error if the entries exist. The user must then remove them manually from the config file for things to work.
2. Merge this PR as-is. If the entries exist it shouldn't really matter.
3. Change this PR so that it removes the entries automatically if they exist and uses the modified JSON file. That's quite an involved change because we don't want to write out to the file (this tool should not change any input files, it should consider them read-only) so we'd either need to write out a temporary file or pass around the modified JSON to the other functions that use it.

I think option (2) is fine.