linux-kernel - RE: [PATCH v3 0/4] media: i2c: ds90ub953: Add back channel GPIO support

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] [day] [month] [year] [list]

Message-ID:
 <AS8SPRMB00234BD00ADD079D8DC7E438FAF0A@AS8SPRMB0023.eurprd04.prod.outlook.com>
Date: Thu, 23 Oct 2025 08:41:23 +0000
From: "G.N. Zhou" <guoniu.zhou@....com>
To: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>, Mauro Carvalho Chehab
	<mchehab@...nel.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
	<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Frank Li
	<frank.li@....com>
CC: "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"imx@...ts.linux.dev" <imx@...ts.linux.dev>
Subject: RE: [PATCH v3 0/4] media: i2c: ds90ub953: Add back channel GPIO
 support

Hi Tomi,

Could you please help to review the patches? Your input will be appreciated.  

Best Regards
G.N Zhou


> -----Original Message-----
> From: G.N. Zhou
> Sent: Tuesday, September 16, 2025 5:44 PM
> To: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>; Mauro Carvalho
> Chehab <mchehab@...nel.org>; Rob Herring <robh@...nel.org>; Krzysztof
> Kozlowski <krzk+dt@...nel.org>; Conor Dooley <conor+dt@...nel.org>; Frank
> Li <frank.li@....com>
> Cc: linux-media@...r.kernel.org; devicetree@...r.kernel.org; linux-
> kernel@...r.kernel.org; imx@...ts.linux.dev; G.N. Zhou
> <guoniu.zhou@....com>
> Subject: [PATCH v3 0/4] media: i2c: ds90ub953: Add back channel GPIO
> support
> 
> Add back channel GPIO support.
>   dt-bindings: media: ti,ds90ub953: Add new range for remote GPIO data
>   media: i2c: ds90ub953: Add back channel GPIO support
> 
> Use helper macro to simplify code.
>   media: i2c: ds90ub953: use devm_mutex_init() to simplify code
>   media: i2c: ds90ub953: use guard() to simplify code
> 
> Signed-off-by: Guoniu Zhou <guoniu.zhou@....com>
> ---
> Changes in v3:
> - Make GPIO range from 0-3 to 0-7 to support GPIO data from remote
>   compatible deserializer suggested by Rob instead of adding third
>   cell for GPIO controller.
> - Update driver to expand GPIO range.
> - Link to v2: https://lore.kernel.org/r/20250911-ds90ub953-v2-0-
> 03ee76eb6b59@....com
> 
> Changes in v2:
> - Remove new property ti,gpio-data added in patch 1.
> - Add third cell for GPIO controller to select GPIO output source.
> - Move PTR_ERR() in dev_err_probe();
> - Link to v1: https://lore.kernel.org/r/20250910-ds90ub953-v1-0-
> a7813ffbdf11@....com
> 
> ---
> Guoniu Zhou (4):
>       dt-bindings: media: ti,ds90ub953: Add new range for remote GPIO data
>       media: i2c: ds90ub953: Add back channel GPIO support
>       media: i2c: ds90ub953: use devm_mutex_init() to simplify code
>       media: i2c: ds90ub953: use guard() to simplify code
> 
>  .../bindings/media/i2c/ti,ds90ub953.yaml           |   8 +-
>  drivers/media/i2c/ds90ub953.c                      | 114 +++++++++++----------
>  2 files changed, 68 insertions(+), 54 deletions(-)
> ---
> base-commit: 34837c444cd42236b2b43ce871f30d83776a3431
> change-id: 20250901-ds90ub953-168628c53a00
> 
> Best regards,
> --
> Guoniu Zhou <guoniu.zhou@....com>