]> Git Repo - linux.git/commitdiff
m68knommu: fix ucsimm sparse warnings
authorGreg Ungerer <[email protected]>
Mon, 17 Jan 2022 07:53:51 +0000 (17:53 +1000)
committerGreg Ungerer <[email protected]>
Mon, 7 Mar 2022 04:51:12 +0000 (14:51 +1000)
Sparse reports the following warnings:

  arch/m68k/68000/ucsimm.c:19:15: sparse: sparse: symbol 'cs8900a_hwaddr' was not declared. Should it be static?
  arch/m68k/68000/ucsimm.c:22:1: sparse: sparse: symbol 'getserialnum' was not declared. Should it be static?
  arch/m68k/68000/ucsimm.c:23:1: sparse: sparse: symbol 'gethwaddr' was not declared. Should it be static?
  arch/m68k/68000/ucsimm.c:24:1: sparse: sparse: symbol 'getbenv' was not declared. Should it be static?

'cs8900a_hwaddr' is not used anywhere else in the kernel, so it can be
removed.

All of 'getserialnum', 'gethwaddr' and 'getbenv' are only used in this
file, so they can all be made static. It turns out that these also cause
compiler warnings like this too:

  arch/m68k/68000/ucsimm.c:22:15: warning: no previous prototype for 'getserialnum' [-Wmissing-prototypes]

Also fixed by making them all static.

Signed-off-by: Greg Ungerer <[email protected]>
Reported-by: kernel test robot <[email protected]>
arch/m68k/68000/ucsimm.c

index 7c6cbf6437126ffe198b70c325e6d0e5fb22be3e..c54fde75eae8efcdf7474ba58dbd3eb400fb8fd7 100644 (file)
 
 #include "m68328.h"
 
-unsigned char *cs8900a_hwaddr;
 static int errno;
 
-_bsc0(char *, getserialnum)
-_bsc1(unsigned char *, gethwaddr, int, a)
-_bsc1(char *, getbenv, char *, a)
+static _bsc0(char *, getserialnum)
+static _bsc1(unsigned char *, gethwaddr, int, a)
+static _bsc1(char *, getbenv, char *, a)
 
 void __init init_ucsimm(char *command, int size)
 {
        char *p;
 
        pr_info("uCsimm/uCdimm serial string [%s]\n", getserialnum());
-       p = cs8900a_hwaddr = gethwaddr(0);
+       p = gethwaddr(0);
        pr_info("uCsimm/uCdimm hwaddr %pM\n", p);
        p = getbenv("APPEND");
        if (p)
This page took 0.051911 seconds and 4 git commands to generate.