On 5/24/2022 3:02 PM, Gerd Hoffmann wrote:
   Hi,

+static int tdvf_parse_section_entry(const TdvfSectionEntry *src,
+                                     TdxFirmwareEntry *entry)

+    /* sanity check */

That is what the whole function is doing.  So rename it to
tdvf_check_section_entry to clarify that?

I will rename it to tdvf_parse_and_check_section_entry() since it first parses the section entries from TDVF to software defined data structure TdxFirmwareEntry

take care,
   Gerd



Reply via email to