Re: [PATCH] drm/i915/selftests: Report missing RC0 power as an error

"Nilawar, Badal" <[email protected]>
Newsgroups org.freedesktop.lists.intel-gfx
Message-ID <[email protected]>
On 29-07-2026 16:54, Sk Anirban wrote:
> The RC0 no-power case is a hard failure (-EINVAL); log it via pr_err
> instead of pr_debug so it is captured in default kernel logs.
>
> Signed-off-by: Sk Anirban <[email protected]>
> ---
>   drivers/gpu/drm/i915/gt/selftest_rc6.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gt/selftest_rc6.c b/drivers/gpu/drm/i915/gt/selftest_rc6.c
> index 90f30b988d68..68fb1830d411 100644
> --- a/drivers/gpu/drm/i915/gt/selftest_rc6.c
> +++ b/drivers/gpu/drm/i915/gt/selftest_rc6.c
> @@ -92,8 +92,8 @@ int live_rc6_manual(void *arg)
>   
>   		if (!rc0_power) {
>   			if (rc0_freq)
> -				pr_debug("No power measured while in RC0! GPU Freq: %uMHz in RC0\n",
> -					 rc0_freq);
> +				pr_err("No power measured while in RC0! GPU Freq: %uMHz in RC0\n",
> +				       rc0_freq);

Looks good to me.

Reviewed-by: Badal Nilawar <[email protected]>

Thanks,
Badal

>   			else
>   				pr_err("No power and freq measured while in RC0\n");
>   			err = -EINVAL;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.