Allow setting a power cap that's lower than recommended

Mario Limonciello mario.limonciello at amd.com
Fri May 3 15:36:50 UTC 2024


On 5/3/2024 07:05, fililip wrote:
> This patch allows setting a low power cap if |ignore_min_pcap|​ is set to 1.
> 
> Signed-off-by: fililip <fililip6483 at protonmail.com>

Rather than an attachment you should send the patch inline.  That would 
mean that your commit message and SoB should be at the top of the patch 
itself.

If you're not aware of it, you should read through:

https://www.kernel.org/doc/html/latest/process/submitting-patches.html

Basically a maintainer should be able to run:

'b4 shazam $URL'

And it would download the patch and apply it.

That being said my comments on the patch content:

As Alex mentioned a concern about the the maintenance of more 
parameters, maybe it's worth making the sysfs file of the limit 
"writable" instead.

Then when it's written the kernel driver can add_taint() for 
TAINT_FIRMWARE_WORKAROUND when in use so we can see it clearly in logs.





More information about the amd-gfx mailing list