[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <56n4r7p63dtzjpafe5wdljwhlkhzahbct5tsocegivvhop4f6a@65pmfavnyqga>
Date: Tue, 10 Dec 2024 23:21:17 +0100
From: Marijn Suijten <marijn.suijten@...ainline.org>
To: Abhinav Kumar <quic_abhinavk@...cinc.com>
Cc: Rob Clark <robdclark@...il.com>,
Dmitry Baryshkov <dmitry.baryshkov@...aro.org>, Sean Paul <sean@...rly.run>, David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Kuogee Hsieh <quic_khsieh@...cinc.com>,
Mahadevan <quic_mahap@...cinc.com>, linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
freedreno@...ts.freedesktop.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/4] dt-bindings: msm/dp: add support for pixel clock to
driver another stream
On 2024-12-02 19:31:38, Abhinav Kumar wrote:
> On some MSM chipsets, the display port controller is capable of supporting
> two streams. To drive the second stream, the pixel clock for the corresponding
> stream needs to be enabled. In order to add the bindings for the pixel clock
> for the second stream, fixup the documentation of some of the bindings to
> clarify exactly which stream they correspond to, then add the new bindings.
>
> In addition, to help out with reviews for dp-controller bindings, add myself
> as the maintainter.
>
> This change was made on top of [1] which fixes a warning on the sa8775p
> bindings.
>
> [1]: https://patchwork.freedesktop.org/patch/624068/
>
> Signed-off-by: Abhinav Kumar <quic_abhinavk@...cinc.com>
> ---
> Abhinav Kumar (4):
> dt-bindings: display: msm: dp-controller: document pixel clock stream
> dt-bindings: display: msm: dp-controller: document clock parents better
> dt-bindings: display/msm: add stream 1 pixel clock binding
> dt-bindings: display: msm: dp: update maintainer entry
Simple nit: any reason why the third patch uses display/msm as subject prefix
while the other 3 patches separate with a colon and space?
- Marijn
> .../bindings/display/msm/dp-controller.yaml | 41 +++++++++++++++++++---
> .../bindings/display/msm/qcom,sa8775p-mdss.yaml | 9 +++--
> 2 files changed, 43 insertions(+), 7 deletions(-)
> ---
> base-commit: 798bb342e0416d846cf67f4725a3428f39bfb96b
> change-id: 20241202-dp_mst_bindings-7536ffc9ae2f
>
> Best regards,
> --
> Abhinav Kumar <quic_abhinavk@...cinc.com>
>
Powered by blists - more mailing lists