Received-SPF: pass (google.com: domain of linux-kernel+bounces-121571-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 27 Mar 2024 16:48:28 +0100
From: Borislav Petkov <bp@alien8.de>
To: Yazen Ghannam <yazen.ghannam@amd.com>
Cc: tony.luck@intel.com, linux-edac@vger.kernel.org,
	linux-kernel@vger.kernel.org, avadhut.naik@amd.com,
	john.allen@amd.com,
	"anthony s . knowles" <akira.2020@protonmail.com>
Subject: Re: [PATCH] RAS: Avoid build errors when CONFIG_DEBUG_FS=n
Message-ID: <20240327154828.GAZgQ_zK1iUbWxuXd0@fat_crate.local>
References: <20240326203252.2699278-1-yazen.ghannam@amd.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20240326203252.2699278-1-yazen.ghannam@amd.com>

On Tue, Mar 26, 2024 at 08:32:52PM +0000, Yazen Ghannam wrote:
> A new helper was introduced for RAS modules to get be able to get the
> RAS subsystem debugfs root directory. The helper is defined in debugfs.c
> which is only built when CONFIG_DEBUG_FS=y.
> 
> However, it's possible that the modules would include debugfs support
> for optional functionality. One current example is the fmpm module. In
> this case, a build error will occur when CONFIG_RAS_FMPM is selected and
> CONFIG_DEBUG_FS=n.
> 
> Add an inline helper function stub for the CONFIG_DEBUG_FS=n case.
> 
> Fixes: 9d2b6fa09d15 ("RAS: Export helper to get ras_debugfs_dir")
> Reported-by: anthony s. knowles <akira.2020@protonmail.com>
> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=218640
> Signed-off-by: Yazen Ghannam <yazen.ghannam@amd.com>
> Tested-by: anthony s. knowles <akira.2020@protonmail.com>
> Link: https://lore.kernel.org/r/20240325183755.776-1-bp@alien8.de
> ---
>  drivers/ras/debugfs.h | 4 ++++
>  1 file changed, 4 insertions(+)

Applied, thanks.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette