ACK: [PATCH] mtrr: remove ioctl header include, it is redundant

ivanhu ivan.hu at canonical.com
Mon Apr 12 06:48:09 UTC 2021



On 4/8/21 6:21 PM, 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: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list