ACK: [PATCH] mtrr: remove ioctl header include, it is redundant
Alex Hung
alex.hung at canonical.com
Thu Apr 8 19:02:07 UTC 2021
On 2021-04-08 4:21 a.m., Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> The ioctl mtrr interface was removed in commit 0a10760a877a
> ("read /proc/mtrr rather than use ioctl() interface") hence the
> include of <sys/ioctl.h> is redundant and can be removed.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/bios/mtrr/mtrr.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/bios/mtrr/mtrr.c b/src/bios/mtrr/mtrr.c
> index 94905c24..1968edc5 100644
> --- a/src/bios/mtrr/mtrr.c
> +++ b/src/bios/mtrr/mtrr.c
> @@ -29,7 +29,6 @@
> #include <stdbool.h>
> #include <sys/types.h>
> #include <sys/stat.h>
> -#include <sys/ioctl.h>
> #include <fcntl.h>
> #include <unistd.h>
> #include <ctype.h>
>
Acked-by: Alex Hung <alex.hung at canonical.com>
More information about the fwts-devel
mailing list