Add check script for new entry to qcom-fitimage.its

[amrianan]

For each configuration subnode in qcom-fitimage.its:

• read its "compatible" property
• strip "qcom," prefix
• split the remaining string on "-"
• verify that each resulting substring exists as a subnode name in qcom-metadata.dts

Output:

• If everything passes: "success"
• If any missing: one line per miss: fail <config_subnode_name>

[NainaMehtaQUIC]

Can we add usage of the script in comments?

[amrianan]

done

[NainaMehtaQUIC]

Can we take the file path from user? If not provided, then fallback to these?

[amrianan]

Analysis:

1. We would have used fdtget, fdtdump commandl-ine utilities instead of awk: but cannot use as these utilities need compiled dtb to run over. Since metadata exists in dts form and its need all required dtbs to get compiled into dtb format. Hence this would be not possible from build infra side to check for IDs.
2. with "fdtget", we could have searched soc , board etc in respective subnodes only but for this also, there would need to maintain a table where we will mark that qcm6490 is a soc. with "awk": we are seaching a particular substring in entire metadata, where ever it is found. Not in specific subnodes as soc, board etc.

[NainaMehtaQUIC]

Checking only "}" for end of the node might be causing issues if the "}" is not properly added in new line?
Can we check number of braces instead?

[amrianan]

Thanks for pointing out. Added support for syntax check for qcom-metadata.dtb before starting string comparison.

[NainaMehtaQUIC]

The syntax for FIT ITS image is not being checked with the current version, otherwise LGTM.

[quic-kaushalk]

Overall changes look fine. Naina, as discussed regarding syntax for FIT ITS image not being checked with the current version, we can keep track of it via a github issue in this project.