From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> To: Marijn Suijten <marijn.suijten@somainline.org> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>, Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>, Abhinav Kumar <quic_abhinavk@quicinc.com>, Stephen Boyd <swboyd@chromium.org>, David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, kernel test robot <lkp@intel.com> Subject: Re: [PATCH v2] drm/msm/dsi: use RMW cycles in dsi_update_dsc_timing Date: Sat, 30 Apr 2022 22:28:42 +0300 [thread overview] Message-ID: <02114b24-f954-f145-4918-01cc3def65ac@linaro.org> (raw) In-Reply-To: <20220430185807.yn2j2coyc77qzx2o@SoMainline.org> On 30/04/2022 21:58, Marijn Suijten wrote: > On 2022-04-30 20:55:33, Dmitry Baryshkov wrote: >> The downstream uses read-modify-write for updating command mode >> compression registers. Let's follow this approach. This also fixes the >> following warning: >> >> drivers/gpu/drm/msm/dsi/dsi_host.c:918:23: warning: variable 'reg_ctrl' set but not used [-Wunused-but-set-variable] >> >> Reported-by: kernel test robot <lkp@intel.com> >> Fixes: 08802f515c3c ("drm/msm/dsi: Add support for DSC configuration") >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > I pointed this out in review multiple times, so you'll obviously get my: I think I might have also pointed this out once (and then forgot to check that the issue was fixed by Vinod). > > Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org> > > (But are you sure there's nothing else to clear in the 1st CTRL > register, only the lowest 16 bits? That should mean `reg` never > contains anything in 0xffff0000) Judging from the downstream the upper half conains the same fields, but used for other virtual channel. I didn't research what's the difference yet. All the dtsi files that I have here at hand use 'qcom,mdss-dsi-virtual-channel-id = <0>;' > > However, this seems to indicate that the DSC patch series has been > approved and merged somehow?? Pending inclusion, yes. If Vinod missed or ignored any other review points, please excuse Abhinav and me not noticing that. Can you please take a look at the latest revision posted, if there are any other missing points. Let's decide if there are grave issues or we can work them through. > >> --- >> >> Changes since v1: >> - Fix c&p error and apply mask clear to reg_ctrl2 instead of reg_ctrl >> (Abhinav) >> >> --- >> drivers/gpu/drm/msm/dsi/dsi_host.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c >> index c983698d1384..a95d5df52653 100644 >> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c >> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c >> @@ -961,10 +961,13 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod >> reg_ctrl = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL); >> reg_ctrl2 = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2); >> >> + reg_ctrl &= ~0xffff; >> reg_ctrl |= reg; >> + >> + reg_ctrl2 &= ~DSI_COMMAND_COMPRESSION_MODE_CTRL2_STREAM0_SLICE_WIDTH__MASK; >> reg_ctrl2 |= DSI_COMMAND_COMPRESSION_MODE_CTRL2_STREAM0_SLICE_WIDTH(bytes_in_slice); >> >> - dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL, reg); >> + dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL, reg_ctrl); >> dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2, reg_ctrl2); >> } else { >> dsi_write(msm_host, REG_DSI_VIDEO_COMPRESSION_MODE_CTRL, reg); >> -- >> 2.35.1 >> -- With best wishes Dmitry
WARNING: multiple messages have this Message-ID (diff)
From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> To: Marijn Suijten <marijn.suijten@somainline.org> Cc: freedreno@lists.freedesktop.org, kernel test robot <lkp@intel.com>, David Airlie <airlied@linux.ie>, linux-arm-msm@vger.kernel.org, Abhinav Kumar <quic_abhinavk@quicinc.com>, dri-devel@lists.freedesktop.org, Stephen Boyd <swboyd@chromium.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, Sean Paul <sean@poorly.run> Subject: Re: [PATCH v2] drm/msm/dsi: use RMW cycles in dsi_update_dsc_timing Date: Sat, 30 Apr 2022 22:28:42 +0300 [thread overview] Message-ID: <02114b24-f954-f145-4918-01cc3def65ac@linaro.org> (raw) In-Reply-To: <20220430185807.yn2j2coyc77qzx2o@SoMainline.org> On 30/04/2022 21:58, Marijn Suijten wrote: > On 2022-04-30 20:55:33, Dmitry Baryshkov wrote: >> The downstream uses read-modify-write for updating command mode >> compression registers. Let's follow this approach. This also fixes the >> following warning: >> >> drivers/gpu/drm/msm/dsi/dsi_host.c:918:23: warning: variable 'reg_ctrl' set but not used [-Wunused-but-set-variable] >> >> Reported-by: kernel test robot <lkp@intel.com> >> Fixes: 08802f515c3c ("drm/msm/dsi: Add support for DSC configuration") >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > > I pointed this out in review multiple times, so you'll obviously get my: I think I might have also pointed this out once (and then forgot to check that the issue was fixed by Vinod). > > Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org> > > (But are you sure there's nothing else to clear in the 1st CTRL > register, only the lowest 16 bits? That should mean `reg` never > contains anything in 0xffff0000) Judging from the downstream the upper half conains the same fields, but used for other virtual channel. I didn't research what's the difference yet. All the dtsi files that I have here at hand use 'qcom,mdss-dsi-virtual-channel-id = <0>;' > > However, this seems to indicate that the DSC patch series has been > approved and merged somehow?? Pending inclusion, yes. If Vinod missed or ignored any other review points, please excuse Abhinav and me not noticing that. Can you please take a look at the latest revision posted, if there are any other missing points. Let's decide if there are grave issues or we can work them through. > >> --- >> >> Changes since v1: >> - Fix c&p error and apply mask clear to reg_ctrl2 instead of reg_ctrl >> (Abhinav) >> >> --- >> drivers/gpu/drm/msm/dsi/dsi_host.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c >> index c983698d1384..a95d5df52653 100644 >> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c >> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c >> @@ -961,10 +961,13 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod >> reg_ctrl = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL); >> reg_ctrl2 = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2); >> >> + reg_ctrl &= ~0xffff; >> reg_ctrl |= reg; >> + >> + reg_ctrl2 &= ~DSI_COMMAND_COMPRESSION_MODE_CTRL2_STREAM0_SLICE_WIDTH__MASK; >> reg_ctrl2 |= DSI_COMMAND_COMPRESSION_MODE_CTRL2_STREAM0_SLICE_WIDTH(bytes_in_slice); >> >> - dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL, reg); >> + dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL, reg_ctrl); >> dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2, reg_ctrl2); >> } else { >> dsi_write(msm_host, REG_DSI_VIDEO_COMPRESSION_MODE_CTRL, reg); >> -- >> 2.35.1 >> -- With best wishes Dmitry
next prev parent reply other threads:[~2022-04-30 19:28 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-30 17:55 [PATCH v2] drm/msm/dsi: use RMW cycles in dsi_update_dsc_timing Dmitry Baryshkov 2022-04-30 17:55 ` Dmitry Baryshkov 2022-04-30 18:42 ` Abhinav Kumar 2022-04-30 18:42 ` Abhinav Kumar 2022-04-30 18:58 ` Marijn Suijten 2022-04-30 18:58 ` Marijn Suijten 2022-04-30 19:25 ` [Freedreno] " Abhinav Kumar 2022-04-30 19:25 ` Abhinav Kumar 2022-05-01 20:06 ` Marijn Suijten 2022-05-01 20:06 ` Marijn Suijten 2022-05-01 23:56 ` Abhinav Kumar 2022-05-01 23:56 ` Abhinav Kumar 2022-05-02 8:02 ` Dmitry Baryshkov 2022-05-02 8:02 ` Dmitry Baryshkov 2022-05-02 8:34 ` Marijn Suijten 2022-05-02 8:34 ` Marijn Suijten 2022-05-02 10:02 ` Dmitry Baryshkov 2022-05-02 10:02 ` Dmitry Baryshkov 2022-05-02 21:56 ` Marijn Suijten 2022-05-02 21:56 ` Marijn Suijten 2022-04-30 19:28 ` Dmitry Baryshkov [this message] 2022-04-30 19:28 ` Dmitry Baryshkov 2022-05-01 20:41 ` Marijn Suijten 2022-05-01 20:41 ` Marijn Suijten 2022-05-01 22:44 ` Dmitry Baryshkov 2022-05-01 22:44 ` Dmitry Baryshkov 2022-05-02 8:43 ` Marijn Suijten 2022-05-02 8:43 ` Marijn Suijten 2022-05-02 9:41 ` Dmitry Baryshkov 2022-05-02 9:41 ` Dmitry Baryshkov 2022-05-02 21:53 ` Marijn Suijten 2022-05-02 21:53 ` Marijn Suijten 2022-05-04 13:41 ` [Freedreno] " Vinod Koul 2022-05-04 13:41 ` Vinod Koul 2022-05-04 13:38 ` Vinod Koul 2022-05-04 13:38 ` Vinod Koul 2022-05-04 13:35 ` Vinod Koul 2022-05-04 13:35 ` Vinod Koul 2022-05-04 13:31 ` [Freedreno] " Vinod Koul 2022-05-04 13:31 ` Vinod Koul
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=02114b24-f954-f145-4918-01cc3def65ac@linaro.org \ --to=dmitry.baryshkov@linaro.org \ --cc=airlied@linux.ie \ --cc=bjorn.andersson@linaro.org \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=freedreno@lists.freedesktop.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=lkp@intel.com \ --cc=marijn.suijten@somainline.org \ --cc=quic_abhinavk@quicinc.com \ --cc=robdclark@gmail.com \ --cc=sean@poorly.run \ --cc=swboyd@chromium.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.