mips: jazz: Renovate coding style
The script checkpatch.pl located in scripts folder was used to detect all errors and warrnings in files: hw/mips/mips_jazz.c hw/display/jazz_led.c hw/dma/rc4030.c All these mips jazz machine files were edited and all the errors and warrings generated by the checkpatch.pl script were corrected and then the script was ran again to make sure there are no more errors and warnings. Signed-off-by: Filip Bozuta <Filip.Bozuta@rt-rk.com> Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com> Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com> Message-Id: <1575640687-20744-2-git-send-email-Filip.Bozuta@rt-rk.com>
This commit is contained in:
parent
084a398bf8
commit
68fa5f552a
|
@ -90,25 +90,25 @@ static void draw_horizontal_line(DisplaySurface *ds,
|
||||||
|
|
||||||
bpp = (surface_bits_per_pixel(ds) + 7) >> 3;
|
bpp = (surface_bits_per_pixel(ds) + 7) >> 3;
|
||||||
d = surface_data(ds) + surface_stride(ds) * posy + bpp * posx1;
|
d = surface_data(ds) + surface_stride(ds) * posy + bpp * posx1;
|
||||||
switch(bpp) {
|
switch (bpp) {
|
||||||
case 1:
|
case 1:
|
||||||
for (x = posx1; x <= posx2; x++) {
|
for (x = posx1; x <= posx2; x++) {
|
||||||
*((uint8_t *)d) = color;
|
*((uint8_t *)d) = color;
|
||||||
d++;
|
d++;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case 2:
|
case 2:
|
||||||
for (x = posx1; x <= posx2; x++) {
|
for (x = posx1; x <= posx2; x++) {
|
||||||
*((uint16_t *)d) = color;
|
*((uint16_t *)d) = color;
|
||||||
d += 2;
|
d += 2;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case 4:
|
case 4:
|
||||||
for (x = posx1; x <= posx2; x++) {
|
for (x = posx1; x <= posx2; x++) {
|
||||||
*((uint32_t *)d) = color;
|
*((uint32_t *)d) = color;
|
||||||
d += 4;
|
d += 4;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -121,25 +121,25 @@ static void draw_vertical_line(DisplaySurface *ds,
|
||||||
|
|
||||||
bpp = (surface_bits_per_pixel(ds) + 7) >> 3;
|
bpp = (surface_bits_per_pixel(ds) + 7) >> 3;
|
||||||
d = surface_data(ds) + surface_stride(ds) * posy1 + bpp * posx;
|
d = surface_data(ds) + surface_stride(ds) * posy1 + bpp * posx;
|
||||||
switch(bpp) {
|
switch (bpp) {
|
||||||
case 1:
|
case 1:
|
||||||
for (y = posy1; y <= posy2; y++) {
|
for (y = posy1; y <= posy2; y++) {
|
||||||
*((uint8_t *)d) = color;
|
*((uint8_t *)d) = color;
|
||||||
d += surface_stride(ds);
|
d += surface_stride(ds);
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case 2:
|
case 2:
|
||||||
for (y = posy1; y <= posy2; y++) {
|
for (y = posy1; y <= posy2; y++) {
|
||||||
*((uint16_t *)d) = color;
|
*((uint16_t *)d) = color;
|
||||||
d += surface_stride(ds);
|
d += surface_stride(ds);
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case 4:
|
case 4:
|
||||||
for (y = posy1; y <= posy2; y++) {
|
for (y = posy1; y <= posy2; y++) {
|
||||||
*((uint32_t *)d) = color;
|
*((uint32_t *)d) = color;
|
||||||
d += surface_stride(ds);
|
d += surface_stride(ds);
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -164,28 +164,28 @@ static void jazz_led_update_display(void *opaque)
|
||||||
if (s->state & REDRAW_SEGMENTS) {
|
if (s->state & REDRAW_SEGMENTS) {
|
||||||
/* set colors according to bpp */
|
/* set colors according to bpp */
|
||||||
switch (surface_bits_per_pixel(surface)) {
|
switch (surface_bits_per_pixel(surface)) {
|
||||||
case 8:
|
case 8:
|
||||||
color_segment = rgb_to_pixel8(0xaa, 0xaa, 0xaa);
|
color_segment = rgb_to_pixel8(0xaa, 0xaa, 0xaa);
|
||||||
color_led = rgb_to_pixel8(0x00, 0xff, 0x00);
|
color_led = rgb_to_pixel8(0x00, 0xff, 0x00);
|
||||||
break;
|
break;
|
||||||
case 15:
|
case 15:
|
||||||
color_segment = rgb_to_pixel15(0xaa, 0xaa, 0xaa);
|
color_segment = rgb_to_pixel15(0xaa, 0xaa, 0xaa);
|
||||||
color_led = rgb_to_pixel15(0x00, 0xff, 0x00);
|
color_led = rgb_to_pixel15(0x00, 0xff, 0x00);
|
||||||
break;
|
break;
|
||||||
case 16:
|
case 16:
|
||||||
color_segment = rgb_to_pixel16(0xaa, 0xaa, 0xaa);
|
color_segment = rgb_to_pixel16(0xaa, 0xaa, 0xaa);
|
||||||
color_led = rgb_to_pixel16(0x00, 0xff, 0x00);
|
color_led = rgb_to_pixel16(0x00, 0xff, 0x00);
|
||||||
break;
|
break;
|
||||||
case 24:
|
case 24:
|
||||||
color_segment = rgb_to_pixel24(0xaa, 0xaa, 0xaa);
|
color_segment = rgb_to_pixel24(0xaa, 0xaa, 0xaa);
|
||||||
color_led = rgb_to_pixel24(0x00, 0xff, 0x00);
|
color_led = rgb_to_pixel24(0x00, 0xff, 0x00);
|
||||||
break;
|
break;
|
||||||
case 32:
|
case 32:
|
||||||
color_segment = rgb_to_pixel32(0xaa, 0xaa, 0xaa);
|
color_segment = rgb_to_pixel32(0xaa, 0xaa, 0xaa);
|
||||||
color_led = rgb_to_pixel32(0x00, 0xff, 0x00);
|
color_led = rgb_to_pixel32(0x00, 0xff, 0x00);
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* display segments */
|
/* display segments */
|
||||||
|
@ -205,8 +205,9 @@ static void jazz_led_update_display(void *opaque)
|
||||||
(s->segments & 0x80) ? color_segment : 0);
|
(s->segments & 0x80) ? color_segment : 0);
|
||||||
|
|
||||||
/* display led */
|
/* display led */
|
||||||
if (!(s->segments & 0x01))
|
if (!(s->segments & 0x01)) {
|
||||||
color_led = 0; /* black */
|
color_led = 0; /* black */
|
||||||
|
}
|
||||||
draw_horizontal_line(surface, 68, 50, 50, color_led);
|
draw_horizontal_line(surface, 68, 50, 50, color_led);
|
||||||
draw_horizontal_line(surface, 69, 49, 51, color_led);
|
draw_horizontal_line(surface, 69, 49, 51, color_led);
|
||||||
draw_horizontal_line(surface, 70, 48, 52, color_led);
|
draw_horizontal_line(surface, 70, 48, 52, color_led);
|
||||||
|
|
|
@ -397,10 +397,11 @@ static void update_jazz_irq(rc4030State *s)
|
||||||
|
|
||||||
pending = s->isr_jazz & s->imr_jazz;
|
pending = s->isr_jazz & s->imr_jazz;
|
||||||
|
|
||||||
if (pending != 0)
|
if (pending != 0) {
|
||||||
qemu_irq_raise(s->jazz_bus_irq);
|
qemu_irq_raise(s->jazz_bus_irq);
|
||||||
else
|
} else {
|
||||||
qemu_irq_lower(s->jazz_bus_irq);
|
qemu_irq_lower(s->jazz_bus_irq);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static void rc4030_irq_jazz_request(void *opaque, int irq, int level)
|
static void rc4030_irq_jazz_request(void *opaque, int irq, int level)
|
||||||
|
@ -588,7 +589,8 @@ static const VMStateDescription vmstate_rc4030 = {
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
static void rc4030_do_dma(void *opaque, int n, uint8_t *buf, int len, int is_write)
|
static void rc4030_do_dma(void *opaque, int n, uint8_t *buf,
|
||||||
|
int len, int is_write)
|
||||||
{
|
{
|
||||||
rc4030State *s = opaque;
|
rc4030State *s = opaque;
|
||||||
hwaddr dma_addr;
|
hwaddr dma_addr;
|
||||||
|
@ -643,8 +645,8 @@ static rc4030_dma *rc4030_allocate_dmas(void *opaque, int n)
|
||||||
struct rc4030DMAState *p;
|
struct rc4030DMAState *p;
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
s = (rc4030_dma *)g_malloc0(sizeof(rc4030_dma) * n);
|
s = (rc4030_dma *)g_new0(rc4030_dma, n);
|
||||||
p = (struct rc4030DMAState *)g_malloc0(sizeof(struct rc4030DMAState) * n);
|
p = (struct rc4030DMAState *)g_new0(struct rc4030DMAState, n);
|
||||||
for (i = 0; i < n; i++) {
|
for (i = 0; i < n; i++) {
|
||||||
p->opaque = opaque;
|
p->opaque = opaque;
|
||||||
p->n = i;
|
p->n = i;
|
||||||
|
|
|
@ -52,8 +52,7 @@
|
||||||
#include "qemu/error-report.h"
|
#include "qemu/error-report.h"
|
||||||
#include "qemu/help_option.h"
|
#include "qemu/help_option.h"
|
||||||
|
|
||||||
enum jazz_model_e
|
enum jazz_model_e {
|
||||||
{
|
|
||||||
JAZZ_MAGNUM,
|
JAZZ_MAGNUM,
|
||||||
JAZZ_PICA61,
|
JAZZ_PICA61,
|
||||||
};
|
};
|
||||||
|
@ -90,16 +89,20 @@ static const MemoryRegionOps rtc_ops = {
|
||||||
static uint64_t dma_dummy_read(void *opaque, hwaddr addr,
|
static uint64_t dma_dummy_read(void *opaque, hwaddr addr,
|
||||||
unsigned size)
|
unsigned size)
|
||||||
{
|
{
|
||||||
/* Nothing to do. That is only to ensure that
|
/*
|
||||||
* the current DMA acknowledge cycle is completed. */
|
* Nothing to do. That is only to ensure that
|
||||||
|
* the current DMA acknowledge cycle is completed.
|
||||||
|
*/
|
||||||
return 0xff;
|
return 0xff;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void dma_dummy_write(void *opaque, hwaddr addr,
|
static void dma_dummy_write(void *opaque, hwaddr addr,
|
||||||
uint64_t val, unsigned size)
|
uint64_t val, unsigned size)
|
||||||
{
|
{
|
||||||
/* Nothing to do. That is only to ensure that
|
/*
|
||||||
* the current DMA acknowledge cycle is completed. */
|
* Nothing to do. That is only to ensure that
|
||||||
|
* the current DMA acknowledge cycle is completed.
|
||||||
|
*/
|
||||||
}
|
}
|
||||||
|
|
||||||
static const MemoryRegionOps dma_dummy_ops = {
|
static const MemoryRegionOps dma_dummy_ops = {
|
||||||
|
@ -109,8 +112,8 @@ static const MemoryRegionOps dma_dummy_ops = {
|
||||||
};
|
};
|
||||||
|
|
||||||
#define MAGNUM_BIOS_SIZE_MAX 0x7e000
|
#define MAGNUM_BIOS_SIZE_MAX 0x7e000
|
||||||
#define MAGNUM_BIOS_SIZE (BIOS_SIZE < MAGNUM_BIOS_SIZE_MAX ? BIOS_SIZE : MAGNUM_BIOS_SIZE_MAX)
|
#define MAGNUM_BIOS_SIZE \
|
||||||
|
(BIOS_SIZE < MAGNUM_BIOS_SIZE_MAX ? BIOS_SIZE : MAGNUM_BIOS_SIZE_MAX)
|
||||||
static void (*real_do_transaction_failed)(CPUState *cpu, hwaddr physaddr,
|
static void (*real_do_transaction_failed)(CPUState *cpu, hwaddr physaddr,
|
||||||
vaddr addr, unsigned size,
|
vaddr addr, unsigned size,
|
||||||
MMUAccessType access_type,
|
MMUAccessType access_type,
|
||||||
|
@ -201,8 +204,9 @@ static void mips_jazz_init(MachineState *machine,
|
||||||
memory_region_add_subregion(address_space, 0xfff00000LL, bios2);
|
memory_region_add_subregion(address_space, 0xfff00000LL, bios2);
|
||||||
|
|
||||||
/* load the BIOS image. */
|
/* load the BIOS image. */
|
||||||
if (bios_name == NULL)
|
if (bios_name == NULL) {
|
||||||
bios_name = BIOS_FILENAME;
|
bios_name = BIOS_FILENAME;
|
||||||
|
}
|
||||||
filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
|
filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
|
||||||
if (filename) {
|
if (filename) {
|
||||||
bios_size = load_image_targphys(filename, 0xfff00000LL,
|
bios_size = load_image_targphys(filename, 0xfff00000LL,
|
||||||
|
@ -229,7 +233,8 @@ static void mips_jazz_init(MachineState *machine,
|
||||||
sysbus_mmio_get_region(sysbus, 0));
|
sysbus_mmio_get_region(sysbus, 0));
|
||||||
memory_region_add_subregion(address_space, 0xf0000000,
|
memory_region_add_subregion(address_space, 0xf0000000,
|
||||||
sysbus_mmio_get_region(sysbus, 1));
|
sysbus_mmio_get_region(sysbus, 1));
|
||||||
memory_region_init_io(dma_dummy, NULL, &dma_dummy_ops, NULL, "dummy_dma", 0x1000);
|
memory_region_init_io(dma_dummy, NULL, &dma_dummy_ops,
|
||||||
|
NULL, "dummy_dma", 0x1000);
|
||||||
memory_region_add_subregion(address_space, 0x8000d000, dma_dummy);
|
memory_region_add_subregion(address_space, 0x8000d000, dma_dummy);
|
||||||
|
|
||||||
/* ISA bus: IO space at 0x90000000, mem space at 0x91000000 */
|
/* ISA bus: IO space at 0x90000000, mem space at 0x91000000 */
|
||||||
|
@ -276,8 +281,9 @@ static void mips_jazz_init(MachineState *machine,
|
||||||
/* Network controller */
|
/* Network controller */
|
||||||
for (n = 0; n < nb_nics; n++) {
|
for (n = 0; n < nb_nics; n++) {
|
||||||
nd = &nd_table[n];
|
nd = &nd_table[n];
|
||||||
if (!nd->model)
|
if (!nd->model) {
|
||||||
nd->model = g_strdup("dp83932");
|
nd->model = g_strdup("dp83932");
|
||||||
|
}
|
||||||
if (strcmp(nd->model, "dp83932") == 0) {
|
if (strcmp(nd->model, "dp83932") == 0) {
|
||||||
qemu_check_nic_model(nd, "dp83932");
|
qemu_check_nic_model(nd, "dp83932");
|
||||||
|
|
||||||
|
@ -338,12 +344,12 @@ static void mips_jazz_init(MachineState *machine,
|
||||||
/* Serial ports */
|
/* Serial ports */
|
||||||
if (serial_hd(0)) {
|
if (serial_hd(0)) {
|
||||||
serial_mm_init(address_space, 0x80006000, 0,
|
serial_mm_init(address_space, 0x80006000, 0,
|
||||||
qdev_get_gpio_in(rc4030, 8), 8000000/16,
|
qdev_get_gpio_in(rc4030, 8), 8000000 / 16,
|
||||||
serial_hd(0), DEVICE_NATIVE_ENDIAN);
|
serial_hd(0), DEVICE_NATIVE_ENDIAN);
|
||||||
}
|
}
|
||||||
if (serial_hd(1)) {
|
if (serial_hd(1)) {
|
||||||
serial_mm_init(address_space, 0x80007000, 0,
|
serial_mm_init(address_space, 0x80007000, 0,
|
||||||
qdev_get_gpio_in(rc4030, 9), 8000000/16,
|
qdev_get_gpio_in(rc4030, 9), 8000000 / 16,
|
||||||
serial_hd(1), DEVICE_NATIVE_ENDIAN);
|
serial_hd(1), DEVICE_NATIVE_ENDIAN);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue