On Jun 25, 2019 12:38 AM, "Philippe Mathieu-Daudé" <f4bug@amsat.org> wrote:
>
> Since we'll move this code around, fix its style first:
>
> ERROR: braces {} are necessary for all arms of this statement
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>
> hw/mips/gt64xxx_pci.c | 27 ++++++++++++++++++---------
> 1 file changed, 18 insertions(+), 9 deletions(-)
>
> diff --git a/hw/mips/gt64xxx_pci.c b/hw/mips/gt64xxx_pci.c
> index bbd719f091..cfd497960c 100644
> --- a/hw/mips/gt64xxx_pci.c
> +++ b/hw/mips/gt64xxx_pci.c
> @@ -257,19 +257,25 @@ static void check_reserved_space(hwaddr *start,
hwaddr *length)
> hwaddr begin = *start;
> hwaddr end = *start + *length;
>
> - if (end >= 0x1e000000LL && end < 0x1f100000LL)
> + if (end >= 0x1e000000LL && end < 0x1f100000LL) {
> end = 0x1e000000LL;
> - if (begin >= 0x1e000000LL && begin < 0x1f100000LL)
> + }
> + if (begin >= 0x1e000000LL && begin < 0x1f100000LL) {
> begin = 0x1f100000LL;
> - if (end >= 0x1fc00000LL && end < 0x1fd00000LL)
> + }
> + if (end >= 0x1fc00000LL && end < 0x1fd00000LL) {
> end = 0x1fc00000LL;
> - if (begin >= 0x1fc00000LL && begin < 0x1fd00000LL)
> + }
> + if (begin >= 0x1fc00000LL && begin < 0x1fd00000LL) {
> begin = 0x1fd00000LL;
> + }
> /* XXX: This is broken when a reserved range splits the requested
range */
> - if (end >= 0x1f100000LL && begin < 0x1e000000LL)
> + if (end >= 0x1f100000LL && begin < 0x1e000000LL) {
> end = 0x1e000000LL;
> - if (end >= 0x1fd00000LL && begin < 0x1fc00000LL)
> + }
> + if (end >= 0x1fd00000LL && begin < 0x1fc00000LL) {
> end = 0x1fc00000LL;
> + }
>
> *start = begin;
> *length = end - begin;
> @@ -385,8 +391,9 @@ static void gt64120_writel (void *opaque, hwaddr addr,
> PCIHostState *phb = PCI_HOST_BRIDGE(s);
> uint32_t saddr;
>
> - if (!(s->regs[GT_CPU] & 0x00001000))
> + if (!(s->regs[GT_CPU] & 0x00001000)) {
> val = bswap32(val);
> + }
>
> saddr = (addr & 0xfff) >> 2;
> switch (saddr) {
> @@ -937,8 +944,9 @@ static uint64_t gt64120_readl (void *opaque,
> break;
> }
>
> - if (!(s->regs[GT_CPU] & 0x00001000))
> + if (!(s->regs[GT_CPU] & 0x00001000)) {
> val = bswap32(val);
> + }
>
> return val;
> }
> @@ -990,8 +998,9 @@ static void gt64120_pci_set_irq(void *opaque, int
irq_num, int level)
> /* The pic level is the logical OR of all the PCI irqs mapped to
it. */
> pic_level = 0;
> for (i = 0; i < 4; i++) {
> - if (pic_irq == piix4_dev->config[0x60 + i])
> + if (pic_irq == piix4_dev->config[0x60 + i]) {
> pic_level |= pci_irq_levels[i];
> + }
> }
> qemu_set_irq(pic[pic_irq], pic_level);
> }
> --
> 2.19.1
>
>