Patchwork [U-Boot,11/14] board/freescale/mpc8610hpcd/mpc8610hpcd.c: Fix GCC 4.6 build warning

login
register
mail settings
Submitter Wolfgang Denk
Date Nov. 30, 2011, 8:17 a.m.
Message ID <1322641077-829-12-git-send-email-wd@denx.de>
Download mbox | patch
Permalink /patch/128450/
State Accepted
Commit e38cc2c7713c3a6f25b75317bb4ddcf290d33dfd
Headers show

Comments

Wolfgang Denk - Nov. 30, 2011, 8:17 a.m.
Fix:
mpc8610hpcd.c: In function 'pci_init_board':
mpc8610hpcd.c:238:15: warning: variable 'pordevsr' set but not used
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Kumar Gala <galak@kernel.crashing.org>
---
total: 0 errors, 0 warnings, 13 lines checked
NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE

 board/freescale/mpc8610hpcd/mpc8610hpcd.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
Kumar Gala - Nov. 30, 2011, 1:55 p.m.
On Nov 30, 2011, at 2:17 AM, Wolfgang Denk wrote:

> Fix:
> mpc8610hpcd.c: In function 'pci_init_board':
> mpc8610hpcd.c:238:15: warning: variable 'pordevsr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Kumar Gala <galak@kernel.crashing.org>
> ---
> total: 0 errors, 0 warnings, 13 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> board/freescale/mpc8610hpcd/mpc8610hpcd.c |    3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)

Acked-by: Kumar Gala <galak@kernel.crashing.org>

- k
Wolfgang Denk - Dec. 1, 2011, 11:16 p.m.
Dear Wolfgang Denk,

In message <1322641077-829-12-git-send-email-wd@denx.de> you wrote:
> Fix:
> mpc8610hpcd.c: In function 'pci_init_board':
> mpc8610hpcd.c:238:15: warning: variable 'pordevsr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> Cc: Kumar Gala <galak@kernel.crashing.org>
> ---
> total: 0 errors, 0 warnings, 13 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/freescale/mpc8610hpcd/mpc8610hpcd.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

Patch

diff --git a/board/freescale/mpc8610hpcd/mpc8610hpcd.c b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
index 5b3b560..2bcd5e6 100644
--- a/board/freescale/mpc8610hpcd/mpc8610hpcd.c
+++ b/board/freescale/mpc8610hpcd/mpc8610hpcd.c
@@ -235,12 +235,11 @@  void pci_init_board(void)
 	volatile immap_t *immap = (immap_t *) CONFIG_SYS_CCSRBAR;
 	volatile ccsr_gur_t *gur = &immap->im_gur;
 	struct fsl_pci_info pci_info;
-	u32 devdisr, pordevsr;
+	u32 devdisr;
 	int first_free_busno;
 	int pci_agent;
 
 	devdisr = in_be32(&gur->devdisr);
-	pordevsr = in_be32(&gur->pordevsr);
 
 	first_free_busno = fsl_pcie_init_board(0);