[v2] Add FDT table support with acpi ged pm register

[PATCH v2 0/2] Add FDT table support with acpi ged pm register

Posted by Bibo Mao 1 year, 5 months ago

ACPI ged is used for power management on LoongArch virt platform, in
general it is parsed from acpi table. However if system boot directly from
elf kernel, no UEFI bios is provided and acpi table cannot be used also.

Here acpi ged pm register is exposed with FDT table, it is compatbile
with syscon method in FDT table, only that acpi ged pm register is accessed
with 8-bit mode, rather with 32-bit mode.

---
v1 ... v2:
  1. Modify name of macro for acpi ged register from ACPI spec, and also add
     comments for macro definition.
---
Bibo Mao (2):
  acpi: ged: Add macro for acpi sleep control register
  hw/loongarch/virt: Add FDT table support with acpi ged pm register

 hw/acpi/generic_event_device.c         |  6 ++--
 hw/i386/acpi-microvm.c                 |  2 +-
 hw/loongarch/acpi-build.c              |  2 +-
 hw/loongarch/virt.c                    | 39 ++++++++++++++++++++++++++
 include/hw/acpi/generic_event_device.h |  9 ++++--
 5 files changed, 51 insertions(+), 7 deletions(-)


base-commit: a66f28df650166ae8b50c992eea45e7b247f4143
-- 
2.39.3

Re: [PATCH v2 0/2] Add FDT table support with acpi ged pm register

Posted by gaosong 1 year, 5 months ago


在 2024/9/11 上午11:09, Bibo Mao 写道:
> ACPI ged is used for power management on LoongArch virt platform, in
> general it is parsed from acpi table. However if system boot directly from
> elf kernel, no UEFI bios is provided and acpi table cannot be used also.
> 
> Here acpi ged pm register is exposed with FDT table, it is compatbile
> with syscon method in FDT table, only that acpi ged pm register is accessed
> with 8-bit mode, rather with 32-bit mode.
> 
> ---
> v1 ... v2:
>    1. Modify name of macro for acpi ged register from ACPI spec, and also add
>       comments for macro definition.
> ---
> Bibo Mao (2):
>    acpi: ged: Add macro for acpi sleep control register
>    hw/loongarch/virt: Add FDT table support with acpi ged pm register
> 


>   hw/acpi/generic_event_device.c         |  6 ++--
>   hw/i386/acpi-microvm.c                 |  2 +-
>   hw/loongarch/acpi-build.c              |  2 +-
>   hw/loongarch/virt.c                    | 39 ++++++++++++++++++++++++++
>   include/hw/acpi/generic_event_device.h |  9 ++++--
>   5 files changed, 51 insertions(+), 7 deletions(-)
> 
> 
> base-commit: a66f28df650166ae8b50c992eea45e7b247f4143
> Reviewed-by: Song Gao <gaosong@loongson.cn>

Applied series to loongarch-next

Thanks
Song Gao

Re: [PATCH v2 0/2] Add FDT table support with acpi ged pm register

Posted by maobibo 1 year, 5 months ago

Song,

On 2024/9/12 下午7:35, gaosong wrote:
> 
> 
> 在 2024/9/11 上午11:09, Bibo Mao 写道:
>> ACPI ged is used for power management on LoongArch virt platform, in
>> general it is parsed from acpi table. However if system boot directly 
>> from
>> elf kernel, no UEFI bios is provided and acpi table cannot be used also.
>>
>> Here acpi ged pm register is exposed with FDT table, it is compatbile
>> with syscon method in FDT table, only that acpi ged pm register is 
>> accessed
>> with 8-bit mode, rather with 32-bit mode.
>>
>> ---
>> v1 ... v2:
>>    1. Modify name of macro for acpi ged register from ACPI spec, and 
>> also add
>>       comments for macro definition.
>> ---
>> Bibo Mao (2):
>>    acpi: ged: Add macro for acpi sleep control register
>>    hw/loongarch/virt: Add FDT table support with acpi ged pm register
>>
> 
> 
>>   hw/acpi/generic_event_device.c         |  6 ++--
>>   hw/i386/acpi-microvm.c                 |  2 +-
>>   hw/loongarch/acpi-build.c              |  2 +-
>>   hw/loongarch/virt.c                    | 39 ++++++++++++++++++++++++++
>>   include/hw/acpi/generic_event_device.h |  9 ++++--
>>   5 files changed, 51 insertions(+), 7 deletions(-)
>>
>>
>> base-commit: a66f28df650166ae8b50c992eea45e7b247f4143
>> Reviewed-by: Song Gao <gaosong@loongson.cn>
> 
> Applied series to loongarch-next

It is not urgent and it needs approval from other maintainers :)

Regards
Bibo Mao
> 
> Thanks
> Song Gao
> 
>