| Message ID | 1676286704-818-1-git-send-email-quic_kalyant@quicinc.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Reserve DSPPs based on user request | expand |
Hi, On Mon, Feb 13, 2023 at 3:11 AM Kalyan Thota <quic_kalyant@quicinc.com> wrote: > > This series will enable color features on sc7280 target which has > primary panel as eDP > > The series removes DSPP allocation based on encoder type and allows > the DSPP reservation based on user request via CTM. > > The series will release/reserve the dpu resources whenever there is > a CTM enable/disable change so that DSPPs are allocated appropriately. > > Kalyan Thota (4): > drm/msm/dpu: clear DSPP reservations in rm release > drm/msm/dpu: add DSPPs into reservation upon a CTM request > drm/msm/dpu: avoid unnecessary check in DPU reservations > drm/msm/dpu: manage DPU resources if CTM is requested > > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 38 ++++++++++++----------------- > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 2 ++ > drivers/gpu/drm/msm/msm_atomic.c | 18 ++++++++++++++ > drivers/gpu/drm/msm/msm_drv.c | 2 +- > drivers/gpu/drm/msm/msm_drv.h | 1 + > 5 files changed, 38 insertions(+), 23 deletions(-) For whatever reason when I use "b4 shazam" on your series it yells: Patch failed at 0002 drm/msm/dpu: add DSPPs into reservation upon a CTM request When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort". error: patch failed: drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:573 error: drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: patch does not apply hint: Use 'git am --show-current-patch=diff' to see the failed patch ...but then I can apply it with `git am --show-current-patch=diff | patch -p1` and it just applies with fuzz. Presumably Abhniav / Dmitry can do the same but it would be nice if your patch applied cleanly to msm-next. In any case, I tried this patch on both a herobrine (sc7280) and trogdor (sc7180) based board running msm-next (including the sc7280 patch [1]). In both cases the night light on the internal display worked fine when using ChromeOS. External night light didn't work on either of them (as expected) because we don't have the compositor support yet. I'm happy enough with: Tested-by: Douglas Anderson <dianders@chromium.org> [1] https://lore.kernel.org/r/1674814487-2112-1-git-send-email-quic_kalyant@quicinc.com/
On Mon, 13 Feb 2023 at 20:12, Doug Anderson <dianders@chromium.org> wrote: > > Hi, > > On Mon, Feb 13, 2023 at 3:11 AM Kalyan Thota <quic_kalyant@quicinc.com> wrote: > > > > This series will enable color features on sc7280 target which has > > primary panel as eDP > > > > The series removes DSPP allocation based on encoder type and allows > > the DSPP reservation based on user request via CTM. > > > > The series will release/reserve the dpu resources whenever there is > > a CTM enable/disable change so that DSPPs are allocated appropriately. > > > > Kalyan Thota (4): > > drm/msm/dpu: clear DSPP reservations in rm release > > drm/msm/dpu: add DSPPs into reservation upon a CTM request > > drm/msm/dpu: avoid unnecessary check in DPU reservations > > drm/msm/dpu: manage DPU resources if CTM is requested > > > > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 38 ++++++++++++----------------- > > drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 2 ++ > > drivers/gpu/drm/msm/msm_atomic.c | 18 ++++++++++++++ > > drivers/gpu/drm/msm/msm_drv.c | 2 +- > > drivers/gpu/drm/msm/msm_drv.h | 1 + > > 5 files changed, 38 insertions(+), 23 deletions(-) > > For whatever reason when I use "b4 shazam" on your series it yells: > > Patch failed at 0002 drm/msm/dpu: add DSPPs into reservation upon a CTM request > When you have resolved this problem, run "git am --continue". > If you prefer to skip this patch, run "git am --skip" instead. > To restore the original branch and stop patching, run "git am --abort". > error: patch failed: drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:573 > error: drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: patch does not apply > hint: Use 'git am --show-current-patch=diff' to see the failed patch > > ...but then I can apply it with `git am --show-current-patch=diff | > patch -p1` and it just applies with fuzz. Presumably Abhniav / Dmitry > can do the same but it would be nice if your patch applied cleanly to > msm-next. Hopefully it is not some 'tested on 5.15' story. > In any case, I tried this patch on both a herobrine (sc7280) and > trogdor (sc7180) based board running msm-next (including the sc7280 > patch [1]). In both cases the night light on the internal display > worked fine when using ChromeOS. External night light didn't work on > either of them (as expected) because we don't have the compositor > support yet. > > I'm happy enough with: > > Tested-by: Douglas Anderson <dianders@chromium.org> Thanks for the testing!
On Mon, 13 Feb 2023 03:11:40 -0800, Kalyan Thota wrote: > This series will enable color features on sc7280 target which has > primary panel as eDP > > The series removes DSPP allocation based on encoder type and allows > the DSPP reservation based on user request via CTM. > > The series will release/reserve the dpu resources whenever there is > a CTM enable/disable change so that DSPPs are allocated appropriately. > > [...] Applied, thanks! [2/4] drm/msm/dpu: add DSPPs into reservation upon a CTM request https://gitlab.freedesktop.org/lumag/msm/-/commit/1a9c3512fbd4 [3/4] drm/msm/dpu: avoid unnecessary check in DPU reservations https://gitlab.freedesktop.org/lumag/msm/-/commit/8b1ed0088e21 [4/4] drm/msm/dpu: manage DPU resources if CTM is requested https://gitlab.freedesktop.org/lumag/msm/-/commit/34c74e76a6a5 Best regards,