hv: io: fix MISRA-C violations related to break

This patch fixes the MISRA-C violations in arch/x86/io.c
 * make the for loop have only one `break`

Tracked-On: #861
Signed-off-by: Shiqing Gao <shiqing.gao@intel.com>
Acked-by: Eddie Dong <eddie.dong@intel.com>
This commit is contained in:
Shiqing Gao 2018-12-25 10:46:23 +08:00 committed by wenlingz
parent 68643b619f
commit 7016244c22
1 changed files with 7 additions and 2 deletions

View File

@ -247,6 +247,7 @@ hv_emulate_mmio(struct acrn_vcpu *vcpu, struct io_request *io_req)
for (idx = 0U; idx < vcpu->vm->emul_mmio_regions; idx++) { for (idx = 0U; idx < vcpu->vm->emul_mmio_regions; idx++) {
uint64_t base, end; uint64_t base, end;
bool emulation_done = false;
mmio_handler = &(vcpu->vm->emul_mmio[idx]); mmio_handler = &(vcpu->vm->emul_mmio[idx]);
base = mmio_handler->range_start; base = mmio_handler->range_start;
@ -257,14 +258,18 @@ hv_emulate_mmio(struct acrn_vcpu *vcpu, struct io_request *io_req)
} else if (!((address >= base) && ((address + size) <= end))) { } else if (!((address >= base) && ((address + size) <= end))) {
pr_fatal("Err MMIO, address:0x%llx, size:%x", address, size); pr_fatal("Err MMIO, address:0x%llx, size:%x", address, size);
status = -EIO; status = -EIO;
break; emulation_done = true;
} else { } else {
/* Handle this MMIO operation */ /* Handle this MMIO operation */
if (mmio_handler->read_write != NULL) { if (mmio_handler->read_write != NULL) {
status = mmio_handler->read_write(io_req, mmio_handler->handler_private_data); status = mmio_handler->read_write(io_req, mmio_handler->handler_private_data);
break; emulation_done = true;
} }
} }
if (emulation_done) {
break;
}
} }
return status; return status;