diff mbox series

[v2] test/power: fix 32-bit build

Message ID 20180713045103.39862-1-pablo.de.lara.guarch@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series [v2] test/power: fix 32-bit build |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

De Lara Guarch, Pablo July 13, 2018, 4:51 a.m. UTC 
  Compilation issue:

test/test/test_power_acpi_cpufreq.c:556:31:
error: format ‘%lx’ expects argument of type ‘long unsigned int’,
but argument 2 has type ‘uint64_t {aka long long unsigned int}’
-Werror=format=]

  printf("ACPI: Capabilities %lx\n", caps.capabilities);
                             ~~^     ~~~~~~~~~~~~~~~~~
                             %llx

Fixes: 39e38d583075 ("test/power: add unit test for get capabilities API")

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: Radu Nicolau <radu.nicolau@intel.com>
---

v2:
- Added missing include

 test/test/test_power_acpi_cpufreq.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Thomas Monjalon July 13, 2018, 4:12 p.m. UTC | #1 
13/07/2018 06:51, Pablo de Lara:
> Compilation issue:
> 
> test/test/test_power_acpi_cpufreq.c:556:31:
> error: format ‘%lx’ expects argument of type ‘long unsigned int’,
> but argument 2 has type ‘uint64_t {aka long long unsigned int}’
> -Werror=format=]
> 
>   printf("ACPI: Capabilities %lx\n", caps.capabilities);
>                              ~~^     ~~~~~~~~~~~~~~~~~
>                              %llx
> 
> Fixes: 39e38d583075 ("test/power: add unit test for get capabilities API")
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> Acked-by: Radu Nicolau <radu.nicolau@intel.com>

Applied, thanks
diff mbox series

Patch

diff --git a/test/test/test_power_acpi_cpufreq.c b/test/test/test_power_acpi_cpufreq.c
index 6410969e0..22e541d65 100644
--- a/test/test/test_power_acpi_cpufreq.c
+++ b/test/test/test_power_acpi_cpufreq.c
@@ -7,6 +7,7 @@ 
 #include <unistd.h>
 #include <limits.h>
 #include <string.h>
+#include <inttypes.h>
 
 #include "test.h"
 
@@ -553,7 +554,7 @@  test_power_acpi_caps(void)
 		return -1;
 	}
 
-	printf("ACPI: Capabilities %lx\n", caps.capabilities);
+	printf("ACPI: Capabilities %"PRIx64"\n", caps.capabilities);
 
 	rte_power_unset_env();
 	return 0;