media: atomisp:use ARRAY_SIZE() instead of open coding
authorAllen Pais <allen.lkml@gmail.com>
Wed, 13 Sep 2017 09:15:58 +0000 (05:15 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Sat, 23 Sep 2017 19:08:38 +0000 (15:08 -0400)
The array_length() macro just duplicates ARRAY_SIZE(), so we can
delete it.

Signed-off-by: Allen Pais <allen.lkml@gmail.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css.c

index e882b55968133c1b63e783686b9c679f90cff860..bee30438e6fddcfb91c456041e2e4dce39437fe8 100644 (file)
@@ -451,8 +451,6 @@ static enum ia_css_frame_format yuv422_copy_formats[] = {
        IA_CSS_FRAME_FORMAT_YUYV
 };
 
-#define array_length(array) (sizeof(array)/sizeof(array[0]))
-
 /* Verify whether the selected output format is can be produced
  * by the copy binary given the stream format.
  * */
@@ -468,7 +466,7 @@ verify_copy_out_frame_format(struct ia_css_pipe *pipe)
        switch (pipe->stream->config.input_config.format) {
        case IA_CSS_STREAM_FORMAT_YUV420_8_LEGACY:
        case IA_CSS_STREAM_FORMAT_YUV420_8:
-               for (i=0; i<array_length(yuv420_copy_formats) && !found; i++)
+               for (i=0; i<ARRAY_SIZE(yuv420_copy_formats) && !found; i++)
                        found = (out_fmt == yuv420_copy_formats[i]);
                break;
        case IA_CSS_STREAM_FORMAT_YUV420_10:
@@ -476,7 +474,7 @@ verify_copy_out_frame_format(struct ia_css_pipe *pipe)
                found = (out_fmt == IA_CSS_FRAME_FORMAT_YUV420_16);
                break;
        case IA_CSS_STREAM_FORMAT_YUV422_8:
-               for (i=0; i<array_length(yuv422_copy_formats) && !found; i++)
+               for (i=0; i<ARRAY_SIZE(yuv422_copy_formats) && !found; i++)
                        found = (out_fmt == yuv422_copy_formats[i]);
                break;
        case IA_CSS_STREAM_FORMAT_YUV422_10: