[PATCH v5 0/6] Move {acpi_}device_init() and device_get_class() to common code

Oleksii Kurochko posted 6 patches 1 month, 3 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1727452451.git.oleksii.kurochko@gmail.com
xen/arch/arm/device.c     | 71 +--------------------------------
xen/arch/arm/xen.lds.S    | 14 +------
xen/arch/ppc/xen.lds.S    |  4 ++
xen/arch/riscv/xen.lds.S  |  2 +
xen/arch/x86/xen.lds.S    |  6 ++-
xen/common/Makefile       |  2 +
xen/common/device.c       | 82 +++++++++++++++++++++++++++++++++++++++
xen/include/xen/xen.lds.h | 22 +++++++++++
8 files changed, 120 insertions(+), 83 deletions(-)
create mode 100644 xen/common/device.c
[PATCH v5 0/6] Move {acpi_}device_init() and device_get_class() to common code
Posted by Oleksii Kurochko 1 month, 3 weeks ago
The {acpi_}device_init() and device_get_class() functions are identical across
Arm and RISC-V, and they are likely to remain the same for other architectures
like PPC.
Since there is no architecture-specific logic within these functions, they are
good candidates to be moved to the common codebase.

This patch series refactors the code by moving these functions to the common
directory, reducing code duplication and simplifying future maintenance.

---
Changes in V5:
 - Rename SIMPLE_DECL_SECTION to DECL_SECTION_WITH_LADDR and update the other
   patches correspondingly.
 - Drop macros argument for  ACPI_DEV_INFO and DT_DEV_INFO.
---
Changes in V4:
 - Introduce SIMPLE_DECL_SECTION to cover the cases when an architecture wants
   to use section declaration without specifying an load address.
 - Refactor macors ACPI_DEV_INFO and DT_DEV_INFO and update their defintion.
---
Changes in V3:
 - drop _SECTIONS from the name of the macros ADEV_INFO and DT_DEV_INFO
 - update the definion of macros ADEV_INFO and DT_DEV_INFO
 - drop DEVICE_INIT config.
---
Changes in v2:
 - Introduce macros for definition of ACPI and Device Tree sections.
 - Introduce CONFIG_DEVICE_INIT to make common/device.o compilable only for Arm,
   PPC and RISC-V.
---

Oleksii Kurochko (6):
  xen: introduce DECL_SECTION_WITH_LADDR
  xen: define ACPI and DT device info sections macros
  xen/arm: use {DT,ACPI}_DEV_INFO for device info sections
  xen/ppc: add section for device information in linker script
  xen/riscv: add section for device information in linker script
  xen/common: move device initialization code to common code

 xen/arch/arm/device.c     | 71 +--------------------------------
 xen/arch/arm/xen.lds.S    | 14 +------
 xen/arch/ppc/xen.lds.S    |  4 ++
 xen/arch/riscv/xen.lds.S  |  2 +
 xen/arch/x86/xen.lds.S    |  6 ++-
 xen/common/Makefile       |  2 +
 xen/common/device.c       | 82 +++++++++++++++++++++++++++++++++++++++
 xen/include/xen/xen.lds.h | 22 +++++++++++
 8 files changed, 120 insertions(+), 83 deletions(-)
 create mode 100644 xen/common/device.c

-- 
2.46.1
Re: [PATCH v5 0/6] Move {acpi_}device_init() and device_get_class() to common code
Posted by oleksii.kurochko@gmail.com 1 month, 2 weeks ago
Hello everyone,

Except the comments ( which Jan wrote could be adjusted while
committing ) for patch 1 everything is properly Acked/Reviewed ( if I
am not missing something ).

I also checked if the patch series rebase/apply on top of current
staging and I hadn't any issue.

Could someone please tell me what should be my next steps? Should I
resend this patch series in case I miss something?

Thanks.

~ Oleksii

On Fri, 2024-09-27 at 18:32 +0200, Oleksii Kurochko wrote:
> The {acpi_}device_init() and device_get_class() functions are
> identical across
> Arm and RISC-V, and they are likely to remain the same for other
> architectures
> like PPC.
> Since there is no architecture-specific logic within these functions,
> they are
> good candidates to be moved to the common codebase.
> 
> This patch series refactors the code by moving these functions to the
> common
> directory, reducing code duplication and simplifying future
> maintenance.
> 
> ---
> Changes in V5:
>  - Rename SIMPLE_DECL_SECTION to DECL_SECTION_WITH_LADDR and update
> the other
>    patches correspondingly.
>  - Drop macros argument for  ACPI_DEV_INFO and DT_DEV_INFO.
> ---
> Changes in V4:
>  - Introduce SIMPLE_DECL_SECTION to cover the cases when an
> architecture wants
>    to use section declaration without specifying an load address.
>  - Refactor macors ACPI_DEV_INFO and DT_DEV_INFO and update their
> defintion.
> ---
> Changes in V3:
>  - drop _SECTIONS from the name of the macros ADEV_INFO and
> DT_DEV_INFO
>  - update the definion of macros ADEV_INFO and DT_DEV_INFO
>  - drop DEVICE_INIT config.
> ---
> Changes in v2:
>  - Introduce macros for definition of ACPI and Device Tree sections.
>  - Introduce CONFIG_DEVICE_INIT to make common/device.o compilable
> only for Arm,
>    PPC and RISC-V.
> ---
> 
> Oleksii Kurochko (6):
>   xen: introduce DECL_SECTION_WITH_LADDR
>   xen: define ACPI and DT device info sections macros
>   xen/arm: use {DT,ACPI}_DEV_INFO for device info sections
>   xen/ppc: add section for device information in linker script
>   xen/riscv: add section for device information in linker script
>   xen/common: move device initialization code to common code
> 
>  xen/arch/arm/device.c     | 71 +--------------------------------
>  xen/arch/arm/xen.lds.S    | 14 +------
>  xen/arch/ppc/xen.lds.S    |  4 ++
>  xen/arch/riscv/xen.lds.S  |  2 +
>  xen/arch/x86/xen.lds.S    |  6 ++-
>  xen/common/Makefile       |  2 +
>  xen/common/device.c       | 82
> +++++++++++++++++++++++++++++++++++++++
>  xen/include/xen/xen.lds.h | 22 +++++++++++
>  8 files changed, 120 insertions(+), 83 deletions(-)
>  create mode 100644 xen/common/device.c
> 
Re: [PATCH v5 0/6] Move {acpi_}device_init() and device_get_class() to common code
Posted by Jan Beulich 1 month, 2 weeks ago
On 08.10.2024 17:25, oleksii.kurochko@gmail.com wrote:
> Except the comments ( which Jan wrote could be adjusted while
> committing ) for patch 1 everything is properly Acked/Reviewed ( if I
> am not missing something ).

I didn't spot a PPC ack so far; did I miss it?

Jan
Re: [PATCH v5 0/6] Move {acpi_}device_init() and device_get_class() to common code
Posted by oleksii.kurochko@gmail.com 1 month, 2 weeks ago
On Tue, 2024-10-08 at 17:30 +0200, Jan Beulich wrote:
> On 08.10.2024 17:25, oleksii.kurochko@gmail.com wrote:
> > Except the comments ( which Jan wrote could be adjusted while
> > committing ) for patch 1 everything is properly Acked/Reviewed ( if
> > I
> > am not missing something ).
> 
> I didn't spot a PPC ack so far; did I miss it?
Yeah, you are right I confused ( I thought that he gave his Ack before
and I just forgot to add it ) with another one patch where Shawn gave
his Ack. I will ask Shawn to look at patch 1.

~ Oleksii