[OPW,kernel] Staging: ft1000: Fix Sparse Warning for Static Declarations in boot.h
diff mbox

Message ID 1382486073-3343-1-git-send-email-ebru.akagunduz@gmail.com
State Accepted
Headers show

Commit Message

Ebru Akagündüz Oct. 22, 2013, 11:54 p.m. UTC
This patch fixes the Sparse Warning: "symbol 'bootimage'
was not declared. Should it be static?" in boot.h

Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com>
---
 drivers/staging/ft1000/ft1000-pcmcia/boot.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Waskiewicz Jr, Peter P Oct. 23, 2013, 1:47 a.m. UTC | #1
On Wed, 2013-10-23 at 02:54 +0300, Ebru Akagunduz wrote:
> This patch fixes the Sparse Warning: "symbol 'bootimage'
> was not declared. Should it be static?" in boot.h
> 
> Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com>

Reviewed-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>

> ---
>  drivers/staging/ft1000/ft1000-pcmcia/boot.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/ft1000/ft1000-pcmcia/boot.h b/drivers/staging/ft1000/ft1000-pcmcia/boot.h
> index 1fc4ac1..9dce54e 100644
> --- a/drivers/staging/ft1000/ft1000-pcmcia/boot.h
> +++ b/drivers/staging/ft1000/ft1000-pcmcia/boot.h
> @@ -27,7 +27,7 @@
>  #define _BOOTH_
>  
>  // Official bootloader
> -unsigned char bootimage [] = {
> +static unsigned char bootimage [] = {
>  0x00,0x00,0x01,0x5E,0x00,0x00
>  ,0x00,0x00,0x00,0x00,0x02,0xD7
>  ,0x00,0x00,0x01,0x5E,0x46,0xB3
> -- 
> 1.8.1.2
>

Patch
diff mbox

diff --git a/drivers/staging/ft1000/ft1000-pcmcia/boot.h b/drivers/staging/ft1000/ft1000-pcmcia/boot.h
index 1fc4ac1..9dce54e 100644
--- a/drivers/staging/ft1000/ft1000-pcmcia/boot.h
+++ b/drivers/staging/ft1000/ft1000-pcmcia/boot.h
@@ -27,7 +27,7 @@ 
 #define _BOOTH_
 
 // Official bootloader
-unsigned char bootimage [] = {
+static unsigned char bootimage [] = {
 0x00,0x00,0x01,0x5E,0x00,0x00
 ,0x00,0x00,0x00,0x00,0x02,0xD7
 ,0x00,0x00,0x01,0x5E,0x46,0xB3