Re: [PATCH 3/3] arm64: qcom: dts: qcm6490: Add JPEG encoder DT properties

[Bryan O'Donoghue <bod@kernel.org>]

On 15/05/2026 12:47, Atanas Filipov wrote:
> Add the required JPEG encoder definitions to the device tree.
> 
> Signed-off-by: Atanas Filipov <atanas.filipov@oss.qualcomm.com>
> ---
>   arch/arm64/boot/dts/qcom/kodiak.dtsi | 52 ++++++++++++++++++++++++++++++++++++
>   1 file changed, 52 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/kodiak.dtsi b/arch/arm64/boot/dts/qcom/kodiak.dtsi
> index 6079e67ea829..dbfc6cc051f3 100644
> --- a/arch/arm64/boot/dts/qcom/kodiak.dtsi
> +++ b/arch/arm64/boot/dts/qcom/kodiak.dtsi
> @@ -5198,6 +5198,58 @@ port@4 {
>   			};
>   		};
> 
> +		qcom_jpeg_enc: qcom,jpegenc@ac4e000 {

This should be a sub-node of the CAMSS block as, it is indeed inside of 
the individually collapsible CAMSS block in qcom documentation.

See how we are instantiating the OPE as a sub-node of CAMSS, along with 
CSIPHY as WIP.

https://patchwork.linuxtv.org/project/linux-media/patch/20260427-camss-isp-ope-v2-14-f430e7485009@oss.qualcomm.com

You can cherry-pick my one-liner into your series to facilitate the 
change in CAMSS and then your sub-nodes will automagically probe.

https://patchwork.linuxtv.org/project/linux-media/patch/20260326-b4-linux-next-25-03-13-dtsi-x1e80100-camss-v11-4-5b93415be6dd@linaro.org/

> +
>   		camcc: clock-controller@ad00000 {
>   			compatible = "qcom,sc7280-camcc";
>   			reg = <0 0x0ad00000 0 0x10000>;
> 
> --
> 2.34.1
>