linux-kernel - Re: [PATCH v2] media: i2c: imx412: wait for NVM read (T7) before programming mode registers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1344ad43-a4c4-4e37-ae4e-eddd2c577b3f@oss.qualcomm.com>
Date: Mon, 12 Jan 2026 11:07:39 +0800
From: Wenmeng Liu <wenmeng.liu@....qualcomm.com>
To: Bryan O'Donoghue <bod@...nel.org>, sakari.ailus@...ux.intel.com,
        mchehab@...nel.org
Cc: linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] media: i2c: imx412: wait for NVM read (T7) before
 programming mode registers



On 1/9/2026 8:32 PM, Bryan O'Donoghue wrote:
> This delay should go at the end of the operation that requires the delay 
> not at the start of the streaming operation.
> 
> The delay after the stream write, should be related to the stream write 
> command, not the antecedent - the command that came before start_streaming.
> 
> Basically I think you need to put your delay into the CCI_ID read NVM 
> parameter load routine so that it guarantees its own completion.
> 
> Because for argument's sake if start_streaming() were not to be the 
> thing to happen after CCI_ID/NVM loading, the logic would no longer work.
> 
> And you need a Fixes: tag for this patch too.

Reading the sensor ID only occurs during the sensor probe process. After 
the probe is completed, the IMX577 will power down. When stream on 
occurs, the driver will power on again and then start streaming, but the 
sensor ID is not read during the stream on process.I have tested this 
change on imx577 modules of different models.

So this change can only happen during power on or stream on.

Hi Bryan, Sakari,
May I ask if you have any suggestions regarding this?


Thanks,
Wenmeng