From: Dan Carpenter Date: Tue, 14 Aug 2012 05:51:20 +0000 (-0300) Subject: [media] it913x-fe: use ARRAY_SIZE() as a cleanup X-Git-Tag: v3.7-rc2~24^2~490 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5359805193dc83e1aab2f8e230d57c649363c5c5;p=platform%2Fkernel%2Flinux-3.10.git [media] it913x-fe: use ARRAY_SIZE() as a cleanup This code looks suspicious, but it turns out that "nv" is an array of u8 so sizeof() is the same as ARRAY_SIZE(). Using ARRAY_SIZE() is more readable though. Signed-off-by: Dan Carpenter Cc: Malcolm Priestley Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/dvb-frontends/it913x-fe.c b/drivers/media/dvb-frontends/it913x-fe.c index 708cbf1..6e1c6eb 100644 --- a/drivers/media/dvb-frontends/it913x-fe.c +++ b/drivers/media/dvb-frontends/it913x-fe.c @@ -199,7 +199,7 @@ static int it913x_init_tuner(struct it913x_fe_state *state) if (reg < 0) return -ENODEV; - else if (reg < sizeof(nv)) + else if (reg < ARRAY_SIZE(nv)) nv_val = nv[reg]; else nv_val = 2;