Re: [PATCH v2 4/4] platform/x86/amd: pmc: Use pm_pr_dbg() for suspend related messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Mario Limonciello <mario.limonciello@xxxxxxx>
Subject: Re: [PATCH v2 4/4] platform/x86/amd: pmc: Use pm_pr_dbg() for suspend related messages
From: andy.shevchenko@xxxxxxxxx
Date: Tue, 23 May 2023 19:56:11 +0300
Cc: rafael@xxxxxxxxxx, hdegoede@xxxxxxxxxx, linus.walleij@xxxxxxxxxx, linux-acpi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-gpio@xxxxxxxxxxxxxxx, platform-driver-x86@xxxxxxxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, Shyam-sundar.S-k@xxxxxxx, Basavaraj.Natikar@xxxxxxx
In-reply-to: <20230522200033.2605-4-mario.limonciello@amd.com>
References: <20230522200033.2605-1-mario.limonciello@amd.com> <20230522200033.2605-4-mario.limonciello@amd.com>

Mon, May 22, 2023 at 03:00:33PM -0500, Mario Limonciello kirjoitti:
> Using pm_pr_dbg() allows users to toggle `/sys/power/pm_debug_messages`
> as a single knob to turn on messages that amd-pmc can emit to aid in
> any s2idle debugging.

It has the same two regressions as I pointed out in previous reply.

-- 
With Best Regards,
Andy Shevchenko

References:
- [PATCH v2 1/4] include/linux/suspend.h: Only show pm_pr_dbg messages at suspend/resume
  - From: Mario Limonciello
- [PATCH v2 4/4] platform/x86/amd: pmc: Use pm_pr_dbg() for suspend related messages
  - From: Mario Limonciello

Prev by Date: Re: [PATCH v2 2/4] ACPI: x86: Add pm_debug_messages for LPS0 _DSM state tracking
Next by Date: Re: [PATCH v2 3/4] pinctrl: amd: Use pm_pr_dbg to show debugging messages
Previous by thread: Re: [PATCH v2 4/4] platform/x86/amd: pmc: Use pm_pr_dbg() for suspend related messages
Next by thread: Re: [PATCH v2 1/4] include/linux/suspend.h: Only show pm_pr_dbg messages at suspend/resume
Index(es):
- Date
- Thread

[Index of Archives] [Linux IBM ACPI] [Linux Power Management] [Linux Kernel] [Linux Laptop] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]