[PATCH] platform/x86/amd: pmc: remove CONFIG_DEBUG_FS checks

[PATCH] platform/x86/amd: pmc: remove CONFIG_DEBUG_FS checks

[Nils Freydank]

commit b37fe34c83099ba5105115f8287c5546af1f0a05 upstream.

This fixes a compilation bug introduced in commit
e9847175b266f12365160e124a207907da3dbe8e (platform/x86/amd: pmc: Read SMU
version during suspend on Cezanne systems).

Signed-off-by: Nils Freydank <nils.freydank@posteo.de>
---

 Backport patch applies and compiles with linux 6.0.6.
 Feel free to use the whole patch or any parts of it at your discretion.

 drivers/platform/x86/amd/pmc.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/drivers/platform/x86/amd/pmc.c b/drivers/platform/x86/amd/pmc.c
index fc326fdf4..6ccb9b384 100644
--- a/drivers/platform/x86/amd/pmc.c
+++ b/drivers/platform/x86/amd/pmc.c
@@ -151,9 +151,7 @@ struct amd_pmc_dev {
 	struct device *dev;
 	struct pci_dev *rdev;
 	struct mutex lock; /* generic mutex lock */
-#if IS_ENABLED(CONFIG_DEBUG_FS)
 	struct dentry *dbgfs_dir;
-#endif /* CONFIG_DEBUG_FS */
 };
 
 static bool enable_stb;
@@ -369,7 +367,6 @@ static void amd_pmc_validate_deepest(struct amd_pmc_dev *pdev)
 }
 #endif
 
-#ifdef CONFIG_DEBUG_FS
 static int smu_fw_info_show(struct seq_file *s, void *unused)
 {
 	struct amd_pmc_dev *dev = s->private;
@@ -504,15 +501,6 @@ static void amd_pmc_dbgfs_register(struct amd_pmc_dev *dev)
 					    &amd_pmc_stb_debugfs_fops);
 	}
 }
-#else
-static inline void amd_pmc_dbgfs_register(struct amd_pmc_dev *dev)
-{
-}
-
-static inline void amd_pmc_dbgfs_unregister(struct amd_pmc_dev *dev)
-{
-}
-#endif /* CONFIG_DEBUG_FS */
 
 static void amd_pmc_dump_registers(struct amd_pmc_dev *dev)
 {
-- 
2.38.1

Re: [PATCH] platform/x86/amd: pmc: remove CONFIG_DEBUG_FS checks

[Greg KH]

On Mon, Oct 31, 2022 at 10:36:53PM +0000, Nils Freydank wrote:
> commit b37fe34c83099ba5105115f8287c5546af1f0a05 upstream.
> 
> This fixes a compilation bug introduced in commit
> e9847175b266f12365160e124a207907da3dbe8e (platform/x86/amd: pmc: Read SMU
> version during suspend on Cezanne systems).
> 
> Signed-off-by: Nils Freydank <nils.freydank@posteo.de>
> ---
> 
>  Backport patch applies and compiles with linux 6.0.6.

This is already in the 6.0.y queue, but thanks for sending it again.

Next time, be sure you keep the original changelog text _AND_ authorship
and signed-off-by lines, and cc: those developers as well.  Stripping
that off is a bit rude of them, don't you think?

thanks,

greg k-h

Re: [PATCH] platform/x86/amd: pmc: remove CONFIG_DEBUG_FS checks

[Nils Freydank]

Am Dienstag, den 01.11.2022 um 05:51:21 Uhr +0100 schrieb Greg KH <greg@kroah.com>:
> On Mon, Oct 31, 2022 at 10:36:53PM +0000, Nils Freydank wrote:
> > commit b37fe34c83099ba5105115f8287c5546af1f0a05 upstream.
> > 
> > This fixes a compilation bug introduced in commit
> > e9847175b266f12365160e124a207907da3dbe8e (platform/x86/amd: pmc: Read SMU
> > version during suspend on Cezanne systems).
> > 
> > Signed-off-by: Nils Freydank <nils.freydank@posteo.de>
> > ---
> > 
> >  Backport patch applies and compiles with linux 6.0.6.
> 
> This is already in the 6.0.y queue, but thanks for sending it again.
> 
> Next time, be sure you keep the original changelog text _AND_ authorship
> and signed-off-by lines, and cc: those developers as well.  Stripping
> that off is a bit rude of them, don't you think?

Thanks for the clear instructions, I'll keep them in mind for the future.

> thanks,
> 
> greg k-h