drivers/edac/octeon_edac-pc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
The comment for co_cache_error_event() starts with "/**" but lacks the
function name line, so scripts/kernel-doc reports "isn't a kernel-doc
comment" and it is silently dropped from the generated documentation.
Add the missing name line and document the parameters.
Signed-off-by: hanzhijian <hanzhijian1991@gmail.com>
---
drivers/edac/octeon_edac-pc.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/edac/octeon_edac-pc.c b/drivers/edac/octeon_edac-pc.c
index aa1219db0..076a3952f 100644
--- a/drivers/edac/octeon_edac-pc.c
+++ b/drivers/edac/octeon_edac-pc.c
@@ -31,9 +31,11 @@ struct co_cache_error {
};
/**
- * EDAC CPU cache error callback
+ * co_cache_error_event() - EDAC CPU cache error callback
*
+ * @this: notifier block
* @event: non-zero if unrecoverable.
+ * @ptr: unused
*/
static int co_cache_error_event(struct notifier_block *this,
unsigned long event, void *ptr)
--
2.43.0
On Sat, Aug 22, 2026 at 07:37:51AM +0800, hanzhijian wrote:
> The comment for co_cache_error_event() starts with "/**" but lacks the
> function name line, so scripts/kernel-doc reports "isn't a kernel-doc
> comment" and it is silently dropped from the generated documentation.
>
> Add the missing name line and document the parameters.
>
> Signed-off-by: hanzhijian <hanzhijian1991@gmail.com>
I'm presuming your legal name is "Zhijian Han"?
Because you should use your proper legal name when you sign off on patches.
You don't have to send a new version - I can fix it up when applying.
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
On Tue, Sep 01, 2026 at 06:23:31PM -0700, Borislav Petkov wrote: > I'm presuming your legal name is "Zhijian Han"? > Because you should use your proper legal name when you sign off on patches. > You don't have to send a new version - I can fix it up when applying. Yes, "Zhijian Han" is correct. Thank you for fixing it up. Regards, Zhijian
© 2016 - 2026 Red Hat, Inc.