From: Konrad Dybcio <konrad.dybcio@linaro.org> To: Marijn Suijten <marijn.suijten@somainline.org>, phone-devel@vger.kernel.org, Rob Clark <robdclark@gmail.com>, Abhinav Kumar <quic_abhinavk@quicinc.com>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org>, Vinod Koul <vkoul@kernel.org> Cc: ~postmarketos/upstreaming@lists.sr.ht, "AngeloGioacchino Del Regno" <angelogioacchino.delregno@somainline.org>, "Konrad Dybcio" <konrad.dybcio@somainline.org>, "Martin Botka" <martin.botka@somainline.org>, "Jami Kettunen" <jami.kettunen@somainline.org>, "Sean Paul" <sean@poorly.run>, "David Airlie" <airlied@gmail.com>, "Daniel Vetter" <daniel@ffwll.ch>, "Stephen Boyd" <swboyd@chromium.org>, "Bjorn Andersson" <andersson@kernel.org>, "Jessica Zhang" <quic_jesszhan@quicinc.com>, "Ville Syrjälä" <ville.syrjala@linux.intel.com>, "Kuogee Hsieh" <quic_khsieh@quicinc.com>, "Jani Nikula" <jani.nikula@intel.com>, sunliming <sunliming@kylinos.cn>, "Sam Ravnborg" <sam@ravnborg.org>, "Haowen Bai" <baihaowen@meizu.com>, "Loic Poulain" <loic.poulain@linaro.org>, "Vinod Polimera" <quic_vpolimer@quicinc.com>, "Douglas Anderson" <dianders@chromium.org>, "Vladimir Lypak" <vladimir.lypak@gmail.com>, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 5/6] drm/msm/dsi: Flip greater-than check for slice_count and slice_per_intf Date: Wed, 14 Dec 2022 01:02:14 +0100 [thread overview] Message-ID: <c5e33d9f-0dc4-fdd2-244a-3d463be1c4e8@linaro.org> (raw) In-Reply-To: <20221213232207.113607-6-marijn.suijten@somainline.org> On 14.12.2022 00:22, Marijn Suijten wrote: > According to downstream /and the comment copied from it/ this comparison > should be the other way around. In other words, when the panel driver > requests to use more slices per packet than what could be sent over this > interface, it is bumped down to only use a single slice per packet (and > strangely not the number of slices that could fit on the interface). > > Fixes: 08802f515c3c ("drm/msm/dsi: Add support for DSC configuration") > --- Missing s-o-b > drivers/gpu/drm/msm/dsi/dsi_host.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c > index 0686c35a6fd4..9bdfa0864cdf 100644 > --- a/drivers/gpu/drm/msm/dsi/dsi_host.c > +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c > @@ -855,11 +855,11 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod > */ > slice_per_intf = DIV_ROUND_UP(hdisplay, dsc->slice_width); > > - /* If slice_per_pkt is greater than slice_per_intf > + /* If slice_count is greater than slice_per_intf > * then default to 1. This can happen during partial > * update. > */ > - if (slice_per_intf > dsc->slice_count) > + if (dsc->slice_count > slice_per_intf) > dsc->slice_count = 1; > > total_bytes_per_intf = dsc->slice_chunk_size * slice_per_intf;
WARNING: multiple messages have this Message-ID (diff)
From: Konrad Dybcio <konrad.dybcio@linaro.org> To: Marijn Suijten <marijn.suijten@somainline.org>, phone-devel@vger.kernel.org, Rob Clark <robdclark@gmail.com>, Abhinav Kumar <quic_abhinavk@quicinc.com>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org>, Vinod Koul <vkoul@kernel.org> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>, Vinod Polimera <quic_vpolimer@quicinc.com>, Sam Ravnborg <sam@ravnborg.org>, Haowen Bai <baihaowen@meizu.com>, Kuogee Hsieh <quic_khsieh@quicinc.com>, Jessica Zhang <quic_jesszhan@quicinc.com>, Jani Nikula <jani.nikula@intel.com>, linux-arm-msm@vger.kernel.org, Stephen Boyd <swboyd@chromium.org>, Martin Botka <martin.botka@somainline.org>, ~postmarketos/upstreaming@lists.sr.ht, Sean Paul <sean@poorly.run>, Loic Poulain <loic.poulain@linaro.org>, Jami Kettunen <jami.kettunen@somainline.org>, Bjorn Andersson <andersson@kernel.org>, Vladimir Lypak <vladimir.lypak@gmail.com>, Douglas Anderson <dianders@chromium.org>, sunliming <sunliming@kylinos.cn>, freedreno@lists.freedesktop.org Subject: Re: [RFC PATCH 5/6] drm/msm/dsi: Flip greater-than check for slice_count and slice_per_intf Date: Wed, 14 Dec 2022 01:02:14 +0100 [thread overview] Message-ID: <c5e33d9f-0dc4-fdd2-244a-3d463be1c4e8@linaro.org> (raw) In-Reply-To: <20221213232207.113607-6-marijn.suijten@somainline.org> On 14.12.2022 00:22, Marijn Suijten wrote: > According to downstream /and the comment copied from it/ this comparison > should be the other way around. In other words, when the panel driver > requests to use more slices per packet than what could be sent over this > interface, it is bumped down to only use a single slice per packet (and > strangely not the number of slices that could fit on the interface). > > Fixes: 08802f515c3c ("drm/msm/dsi: Add support for DSC configuration") > --- Missing s-o-b > drivers/gpu/drm/msm/dsi/dsi_host.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c > index 0686c35a6fd4..9bdfa0864cdf 100644 > --- a/drivers/gpu/drm/msm/dsi/dsi_host.c > +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c > @@ -855,11 +855,11 @@ static void dsi_update_dsc_timing(struct msm_dsi_host *msm_host, bool is_cmd_mod > */ > slice_per_intf = DIV_ROUND_UP(hdisplay, dsc->slice_width); > > - /* If slice_per_pkt is greater than slice_per_intf > + /* If slice_count is greater than slice_per_intf > * then default to 1. This can happen during partial > * update. > */ > - if (slice_per_intf > dsc->slice_count) > + if (dsc->slice_count > slice_per_intf) > dsc->slice_count = 1; > > total_bytes_per_intf = dsc->slice_chunk_size * slice_per_intf;
next prev parent reply other threads:[~2022-12-14 0:02 UTC|newest] Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-12-13 23:22 [RFC PATCH 0/6] drm/msm: DSC Electric Boogaloo for sm8[12]50 Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-13 23:22 ` [RFC PATCH 1/6] drm/msm/dpu1: Implement DSC binding to PP block for CTL V1 Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 18:40 ` Dmitry Baryshkov 2022-12-14 18:40 ` Dmitry Baryshkov 2022-12-16 20:51 ` Abhinav Kumar 2022-12-16 20:51 ` Abhinav Kumar 2022-12-13 23:22 ` [RFC PATCH 2/6] drm/msm/dpu1: Add DSC config for sm8150 and sm8250 Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 18:42 ` Dmitry Baryshkov 2022-12-14 18:42 ` Dmitry Baryshkov 2022-12-16 20:53 ` Abhinav Kumar 2022-12-16 20:53 ` Abhinav Kumar 2022-12-13 23:22 ` [RFC PATCH 3/6] drm/msm/dpu1: Wire up DSC mask for active CTL configuration Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 18:43 ` Dmitry Baryshkov 2022-12-14 18:43 ` Dmitry Baryshkov 2022-12-14 19:30 ` Marijn Suijten 2022-12-14 19:30 ` Marijn Suijten 2022-12-15 1:08 ` Dmitry Baryshkov 2022-12-16 22:20 ` Abhinav Kumar 2022-12-20 22:32 ` Marijn Suijten 2022-12-20 22:32 ` Marijn Suijten 2022-12-13 23:22 ` [RFC PATCH 4/6] drm/msm/dsi: Use DSC slice(s) packet size to compute word count Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 18:52 ` Dmitry Baryshkov 2022-12-14 18:52 ` Dmitry Baryshkov 2022-12-16 23:01 ` Abhinav Kumar 2022-12-16 23:01 ` Abhinav Kumar 2022-12-13 23:22 ` [RFC PATCH 5/6] drm/msm/dsi: Flip greater-than check for slice_count and slice_per_intf Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 0:02 ` Konrad Dybcio [this message] 2022-12-14 0:02 ` Konrad Dybcio 2022-12-14 8:38 ` Marijn Suijten 2022-12-14 8:38 ` Marijn Suijten 2022-12-14 18:53 ` Dmitry Baryshkov 2022-12-14 18:53 ` Dmitry Baryshkov 2022-12-17 0:31 ` Abhinav Kumar 2022-12-17 0:31 ` Abhinav Kumar 2022-12-13 23:22 ` [RFC PATCH 6/6] drm/msm/dpu: Disallow unallocated (DSC) resources to be returned Marijn Suijten 2022-12-13 23:22 ` Marijn Suijten 2022-12-14 18:56 ` Dmitry Baryshkov 2022-12-14 18:56 ` Dmitry Baryshkov 2022-12-14 19:31 ` Marijn Suijten 2022-12-14 19:31 ` Marijn Suijten 2022-12-14 18:40 ` [RFC PATCH 0/6] drm/msm: DSC Electric Boogaloo for sm8[12]50 Dmitry Baryshkov 2022-12-14 18:40 ` Dmitry Baryshkov 2022-12-14 19:23 ` Marijn Suijten 2022-12-14 19:23 ` Marijn Suijten 2022-12-15 0:52 ` Dmitry Baryshkov 2022-12-20 22:35 ` Marijn Suijten 2022-12-20 22:35 ` Marijn Suijten
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=c5e33d9f-0dc4-fdd2-244a-3d463be1c4e8@linaro.org \ --to=konrad.dybcio@linaro.org \ --cc=airlied@gmail.com \ --cc=andersson@kernel.org \ --cc=angelogioacchino.delregno@somainline.org \ --cc=baihaowen@meizu.com \ --cc=daniel@ffwll.ch \ --cc=dianders@chromium.org \ --cc=dmitry.baryshkov@linaro.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=freedreno@lists.freedesktop.org \ --cc=jami.kettunen@somainline.org \ --cc=jani.nikula@intel.com \ --cc=konrad.dybcio@somainline.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=loic.poulain@linaro.org \ --cc=marijn.suijten@somainline.org \ --cc=martin.botka@somainline.org \ --cc=phone-devel@vger.kernel.org \ --cc=quic_abhinavk@quicinc.com \ --cc=quic_jesszhan@quicinc.com \ --cc=quic_khsieh@quicinc.com \ --cc=quic_vpolimer@quicinc.com \ --cc=robdclark@gmail.com \ --cc=sam@ravnborg.org \ --cc=sean@poorly.run \ --cc=sunliming@kylinos.cn \ --cc=swboyd@chromium.org \ --cc=ville.syrjala@linux.intel.com \ --cc=vkoul@kernel.org \ --cc=vladimir.lypak@gmail.com \ --cc=~postmarketos/upstreaming@lists.sr.ht \ /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.