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(-)
--
2.7.4
Hi,
On Mon, Feb 13, 2023 at 3:11 AM Kalyan Thota <[email protected]> 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 <[email protected]>
[1] https://lore.kernel.org/r/[email protected]/
On Mon, 13 Feb 2023 at 20:12, Doug Anderson <[email protected]> wrote:
>
> Hi,
>
> On Mon, Feb 13, 2023 at 3:11 AM Kalyan Thota <[email protected]> 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 <[email protected]>
Thanks for the testing!
--
With best wishes
Dmitry
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,
--
Dmitry Baryshkov <[email protected]>