linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] media: dvb-frontends: Remove unused SIZEOF_ARRAY
@ 2013-08-31 19:31 Christoph Jaeger
  2013-09-05 20:49 ` Sakari Ailus
  0 siblings, 1 reply; 2+ messages in thread
From: Christoph Jaeger @ 2013-08-31 19:31 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: linux-media, Christoph Jaeger

SIZEOF_ARRAY is not used (anymore). Besides, ARRAY_SIZE, defined in
include/linux/kernel.h, should be used rather than explicitly coding some
variant of it.

Signed-off-by: Christoph Jaeger <christophjaeger@linux.com>
---
 drivers/media/dvb-frontends/drxd_hard.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/media/dvb-frontends/drxd_hard.c b/drivers/media/dvb-frontends/drxd_hard.c
index 9a213479..cbd7c92 100644
--- a/drivers/media/dvb-frontends/drxd_hard.c
+++ b/drivers/media/dvb-frontends/drxd_hard.c
@@ -46,10 +46,6 @@
 #define DRX_I2C_MODEFLAGS     0xC0
 #define DRX_I2C_FLAGS         0xF0
 
-#ifndef SIZEOF_ARRAY
-#define SIZEOF_ARRAY(array) (sizeof((array))/sizeof((array)[0]))
-#endif
-
 #define DEFAULT_LOCK_TIMEOUT    1100
 
 #define DRX_CHANNEL_AUTO 0
-- 
1.8.3.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] media: dvb-frontends: Remove unused SIZEOF_ARRAY
  2013-08-31 19:31 [PATCH] media: dvb-frontends: Remove unused SIZEOF_ARRAY Christoph Jaeger
@ 2013-09-05 20:49 ` Sakari Ailus
  0 siblings, 0 replies; 2+ messages in thread
From: Sakari Ailus @ 2013-09-05 20:49 UTC (permalink / raw)
  To: Christoph Jaeger; +Cc: Mauro Carvalho Chehab, linux-media

On Sat, Aug 31, 2013 at 09:31:07PM +0200, Christoph Jaeger wrote:
> SIZEOF_ARRAY is not used (anymore). Besides, ARRAY_SIZE, defined in
> include/linux/kernel.h, should be used rather than explicitly coding some
> variant of it.
> 
> Signed-off-by: Christoph Jaeger <christophjaeger@linux.com>

Reviewed-by: Sakari Ailus <sakari.ailus@iki.fi>

-- 
Sakari Ailus
e-mail: sakari.ailus@iki.fi	XMPP: sailus@retiisi.org.uk

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-09-05 20:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-31 19:31 [PATCH] media: dvb-frontends: Remove unused SIZEOF_ARRAY Christoph Jaeger
2013-09-05 20:49 ` Sakari Ailus

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).