[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK9yfHye4YShGSu9fBqgAP+cEpw-fedpJ7jUJpj+jwo0_V4tng@mail.gmail.com>
Date: Sat, 9 Jun 2012 18:15:49 +0530
From: Sachin Kamat <sachin.kamat@...aro.org>
To: Devendra Naga <devendra.aaru@...il.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Sage Ahn <syahn@...semi.com>, Ben Chan <benchan@...omium.org>,
Paul Stewart <pstew@...omium.org>,
Geert Uytterhoeven <geert@...ux-m68k.org>,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging/gdm72xx: Remove version.h includes
Hi Devendra,
I have already submitted a similar patch for the same.
http://permalink.gmane.org/gmane.linux.drivers.driver-project.devel/27044
Regards
Sachin
On 9 June 2012 10:20, Devendra Naga <devendra.aaru@...il.com> wrote:
> here are the version.h warnings that are reported when ran with
> make versioncheck
>
> /home/devendra/linux/drivers/staging/gdm72xx/gdm_qos.c: 14 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/gdm_sdio.c: 17 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/gdm_usb.c: 15 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/gdm_wimax.c: 14 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/gdm_wimax.h: 18 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/netlink_k.c: 14 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/sdio_boot.c: 15 linux/version.h not needed.
> /home/devendra/linux/drivers/staging/gdm72xx/usb_boot.c: 16 linux/version.h not needed.
>
> Signed-off-by: Devendra Naga <devendra.aaru@...il.com>
> ---
> drivers/staging/gdm72xx/gdm_qos.c | 1 -
> drivers/staging/gdm72xx/gdm_sdio.c | 1 -
> drivers/staging/gdm72xx/gdm_usb.c | 1 -
> drivers/staging/gdm72xx/gdm_wimax.c | 1 -
> drivers/staging/gdm72xx/gdm_wimax.h | 1 -
> drivers/staging/gdm72xx/netlink_k.c | 1 -
> drivers/staging/gdm72xx/sdio_boot.c | 1 -
> drivers/staging/gdm72xx/usb_boot.c | 1 -
> 8 files changed, 8 deletions(-)
>
> diff --git a/drivers/staging/gdm72xx/gdm_qos.c b/drivers/staging/gdm72xx/gdm_qos.c
> index 0217680..e7134e2 100644
> --- a/drivers/staging/gdm72xx/gdm_qos.c
> +++ b/drivers/staging/gdm72xx/gdm_qos.c
> @@ -11,7 +11,6 @@
> * GNU General Public License for more details.
> */
>
> -#include <linux/version.h>
> #include <linux/etherdevice.h>
> #include <asm/byteorder.h>
>
> diff --git a/drivers/staging/gdm72xx/gdm_sdio.c b/drivers/staging/gdm72xx/gdm_sdio.c
> index e1a3dd2..5e19b09 100644
> --- a/drivers/staging/gdm72xx/gdm_sdio.c
> +++ b/drivers/staging/gdm72xx/gdm_sdio.c
> @@ -14,7 +14,6 @@
> #include <linux/module.h>
> #include <linux/init.h>
> #include <linux/kernel.h>
> -#include <linux/version.h>
>
> #include <linux/mmc/core.h>
> #include <linux/mmc/card.h>
> diff --git a/drivers/staging/gdm72xx/gdm_usb.c b/drivers/staging/gdm72xx/gdm_usb.c
> index 1e9dc0d..880f67d 100644
> --- a/drivers/staging/gdm72xx/gdm_usb.c
> +++ b/drivers/staging/gdm72xx/gdm_usb.c
> @@ -12,7 +12,6 @@
> */
>
> #include <linux/module.h>
> -#include <linux/version.h>
> #include <linux/kernel.h>
> #include <linux/usb.h>
> #include <asm/byteorder.h>
> diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging/gdm72xx/gdm_wimax.c
> index f1936b9..110bdbc 100644
> --- a/drivers/staging/gdm72xx/gdm_wimax.c
> +++ b/drivers/staging/gdm72xx/gdm_wimax.c
> @@ -11,7 +11,6 @@
> * GNU General Public License for more details.
> */
>
> -#include <linux/version.h>
> #include <linux/etherdevice.h>
> #include <asm/byteorder.h>
> #include <linux/ip.h>
> diff --git a/drivers/staging/gdm72xx/gdm_wimax.h b/drivers/staging/gdm72xx/gdm_wimax.h
> index 023e649..6ec0ab4 100644
> --- a/drivers/staging/gdm72xx/gdm_wimax.h
> +++ b/drivers/staging/gdm72xx/gdm_wimax.h
> @@ -15,7 +15,6 @@
> #define __GDM_WIMAX_H__
>
> #include <linux/netdevice.h>
> -#include <linux/version.h>
> #include <linux/types.h>
> #include "wm_ioctl.h"
> #if defined(CONFIG_WIMAX_GDM72XX_QOS)
> diff --git a/drivers/staging/gdm72xx/netlink_k.c b/drivers/staging/gdm72xx/netlink_k.c
> index 292af0f..1c74b08 100644
> --- a/drivers/staging/gdm72xx/netlink_k.c
> +++ b/drivers/staging/gdm72xx/netlink_k.c
> @@ -11,7 +11,6 @@
> * GNU General Public License for more details.
> */
>
> -#include <linux/version.h>
> #include <linux/module.h>
> #include <linux/etherdevice.h>
> #include <linux/netlink.h>
> diff --git a/drivers/staging/gdm72xx/sdio_boot.c b/drivers/staging/gdm72xx/sdio_boot.c
> index 6ff4dc3..760efee 100644
> --- a/drivers/staging/gdm72xx/sdio_boot.c
> +++ b/drivers/staging/gdm72xx/sdio_boot.c
> @@ -12,7 +12,6 @@
> */
>
> #include <linux/module.h>
> -#include <linux/version.h>
> #include <linux/kernel.h>
> #include <linux/init.h>
> #include <linux/mm.h>
> diff --git a/drivers/staging/gdm72xx/usb_boot.c b/drivers/staging/gdm72xx/usb_boot.c
> index 5a0e030..c163bcc 100644
> --- a/drivers/staging/gdm72xx/usb_boot.c
> +++ b/drivers/staging/gdm72xx/usb_boot.c
> @@ -13,7 +13,6 @@
>
> #include <linux/uaccess.h>
> #include <linux/module.h>
> -#include <linux/version.h>
> #include <linux/kernel.h>
> #include <linux/mm.h>
> #include <linux/usb.h>
> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists