arm64: dts: qcom: qcs615: Add support for camss

[PATCH 0/2] arm64: dts: qcom: qcs615: Add support for camss

Posted by Wenmeng Liu 7 months ago

This series adds support to bring up the CSIPHY, CSID, VFE/RDI interfaces 
in QCS615. Tested this on QCS615 ADP AIR board with CSID TPG.

Tested with following commands:
media-ctl --reset
v4l2-ctl -d /dev/v4l-subdev3 -c test_pattern=0
media-ctl -V '"msm_csid0":0[fmt:SRGGB10/1920x1080 field:none]'
media-ctl -V '"msm_vfe0_rdi0":0[fmt:SRGGB10/1920x1080 field:none]'
media-ctl -l '"msm_csid0":1->"msm_vfe0_rdi0":0[1]'
v4l2-ctl -d /dev/v4l-subdev3 -c test_pattern=9
yavta -B capture-mplane -n 5 -f SRGGB10P -s 1920x1080 /dev/video0 --capture=7

This patch series depends on patch series:
https://lore.kernel.org/all/20250518-qcs615_camss-v1-0-12723e26ea3e@quicinc.com/
https://lore.kernel.org/all/20250518-qcs615_camss-v1-0-12723e26ea3e@quicinc.com/
https://lore.kernel.org/all/20250424-qcs615-mm-v7-clock-controllers-v8-0-bacad5b3659a@quicinc.com/

Signed-off-by: Wenmeng Liu <quic_wenmliu@quicinc.com>
---
Wenmeng Liu (2):
      arm64: dts: qcom: qcs615: Add support for camss
      arm64: dts: qcom: qcs615: Enable camss for qcs615-adp-air

 arch/arm64/boot/dts/qcom/qcs615-ride.dts |   7 ++
 arch/arm64/boot/dts/qcom/qcs615.dtsi     | 133 +++++++++++++++++++++++++++++++
 2 files changed, 140 insertions(+)
---
base-commit: 5840726e19bb6e86566a9fa68c2dc21ed64761d4
change-id: 20250520-qcs615-adp-air-camss-4109fc2ce430

Best regards,
-- 
Wenmeng Liu <quic_wenmliu@quicinc.com>

Re: [PATCH 0/2] arm64: dts: qcom: qcs615: Add support for camss

Posted by Krzysztof Kozlowski 7 months ago

On 20/05/2025 10:56, Wenmeng Liu wrote:
> This series adds support to bring up the CSIPHY, CSID, VFE/RDI interfaces 
> in QCS615. Tested this on QCS615 ADP AIR board with CSID TPG.
> 
> Tested with following commands:
> media-ctl --reset
> v4l2-ctl -d /dev/v4l-subdev3 -c test_pattern=0
> media-ctl -V '"msm_csid0":0[fmt:SRGGB10/1920x1080 field:none]'
> media-ctl -V '"msm_vfe0_rdi0":0[fmt:SRGGB10/1920x1080 field:none]'
> media-ctl -l '"msm_csid0":1->"msm_vfe0_rdi0":0[1]'
> v4l2-ctl -d /dev/v4l-subdev3 -c test_pattern=9
> yavta -B capture-mplane -n 5 -f SRGGB10P -s 1920x1080 /dev/video0 --capture=7
> 
> This patch series depends on patch series:
> https://lore.kernel.org/all/20250518-qcs615_camss-v1-0-12723e26ea3e@quicinc.com/
> https://lore.kernel.org/all/20250518-qcs615_camss-v1-0-12723e26ea3e@quicinc.com/
> https://lore.kernel.org/all/20250424-qcs615-mm-v7-clock-controllers-v8-0-bacad5b3659a@quicinc.com/

Three dependencies? So this cannot be tested in any close future. You
need to rework the way you upstream, to avoid multiple dependencies.
This was already communicated multiple times.

Best regards,
Krzysztof