media: i2c: adv748x: Write initial DV timings to device
authorNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Wed, 22 Feb 2023 22:12:48 +0000 (23:12 +0100)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Tue, 11 Apr 2023 15:10:10 +0000 (17:10 +0200)
commit8a7bc891b9ebe4ba65154d38bbe1b8b43bbb6ef4
treec91dfce9bf3268357e401de1cd521b9c27a74dac
parent1e293874fabf41fb34d58e22153766fd2bf8c6aa
media: i2c: adv748x: Write initial DV timings to device

When initializing the HDMI block during probe an initial set of timings
are selected. These timings are stored in the drivers private data, but
not written to the device.

This in itself is not bad, but in s_dv_timings() the timings stored in
the drivers private data are compared to the new timings, if they match
no action is taken. This creates the corner-case where the timing
selected at initialization is the first timings a user want to use as
the driver then never writes it to the device preventing proper
operation.

Fix this by writing the timings to the device at initialization in
addition to storing them in the drivers private data.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/i2c/adv748x/adv748x-hdmi.c