[Public] Appreciate for the feedback!
Regards, Wayne > -----Original Message----- > From: Christian König <[email protected]> > Sent: Tuesday, March 12, 2024 6:21 PM > To: Lin, Wayne <[email protected]>; [email protected] > Cc: Wentland, Harry <[email protected]>; Li, Sun peng (Leo) > <[email protected]>; Siqueira, Rodrigo <[email protected]>; > Pillai, Aurabindo <[email protected]>; Li, Roman > <[email protected]>; Gutierrez, Agustin <[email protected]>; > Chung, ChiaHsuan (Tom) <[email protected]>; Wu, Hersen > <[email protected]>; Zuo, Jerry <[email protected]> > Subject: Re: [PATCH 03/43] drm/amd/display: Enable 2to1 ODM policy for > DCN35 > > Just another general comment on how to upstream patches. > > When publishing a large set of patches it is usually good convention to sort > them: > 1. Bug fixes which might even get backported 2. Comment and other non > function cleanups 3. Functional cleanups 4. New features > > One good reason for that is that it usually makes fixes much easier to port to > older kernel versions, but it also makes things easier to review. > > If you are in doubt if a patch set is still fully compiling after re-ordering > things > you can use the command > > git rebase -x make base_branch > > On your branch and git will run a make between after applying each patch. This > way you can double check that everything still builds fine. > > Working like that is not a must have, but really good practice. > > Regards, > Christian. > > Am 12.03.24 um 10:19 schrieb Wayne Lin: > > From: Rodrigo Siqueira <[email protected]> > > > > [Why & How] > > Enable 2to1 ODM policy for DCN35 > > > > Acked-by: Wayne Lin <[email protected]> > > Signed-off-by: Rodrigo Siqueira <[email protected]> > > --- > > drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git > > a/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c > > b/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c > > index 5d52853cac96..a8f4023ff3b1 100644 > > --- a/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c > > @@ -769,7 +769,7 @@ static const struct dc_debug_options > debug_defaults_drv = { > > .support_eDP1_5 = true, > > .enable_hpo_pg_support = false, > > .enable_legacy_fast_update = true, > > - .enable_single_display_2to1_odm_policy = false, > > + .enable_single_display_2to1_odm_policy = true, > > .disable_idle_power_optimizations = false, > > .dmcub_emulation = false, > > .disable_boot_optimizations = false,
