[PATCH 1/2] S3C64XX: Ensure samsung_keypad_cfg_gpio() is prototyped

Kukjin Kim kgene.kim at samsung.com
Wed Oct 13 04:48:48 EDT 2010


Mark Brown wrote:
> 
> On Wed, Oct 13, 2010 at 05:14:37PM +0900, Kukjin Kim wrote:
> > Mark Brown wrote:
> 
> > > Since it's exported we should make sure we're using the prototype
> > > others see.
> 
> > Hmm...I'm not sure that we need really this. Of course, it can be
helpful.
> > But if so, we have to apply this policy/rule to all of ours...but I
think we
> > don't need it now.
> 
> sparse warns about this issue so fixing issues like this is required to
> get a clean build for sparse users (I do pretty much all my builds with
> sparse).  I am sending fixes for everything I build, though obviously I
> don't have access to all Samsung systems, and it does make life easier
> when doing updates on the arch code.

You mean following?
arch/arm/mach-s3c64xx/setup-keypad.c:16:6: warning: symbol
'samsung_keypad_cfg_gpio' was not declared. Should it be static?

I didn't check sparse :-(
Ok...Ben seems to be busy now...so will apply in my tree.
And I will also fix it our Samsung stuff.

Ben, if any problems, please let me know.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




More information about the linux-arm-kernel mailing list