> >> > >> On 1/10/19 3:07 PM, Wei Yongjun wrote: > >>> Fixes the following sparse warning: > >>> > >>> drivers/gpio/gpio-pca953x.c:292:28: warning: > >>> symbol 'pca953x_i2c_regmap' was not declared. Should it be static? > >>> > >>> Fixes: 49427232764d ("gpio: pca953x: Perform basic regmap conversion") > >>> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> > >> > >> Nice, thanks. > >> Reviewed-by: Marek Vasut <marex@xxxxxxx> > >> > >> For future reference, how did you run sparse to detect this ? > > > > > > I have a script which run sparse everyday on each changed file > > of linux-next.git tree. So most of this issues can be detected. > > Can you share that script ? I'm sure others can benefit from that too. Sure, the script exists in my github, as a part of our automated patch system. https://github.com/weiyj/dpatch-devel/blob/master/dpatch/core/checksparse.py We are considering making it as a public system, but still not ready. > > -- > Best regards, > Marek Vasut