Re: [PATCH 7/8] drivers/staging/vt6656: use ARRAY_SIZE macro inchannel.c

From: Charles Clément
Date: Thu Sep 09 2010 - 05:29:22 EST


I think this definition is not used anywhere in the driver and thus can
be removed.

On Wed, Sep 08, 2010 at 10:27:55PM +0100, Nikitas Angelinas wrote:
> Replace (sizeof(ChannelRuleTab) / sizeof(ChannelRuleTab[0])) with
> ARRAY_SIZE(ChannelRuleTab) in drivers/staging/vt6656/channel.c
>
> Signed-off-by: Nikitas Angelinas <nikitasangelinas@xxxxxxxxx>
> ---
> drivers/staging/vt6656/channel.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/staging/vt6656/channel.c b/drivers/staging/vt6656/channel.c
> index 6ad03e4..a2460ae 100644
> --- a/drivers/staging/vt6656/channel.c
> +++ b/drivers/staging/vt6656/channel.c
> @@ -34,6 +34,7 @@
> *
> */
>
> +#include <linux/kernel.h>
> #include "country.h"
> #include "channel.h"
> #include "rf.h"
> @@ -367,7 +368,7 @@ static struct
> /* 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 */
> };
>
> -#define NUM_RULES (sizeof(ChannelRuleTab) / sizeof(ChannelRuleTab[0]))
> +#define NUM_RULES ARRAY_SIZE(ChannelRuleTab)
>
> /*--------------------- Export function -------------------------*/
> /************************************************************************
> --
> 1.7.2.3
>
> _______________________________________________
> devel mailing list
> devel@xxxxxxxxxxxxxxxxxxxxxx
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

--
Charles Clément
--
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/