diff mbox series

[for-4.15,1/3] hvmloader: do not include inttypes.h

Message ID 20210226085908.21254-2-roger.pau@citrix.com (mailing list archive)
State Superseded
Headers show
Series firmware: fix build on Alpine | expand

Commit Message

Roger Pau Monne Feb. 26, 2021, 8:59 a.m. UTC 
elfstructs.h doesn't require anything from inttypes.h: it's more
appropriate to include stdint.h instead which contains the type
declarations required for the ELF types.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 tools/firmware/hvmloader/32bitbios_support.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jan Beulich Feb. 26, 2021, 9:11 a.m. UTC | #1 
On 26.02.2021 09:59, Roger Pau Monne wrote:
> elfstructs.h doesn't require anything from inttypes.h: it's more
> appropriate to include stdint.h instead which contains the type
> declarations required for the ELF types.
> 
> Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>

Let's go with Andrew's slightly larger change here.

Jan
diff mbox series

Patch

diff --git a/tools/firmware/hvmloader/32bitbios_support.c b/tools/firmware/hvmloader/32bitbios_support.c
index e726946a7b..d1ead1ec11 100644
--- a/tools/firmware/hvmloader/32bitbios_support.c
+++ b/tools/firmware/hvmloader/32bitbios_support.c
@@ -20,7 +20,7 @@ 
  * this program; If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include <inttypes.h>
+#include <stdint.h>
 #include <xen/libelf/elfstructs.h>
 #ifdef __sun__
 #include <sys/machelf.h>