[PATCH v2 1/2] sh: cache: remove unused but set variables in sh4_flush_cache_range

From: Thorsten Blum

Date: Thu Apr 23 2026 - 12:58:08 EST


Remove 'start' and 'end' to avoid two "variable set but not used"
warnings triggered by -Wunused-but-set-variable. Update the function
comment accordingly.

Reviewed-by: Zi Yan <ziy@xxxxxxxxxx>
Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx>
---
Changes in v2:
- Update the comment as suggested by Zi Yan (thanks!)
- v1: https://lore.kernel.org/lkml/20260423145301.245374-4-thorsten.blum@xxxxxxxxx/
---
arch/sh/mm/cache-sh4.c | 5 -----
1 file changed, 5 deletions(-)

diff --git a/arch/sh/mm/cache-sh4.c b/arch/sh/mm/cache-sh4.c
index 83fb34b39ca7..c7635beb4f1f 100644
--- a/arch/sh/mm/cache-sh4.c
+++ b/arch/sh/mm/cache-sh4.c
@@ -274,8 +274,6 @@ static void sh4_flush_cache_page(void *args)
/*
* Write back and invalidate D-caches.
*
- * START, END: Virtual Address (U0 address)
- *
* NOTE: We need to flush the _physical_ page entry.
* Flushing the cache lines for U0 only isn't enough.
* We need to flush for P1 too, which may contain aliases.
@@ -284,11 +282,8 @@ static void sh4_flush_cache_range(void *args)
{
struct flusher_data *data = args;
struct vm_area_struct *vma;
- unsigned long start, end;

vma = data->vma;
- start = data->addr1;
- end = data->addr2;

if (cpu_context(smp_processor_id(), vma->vm_mm) == NO_CONTEXT)
return;