Re: [PATCH v2 0/3] media: qcom: camss: Add sm6150 camss support

[Wenmeng Liu <wenmeng.liu@oss.qualcomm.com>]

On 12/23/2025 9:35 PM, Krzysztof Kozlowski wrote:
> On Mon, Dec 22, 2025 at 04:28:38PM +0800, Wenmeng Liu wrote:
>> SM6150 is a Qualcomm flagship SoC. This series adds support to
>> the CSIPHY, CSID, VFE/RDI interfaces in SM6150.
>>
>> The SM6150 platform provides:
>>   - 2 x VFE (version 170), each with 3 RDI
>>   - 1 x VFE Lite (version 170), each with 4 RDI
>>   - 2 x CSID (version 170)
>>   - 1 x CSID Lite (version 170)
>>   - 3 x CSIPHY (version 2.0.0)
>>   - 1 x BPS (Bayer Processing Segment)
>>   - 1 x ICP (Imaging Control Processor)
>>   - 1 x IPE (Image Postprocessing Engine)
>>   - 1 x JPEG Encoder/Decoder
>>   - 1 x LRME (Low Resolution Motion Estimation)
>>
>> Tested on Talos EVK board.
>>
>> Tested with following commands:
>> media-ctl -d /dev/media0 --reset
>> media-ctl -d /dev/media0 -V '"imx577 9-001a":0[fmt:SRGGB10/4056x3040 field:none]'
>> media-ctl -d /dev/media0 -V '"msm_csid0":0[fmt:SRGGB10/4056x3040]'
>> media-ctl -d /dev/media0 -V '"msm_vfe0_rdi0":0[fmt:SRGGB10/4056x3040]'
>> media-ctl -d /dev/media0 -l '"msm_csiphy1":1->"msm_csid0":0[1]'
>> media-ctl -d /dev/media0 -l '"msm_csid0":1->"msm_vfe0_rdi0":0[1]'
>> yavta -B capture-mplane  -n 5 -f SRGGB10P -s 4056x3040 -F /dev/video0 --capture=5
>>
>> Changes in v2:
>> - Add more resources(BPS, ICP, IPE, JPEG, LRME) and details for CAMSS.
>> - Correct the order of the header files. - bob
>> - Add Csiphy id num, improve the resource name. - Vladimir
>> - Link to v1:
>>    https://lore.kernel.org/all/20251016-sm6150-camss-v1-0-e7f64ac32370@oss.qualcomm.com/
> 
> So no changes done in response to what I asked?
> 
> Best regards,
> Krzysztof
> 

I referred to the comments you raised and the latest reviewed 
DT-binding, but since this was not explicitly mentioned in my previous 
patch, I did not include it in the change description.


Thanks,
Wenmeng