RE: [PATCH -next] gpio: pca953x: Make symbol 'pca953x_i2c_regmap' static

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> >>
> >> 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




[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux