[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