* [PATCH v2] Add PP_FEATURE_MASK comments
@ 2020-09-22 15:55 Ryan Taylor
2020-09-23 14:40 ` Luben Tuikov
0 siblings, 1 reply; 5+ messages in thread
From: Ryan Taylor @ 2020-09-22 15:55 UTC (permalink / raw)
To: amd-gfx; +Cc: Alex Deucher, Ryan Taylor
Documents PP_FEATURE_MASK enum.
Provides instructions on how to use ppfeaturemasks.
v2: Improved enum definitions. Adds kernel command line
parameters to ppfeaturemask instructions.
Signed-off-by: Ryan Taylor <ryan.taylor@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
---
drivers/gpu/drm/amd/include/amd_shared.h | 28 ++++++++++++++++++++++++
1 file changed, 28 insertions(+)
diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h
index e98c84ef206f..7c84d5beb600 100644
--- a/drivers/gpu/drm/amd/include/amd_shared.h
+++ b/drivers/gpu/drm/amd/include/amd_shared.h
@@ -128,6 +128,34 @@ enum amd_powergating_state {
#define AMD_PG_SUPPORT_ATHUB (1 << 16)
#define AMD_PG_SUPPORT_JPEG (1 << 17)
+/**
+* enum PP_FEATURE_MASK - Used to mask power play features.
+*
+* @PP_SCLK_DPM_MASK: Dynamic adjustment of the system (graphics) clock.
+* @PP_MCLK_DPM_MASK: Dynamic adjustment of the memory clock.
+* @PP_PCIE_DPM_MASK: Dynamic adjustment of PCIE clocks and lanes.
+* @PP_SCLK_DEEP_SLEEP_MASK: System (graphics) clock deep sleep.
+* @PP_POWER_CONTAINMENT_MASK: Power containment.
+* @PP_UVD_HANDSHAKE_MASK: Unified video decoder handshake.
+* @PP_SMC_VOLTAGE_CONTROL_MASK: Dynamic voltage control.
+* @PP_VBI_TIME_SUPPORT_MASK: Vertical blank interval support.
+* @PP_ULV_MASK: Ultra low voltage.
+* @PP_ENABLE_GFX_CG_THRU_SMU: SMU control of GFX engine clockgating.
+* @PP_CLOCK_STRETCH_MASK: Clock stretching.
+* @PP_OD_FUZZY_FAN_CONTROL_MASK: Overdrive fuzzy fan control.
+* @PP_SOCCLK_DPM_MASK: Dynamic adjustment of the SoC clock.
+* @PP_DCEFCLK_DPM_MASK: Dynamic adjustment of the Display Controller Engine Fabric clock.
+* @PP_OVERDRIVE_MASK: Over- and under-clocking support.
+* @PP_GFXOFF_MASK: Dynamic graphics engine power control.
+* @PP_ACG_MASK: Adaptive clock generator.
+* @PP_STUTTER_MODE: Stutter mode.
+* @PP_AVFS_MASK: Adaptive voltage and frequency scaling.
+*
+* To override these settings on boot, append amdgpu.ppfeaturemask=<mask> to
+* the kernel's command line parameters. This is usually done through a system's
+* boot loader (E.g. GRUB). If manually loading the driver, pass
+* ppfeaturemask=<mask> as a modprobe parameter.
+*/
enum PP_FEATURE_MASK {
PP_SCLK_DPM_MASK = 0x1,
PP_MCLK_DPM_MASK = 0x2,
--
2.28.0
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v2] Add PP_FEATURE_MASK comments 2020-09-22 15:55 [PATCH v2] Add PP_FEATURE_MASK comments Ryan Taylor @ 2020-09-23 14:40 ` Luben Tuikov 2020-09-23 16:31 ` Ryan Taylor 0 siblings, 1 reply; 5+ messages in thread From: Luben Tuikov @ 2020-09-23 14:40 UTC (permalink / raw) To: Ryan Taylor, amd-gfx; +Cc: Alex Deucher On 2020-09-22 11:55, Ryan Taylor wrote: > Documents PP_FEATURE_MASK enum. > Provides instructions on how to use ppfeaturemasks. > > v2: Improved enum definitions. Adds kernel command line > parameters to ppfeaturemask instructions. > > Signed-off-by: Ryan Taylor <ryan.taylor@amd.com> > Reviewed-by: Alex Deucher <alexander.deucher@amd.com> > --- > drivers/gpu/drm/amd/include/amd_shared.h | 28 ++++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h > index e98c84ef206f..7c84d5beb600 100644 > --- a/drivers/gpu/drm/amd/include/amd_shared.h > +++ b/drivers/gpu/drm/amd/include/amd_shared.h > @@ -128,6 +128,34 @@ enum amd_powergating_state { > #define AMD_PG_SUPPORT_ATHUB (1 << 16) > #define AMD_PG_SUPPORT_JPEG (1 << 17) > > +/** > +* enum PP_FEATURE_MASK - Used to mask power play features. > +* > +* @PP_SCLK_DPM_MASK: Dynamic adjustment of the system (graphics) clock. > +* @PP_MCLK_DPM_MASK: Dynamic adjustment of the memory clock. > +* @PP_PCIE_DPM_MASK: Dynamic adjustment of PCIE clocks and lanes. > +* @PP_SCLK_DEEP_SLEEP_MASK: System (graphics) clock deep sleep. > +* @PP_POWER_CONTAINMENT_MASK: Power containment. > +* @PP_UVD_HANDSHAKE_MASK: Unified video decoder handshake. > +* @PP_SMC_VOLTAGE_CONTROL_MASK: Dynamic voltage control. > +* @PP_VBI_TIME_SUPPORT_MASK: Vertical blank interval support. > +* @PP_ULV_MASK: Ultra low voltage. > +* @PP_ENABLE_GFX_CG_THRU_SMU: SMU control of GFX engine clockgating. > +* @PP_CLOCK_STRETCH_MASK: Clock stretching. > +* @PP_OD_FUZZY_FAN_CONTROL_MASK: Overdrive fuzzy fan control. > +* @PP_SOCCLK_DPM_MASK: Dynamic adjustment of the SoC clock. > +* @PP_DCEFCLK_DPM_MASK: Dynamic adjustment of the Display Controller Engine Fabric clock. > +* @PP_OVERDRIVE_MASK: Over- and under-clocking support. > +* @PP_GFXOFF_MASK: Dynamic graphics engine power control. > +* @PP_ACG_MASK: Adaptive clock generator. > +* @PP_STUTTER_MODE: Stutter mode. > +* @PP_AVFS_MASK: Adaptive voltage and frequency scaling. > +* > +* To override these settings on boot, append amdgpu.ppfeaturemask=<mask> to > +* the kernel's command line parameters. This is usually done through a system's > +* boot loader (E.g. GRUB). If manually loading the driver, pass > +* ppfeaturemask=<mask> as a modprobe parameter. > +*/ Alignment should be: /** * * ... */ Also, you can use Git's own patch version setting, but I've seen here we usually put it in the commit title. (Remember, titles do not end with a period.) Something like Commit title (v2) I don't mind using either. Not sure what the direction is on this one. Regards, Luben > enum PP_FEATURE_MASK { > PP_SCLK_DPM_MASK = 0x1, > PP_MCLK_DPM_MASK = 0x2, > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] Add PP_FEATURE_MASK comments 2020-09-23 14:40 ` Luben Tuikov @ 2020-09-23 16:31 ` Ryan Taylor 2020-09-23 17:13 ` Luben Tuikov 0 siblings, 1 reply; 5+ messages in thread From: Ryan Taylor @ 2020-09-23 16:31 UTC (permalink / raw) To: Luben Tuikov, amd-gfx; +Cc: Alex Deucher Thanks Luben, I'll make these revisions and try out Git's patch version setting. Best, Ryan On 9/23/20 7:40 AM, Luben Tuikov wrote: > On 2020-09-22 11:55, Ryan Taylor wrote: >> Documents PP_FEATURE_MASK enum. >> Provides instructions on how to use ppfeaturemasks. >> >> v2: Improved enum definitions. Adds kernel command line >> parameters to ppfeaturemask instructions. >> >> Signed-off-by: Ryan Taylor <ryan.taylor@amd.com> >> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> >> --- >> drivers/gpu/drm/amd/include/amd_shared.h | 28 ++++++++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h >> index e98c84ef206f..7c84d5beb600 100644 >> --- a/drivers/gpu/drm/amd/include/amd_shared.h >> +++ b/drivers/gpu/drm/amd/include/amd_shared.h >> @@ -128,6 +128,34 @@ enum amd_powergating_state { >> #define AMD_PG_SUPPORT_ATHUB (1 << 16) >> #define AMD_PG_SUPPORT_JPEG (1 << 17) >> >> +/** >> +* enum PP_FEATURE_MASK - Used to mask power play features. >> +* >> +* @PP_SCLK_DPM_MASK: Dynamic adjustment of the system (graphics) clock. >> +* @PP_MCLK_DPM_MASK: Dynamic adjustment of the memory clock. >> +* @PP_PCIE_DPM_MASK: Dynamic adjustment of PCIE clocks and lanes. >> +* @PP_SCLK_DEEP_SLEEP_MASK: System (graphics) clock deep sleep. >> +* @PP_POWER_CONTAINMENT_MASK: Power containment. >> +* @PP_UVD_HANDSHAKE_MASK: Unified video decoder handshake. >> +* @PP_SMC_VOLTAGE_CONTROL_MASK: Dynamic voltage control. >> +* @PP_VBI_TIME_SUPPORT_MASK: Vertical blank interval support. >> +* @PP_ULV_MASK: Ultra low voltage. >> +* @PP_ENABLE_GFX_CG_THRU_SMU: SMU control of GFX engine clockgating. >> +* @PP_CLOCK_STRETCH_MASK: Clock stretching. >> +* @PP_OD_FUZZY_FAN_CONTROL_MASK: Overdrive fuzzy fan control. >> +* @PP_SOCCLK_DPM_MASK: Dynamic adjustment of the SoC clock. >> +* @PP_DCEFCLK_DPM_MASK: Dynamic adjustment of the Display Controller Engine Fabric clock. >> +* @PP_OVERDRIVE_MASK: Over- and under-clocking support. >> +* @PP_GFXOFF_MASK: Dynamic graphics engine power control. >> +* @PP_ACG_MASK: Adaptive clock generator. >> +* @PP_STUTTER_MODE: Stutter mode. >> +* @PP_AVFS_MASK: Adaptive voltage and frequency scaling. >> +* >> +* To override these settings on boot, append amdgpu.ppfeaturemask=<mask> to >> +* the kernel's command line parameters. This is usually done through a system's >> +* boot loader (E.g. GRUB). If manually loading the driver, pass >> +* ppfeaturemask=<mask> as a modprobe parameter. >> +*/ > > Alignment should be: > > /** > * > * > ... > */ > > Also, you can use Git's own patch version setting, > but I've seen here we usually put it in the commit title. > (Remember, titles do not end with a period.) Something like > > Commit title (v2) > > I don't mind using either. Not sure what the direction is > on this one. > > Regards, > Luben > >> enum PP_FEATURE_MASK { >> PP_SCLK_DPM_MASK = 0x1, >> PP_MCLK_DPM_MASK = 0x2, >> > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] Add PP_FEATURE_MASK comments 2020-09-23 16:31 ` Ryan Taylor @ 2020-09-23 17:13 ` Luben Tuikov 2020-09-23 17:27 ` Ryan Taylor 0 siblings, 1 reply; 5+ messages in thread From: Luben Tuikov @ 2020-09-23 17:13 UTC (permalink / raw) To: Ryan Taylor, amd-gfx; +Cc: Alex Deucher On 2020-09-23 12:31, Ryan Taylor wrote: > Thanks Luben, > > I'll make these revisions and try out Git's patch version setting. Well, Git's own patch revision command line option (-v of git-format-patch) would put it in the "[PATCH v<N>] Title text" as you had it here, and I'd presumed that you'd used that command line option. Our practice here is to manually do "[PATCH] Title text (v<N>)", where the patch version is postfixed. Regards, Luben > > Best, > Ryan > > On 9/23/20 7:40 AM, Luben Tuikov wrote: >> On 2020-09-22 11:55, Ryan Taylor wrote: >>> Documents PP_FEATURE_MASK enum. >>> Provides instructions on how to use ppfeaturemasks. >>> >>> v2: Improved enum definitions. Adds kernel command line >>> parameters to ppfeaturemask instructions. >>> >>> Signed-off-by: Ryan Taylor <ryan.taylor@amd.com> >>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> >>> --- >>> drivers/gpu/drm/amd/include/amd_shared.h | 28 ++++++++++++++++++++++++ >>> 1 file changed, 28 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h >>> index e98c84ef206f..7c84d5beb600 100644 >>> --- a/drivers/gpu/drm/amd/include/amd_shared.h >>> +++ b/drivers/gpu/drm/amd/include/amd_shared.h >>> @@ -128,6 +128,34 @@ enum amd_powergating_state { >>> #define AMD_PG_SUPPORT_ATHUB (1 << 16) >>> #define AMD_PG_SUPPORT_JPEG (1 << 17) >>> >>> +/** >>> +* enum PP_FEATURE_MASK - Used to mask power play features. >>> +* >>> +* @PP_SCLK_DPM_MASK: Dynamic adjustment of the system (graphics) clock. >>> +* @PP_MCLK_DPM_MASK: Dynamic adjustment of the memory clock. >>> +* @PP_PCIE_DPM_MASK: Dynamic adjustment of PCIE clocks and lanes. >>> +* @PP_SCLK_DEEP_SLEEP_MASK: System (graphics) clock deep sleep. >>> +* @PP_POWER_CONTAINMENT_MASK: Power containment. >>> +* @PP_UVD_HANDSHAKE_MASK: Unified video decoder handshake. >>> +* @PP_SMC_VOLTAGE_CONTROL_MASK: Dynamic voltage control. >>> +* @PP_VBI_TIME_SUPPORT_MASK: Vertical blank interval support. >>> +* @PP_ULV_MASK: Ultra low voltage. >>> +* @PP_ENABLE_GFX_CG_THRU_SMU: SMU control of GFX engine clockgating. >>> +* @PP_CLOCK_STRETCH_MASK: Clock stretching. >>> +* @PP_OD_FUZZY_FAN_CONTROL_MASK: Overdrive fuzzy fan control. >>> +* @PP_SOCCLK_DPM_MASK: Dynamic adjustment of the SoC clock. >>> +* @PP_DCEFCLK_DPM_MASK: Dynamic adjustment of the Display Controller Engine Fabric clock. >>> +* @PP_OVERDRIVE_MASK: Over- and under-clocking support. >>> +* @PP_GFXOFF_MASK: Dynamic graphics engine power control. >>> +* @PP_ACG_MASK: Adaptive clock generator. >>> +* @PP_STUTTER_MODE: Stutter mode. >>> +* @PP_AVFS_MASK: Adaptive voltage and frequency scaling. >>> +* >>> +* To override these settings on boot, append amdgpu.ppfeaturemask=<mask> to >>> +* the kernel's command line parameters. This is usually done through a system's >>> +* boot loader (E.g. GRUB). If manually loading the driver, pass >>> +* ppfeaturemask=<mask> as a modprobe parameter. >>> +*/ >> >> Alignment should be: >> >> /** >> * >> * >> ... >> */ >> >> Also, you can use Git's own patch version setting, >> but I've seen here we usually put it in the commit title. >> (Remember, titles do not end with a period.) Something like >> >> Commit title (v2) >> >> I don't mind using either. Not sure what the direction is >> on this one. >> >> Regards, >> Luben >> >>> enum PP_FEATURE_MASK { >>> PP_SCLK_DPM_MASK = 0x1, >>> PP_MCLK_DPM_MASK = 0x2, >>> >> _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] Add PP_FEATURE_MASK comments 2020-09-23 17:13 ` Luben Tuikov @ 2020-09-23 17:27 ` Ryan Taylor 0 siblings, 0 replies; 5+ messages in thread From: Ryan Taylor @ 2020-09-23 17:27 UTC (permalink / raw) To: Luben Tuikov, amd-gfx; +Cc: Alex Deucher I see, I'll add it to the end next time then. Ryan On 9/23/20 10:13 AM, Luben Tuikov wrote: > On 2020-09-23 12:31, Ryan Taylor wrote: >> Thanks Luben, >> >> I'll make these revisions and try out Git's patch version setting. > > Well, Git's own patch revision command line option (-v of git-format-patch) > would put it in the "[PATCH v<N>] Title text" as you had it here, and I'd > presumed that you'd used that command line option. Our practice > here is to manually do "[PATCH] Title text (v<N>)", where the patch version > is postfixed. > > Regards, > Luben > >> >> Best, >> Ryan >> >> On 9/23/20 7:40 AM, Luben Tuikov wrote: >>> On 2020-09-22 11:55, Ryan Taylor wrote: >>>> Documents PP_FEATURE_MASK enum. >>>> Provides instructions on how to use ppfeaturemasks. >>>> >>>> v2: Improved enum definitions. Adds kernel command line >>>> parameters to ppfeaturemask instructions. >>>> >>>> Signed-off-by: Ryan Taylor <ryan.taylor@amd.com> >>>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> >>>> --- >>>> drivers/gpu/drm/amd/include/amd_shared.h | 28 ++++++++++++++++++++++++ >>>> 1 file changed, 28 insertions(+) >>>> >>>> diff --git a/drivers/gpu/drm/amd/include/amd_shared.h b/drivers/gpu/drm/amd/include/amd_shared.h >>>> index e98c84ef206f..7c84d5beb600 100644 >>>> --- a/drivers/gpu/drm/amd/include/amd_shared.h >>>> +++ b/drivers/gpu/drm/amd/include/amd_shared.h >>>> @@ -128,6 +128,34 @@ enum amd_powergating_state { >>>> #define AMD_PG_SUPPORT_ATHUB (1 << 16) >>>> #define AMD_PG_SUPPORT_JPEG (1 << 17) >>>> >>>> +/** >>>> +* enum PP_FEATURE_MASK - Used to mask power play features. >>>> +* >>>> +* @PP_SCLK_DPM_MASK: Dynamic adjustment of the system (graphics) clock. >>>> +* @PP_MCLK_DPM_MASK: Dynamic adjustment of the memory clock. >>>> +* @PP_PCIE_DPM_MASK: Dynamic adjustment of PCIE clocks and lanes. >>>> +* @PP_SCLK_DEEP_SLEEP_MASK: System (graphics) clock deep sleep. >>>> +* @PP_POWER_CONTAINMENT_MASK: Power containment. >>>> +* @PP_UVD_HANDSHAKE_MASK: Unified video decoder handshake. >>>> +* @PP_SMC_VOLTAGE_CONTROL_MASK: Dynamic voltage control. >>>> +* @PP_VBI_TIME_SUPPORT_MASK: Vertical blank interval support. >>>> +* @PP_ULV_MASK: Ultra low voltage. >>>> +* @PP_ENABLE_GFX_CG_THRU_SMU: SMU control of GFX engine clockgating. >>>> +* @PP_CLOCK_STRETCH_MASK: Clock stretching. >>>> +* @PP_OD_FUZZY_FAN_CONTROL_MASK: Overdrive fuzzy fan control. >>>> +* @PP_SOCCLK_DPM_MASK: Dynamic adjustment of the SoC clock. >>>> +* @PP_DCEFCLK_DPM_MASK: Dynamic adjustment of the Display Controller Engine Fabric clock. >>>> +* @PP_OVERDRIVE_MASK: Over- and under-clocking support. >>>> +* @PP_GFXOFF_MASK: Dynamic graphics engine power control. >>>> +* @PP_ACG_MASK: Adaptive clock generator. >>>> +* @PP_STUTTER_MODE: Stutter mode. >>>> +* @PP_AVFS_MASK: Adaptive voltage and frequency scaling. >>>> +* >>>> +* To override these settings on boot, append amdgpu.ppfeaturemask=<mask> to >>>> +* the kernel's command line parameters. This is usually done through a system's >>>> +* boot loader (E.g. GRUB). If manually loading the driver, pass >>>> +* ppfeaturemask=<mask> as a modprobe parameter. >>>> +*/ >>> >>> Alignment should be: >>> >>> /** >>> * >>> * >>> ... >>> */ >>> >>> Also, you can use Git's own patch version setting, >>> but I've seen here we usually put it in the commit title. >>> (Remember, titles do not end with a period.) Something like >>> >>> Commit title (v2) >>> >>> I don't mind using either. Not sure what the direction is >>> on this one. >>> >>> Regards, >>> Luben >>> >>>> enum PP_FEATURE_MASK { >>>> PP_SCLK_DPM_MASK = 0x1, >>>> PP_MCLK_DPM_MASK = 0x2, >>>> >>> > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-09-23 17:28 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-09-22 15:55 [PATCH v2] Add PP_FEATURE_MASK comments Ryan Taylor 2020-09-23 14:40 ` Luben Tuikov 2020-09-23 16:31 ` Ryan Taylor 2020-09-23 17:13 ` Luben Tuikov 2020-09-23 17:27 ` Ryan Taylor
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox