[PATCH] acpi: include missing header in <acpi.h>

[PATCH] acpi: include missing header in <acpi.h>

[Ahmad Fatoum]

acpi_sigcmp defined in the header uses memcmp, but no included headers
pull in memcmp, which can lead to compile errors in new code, depending
on include order. Fix this.

Reported-by: @healytpk, https://github.com/saschahauer/barebox/issues/7
Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
---
 include/acpi.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/acpi.h b/include/acpi.h
index 2d5fd3086a89..b8e73b35df99 100644
--- a/include/acpi.h
+++ b/include/acpi.h
@@ -6,6 +6,7 @@
 #ifndef __ACPI_H_
 #define __ACPI_H_
 
+#include <linux/string.h>
 #include <linux/types.h>
 #include <driver.h>
 
-- 
2.20.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] acpi: include missing header in <acpi.h>

[Sascha Hauer]

On Mon, Feb 03, 2020 at 09:21:59AM +0100, Ahmad Fatoum wrote:
> acpi_sigcmp defined in the header uses memcmp, but no included headers
> pull in memcmp, which can lead to compile errors in new code, depending
> on include order. Fix this.
> 
> Reported-by: @healytpk, https://github.com/saschahauer/barebox/issues/7
> Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
> ---
>  include/acpi.h | 1 +
>  1 file changed, 1 insertion(+)

Applied, thanks

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox