* [RFC,v4,1/3] EDAC, GHES: Remove unused argument to ghes_edac_report_mem_error
@ 2018-04-30 21:33 Alexandru Gagniuc
0 siblings, 0 replies; 2+ messages in thread
From: Alexandru Gagniuc @ 2018-04-30 21:33 UTC (permalink / raw)
To: bp
Cc: alex_gagniuc, austin_bolen, shyam_iyer, Alexandru Gagniuc,
Rafael J. Wysocki, Len Brown, Tony Luck, Mauro Carvalho Chehab,
Robert Moore, Erik Schmauss, Tyler Baicar, Will Deacon,
James Morse, Shiju Jose, Jonathan (Zhixiong) Zhang, Dongjiu Geng,
linux-acpi, linux-kernel, linux-edac, devel
The use of the 'ghes' argument was removed in a previous commit, but
function signature was not updated to reflect this.
Fixes: 0fe5f281f749 ("EDAC, ghes: Model a single, logical memory controller")
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
---
drivers/acpi/apei/ghes.c | 2 +-
drivers/edac/ghes_edac.c | 3 +--
include/acpi/ghes.h | 5 ++---
3 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/acpi/apei/ghes.c b/drivers/acpi/apei/ghes.c
index 1efefe919555..f9b53a6f55f3 100644
--- a/drivers/acpi/apei/ghes.c
+++ b/drivers/acpi/apei/ghes.c
@@ -481,7 +481,7 @@ static void ghes_do_proc(struct ghes *ghes,
if (guid_equal(sec_type, &CPER_SEC_PLATFORM_MEM)) {
struct cper_sec_mem_err *mem_err = acpi_hest_get_payload(gdata);
- ghes_edac_report_mem_error(ghes, sev, mem_err);
+ ghes_edac_report_mem_error(sev, mem_err);
arch_apei_report_mem_error(sev, mem_err);
ghes_handle_memory_failure(gdata, sev);
diff --git a/drivers/edac/ghes_edac.c b/drivers/edac/ghes_edac.c
index 68b6ee18bea6..32bb8c5f47dc 100644
--- a/drivers/edac/ghes_edac.c
+++ b/drivers/edac/ghes_edac.c
@@ -172,8 +172,7 @@ static void ghes_edac_dmidecode(const struct dmi_header *dh, void *arg)
}
}
-void ghes_edac_report_mem_error(struct ghes *ghes, int sev,
- struct cper_sec_mem_err *mem_err)
+void ghes_edac_report_mem_error(int sev, struct cper_sec_mem_err *mem_err)
{
enum hw_event_mc_err_type type;
struct edac_raw_error_desc *e;
diff --git a/include/acpi/ghes.h b/include/acpi/ghes.h
index 8feb0c866ee0..e096a4e7f611 100644
--- a/include/acpi/ghes.h
+++ b/include/acpi/ghes.h
@@ -55,15 +55,14 @@ enum {
/* From drivers/edac/ghes_edac.c */
#ifdef CONFIG_EDAC_GHES
-void ghes_edac_report_mem_error(struct ghes *ghes, int sev,
- struct cper_sec_mem_err *mem_err);
+void ghes_edac_report_mem_error(int sev, struct cper_sec_mem_err *mem_err);
int ghes_edac_register(struct ghes *ghes, struct device *dev);
void ghes_edac_unregister(struct ghes *ghes);
#else
-static inline void ghes_edac_report_mem_error(struct ghes *ghes, int sev,
+static inline void ghes_edac_report_mem_error(int sev,
struct cper_sec_mem_err *mem_err)
{
}
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [RFC,v4,1/3] EDAC, GHES: Remove unused argument to ghes_edac_report_mem_error
@ 2018-05-12 9:00 Borislav Petkov
0 siblings, 0 replies; 2+ messages in thread
From: Borislav Petkov @ 2018-05-12 9:00 UTC (permalink / raw)
To: Alexandru Gagniuc
Cc: alex_gagniuc, austin_bolen, shyam_iyer, Rafael J. Wysocki,
Len Brown, Tony Luck, Mauro Carvalho Chehab, Robert Moore,
Erik Schmauss, Tyler Baicar, Will Deacon, James Morse, Shiju Jose,
Jonathan (Zhixiong) Zhang, Dongjiu Geng, linux-acpi, linux-kernel,
linux-edac, devel
On Mon, Apr 30, 2018 at 04:33:50PM -0500, Alexandru Gagniuc wrote:
> The use of the 'ghes' argument was removed in a previous commit, but
> function signature was not updated to reflect this.
>
> Fixes: 0fe5f281f749 ("EDAC, ghes: Model a single, logical memory controller")
> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
> ---
> drivers/acpi/apei/ghes.c | 2 +-
> drivers/edac/ghes_edac.c | 3 +--
> include/acpi/ghes.h | 5 ++---
> 3 files changed, 4 insertions(+), 6 deletions(-)
Applied, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-05-12 9:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-12 9:00 [RFC,v4,1/3] EDAC, GHES: Remove unused argument to ghes_edac_report_mem_error Borislav Petkov
-- strict thread matches above, loose matches on Subject: below --
2018-04-30 21:33 Alexandru Gagniuc
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox