mbox series

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

Message ID 20241202-dp_mst_bindings-v1-0-9a9a43b0624a@quicinc.com (mailing list archive)
Headers show
Series dt-bindings: msm/dp: add support for pixel clock to driver another stream | expand

Message

Abhinav Kumar Dec. 3, 2024, 3:31 a.m. UTC
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,

Comments

Marijn Suijten Dec. 10, 2024, 10:21 p.m. UTC | #1
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>
>