V/r,
Daniel P. Smith
Apertus Solutions, LLC

On 6/13/25 11:13, Alejandro Vallejo wrote:
... in alignment with the new coding style on word splitting for type
names.

This aligns its name with the largely duplicate boot_module struct
in x86. While there's no equivalent to "struct bootmodules" in x86,
changing one and not the other is just confusing. Same with various
comments and function names.

Rather than making a long subfield name even longer, remove the
_bootmodule suffix in the kernel, initrd and dtb subfields.

Not a functional change.

Signed-off-by: Alejandro Vallejo <agarc...@amd.com>
---
  xen/arch/arm/domain_build.c             |  4 +-
  xen/arch/arm/efi/efi-boot.h             |  6 +--
  xen/arch/arm/kernel.c                   |  4 +-
  xen/arch/arm/llc-coloring.c             |  8 ++--
  xen/arch/arm/mmu/setup.c                | 10 ++---
  xen/arch/arm/setup.c                    | 10 ++---
  xen/common/device-tree/bootfdt.c        |  4 +-
  xen/common/device-tree/bootinfo.c       | 52 ++++++++++++-------------
  xen/common/device-tree/dom0less-build.c | 25 ++++++------
  xen/common/device-tree/domain-build.c   |  2 +-
  xen/common/device-tree/kernel.c         | 22 +++++------
  xen/include/xen/bootfdt.h               | 26 ++++++-------
  xen/include/xen/fdt-kernel.h            |  7 ++--
  xen/xsm/xsm_policy.c                    |  2 +-
  14 files changed, 90 insertions(+), 92 deletions(-)


<snip/>

diff --git a/xen/xsm/xsm_policy.c b/xen/xsm/xsm_policy.c
index 7f70d860bd..1f88b4fc5a 100644
--- a/xen/xsm/xsm_policy.c
+++ b/xen/xsm/xsm_policy.c
@@ -68,7 +68,7 @@ int __init xsm_multiboot_policy_init(
  #ifdef CONFIG_HAS_DEVICE_TREE
  int __init xsm_dt_policy_init(void **policy_buffer, size_t *policy_size)
  {
-    struct bootmodule *mod = boot_module_find_by_kind(BOOTMOD_XSM);
+    struct boot_module *mod = boot_module_find_by_kind(BOOTMOD_XSM);
      paddr_t paddr, len;
if ( !mod || !mod->size )

Acked-By: Daniel P. Smith <dpsm...@apertussolutions.com>

Reply via email to