diff options
| author | Maximilian Brune <maximilian.brune@9elements.com> | 2023-05-05 15:10:24 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-05-24 11:49:51 +0000 |
| commit | 9d475bf6de63f4e05bba39e4771e0d32c07c0c9b (patch) | |
| tree | 715cafa4621b060250b79cc9a890df603f90c804 /payloads/libpayload/arch/arm | |
| parent | 00d9107e02b0327ae16eb2b08605d9942ae0ab18 (diff) | |
libpayload/arch/x86: Update API handling of CBTABLE handoff
The payload API of coreboot described in
https://www.coreboot.org/Payload_API does not reflect the current
handoff mechanism to hand the coreboot tables off. Therefore the
arguments supplied by coreboot (cbtable) will currently never be parsed
correctly and libpayload has to search for the coreboot tables by
iterating through memory.
This patch removes the old payload API implementation and just takes the
coreboot table pointer from the first argument on the stack.
Tested: started prodrive/atlas with coreinfo payload
Signed-off-by: Maximilian Brune <maximilian.brune@9elements.com>
Change-Id: I51fb0cfc81043cbfe3fc9c8ea0776add2d6a42b2
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74965
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'payloads/libpayload/arch/arm')
| -rw-r--r-- | payloads/libpayload/arch/arm/main.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/payloads/libpayload/arch/arm/main.c b/payloads/libpayload/arch/arm/main.c index 434d946031..8f663e5c2b 100644 --- a/payloads/libpayload/arch/arm/main.c +++ b/payloads/libpayload/arch/arm/main.c @@ -29,11 +29,6 @@ #include <exception.h> #include <libpayload.h> -int main_argc; /**< The argc value to pass to main() */ - -/** The argv value to pass to main() */ -char *main_argv[MAX_ARGC_COUNT]; - /** * This is our C entry function - set up the system * and jump into the payload entry point. @@ -63,7 +58,7 @@ void start_main(void) * In the future we may care about the return value. */ - (void) main(main_argc, (main_argc != 0) ? main_argv : NULL); + (void) main(0, NULL); /* * Returning here will go to the _leave function to return |