[PATCH 12/15] drm/amd/display: Use hw lock mgr

[Wood, Wyatt]

The condition in which it’s enabled is hidden from upstream.

From: Brandon Wright <bearoso at gmail.com>
Sent: Saturday, August 8, 2020 10:11 PM
To: Brol, Eryk <Eryk.Brol at amd.com>; amd-gfx at lists.freedesktop.org; Wood, Wyatt <Wyatt.Wood at amd.com>
Subject: Re: [PATCH 12/15] drm/amd/display: Use hw lock mgr

Just curious, but I noticed this new lock manager isn't being used because of the definitive false return value in display/dc/dce/dmub_hw_lock_mgr.c:should_use_dmub_lock.
Was this supposed to be enabled?


From: Wyatt Wood <wyatt.w... at amd.com<mailto:wyatt.w... at amd.com>>



[Why]

Feature requires synchronization of dig, pipe, and cursor locking

between driver and fw.

[How]

Set flag to force psr to use hw lock mgr.



Signed-off-by: Wyatt Wood <wyatt.w... at amd.com<mailto:wyatt.w... at amd.com>>

Reviewed-by: Anthony Koo <anthony.... at amd.com<mailto:anthony.... at amd.com>>

Acked-by: Eryk Brol <eryk.b... at amd.com<mailto:eryk.b... at amd.com>>

---

 drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c

b/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c

index 82e67bd81f2d..5167d6b8a48d 100644

--- a/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c

+++ b/drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c

@@ -233,8 +233,8 @@ static bool dmub_psr_copy_settings(struct dmub_psr *dmub,

        copy_settings_data->frame_cap_ind                       =

psr_context->psrFrameCaptureIndicationReq;

        copy_settings_data->debug.bitfields.visual_confirm      =

dc->dc->debug.visual_confirm == VISUAL_CONFIRM_PSR ?

                                                                        true :

false;

+       copy_settings_data->debug.bitfields.use_hw_lock_mgr     = 1;

        copy_settings_data->init_sdp_deadline                   =

psr_context->sdpTransmitLineNumDeadline;

-       copy_settings_data->debug.bitfields.use_hw_lock_mgr     = 0;



        dc_dmub_srv_cmd_queue(dc->dmub_srv, &cmd);

        dc_dmub_srv_cmd_execute(dc->dmub_srv);

--

2.25.1
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20200810/6e24c805/attachment.htm>