linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message
@ 2025-07-11 14:00 Sunil V L
  2025-07-11 16:24 ` Anup Patel
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Sunil V L @ 2025-07-11 14:00 UTC (permalink / raw)
  To: linux-acpi, linux-riscv, linux-kernel
  Cc: Rafael J . Wysocki, Len Brown, Paul Walmsley, Palmer Dabbelt,
	Alexandre Ghiti, Anup Patel, Sunil V L

The presence or absence of the CPPC SBI extension is currently logged
on every boot. This message is not particularly useful and can clutter
the boot log. Remove this debug message to reduce noise during boot.

This change has no functional impact.

Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
---
 drivers/acpi/riscv/cppc.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/acpi/riscv/cppc.c b/drivers/acpi/riscv/cppc.c
index 4cdff387deff..440cf9fb91aa 100644
--- a/drivers/acpi/riscv/cppc.c
+++ b/drivers/acpi/riscv/cppc.c
@@ -37,10 +37,8 @@ static int __init sbi_cppc_init(void)
 {
 	if (sbi_spec_version >= sbi_mk_version(2, 0) &&
 	    sbi_probe_extension(SBI_EXT_CPPC) > 0) {
-		pr_info("SBI CPPC extension detected\n");
 		cppc_ext_present = true;
 	} else {
-		pr_info("SBI CPPC extension NOT detected!!\n");
 		cppc_ext_present = false;
 	}
 
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message
  2025-07-11 14:00 [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message Sunil V L
@ 2025-07-11 16:24 ` Anup Patel
  2025-07-11 17:45 ` Drew Fustini
  2025-07-16 20:00 ` patchwork-bot+linux-riscv
  2 siblings, 0 replies; 4+ messages in thread
From: Anup Patel @ 2025-07-11 16:24 UTC (permalink / raw)
  To: Sunil V L
  Cc: linux-acpi, linux-riscv, linux-kernel, Rafael J . Wysocki,
	Len Brown, Paul Walmsley, Palmer Dabbelt, Alexandre Ghiti,
	Anup Patel

On Fri, Jul 11, 2025 at 7:30 PM Sunil V L <sunilvl@ventanamicro.com> wrote:
>
> The presence or absence of the CPPC SBI extension is currently logged
> on every boot. This message is not particularly useful and can clutter
> the boot log. Remove this debug message to reduce noise during boot.
>
> This change has no functional impact.
>
> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>

LGTM.

Reviewed-by: Anup Patel <anup@brainfault.org>

Regards,
Anup

> ---
>  drivers/acpi/riscv/cppc.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/acpi/riscv/cppc.c b/drivers/acpi/riscv/cppc.c
> index 4cdff387deff..440cf9fb91aa 100644
> --- a/drivers/acpi/riscv/cppc.c
> +++ b/drivers/acpi/riscv/cppc.c
> @@ -37,10 +37,8 @@ static int __init sbi_cppc_init(void)
>  {
>         if (sbi_spec_version >= sbi_mk_version(2, 0) &&
>             sbi_probe_extension(SBI_EXT_CPPC) > 0) {
> -               pr_info("SBI CPPC extension detected\n");
>                 cppc_ext_present = true;
>         } else {
> -               pr_info("SBI CPPC extension NOT detected!!\n");
>                 cppc_ext_present = false;
>         }
>
> --
> 2.43.0
>
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message
  2025-07-11 14:00 [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message Sunil V L
  2025-07-11 16:24 ` Anup Patel
@ 2025-07-11 17:45 ` Drew Fustini
  2025-07-16 20:00 ` patchwork-bot+linux-riscv
  2 siblings, 0 replies; 4+ messages in thread
From: Drew Fustini @ 2025-07-11 17:45 UTC (permalink / raw)
  To: Sunil V L
  Cc: linux-acpi, linux-riscv, linux-kernel, Rafael J . Wysocki,
	Len Brown, Paul Walmsley, Palmer Dabbelt, Alexandre Ghiti,
	Anup Patel

On Fri, Jul 11, 2025 at 07:30:13PM +0530, Sunil V L wrote:
> The presence or absence of the CPPC SBI extension is currently logged
> on every boot. This message is not particularly useful and can clutter
> the boot log. Remove this debug message to reduce noise during boot.
> 
> This change has no functional impact.
> 
> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>

Thanks, I'm pleased to no longer see the "NOT dectected!!" when testing
defconfig on systems which don't have the CPPC extension.

Tested-by: Drew Fustini <fustini@kernel.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message
  2025-07-11 14:00 [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message Sunil V L
  2025-07-11 16:24 ` Anup Patel
  2025-07-11 17:45 ` Drew Fustini
@ 2025-07-16 20:00 ` patchwork-bot+linux-riscv
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+linux-riscv @ 2025-07-16 20:00 UTC (permalink / raw)
  To: Sunil V L
  Cc: linux-riscv, linux-acpi, linux-kernel, apatel, alex, rafael,
	palmer, paul.walmsley, lenb

Hello:

This patch was applied to riscv/linux.git (fixes)
by Palmer Dabbelt <palmer@dabbelt.com>:

On Fri, 11 Jul 2025 19:30:13 +0530 you wrote:
> The presence or absence of the CPPC SBI extension is currently logged
> on every boot. This message is not particularly useful and can clutter
> the boot log. Remove this debug message to reduce noise during boot.
> 
> This change has no functional impact.
> 
> Signed-off-by: Sunil V L <sunilvl@ventanamicro.com>
> 
> [...]

Here is the summary with links:
  - ACPI: RISC-V: Remove unnecessary CPPC debug message
    https://git.kernel.org/riscv/c/16d743606dba

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-07-16 19:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-11 14:00 [PATCH] ACPI: RISC-V: Remove unnecessary CPPC debug message Sunil V L
2025-07-11 16:24 ` Anup Patel
2025-07-11 17:45 ` Drew Fustini
2025-07-16 20:00 ` patchwork-bot+linux-riscv

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).