media: dvb-usb: m920x: make read-only arrays static const
authorColin Ian King <colin.i.king@gmail.com>
Wed, 30 Nov 2022 18:04:58 +0000 (18:04 +0000)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 2 Dec 2022 14:36:44 +0000 (15:36 +0100)
Don't populate the arrays on the stack, instead make them static
const. Also makes the object code smaller.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/usb/dvb-usb/m920x.c

index 548199cd86f60be48aa9f3d80594ece8747a328d..fea5bcf72a31a608ac65f4945ce53cc89c44860f 100644 (file)
@@ -485,14 +485,14 @@ static int m920x_identify_state(struct usb_device *udev,
 static int m920x_mt352_demod_init(struct dvb_frontend *fe)
 {
        int ret;
-       u8 config[] = { CONFIG, 0x3d };
-       u8 clock[] = { CLOCK_CTL, 0x30 };
-       u8 reset[] = { RESET, 0x80 };
-       u8 adc_ctl[] = { ADC_CTL_1, 0x40 };
-       u8 agc[] = { AGC_TARGET, 0x1c, 0x20 };
-       u8 sec_agc[] = { 0x69, 0x00, 0xff, 0xff, 0x40, 0xff, 0x00, 0x40, 0x40 };
-       u8 unk1[] = { 0x93, 0x1a };
-       u8 unk2[] = { 0xb5, 0x7a };
+       static const u8 config[] = { CONFIG, 0x3d };
+       static const u8 clock[] = { CLOCK_CTL, 0x30 };
+       static const u8 reset[] = { RESET, 0x80 };
+       static const u8 adc_ctl[] = { ADC_CTL_1, 0x40 };
+       static const u8 agc[] = { AGC_TARGET, 0x1c, 0x20 };
+       static const u8 sec_agc[] = { 0x69, 0x00, 0xff, 0xff, 0x40, 0xff, 0x00, 0x40, 0x40 };
+       static const u8 unk1[] = { 0x93, 0x1a };
+       static const u8 unk2[] = { 0xb5, 0x7a };
 
        deb("Demod init!\n");