Re: [PATCH v4 3/5] dt-bindings: media: qcom: Add JPEG encoder binding

[Gjorgji Rosikopulos (Consultant)]

Hi Bjorn,

On 7/13/2026 11:38 AM, Atanas Filipov wrote:
> On 7/12/2026 5:55 PM, Bjorn Andersson wrote:
>> On Mon, Jul 06, 2026 at 10:11:11AM +0300, Atanas Filipov wrote:
>>> Add device-tree binding for the Qualcomm JPEG encoder hardware block
>>> present in SM8250 (Kona) SoCs.
>>>
>>> The JPEG encoder is a standalone hardware IP within the camera subsystem
>>> that performs JPEG compression in memory-to-memory fashion.  It is
>>> separate from the CAMSS ISP pipeline and has its own register space,
>>> interrupt, clocks, power domain, IOMMU streams, and interconnect paths.
>>>
>>> Signed-off-by: Atanas Filipov <atanas.filipov@xxxxxxxxxxxxxxxx>
>>> ---
>>>   .../bindings/media/qcom,jpeg-encoder.yaml     | 151 ++++++++++++++++++
>>>   1 file changed, 151 insertions(+)
>>>   create mode 100644 Documentation/devicetree/bindings/media/qcom,jpeg-encoder.yaml
>>>
>>> diff --git a/Documentation/devicetree/bindings/media/qcom,jpeg-encoder.yaml b/Documentation/devicetree/bindings/media/qcom,jpeg-encoder.yaml
>>> new file mode 100644
>>> index 000000000000..e4c16388ef07
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/media/qcom,jpeg-encoder.yaml
>>> @@ -0,0 +1,151 @@
>>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>>> +%YAML 1.2
>>> +---
>>> +$id: http://devicetree.org/schemas/media/qcom,jpeg-encoder.yaml#
>>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>>> +
>>> +title: Qualcomm JPEG Encoder
>>> +
>>> +maintainers:
>>> +  - Atanas Filipov <atanas.filipov@xxxxxxxxxxxxxxxx>
>>> +
>>> +description:
>>> +  Qualcomm JPEG Encoder is the JPEG encode hardware present in Qualcomm SoCs.
>>> +
>>> +properties:
>>> +  compatible:
>>> +    enum:
>>> +      - qcom,sm8250-jenc
>>
>> I suspect you find it obvious
>> that this is the block than is used to enc the js, but I don't.
>>
>> We have space, can we please use some more letters here? How about
>> inflating this to qcom,sm8250-jpeg-enc?
>>
>> Is there a qcom,sm8250-jdec as well? Is that a completely separate
>> block, or is this IP-block actually the qcom,scm8250-jpeg block doing
>> both encoding and decoding?
>>
Just to add that it is separate block and is doing only encoding, jpeg
decoder i believe is not available as hw block for this Soc.

~Gjorgji