From b0dbc1cbfe125cc9ae9f73f7755f74ae8e8b1862 Mon Sep 17 00:00:00 2001 From: Junjie Mao Date: Fri, 29 Oct 2021 10:56:11 +0800 Subject: [PATCH] HV: treewide: fix violations of coding guideline C-ST-02 The coding guideline rule C-ST-02 requires that 'the loop body shall be enclosed with brackets', or more specifically, braces. This patch adds braces to the single-line loop bodies. This patch has no semantic change. Tracked-On: #6776 Signed-off-by: Junjie Mao Acked-by: Eddie Dong --- hypervisor/acpi_parser/dmar_parse.c | 6 ++++-- hypervisor/dm/vpci/vmsix.c | 3 ++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/hypervisor/acpi_parser/dmar_parse.c b/hypervisor/acpi_parser/dmar_parse.c index ec9221765..ef2fdadd9 100644 --- a/hypervisor/acpi_parser/dmar_parse.c +++ b/hypervisor/acpi_parser/dmar_parse.c @@ -93,8 +93,9 @@ static uint32_t get_drhd_dev_scope_cnt(struct acpi_dmar_hardware_unit *drhd) while (start < end) { scope = (struct acpi_dmar_device_scope *)start; if ((scope->entry_type != ACPI_DMAR_SCOPE_TYPE_NOT_USED) && - (scope->entry_type < ACPI_DMAR_SCOPE_TYPE_RESERVED)) + (scope->entry_type < ACPI_DMAR_SCOPE_TYPE_RESERVED)) { count++; + } start += scope->length; } return count; @@ -140,8 +141,9 @@ static int32_t handle_one_drhd(struct acpi_dmar_hardware_unit *acpi_drhd, struct } } - if (consumed <= 0) + if (consumed <= 0) { break; + } remaining -= consumed; /* skip IOAPIC & HPET */ diff --git a/hypervisor/dm/vpci/vmsix.c b/hypervisor/dm/vpci/vmsix.c index d0109f7df..990a12b1c 100644 --- a/hypervisor/dm/vpci/vmsix.c +++ b/hypervisor/dm/vpci/vmsix.c @@ -130,8 +130,9 @@ int32_t add_vmsix_capability(struct pci_vdev *vdev, uint32_t entry_num, uint8_t vdev->msix.table_count = entry_num; /* set mask bit of vector control register */ - for (i = 0; i < entry_num; i++) + for (i = 0; i < entry_num; i++) { vdev->msix.table_entries[i].vector_control |= PCIM_MSIX_VCTRL_MASK; + } (void)memset(&msixcap, 0U, sizeof(struct msixcap));