[v1] dt-bindings: msm/dp: add support for pixel clock to driver another stream

[PATCH 0/4] dt-bindings: msm/dp: add support for pixel clock to driver another stream

Posted by Abhinav Kumar 1 year, 2 months ago

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@quicinc.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

 .../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@quicinc.com>

Re: [PATCH 0/4] dt-bindings: msm/dp: add support for pixel clock to driver another stream

Posted by Dmitry Baryshkov 1 year, 1 month ago

On Mon, 02 Dec 2024 19:31:38 -0800, 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.
> 
> [...]

Applied, thanks!

[4/4] dt-bindings: display: msm: dp: update maintainer entry
      https://gitlab.freedesktop.org/lumag/msm/-/commit/c36c60d1f742

Best regards,
-- 
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Re: [PATCH 0/4] dt-bindings: msm/dp: add support for pixel clock to driver another stream

Posted by Marijn Suijten 1 year, 2 months ago

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@quicinc.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@quicinc.com>
>