Hi, On Mon, Mar 14, 2022 at 7:47 AM Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> wrote: > > Drop the assigned clock rate property and vote on the mdp clock as per > calculated value during the usecase. > > This patch is dependent on below patch > https://patchwork.kernel.org/patch/12774067/ Some nits on how you're referring to the dependent patch: 1. In the commit message it's really nice to also include the subject line of the patch so someone looking at the commit after it lands can more easily identify the patch you depend on. 2. It's better to use links that have the message ID in them. In the past patchwork's magic IDs have been list. So I'd write: This patch is dependent on the patch ("drm/msm/disp/dpu1: set mdp clk to the maximum frequency in opp table during probe") [1]. [1] https://lore.kernel.org/r/1647269217-14064-2-git-send-email-quic_vpolimer@xxxxxxxxxxx/ > Signed-off-by: Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> > Reviewed-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> > --- > arch/arm64/boot/dts/qcom/sc7280.dtsi | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-) Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
