On Tue, 15 Oct 2024 at 10:02, Soutrik Mukhopadhyay <quic_mukhopad@xxxxxxxxxxx> wrote: > > > On 10/6/2024 8:30 PM, Dmitry Baryshkov wrote: > > On Fri, 4 Oct 2024 at 12:30, Soutrik Mukhopadhyay > > <quic_mukhopad@xxxxxxxxxxx> wrote: > >> This series adds support for the DisplayPort controller > >> and eDP PHY v5 found on the Qualcomm SA8775P platform. > >> > >> --- > >> v2: Fixed review comments from Dmitry and Bjorn > >> - Made aux_cfg array as const. > >> - Reused edp_swing_hbr_rbr and edp_swing_hbr2_hbr3 for v5. > >> > >> v3: Fixed review comments from Dmitry, Konrad and Bjorn > >> - Used a for loop to write the dp_phy_aux_cfg registers. > >> - Pre-defined the aux_cfg size to prevent any magic numbers. > >> - Added all the necessary DPTX controllers for this platform. > >> > >> v4: Fixed review comments from Dmitry and Krzysztof > >> - Updated commit message. > > For which patches? How? > > > We have removed the "reviewed by" and kept only the "acked by" for patch > 1. We have updated > > the commit message of patch 5 , to mention specifically about the > validation of 'only' MDSS0 DPTX0 and > > DPTX1. All of this must be a part of the changelog. It's not enough to say "updated messages" or "fixed comments". At least please let the reviewers know which patches have changed and which didn't. > > > > > >> --- > >> > >> Soutrik Mukhopadhyay (5): > >> dt-bindings: phy: Add eDP PHY compatible for sa8775p > >> phy: qcom: edp: Introduce aux_cfg array for version specific aux > >> settings > >> phy: qcom: edp: Add support for eDP PHY on SA8775P > >> dt-bindings: display: msm: dp-controller: document SA8775P compatible > >> drm/msm/dp: Add DisplayPort controller for SA8775P > >> > >> .../bindings/display/msm/dp-controller.yaml | 1 + > >> .../devicetree/bindings/phy/qcom,edp-phy.yaml | 1 + > >> drivers/gpu/drm/msm/dp/dp_display.c | 9 +++ > >> drivers/phy/qualcomm/phy-qcom-edp.c | 74 +++++++++++++------ > >> 4 files changed, 61 insertions(+), 24 deletions(-) > >> > >> -- > >> 2.17.1 > >> > > -- With best wishes Dmitry
