On 11/25/24 18:34, Jason Andryuk wrote:
On 2024-11-23 13:20, Daniel P. Smith wrote:
Look for a subnode of type `multiboot,ramdisk` within a domain node. If
found, process the reg property for the MB1 module index.

Signed-off-by: Daniel P. Smith <dpsm...@apertussolutions.com>
---
  xen/arch/x86/domain_builder/fdt.c | 25 ++++++++++++++++++++++
  xen/arch/x86/setup.c              | 35 +++++++++++++++++--------------
  2 files changed, 44 insertions(+), 16 deletions(-)

diff --git a/xen/arch/x86/domain_builder/fdt.c b/xen/arch/x86/ domain_builder/fdt.c
index f8ddb11b339e..bc8054a80ec1 100644
--- a/xen/arch/x86/domain_builder/fdt.c
+++ b/xen/arch/x86/domain_builder/fdt.c
@@ -152,6 +152,31 @@ static int __init process_domain_node(
                  if ( ret > 0 )
                      bd->kernel->fdt_cmdline = true;
              }
+
+            continue;
+        }
+        if ( fdt_node_check_compatible(fdt, node, "multiboot,ramdisk") == 0 )

I think
         continue;
     }
     if

should change to

     } else if

?

Yah, I can make it a nested set of ifs.

Also "module,ramdisk"/"module,index"?

Yes, will be updated to new format.

v/r,
dps


Reply via email to