From: Xuewen Wang Three function definitions terminate with '};' instead of '}', which is unnecessary and inconsistent with kernel coding style: - damon_pa_initcall() in paddr.c - damon_va_initcall() in vaddr.c - damos_get_some_mem_psi_total() in core.c No functional change intended. Signed-off-by: Xuewen Wang Reviewed-by: SJ Park Signed-off-by: SJ Park --- Changes from v1 - v1: https://lore.kernel.org/20260714075358.1592393-1-wangxuewen@kylinos.cn - Collect R-b: from SJ. - Rebase to the latest mm-new. mm/damon/core.c | 2 +- mm/damon/paddr.c | 2 +- mm/damon/vaddr.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/mm/damon/core.c b/mm/damon/core.c index e0122376f43d4..644daf5a16560 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -2791,7 +2791,7 @@ static u64 damos_get_some_mem_psi_total(void) static inline u64 damos_get_some_mem_psi_total(void) { return 0; -}; +} #endif /* CONFIG_PSI */ diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c index e4f98d67461f5..5a6a78054784a 100644 --- a/mm/damon/paddr.c +++ b/mm/damon/paddr.c @@ -447,6 +447,6 @@ static int __init damon_pa_initcall(void) }; return damon_register_ops(&ops); -}; +} subsys_initcall(damon_pa_initcall); diff --git a/mm/damon/vaddr.c b/mm/damon/vaddr.c index d487b7a4a1042..4976775fe2762 100644 --- a/mm/damon/vaddr.c +++ b/mm/damon/vaddr.c @@ -959,7 +959,7 @@ static int __init damon_va_initcall(void) if (err) return err; return damon_register_ops(&ops_fvaddr); -}; +} subsys_initcall(damon_va_initcall); base-commit: 4304be1e26c22e99b8719c5093ece877708b53e7 -- 2.47.3