linux-kernel - Re: [PATCH v4 1/2] media: i2c: imx412: fix sensor power-on timing

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:
 <PN3P287MB1829EDDB642D6CFE9FD7EDBC8B96A@PN3P287MB1829.INDP287.PROD.OUTLOOK.COM>
Date: Wed, 21 Jan 2026 08:47:27 +0000
From: Tarang Raval <tarang.raval@...iconsignals.io>
To: Wenmeng Liu <wenmeng.liu@....qualcomm.com>
CC: "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>, Sakari Ailus
	<sakari.ailus@...ux.intel.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, Mauro Carvalho Chehab <mchehab@...nel.org>
Subject: Re: [PATCH v4 1/2] media: i2c: imx412: fix sensor power-on timing

Hi Wenmeng,

> Assert the reset GPIO before first power up. This avoids a mismatch where     
> the first power up (when the reset GPIO defaults deasserted) differs from     
> subsequent cycles.                                                            
>                                                                               
> Signed-off-by: Wenmeng Liu <wenmeng.liu@....qualcomm.com>                     
> ---                                                                           
>  drivers/media/i2c/imx412.c | 2 ++                                            
>  1 file changed, 2 insertions(+)                                              
>                                                                               
> diff --git a/drivers/media/i2c/imx412.c b/drivers/media/i2c/imx412.c          
> index b3826f80354703b17b416dc233854da3f5736e38..5cb55deb125edb218779b076429f9fff93e11a08 100644
> --- a/drivers/media/i2c/imx412.c                                              
> +++ b/drivers/media/i2c/imx412.c                                              
> @@ -1188,6 +1188,8 @@ static int imx412_probe(struct i2c_client *client)      
>                                                                               
>         mutex_init(&imx412->mutex);                                           
>                                                                               
> +       gpiod_set_value_cansleep(imx412->reset_gpio, 1);                      
> +                                                                             
                                                                                
Just logic needs to be set to high when requesting this GPIO in                 
imx412_parse_hw_config.                                                         
                                                                                
Best regards,                                                                   
Tarang