]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: zoran: constify struct tvnorm
authorCorentin Labbe <clabbe@baylibre.com>
Fri, 25 Sep 2020 18:30:43 +0000 (20:30 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Thu, 1 Oct 2020 07:57:48 +0000 (09:57 +0200)
The structure tvnorm could be consified.

Signed-off-by: Corentin Labbe <clabbe@baylibre.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/zoran/videocodec.h
drivers/staging/media/zoran/zoran.h
drivers/staging/media/zoran/zoran_card.c
drivers/staging/media/zoran/zoran_device.c
drivers/staging/media/zoran/zr36016.c
drivers/staging/media/zoran/zr36050.c
drivers/staging/media/zoran/zr36060.c

index 553af4ad431e0da1b8c7a761456b531808d2e6df..a049f3c2c68a28a881540b2457e56b22b8be5f95 100644 (file)
@@ -256,7 +256,7 @@ struct videocodec {
        // set compression or decompression (or freeze, stop, standby, etc)
        int (*set_mode)(struct videocodec *codec, int mode);
        // setup picture size and norm (for the codec's video frontend)
-       int (*set_video)(struct videocodec *codec, struct tvnorm *norm,
+       int (*set_video)(struct videocodec *codec, const struct tvnorm *norm,
                         struct vfe_settings *cap, struct vfe_polarity *pol);
        // other control commands, also mmap setup etc.
        int (*control)(struct videocodec *codec, int type, int size, void *data);
index c73b7e7fd5e106ec425b7bdac5ddca9b46cd41d1..727ba67afd5681d16d1f4200c791c210e1b6b944 100644 (file)
@@ -227,7 +227,7 @@ struct card_info {
        } input[BUZ_MAX_INPUT];
 
        v4l2_std_id norms;
-       struct tvnorm *tvn[3];  /* supported TV norms */
+       const struct tvnorm *tvn[3];    /* supported TV norms */
 
        u32 jpeg_int;           /* JPEG interrupt */
        u32 vsync_int;          /* VSYNC interrupt */
@@ -266,7 +266,7 @@ struct zoran {
        u8 initialized;         /* flag if zoran has been correctly initialized */
        int user;               /* number of current users */
        struct card_info card;
-       struct tvnorm *timing;
+       const struct tvnorm *timing;
 
        unsigned short id;      /* number of this device */
        char name[32];          /* name of this device */
index 756705a34e2b82a7ca4592539f54d6049b6eb2df..79783cbb919a898e872b133d4ab7cec7d383e61c 100644 (file)
@@ -277,24 +277,24 @@ static char *codecid_to_modulename(u16 codecid)
 //      u16 Wt, Wa, HStart, HSyncStart, Ht, Ha, VStart;
 // };
 
-static struct tvnorm f50sqpixel = { 944, 768, 83, 880, 625, 576, 16 };
-static struct tvnorm f60sqpixel = { 780, 640, 51, 716, 525, 480, 12 };
-static struct tvnorm f50ccir601 = { 864, 720, 75, 804, 625, 576, 18 };
-static struct tvnorm f60ccir601 = { 858, 720, 57, 788, 525, 480, 16 };
+static const struct tvnorm f50sqpixel = { 944, 768, 83, 880, 625, 576, 16 };
+static const struct tvnorm f60sqpixel = { 780, 640, 51, 716, 525, 480, 12 };
+static const struct tvnorm f50ccir601 = { 864, 720, 75, 804, 625, 576, 18 };
+static const struct tvnorm f60ccir601 = { 858, 720, 57, 788, 525, 480, 16 };
 
-static struct tvnorm f50ccir601_lml33 = { 864, 720, 75 + 34, 804, 625, 576, 18 };
-static struct tvnorm f60ccir601_lml33 = { 858, 720, 57 + 34, 788, 525, 480, 16 };
+static const struct tvnorm f50ccir601_lml33 = { 864, 720, 75 + 34, 804, 625, 576, 18 };
+static const struct tvnorm f60ccir601_lml33 = { 858, 720, 57 + 34, 788, 525, 480, 16 };
 
 /* The DC10 (57/16/50) uses VActive as HSync, so HStart must be 0 */
-static struct tvnorm f50sqpixel_dc10 = { 944, 768, 0, 880, 625, 576, 0 };
-static struct tvnorm f60sqpixel_dc10 = { 780, 640, 0, 716, 525, 480, 12 };
+static const struct tvnorm f50sqpixel_dc10 = { 944, 768, 0, 880, 625, 576, 0 };
+static const struct tvnorm f60sqpixel_dc10 = { 780, 640, 0, 716, 525, 480, 12 };
 
 /*
  * FIXME: I cannot swap U and V in saa7114, so i do one pixel left shift in zoran (75 -> 74)
  * (Maxim Yevtyushkin <max@linuxmedialabs.com>)
  */
-static struct tvnorm f50ccir601_lm33r10 = { 864, 720, 74 + 54, 804, 625, 576, 18 };
-static struct tvnorm f60ccir601_lm33r10 = { 858, 720, 56 + 54, 788, 525, 480, 16 };
+static const struct tvnorm f50ccir601_lm33r10 = { 864, 720, 74 + 54, 804, 625, 576, 18 };
+static const struct tvnorm f60ccir601_lm33r10 = { 858, 720, 56 + 54, 788, 525, 480, 16 };
 
 /*
  * FIXME: The ks0127 seem incapable of swapping U and V, too, which is why I copy Maxim's left
@@ -303,8 +303,8 @@ static struct tvnorm f60ccir601_lm33r10 = { 858, 720, 56 + 54, 788, 525, 480, 16
  * Christer's driver used the unshifted norms, though...
  * /Sam
  */
-static struct tvnorm f50ccir601_avs6eyes = { 864, 720, 74, 804, 625, 576, 18 };
-static struct tvnorm f60ccir601_avs6eyes = { 858, 720, 56, 788, 525, 480, 16 };
+static const struct tvnorm f50ccir601_avs6eyes = { 864, 720, 74, 804, 625, 576, 18 };
+static const struct tvnorm f60ccir601_avs6eyes = { 858, 720, 56, 788, 525, 480, 16 };
 
 static const unsigned short vpx3220_addrs[] = { 0x43, 0x47, I2C_CLIENT_END };
 static const unsigned short saa7110_addrs[] = { 0x4e, 0x4f, I2C_CLIENT_END };
index 4c9f6eafd130ee443bf001facb0ca949d97478c4..fb318449d784b3a03fbf9feeee5875ee2d25dfc9 100644 (file)
@@ -292,7 +292,7 @@ static void zr36057_adjust_vfe(struct zoran *zr, enum zoran_codec_mode mode)
 static void zr36057_set_vfe(struct zoran *zr, int video_width, int video_height,
                            const struct zoran_format *format)
 {
-       struct tvnorm *tvn;
+       const struct tvnorm *tvn;
        unsigned int HStart, HEnd, VStart, VEnd;
        unsigned int DispMode;
        unsigned int VidWinWid, VidWinHt;
@@ -510,7 +510,7 @@ static void init_jpeg_queue(struct zoran *zr)
 
 static void zr36057_set_jpg(struct zoran *zr, enum zoran_codec_mode mode)
 {
-       struct tvnorm *tvn;
+       const struct tvnorm *tvn;
        u32 reg;
 
        tvn = zr->timing;
index a8190005bc4dee1bd498b3fafca098ed25aea568..046cffb92f5f92f744a41cbe40256ad7028d18aa 100644 (file)
@@ -238,8 +238,8 @@ static int zr36016_set_mode(struct videocodec *codec, int mode)
 }
 
 /* set picture size */
-static int zr36016_set_video(struct videocodec *codec, struct tvnorm *norm,
-                  struct vfe_settings *cap, struct vfe_polarity *pol)
+static int zr36016_set_video(struct videocodec *codec, const struct tvnorm *norm,
+                            struct vfe_settings *cap, struct vfe_polarity *pol)
 {
        struct zr36016 *ptr = (struct zr36016 *)codec->data;
 
index f551631c18cdcc878c2647b49ebacaef9d429883..348b9f0ab74c52f3057f243621d7bff8a284fdf9 100644 (file)
@@ -554,8 +554,8 @@ static int zr36050_set_mode(struct videocodec *codec, int mode)
 }
 
 /* set picture size (norm is ignored as the codec doesn't know about it) */
-static int zr36050_set_video(struct videocodec *codec, struct tvnorm *norm,
-                  struct vfe_settings *cap, struct vfe_polarity *pol)
+static int zr36050_set_video(struct videocodec *codec, const struct tvnorm *norm,
+                            struct vfe_settings *cap, struct vfe_polarity *pol)
 {
        struct zr36050 *ptr = (struct zr36050 *)codec->data;
        int size;
index 0a0916f5f8cdc8207a8e053f3dbdf04acc002b4e..773de0e64daf3165d62784e6056c36f9054db1a2 100644 (file)
@@ -477,7 +477,7 @@ static int zr36060_set_mode(struct videocodec *codec, int mode)
 }
 
 /* set picture size (norm is ignored as the codec doesn't know about it) */
-static int zr36060_set_video(struct videocodec *codec, struct tvnorm *norm,
+static int zr36060_set_video(struct videocodec *codec, const struct tvnorm *norm,
                             struct vfe_settings *cap, struct vfe_polarity *pol)
 {
        struct zr36060 *ptr = (struct zr36060 *)codec->data;