Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/33329 )
Change subject: device/cpu_device.c: Use 'unsigned int' instead of 'unsigned' ......................................................................
device/cpu_device.c: Use 'unsigned int' instead of 'unsigned'
Change-Id: I9f96d8c8e6cf2715e62ab3715da83740db8cce40 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/33329 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Arthur Heymans arthur@aheymans.xyz Reviewed-by: Vlado Cibic --- M src/device/cpu_device.c 1 file changed, 4 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved Vlado Cibic: Looks good to me, but someone else must approve
diff --git a/src/device/cpu_device.c b/src/device/cpu_device.c index b07b555..f59cce3 100644 --- a/src/device/cpu_device.c +++ b/src/device/cpu_device.c @@ -17,7 +17,7 @@ #include <device/device.h> #include <console/console.h>
-struct device *add_cpu_device(struct bus *cpu_bus, unsigned apic_id, +struct device *add_cpu_device(struct bus *cpu_bus, unsigned int apic_id, int enabled) { struct device_path cpu_path; @@ -42,8 +42,9 @@ return cpu; }
-void set_cpu_topology(struct device *cpu, unsigned node, unsigned package, - unsigned core, unsigned thread) +void set_cpu_topology(struct device *cpu, unsigned int node, + unsigned int package, unsigned int core, + unsigned int thread) { cpu->path.apic.node_id = node; cpu->path.apic.package_id = package;