[PATCH 1/5] SPEAr3xx: Make local structres static

Ryan Mallon ryan at bluewatersys.com
Mon Jun 14 01:31:30 EDT 2010


Viresh KUMAR wrote:
> On 6/5/2010 11:35 AM, Ryan Mallon wrote:
>> Several structures in arch/arm/mach-spear3xx are not marked static
>> like they should be. Fix this.
>>
>> Signed-off-by: Ryan Mallon <ryan at bluewatersys.com>
>> ---
>>  arch/arm/mach-spear3xx/spear300.c |   33 ++++++++++----------
>>  arch/arm/mach-spear3xx/spear310.c |   16 +++++-----
>>  arch/arm/mach-spear3xx/spear320.c |   56 +++++++++++++++++-----------------
>>  arch/arm/mach-spear3xx/spear3xx.c |   60 ++++++++++++++++++------------------
>>  4 files changed, 83 insertions(+), 82 deletions(-)
>>

<snip>

>> -struct pmx_dev_mode pmx_plgpio_45_46_49_50_modes[] = {
>> +static struct pmx_dev_mode pmx_plgpio_45_46_49_50_modes[] = {
>>  	{
>>  		.ids = 0x00,
>>  		.mask = PMX_TIMER_3_4_MASK,
> 
> These structures are not local, they are used in spear3**_evb.c files,
> and hence can't be static.

The pmx_dev structures are used by the spear3**_evb.c files, but the
pmx_dev_mode structures are local. This patch is only making the latter
static. I have done a build test which includes all three spear3**_evb
board files and got no compilation warnings/errors.

~Ryan




More information about the linux-arm-kernel mailing list