On Mon, Jul 07, 2025 at 09:31:45AM +0800, shangyao lin wrote:
From: "shangyao.lin" shangyao.lin@mediatek.com
Add camera isp7x module device document.
Changes in v2:
- Rename binding file to mediatek,mt8188-cam-yuv.yaml
- Various fixes per review comments
- Update maintainers list
Where did you post v1?
Please use standard email subjects, so with the PATCH keyword in the title. 'git format-patch -vX' helps here to create proper versioned patches. Another useful tool is b4. Skipping the PATCH keyword makes filtering of emails more difficult thus making the review process less convenient.
A nit, subject: drop second/last, redundant "bindings". The "dt-bindings" prefix is already stating that these are bindings. See also: https://elixir.bootlin.com/linux/v6.7-rc8/source/Documentation/devicetree/bi...
+properties:
- compatible:
- const: mediatek,mt8188-cam-yuv
- reg:
- minItems: 1
What, why?
Look at other bindings.
- maxItems: 2
- description:
Base address and optional inner base address of the cam-yuv hardware block.
Why are you stating obvious? From where did you take it?
- reg-names:
- items:
- const: base
- const: inner_base
- minItems: 1
- maxItems: 2
No, really no. You did not follow any existing patterns and this binding does not look at all as anything else. Why making this things up? Just use recently reviewed binding as starting point.
Best regards, Krzysztof
linaro-mm-sig@lists.linaro.org