On 10/1/24 4:56 PM, Michael Tretter wrote:
On Sat, 13 Jul 2024 17:41:54 +0200, Marek Vasut wrote:
The VACTIVE and HDELAY shadow registers must be enabled for per channel
NTSC/PAL detection to work, enable them.
I think the description is wrong as the change doesn't affect the
VACTIVE and HDELAY registers. The change sets the standard selection
register to auto detection to always trigger a standard detection during
initialization instead of defaulting to NTSC. querystd explicitly
triggers this behavior.
It does, commit message fixed.
If the driver always runs standard detection during init, the driver
should also wait for the detection to be finished and update
isl7998x->norm accordingly, because otherwise the standard will be reset
to NTSC.
Maybe we can also get rid of this part from the init sequence and run
the standard detection during the initialization in code?
Maybe the init sequence is OK as-is and this and the follow up patches
should be discarded ? Users can run QUERYSTD to trigger the standard
detection when needed ?
