[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [EXTERNAL][PATCH v8 16/37] target/mips: Style improveme
From: |
Aleksandar Rikalo |
Subject: |
Re: [Qemu-devel] [EXTERNAL][PATCH v8 16/37] target/mips: Style improvements in mips_mipssim.c |
Date: |
Mon, 19 Aug 2019 14:48:15 +0000 |
> From: Aleksandar Markovic <address@hidden>
> Sent: Monday, August 19, 2019 2:07 PM
> To: address@hidden <address@hidden>
> Cc: address@hidden <address@hidden>; Aleksandar Markovic <address@hidden>;
> Aleksandar Rikalo <address@hidden>
> Subject: [EXTERNAL][PATCH v8 16/37] target/mips: Style improvements in
> mips_mipssim.c
>
> From: Aleksandar Markovic <address@hidden>
>
> Fixes mostly errors and warnings reported by 'checkpatch.pl -f'.
>
> Signed-off-by: Aleksandar Markovic <address@hidden>
> ---
> hw/mips/mips_mipssim.c | 19 ++++++++++++-------
> 1 file changed, 12 insertions(+), 7 deletions(-)
>
> diff --git a/hw/mips/mips_mipssim.c b/hw/mips/mips_mipssim.c
> index 824abda..b3155e17 100644
> --- a/hw/mips/mips_mipssim.c
> +++ b/hw/mips/mips_mipssim.c
> @@ -3,7 +3,7 @@
> *
> * Emulates a very simple machine model similar to the one used by the
> * proprietary MIPS emulator.
> - *
> + *
> * Copyright (c) 2007 Thiemo Seufer
> *
> * Permission is hereby granted, free of charge, to any person obtaining a
> copy
> @@ -75,8 +75,9 @@ static int64_t load_kernel(void)
> (uint64_t *)&kernel_high, big_endian,
> EM_MIPS, 1, 0);
> if (kernel_size >= 0) {
> - if ((entry & ~0x7fffffffULL) == 0x80000000)
> + if ((entry & ~0x7fffffffULL) == 0x80000000) {
> entry = (int32_t)entry;
> + }
> } else {
> error_report("could not load kernel '%s': %s",
> loaderparams.kernel_filename,
> @@ -88,9 +89,10 @@ static int64_t load_kernel(void)
> initrd_size = 0;
> initrd_offset = 0;
> if (loaderparams.initrd_filename) {
> - initrd_size = get_image_size (loaderparams.initrd_filename);
> + initrd_size = get_image_size(loaderparams.initrd_filename);
> if (initrd_size > 0) {
> - initrd_offset = (kernel_high + ~INITRD_PAGE_MASK) &
> INITRD_PAGE_MASK;
> + initrd_offset = (kernel_high + ~INITRD_PAGE_MASK) &
> + INITRD_PAGE_MASK;
> if (initrd_offset + initrd_size > loaderparams.ram_size) {
> error_report("memory too small for initial ram disk '%s'",
> loaderparams.initrd_filename);
> @@ -174,8 +176,9 @@ mips_mipssim_init(MachineState *machine)
> /* Map the BIOS / boot exception handler. */
> memory_region_add_subregion(address_space_mem, 0x1fc00000LL, bios);
> /* Load a BIOS / boot exception handler image. */
> - if (bios_name == NULL)
> + if (bios_name == NULL) {
> bios_name = BIOS_FILENAME;
> + }
> filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
> if (filename) {
> bios_size = load_image_targphys(filename, 0x1fc00000LL, BIOS_SIZE);
> @@ -211,8 +214,10 @@ mips_mipssim_init(MachineState *machine)
> get_system_io(), 0, 0x00010000);
> memory_region_add_subregion(get_system_memory(), 0x1fd00000, isa);
>
> - /* A single 16450 sits at offset 0x3f8. It is attached to
> - MIPS CPU INT2, which is interrupt 4. */
> + /*
> + * A single 16450 sits at offset 0x3f8. It is attached to
> + * MIPS CPU INT2, which is interrupt 4.
> + */
> if (serial_hd(0))
> serial_init(0x3f8, env->irq[4], 115200, serial_hd(0),
> get_system_io());
> --
> 2.7.4
Reviewed-by: Aleksandar Rikalo <address@hidden>
- [Qemu-devel] [PATCH v8 08/37] target/mips: Style improvements in internal.h, (continued)
- [Qemu-devel] [PATCH v8 08/37] target/mips: Style improvements in internal.h, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 01/37] target/mips: Add support for DSPRAM, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 10/37] target/mips: Style improvements in helper.c, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 04/37] target/mips: Add support for emulation of GINVT instruction, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 12/37] target/mips: Style improvements in cps.c, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 16/37] target/mips: Style improvements in mips_mipssim.c, Aleksandar Markovic, 2019/08/19
- Re: [Qemu-devel] [EXTERNAL][PATCH v8 16/37] target/mips: Style improvements in mips_mipssim.c,
Aleksandar Rikalo <=
- [Qemu-devel] [PATCH v8 13/37] target/mips: Style improvements in mips_fulong2e.c, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 05/37] target/mips: Add support for emulation of CRC32 group of instructions, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 14/37] target/mips: Style improvements in mips_int.c, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 23/37] target/mips: Clean up handling of CP0 register 8, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 06/37] target/mips: Style improvements in cp0_timer.c, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 25/37] target/mips: Clean up handling of CP0 register 11, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 22/37] target/mips: Clean up handling of CP0 register 7, Aleksandar Markovic, 2019/08/19
- [Qemu-devel] [PATCH v8 18/37] target/mips: Clean up handling of CP0 register 1, Aleksandar Markovic, 2019/08/19