Skip to content

Commit 0bed257

Browse files
yuchuyanglijinxia
authored andcommitted
HV: Using one assignment in for loop statement
To follow Misra-C standard, only one assignment is allowed in both initialization and update statement. Noncompliant example: for (i = 0, a = arry[0]; ...; i++, a = arry[i]){...} Signed-off-by: Yang, Yu-chu <yu-chu.yang@intel.com>
1 parent dd695f3 commit 0bed257

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

hypervisor/arch/x86/mmu.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -620,9 +620,8 @@ void init_paging(void)
620620
attr_uc);
621621

622622
/* Modify WB attribute for E820_TYPE_RAM */
623-
for (i = 0U, entry = &e820[0];
624-
i < e820_entries;
625-
i++, entry = &e820[i]) {
623+
for (i = 0U; i < e820_entries; i++) {
624+
entry = &e820[i];
626625
if (entry->type == E820_TYPE_RAM) {
627626
modify_mem(&map_params, (void *)entry->baseaddr,
628627
(void *)entry->baseaddr,

0 commit comments

Comments
 (0)