Re: [PATCH 1/6] dt-bindings: media: Add bindings for qcom,nord-camss

[Wenmeng Liu]

On 9/18/2026 2:47 PM, Krzysztof Kozlowski wrote:
> On 17/09/2026 14:37, Wenmeng Liu wrote:
> > Hi Krzysztof,
> >
> > On 9/17/2026 6:13 PM, Krzysztof Kozlowski wrote:
> > > On Tue, Sep 15, 2026 at 05:34:11PM +0800, Wenmeng Liu wrote:
> > > > Add bindings for the camera subsystem (CAMSS) on the Qualcomm Nord
> > > > platform.
> > > >
> > > > The Nord platform provides:
> > > > - 3 x TPG (Test Pattern Generator)
> > > > - 3 x CSID (CSI Decoder)
> > > > - 10 x CSID Lite
> > > > - 3 x VFE (Video Front End)
> > > > - 10 x VFE Lite
> > > >
> > > > Signed-off-by: Wenmeng Liu <wenmeng.liu@xxxxxxxxxxxxxxxx>
> > > > ---
> > > >    .../devicetree/bindings/media/qcom,nord-camss.yaml | 393 +++++++++++++++++++++
> > > >    1 file changed, 393 insertions(+)
> > >
> > > A nit, subject: drop second/last, redundant "bindings for". The
> > > "dt-bindings" prefix is already stating that these are bindings.
> > > See also:
> > > https://elixir.bootlin.com/linux/v7.1-rc7/source/Documentation/devicetree/bindings/submitting-patches.rst#L23
> > ACK
> >
> > > ...
> > >
> > > > +        description:
> > > > +          Input port for receiving CSI data.
> > > > +
> > > > +        properties:
> > > > +          endpoint:
> > > > +            $ref: video-interfaces.yaml#
> > > > +            unevaluatedProperties: false
> > > > +
> > > > +            properties:
> > > > +              data-lanes:
> > > > +                minItems: 1
> > > > +                maxItems: 4
> > > > +
> > > > +              bus-type:
> > > > +                enum:
> > > > +                  - 1 # MEDIA_BUS_TYPE_CSI2_CPHY
> > > > +                  - 4 # MEDIA_BUS_TYPE_CSI2_DPHY
> > > > +
> > > > +            required:
> > > > +              - data-lanes
> > > > +
> > >
> > > Missing supplies, altough maybe this is related to some other ongoing
> > > work, so would be useful to see explanation in commit msg.
> >
> > At the moment, the Nord platform only supports TPG. Perhaps I should
> > remove this part of the change for now and reintroduce it later when
> > CSIPHY support is available, once the EVK board is ready.
>
> Hm? So you added hardware for TPG? I do not see how such hardware is
> useful at all, thus I don't think we want the bindings for it. Bring
> real hardware, since the bindings should be complete.
>
> Best regards,
> Krzysztof

CSIPHY support will be added later on top of Bryan's CSIPHY rework, and 
the current bindings are already complete with new csiphy rework.
At the moment, no suitable hardware is available, so only the TPG -> 
CSID -> VFE path is supported.

Thanks,
Wenmeng