[media] vivid: don't use more than 1024 bytes of stack
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 5 Jun 2015 10:24:22 +0000 (07:24 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 5 Jun 2015 10:41:54 +0000 (07:41 -0300)
Remove the following compilation warnings:

drivers/media/platform/vivid/vivid-tpg.c: In function 'tpg_gen_text':
drivers/media/platform/vivid/vivid-tpg.c:1562:1: warning: the frame size of 1308 bytes is larger than 1024 bytes [-Wframe-larger-than=]
 }
 ^

This seems to be due to some bad optimization done by gcc.

Moving the for() loop to happen inside the macro solves the
issue.

While here, fix CodingStyle at the switch().

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
drivers/media/platform/vivid/vivid-tpg.c

index b1147f2df26c55173e260d21e4578bb898d9e4bb..7a3ed580626abe5bd6e27a486be0802279a9088c 100644 (file)
@@ -1492,12 +1492,10 @@ void tpg_gen_text(const struct tpg_data *tpg, u8 *basep[TPG_MAX_PLANES][2],
        else if (tpg->field == V4L2_FIELD_SEQ_TB || tpg->field == V4L2_FIELD_SEQ_BT)
                div = 2;
 
-       for (p = 0; p < tpg->planes; p++) {
-               unsigned vdiv = tpg->vdownsampling[p];
-               unsigned hdiv = tpg->hdownsampling[p];
-
-               /* Print text */
-#define PRINTSTR(PIXTYPE) do { \
+       /* Print text */
+#define PRINTSTR(PIXTYPE) for (p = 0; p < tpg->planes; p++) {  \
+       unsigned vdiv = tpg->vdownsampling[p];  \
+       unsigned hdiv = tpg->hdownsampling[p];  \
        PIXTYPE fg;     \
        PIXTYPE bg;     \
        memcpy(&fg, tpg->textfg[p], sizeof(PIXTYPE));   \
@@ -1548,16 +1546,19 @@ void tpg_gen_text(const struct tpg_data *tpg, u8 *basep[TPG_MAX_PLANES][2],
        }       \
 } while (0)
 
-               switch (tpg->twopixelsize[p]) {
-               case 2:
-                       PRINTSTR(u8); break;
-               case 4:
-                       PRINTSTR(u16); break;
-               case 6:
-                       PRINTSTR(x24); break;
-               case 8:
-                       PRINTSTR(u32); break;
-               }
+       switch (tpg->twopixelsize[p]) {
+       case 2:
+               PRINTSTR(u8);
+               break;
+       case 4:
+               PRINTSTR(u16);
+               break;
+       case 6:
+               PRINTSTR(x24);
+               break;
+       case 8:
+               PRINTSTR(u32);
+               break;
        }
 }