Re: [PATCH v4 45/79] linux/if.h linux/hdlc/ioctl.h: move IFNAMSIZ definition to hdlc/ioctl.h

[Frans Klaver]

Hi,

On Thu, Oct 15, 2015 at 7:56 AM, Mikko Rapeli <mikko.rapeli@xxxxxx> wrote:
> And include linux/hdlc/ioctl.h from linux/if.h.

That appears to have already been the case before this patch. You just
add a comment behind the include statement.


> diff --git a/include/uapi/linux/if.h b/include/uapi/linux/if.h
> index 9cf2394..0f98f0a 100644
> --- a/include/uapi/linux/if.h
> +++ b/include/uapi/linux/if.h
> @@ -22,10 +22,8 @@
>  #include <linux/types.h>               /* for "__kernel_caddr_t" et al */
>  #include <linux/socket.h>              /* for "struct sockaddr" et al  */
>  #include <linux/compiler.h>            /* for "__user" et al           */
> -
> -#define        IFNAMSIZ        16
> -#define        IFALIASZ        256
> -#include <linux/hdlc/ioctl.h>
> +#include <linux/hdlc/ioctl.h>           /* for IFNAMSIZ                 */
> +#define        IFALIASZ        256

Thanks,
Frans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/