Re: [PATCH 3/8] dt-bindings: media: i2c: Add Samsung S5K3M3 image sensor

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: media: samsung,s5k3m3: description refers to driver
  support
--

commit 4922a603dc5707d1a4718e4be230c423c1f7953c
Author: Gaurav Suman <gauravsuman007@gmail.com>

dt-bindings: media: i2c: Add Samsung S5K3M3 image sensor

This commit adds the Devicetree binding schema for the Samsung S5K3M3 13MP
image sensor.

> diff --git a/Documentation/devicetree/bindings/media/i2c/samsung,s5k3m3.yaml b/Documentation/devicetree/bindings/media/i2c/samsung,s5k3m3.yaml
> new file mode 100644
> index 0000000000000..434f5d4326b72
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/media/i2c/samsung,s5k3m3.yaml
> @@ -0,0 +1,104 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/media/i2c/samsung,s5k3m3.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Samsung S5K3M3 Image Sensor
> +
> +description:
> +  Samsung S5K3M3 (ISOCELL 3M3) is a 13MP image sensor controlled over a
> +  serial camera control bus. The driver supports 4160x3120 at 30 frames per
> +  second, RAW10 over a 4-lane MIPI D-PHY interface.

[Severity: Low]
Should this description describe the hardware capabilities of the sensor
rather than what the driver supports, per
Documentation/devicetree/bindings/writing-bindings.rst?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261109-sm8150-camera-v1-0-3d2e7b407516@gmail.com?part=3