hw/loongarch/virt: rename PCH_PIC_IRQ_OFFSET with VIRT_GSI_BASE

In theory gsi base can start from 0 on loongarch virt machine,
however gsi base is hard-coded in linux kernel loongarch system,
else system fails to boot.

This patch renames macro PCH_PIC_IRQ_OFFSET with VIRT_GSI_BASE,
keeps value unchanged. GSI base is common concept in acpi spec
and easy to understand.

Signed-off-by: Bibo Mao <maobibo@loongson.cn>
---
 hw/loongarch/acpi-build.c  |  2 +-
 hw/loongarch/virt.c        |  8 ++++----
 include/hw/pci-host/ls7a.h | 17 +++++++++--------
 3 files changed, 14 insertions(+), 13 deletions(-)

在 2022/12/28 上午11:07, Bibo Mao 写道:
> In theory gsi base can start from 0 on loongarch virt machine,
> however gsi base is hard-coded in linux kernel loongarch system,
> else system fails to boot.
>
> This patch renames macro PCH_PIC_IRQ_OFFSET with VIRT_GSI_BASE,
> keeps value unchanged. GSI base is common concept in acpi spec
> and easy to understand.
>
> Signed-off-by: Bibo Mao <maobibo@loongson.cn>
> ---
>   hw/loongarch/acpi-build.c  |  2 +-
>   hw/loongarch/virt.c        |  8 ++++----
>   include/hw/pci-host/ls7a.h | 17 +++++++++--------
>   3 files changed, 14 insertions(+), 13 deletions(-)

Reviewed-by: Song Gao <gaosong@loongson.cn>

Thanks.
Song Gao
> diff --git a/hw/loongarch/acpi-build.c b/hw/loongarch/acpi-build.c
> index c2b237736d..33e04e4b76 100644
> --- a/hw/loongarch/acpi-build.c
> +++ b/hw/loongarch/acpi-build.c
> @@ -272,7 +272,7 @@ static void build_pci_device_aml(Aml *scope, LoongArchMachineState *lams)
>           .pio.size    = VIRT_PCI_IO_SIZE,
>           .ecam.base   = VIRT_PCI_CFG_BASE,
>           .ecam.size   = VIRT_PCI_CFG_SIZE,
> -        .irq         = PCH_PIC_IRQ_OFFSET + VIRT_DEVICE_IRQS,
> +        .irq         = VIRT_GSI_BASE + VIRT_DEVICE_IRQS,
>           .bus         = lams->pci_bus,
>       };
>   
> diff --git a/hw/loongarch/virt.c b/hw/loongarch/virt.c
> index c8a495ea30..3754e2151f 100644
> --- a/hw/loongarch/virt.c
> +++ b/hw/loongarch/virt.c
> @@ -432,7 +432,7 @@ static DeviceState *create_acpi_ged(DeviceState *pch_pic, LoongArchMachineState
>       sysbus_mmio_map(SYS_BUS_DEVICE(dev), 2, VIRT_GED_REG_ADDR);
>   
>       sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0,
> -                       qdev_get_gpio_in(pch_pic, VIRT_SCI_IRQ - PCH_PIC_IRQ_OFFSET));
> +                       qdev_get_gpio_in(pch_pic, VIRT_SCI_IRQ - VIRT_GSI_BASE));
>       sysbus_realize_and_unref(SYS_BUS_DEVICE(dev), &error_fatal);
>       return dev;
>   }
> @@ -452,7 +452,7 @@ static DeviceState *create_platform_bus(DeviceState *pch_pic)
>   
>       sysbus = SYS_BUS_DEVICE(dev);
>       for (i = 0; i < VIRT_PLATFORM_BUS_NUM_IRQS; i++) {
> -        irq = VIRT_PLATFORM_BUS_IRQ - PCH_PIC_IRQ_OFFSET + i;
> +        irq = VIRT_PLATFORM_BUS_IRQ - VIRT_GSI_BASE + i;
>           sysbus_connect_irq(sysbus, i, qdev_get_gpio_in(pch_pic, irq));
>       }
>   
> @@ -509,7 +509,7 @@ static void loongarch_devices_init(DeviceState *pch_pic, LoongArchMachineState *
>   
>       serial_mm_init(get_system_memory(), VIRT_UART_BASE, 0,
>                      qdev_get_gpio_in(pch_pic,
> -                                    VIRT_UART_IRQ - PCH_PIC_IRQ_OFFSET),
> +                                    VIRT_UART_IRQ - VIRT_GSI_BASE),
>                      115200, serial_hd(0), DEVICE_LITTLE_ENDIAN);
>       fdt_add_uart_node(lams);
>   
> @@ -531,7 +531,7 @@ static void loongarch_devices_init(DeviceState *pch_pic, LoongArchMachineState *
>       create_unimplemented_device("pci-dma-cfg", 0x1001041c, 0x4);
>       sysbus_create_simple("ls7a_rtc", VIRT_RTC_REG_BASE,
>                            qdev_get_gpio_in(pch_pic,
> -                         VIRT_RTC_IRQ - PCH_PIC_IRQ_OFFSET));
> +                         VIRT_RTC_IRQ - VIRT_GSI_BASE));
>       fdt_add_rtc_node(lams);
>   
>       pm_mem = g_new(MemoryRegion, 1);
> diff --git a/include/hw/pci-host/ls7a.h b/include/hw/pci-host/ls7a.h
> index df7fa55a30..194aac905e 100644
> --- a/include/hw/pci-host/ls7a.h
> +++ b/include/hw/pci-host/ls7a.h
> @@ -28,24 +28,25 @@
>   #define VIRT_PCH_MSI_ADDR_LOW    0x2FF00000UL
>   
>   /*
> - * According to the kernel pch irq start from 64 offset
> - * 0 ~ 16 irqs used for non-pci device while 16 ~ 64 irqs
> - * used for pci device.
> + * GSI_BASE is hard-coded with 64 in linux kernel, else kernel fails to boot
> + * 0  - 15  GSI for ISA devices even if there is no ISA devices
> + * 16 - 63  GSI for CPU devices such as timers/perf monitor etc
> + * 64 -     GSI for external devices
>    */
> -#define PCH_PIC_IRQ_OFFSET       64
> +#define VIRT_GSI_BASE            64
>   #define VIRT_DEVICE_IRQS         16
>   #define VIRT_PCI_IRQS            48
> -#define VIRT_UART_IRQ            (PCH_PIC_IRQ_OFFSET + 2)
> +#define VIRT_UART_IRQ            (VIRT_GSI_BASE + 2)
>   #define VIRT_UART_BASE           0x1fe001e0
>   #define VIRT_UART_SIZE           0X100
> -#define VIRT_RTC_IRQ             (PCH_PIC_IRQ_OFFSET + 3)
> +#define VIRT_RTC_IRQ             (VIRT_GSI_BASE + 3)
>   #define VIRT_MISC_REG_BASE       (VIRT_PCH_REG_BASE + 0x00080000)
>   #define VIRT_RTC_REG_BASE        (VIRT_MISC_REG_BASE + 0x00050100)
>   #define VIRT_RTC_LEN             0x100
> -#define VIRT_SCI_IRQ             (PCH_PIC_IRQ_OFFSET + 4)
> +#define VIRT_SCI_IRQ             (VIRT_GSI_BASE + 4)
>   
>   #define VIRT_PLATFORM_BUS_BASEADDRESS   0x16000000
>   #define VIRT_PLATFORM_BUS_SIZE          0x2000000
>   #define VIRT_PLATFORM_BUS_NUM_IRQS      2
> -#define VIRT_PLATFORM_BUS_IRQ           69
> +#define VIRT_PLATFORM_BUS_IRQ           (VIRT_GSI_BASE + 5)
>   #endif