summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Tretter <m.tretter@pengutronix.de>2019-11-06 16:36:37 +0100
committerMichael Tretter <m.tretter@pengutronix.de>2020-03-03 10:58:34 +0100
commit35721c5b6695c1a893da4f17515d1d597de8c1eb (patch)
tree771933b5fb7dddae4429a84e836b41a816565863
parent3babc970f37700ead4ee8b8742f62deb69b71b4c (diff)
downloadlinux-0-day-35721c5b6695c1a893da4f17515d1d597de8c1eb.tar.gz
linux-0-day-35721c5b6695c1a893da4f17515d1d597de8c1eb.tar.xz
media: allegro: remove unknown39 field from create_channel
The subframe_latency and lda_control_mode fields directly follow freq_golden_ref field and there is no unknown field in between. The unknown field it at the end of the message. Reorder the fields accordingly. This further allows to drop the hard coded value from the lda_control_mode field and set the mode to 0. Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
-rw-r--r--drivers/staging/media/allegro-dvt/allegro-core.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/staging/media/allegro-dvt/allegro-core.c b/drivers/staging/media/allegro-dvt/allegro-core.c
index 6e3f28e637af1..d67f84bf38d3e 100644
--- a/drivers/staging/media/allegro-dvt/allegro-core.c
+++ b/drivers/staging/media/allegro-dvt/allegro-core.c
@@ -397,10 +397,9 @@ struct mcu_msg_create_channel {
u8 num_b;
u8 freq_golden_ref;
- u32 unknown39;
-
u32 subframe_latency;
u32 lda_control_mode;
+ u32 unknown41;
} __attribute__ ((__packed__));
struct mcu_msg_create_channel_response {
@@ -1121,7 +1120,6 @@ static int allegro_mcu_send_create_channel(struct allegro_dev *dev,
msg.gdr_mode = 0x00000000;
msg.gop_length = channel->gop_size;
msg.subframe_latency = 0x00000000;
- msg.lda_control_mode = 0x700d0000;
allegro_mbox_write(dev, &dev->mbox_command, &msg, sizeof(msg));
allegro_mcu_interrupt(dev);