On Fri, Dec 17, 2021 at 4:51 PM Rodrigo Siqueira Jordao <rjordrigo@xxxxxxx> wrote: > > > > On 2021-12-17 4:36 p.m., Deucher, Alexander wrote: > > [AMD Official Use Only] > > > > > > Maybe add Bug links for: > > https://gitlab.freedesktop.org/drm/amd/-/issues/1522 > > <https://gitlab.freedesktop.org/drm/amd/-/issues/1522> > > https://gitlab.freedesktop.org/drm/amd/-/issues/1709 > > <https://gitlab.freedesktop.org/drm/amd/-/issues/1709> > > https://gitlab.freedesktop.org/drm/amd/-/issues/1655 > > <https://gitlab.freedesktop.org/drm/amd/-/issues/1655> > > https://gitlab.freedesktop.org/drm/amd/-/issues/1403 > > <https://gitlab.freedesktop.org/drm/amd/-/issues/1403> > > Sure, I'll update the commit message before apply this patch. > > Thanks. > > > > > > > > > ------------------------------------------------------------------------ > > *From:* amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of > > Rodrigo Siqueira <Rodrigo.Siqueira@xxxxxxx> > > *Sent:* Friday, December 17, 2021 4:23 PM > > *To:* amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> > > *Cc:* Wang, Chao-kai (Stylon) <Stylon.Wang@xxxxxxx>; Cyr, Aric > > <Aric.Cyr@xxxxxxx>; Li, Sun peng (Leo) <Sunpeng.Li@xxxxxxx>; Wentland, > > Harry <Harry.Wentland@xxxxxxx>; Zhuo, Qingqing (Lillian) > > <Qingqing.Zhuo@xxxxxxx>; Siqueira, Rodrigo <Rodrigo.Siqueira@xxxxxxx>; > > Li, Roman <Roman.Li@xxxxxxx>; Chiu, Solomon <Solomon.Chiu@xxxxxxx>; > > Pillai, Aurabindo <Aurabindo.Pillai@xxxxxxx>; Wang, Angus > > <Angus.Wang@xxxxxxx>; Lin, Wayne <Wayne.Lin@xxxxxxx>; Lipski, Mikita > > <Mikita.Lipski@xxxxxxx>; Lakha, Bhawanpreet <Bhawanpreet.Lakha@xxxxxxx>; > > Gutierrez, Agustin <Agustin.Gutierrez@xxxxxxx>; Kotarac, Pavle > > <Pavle.Kotarac@xxxxxxx> > > *Subject:* [PATCH 10/19] drm/amd/display: Changed pipe split policy to > > allow for multi-display pipe split > > From: Angus Wang <angus.wang@xxxxxxx> > > > > [WHY] > > Current implementation of pipe split policy prevents pipe split with > > multiple displays connected, which caused the MCLK speed to be stuck at > > max > > > > [HOW] > > Changed the pipe split policies so that pipe split is allowed for > > multi-display configurations > > > > Reviewed-by: Aric Cyr <Aric.Cyr@xxxxxxx> > > Acked-by: Rodrigo Siqueira <Rodrigo.Siqueira@xxxxxxx> > > Signed-off-by: Angus Wang <angus.wang@xxxxxxx> > > --- > > drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c | 2 +- > > drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c | 2 +- Also, it looks like dcn10_resource.c was missed. Was that intentional? Alex > > 8 files changed, 8 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c > > index 2a72517e2b28..2bc93df023ad 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c > > @@ -1069,7 +1069,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c > > index d6acf9a8590a..0bb7d3dd53fa 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c > > @@ -603,7 +603,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c > > index ca1bbc942fd4..e5cc6bf45743 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c > > @@ -873,7 +873,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > .min_disp_clk_khz = 100000, > > - .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c > > index 369ceeeddc7e..e12660c609ee 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c > > @@ -840,7 +840,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c > > index b4001233867c..c1c6e602b06c 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c > > @@ -686,7 +686,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .disable_clock_gate = true, > > .disable_pplib_clock_request = true, > > .disable_pplib_wm_range = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c > > index 003e95368672..2e9cbfa7663b 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c > > @@ -211,7 +211,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c > > index 01ba9d656c72..2de687f64cf6 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c > > @@ -193,7 +193,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = true, > > - .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > diff --git a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c > > b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c > > index 6d07dcecc953..7c64317a56f7 100644 > > --- a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c > > @@ -1003,7 +1003,7 @@ static const struct dc_debug_options > > debug_defaults_drv = { > > .timing_trace = false, > > .clock_trace = true, > > .disable_pplib_clock_request = false, > > - .pipe_split_policy = MPC_SPLIT_AVOID, > > + .pipe_split_policy = MPC_SPLIT_DYNAMIC, > > .force_single_disp_pipe_split = false, > > .disable_dcc = DCC_ENABLE, > > .vsr_support = true, > > -- > > 2.25.1 > > >
