RE: [PATCH upstream review v6 0/4] media: i2c: ds90ub953: Add back channel GPIO support

[G.N. Zhou]

Please disregard this incomplete patch series. I discovered an issue during submission
and only patches 0-1 were sent.

I will send a corrected v6 series shortly.

Sorry for the noise.

> -----Original Message-----
> From: G.N. Zhou (OSS) <guoniu.zhou@xxxxxxxxxxx>
> Sent: Friday, April 24, 2026 9:35 AM
> To: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx>; Mauro Carvalho
> Chehab <mchehab@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof
> Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Frank Li
> <frank.li@xxxxxxx>; Vladimir Zapolskiy <vz@xxxxxxxxx>; Linus Walleij
> <linusw@xxxxxxxxxx>; Bartosz Golaszewski <brgl@xxxxxxxxxx>
> Cc: linux-media@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; imx@xxxxxxxxxxxxxxx; linux-gpio@xxxxxxxxxxxxxxx; G.N.
> Zhou <guoniu.zhou@xxxxxxx>
> Subject: [PATCH upstream review v6 0/4] media: i2c: ds90ub953: Add back
> channel GPIO support
> 
> Add back channel GPIO support.
>   dt-bindings: media: ti,ds90ub953: Add support for remote GPIO data source
>   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@xxxxxxx>
> ---
> Changes in v6:
> - Rebased to latest media/next
> - Replaced label "err" with "out"
> - Changed approach from extending GPIO range (v5) to using a custom GPIO
>   flag (GPIO_DATA_SOURCE_REMOTE) as suggested by the driver maintainer
> - Dropped Reviewed-by tag from Rob Herring due to significant binding change
> - Added GPIO_DATA_SOURCE_REMOTE flag to dt-bindings/gpio/gpio.h
> - Updated dt-bindings documentation accordingly
> - Added Reviewed-by tag from Tomi Valkeinen
> - Detailed changes can be found in each patch's changelog
> - Link to v5: https://lore.kernel.org/r/20260228-ds90ub953-v5-0-
> 056cf07cc8f1@xxxxxxx
> 
> Changes in v5:
> - Improve the description for "#gpio-cells" as commented by Conor.
> - Detail change see each patches's change log.
> - Link to v4: https://lore.kernel.org/r/20260227-ds90ub953-v4-0-
> dea596205f9e@xxxxxxx
> 
> Changes in v4:
> - Rebase to 7.0-rc1
> - Use folder block instead of literal block for #gpio-cell property description.
> - Only log GPIO 0-3 stats since remote GPIO 4-7 reuse GPIO 0-3 pins.
> - Link to v3: https://lore.kernel.org/r/20250916-ds90ub953-v3-0-
> bb7bc6545da7@xxxxxxx
> 
> 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@xxxxxxx
> 
> 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();
> - Parse gpio third cell to select which GPIO pin the data from remote compatible
> deserializer.
> - Link to v1: https://lore.kernel.org/r/20250910-ds90ub953-v1-0-
> a7813ffbdf11@xxxxxxx
> 
> ---
> Guoniu Zhou (4):
>       dt-bindings: media: ti,ds90ub953: Add support for remote GPIO data source
>       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           |   6 +-
>  drivers/media/i2c/ds90ub953.c                      | 118 +++++++++++++--------
>  include/dt-bindings/gpio/gpio.h                    |   8 ++
>  3 files changed, 83 insertions(+), 49 deletions(-)
> ---
> base-commit: 4fbeef21f5387234111b5d52924e77757626faa5
> change-id: 20250901-ds90ub953-168628c53a00
> 
> Best regards,
> --
> Guoniu Zhou <guoniu.zhou@xxxxxxxxxxx>