All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 40/56] board/mbx8xx/mbx8xx.c: Fix GCC 4.6 build warnings
Date: Sat,  5 Nov 2011 02:55:44 +0100	[thread overview]
Message-ID: <1320458160-23136-41-git-send-email-wd@denx.de> (raw)
In-Reply-To: <1320458160-23136-1-git-send-email-wd@denx.de>

Fix:
mbx8xx.c: In function 'mbx_init':
mbx8xx.c:127:15: warning: variable 'refclock' set but not used
[-Wunused-but-set-variable]
mbx8xx.c: In function 'board_get_cpufreq':
mbx8xx.c:231:2: warning: dereferencing type-punned pointer will break
strict-aliasing rules [-Wstrict-aliasing]
mbx8xx.c: In function 'get_reffreq':
mbx8xx.c:242:2: warning: dereferencing type-punned pointer will break
strict-aliasing rules [-Wstrict-aliasing]

Signed-off-by: Wolfgang Denk <wd@denx.de>
---
 board/mbx8xx/mbx8xx.c |   17 ++++++++++++-----
 1 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/board/mbx8xx/mbx8xx.c b/board/mbx8xx/mbx8xx.c
index 255796b..0f014e1 100644
--- a/board/mbx8xx/mbx8xx.c
+++ b/board/mbx8xx/mbx8xx.c
@@ -117,14 +117,16 @@ static const uint sdram_table_50[] = {
 
 /* ------------------------------------------------------------------------- */
 
+#ifdef CONFIG_SYS_USE_OSCCLK
 static unsigned int get_reffreq(void);
+#endif
 static unsigned int board_get_cpufreq(void);
 
 void mbx_init (void)
 {
 	volatile immap_t *immr = (immap_t *) CONFIG_SYS_IMMR;
 	volatile memctl8xx_t *memctl = &immr->im_memctl;
-	ulong speed, refclock, plprcr, sccr;
+	ulong speed, plprcr, sccr;
 	ulong br0_32 = memctl->memc_br0 & 0x400;
 
 	/* real-time clock status and control register */
@@ -152,7 +154,6 @@ void mbx_init (void)
 	immr->im_clkrst.car_sccr = sccr;
 
 	speed = board_get_cpufreq ();
-	refclock = get_reffreq ();
 
 #if ((CONFIG_SYS_PLPRCR & PLPRCR_MF_MSK) != 0)
 	plprcr = CONFIG_SYS_PLPRCR;
@@ -163,7 +164,7 @@ void mbx_init (void)
 #endif
 
 #ifdef CONFIG_SYS_USE_OSCCLK			/* See doc/README.MBX ! */
-	plprcr |= ((speed + refclock / 2) / refclock - 1) << 20;
+	plprcr |= ((speed + get_reffreq() / 2) / refclock - 1) << 20;
 #endif
 
 	immr->im_clkrstk.cark_plprcrk = KAPWR_KEY;
@@ -226,21 +227,27 @@ static unsigned int board_get_cpufreq (void)
 {
 #ifndef CONFIG_8xx_GCLK_FREQ
 	vpd_packet_t *packet;
+	ulong *p;
 
 	packet = vpd_find_packet (VPD_PID_ICS);
-	return *((ulong *) packet->data);
+	p = (ulong *)packet->data;
+	return *p;
 #else
 	return((unsigned int)CONFIG_8xx_GCLK_FREQ );
 #endif /* CONFIG_8xx_GCLK_FREQ */
 }
 
+#ifdef CONFIG_SYS_USE_OSCCLK
 static unsigned int get_reffreq (void)
 {
 	vpd_packet_t *packet;
+	ulong *p;
 
 	packet = vpd_find_packet (VPD_PID_RCS);
-	return *((ulong *) packet->data);
+	p = (ulong *)packet->data;
+	return *p;
 }
+#endif
 
 static void board_get_enetaddr(uchar *addr)
 {
-- 
1.7.6.4

  parent reply	other threads:[~2011-11-05  1:55 UTC|newest]

Thread overview: 163+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-05  1:55 [U-Boot] [PATCH 00/56] [PPC] Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 01/56] board/mpl/common/flash.c: fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:09   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 02/56] arch/powerpc/cpu/mpc5xxx/i2c.c: fix GC 4.6 build warnings Wolfgang Denk
2011-11-07 21:09   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 03/56] arch/powerpc/lib/board.c: make (mostly) checkpatch-clean Wolfgang Denk
2011-11-07 21:09   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 04/56] arch/powerpc/lib/board.c: fix build warning Wolfgang Denk
2011-11-07 21:09   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 05/56] board/tqc/tqm5200/cmd_stk52xx.c: fix GC 4.6 build warnings Wolfgang Denk
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 06/56] arch/powerpc/cpu/mpc5xxx/usb_ohci.c: fix GCC " Wolfgang Denk
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 07/56] board/mcc200/auto_update.c: fixx " Wolfgang Denk
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 08/56] drivers/rtc/ds3231.c: fix " Wolfgang Denk
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 09/56] drivers/rtc/rv3029.c: " Wolfgang Denk
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 10/56] drivers/video/videomodes.c: fix GCC 4.6 build warning Wolfgang Denk
2011-11-06 19:44   ` Anatolij Gustschin
2011-11-07 21:10   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 11/56] fs/fat/fat.c: Fix " Wolfgang Denk
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 12/56] PM520: add missing enable_interrupts() Wolfgang Denk
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 13/56] board/esd/cpci5200/strataflash.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 14/56] board/esd/pf5200/pf5200.c: " Wolfgang Denk
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 15/56] board/tqc/tqm5200/cam5200_flash.c: " Wolfgang Denk
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 16/56] drivers/video/cfb_console.c:Fix " Wolfgang Denk
2011-11-06 19:31   ` Anatolij Gustschin
2011-11-07 21:11   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 17/56] mpc8xx/fec.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 18/56] drivers/pcmcia/mpc8xx_pcmcia.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 19/56] board/c2mon/pcmcia.c: " Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 20/56] board/icu862/pcmcia.c: " Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 21/56] board/kup/common/pcmcia.c: Fix GCC 4.6 compiler warnings Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 22/56] board/lwmon/pcmcia.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 23/56] board/manroland/uc100/pcmcia.c: " Wolfgang Denk
2011-11-07 21:12   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 24/56] board/mbx8xx/pcmcia.c: " Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 25/56] board/netta/codec.c: " Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 26/56] board/netta/pcmcia.c: " Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 27/56] board/r360mpi/pcmcia.c: " Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 28/56] mpc8xx/cpu.c: " Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 29/56] ICU862: Fix incorrect flash handling Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 30/56] board/kup/kup4k/kup4k.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:13   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 31/56] board/eltec/mhpc/flash.c: " Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 32/56] mpc8xx/i2c.c: CodungStyle cleanup Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 33/56] mpc8xx/i2c.c: replace PRINTD() by debug() Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 34/56] board/etx094/flash.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 35/56] board/etx094/flash.c: Fix flash initialization Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 36/56] board/genietv/flash.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 37/56] board/genietv/flash.c: Fix flash initialization Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 38/56] net/net.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:14   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 39/56] mpc8xx/spi.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` Wolfgang Denk [this message]
2011-11-07 21:15   ` [U-Boot] [PATCH 40/56] board/mbx8xx/mbx8xx.c: " Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 41/56] board/fads/fads.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 42/56] board/rbc823/flash.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 43/56] board/rbc823/flash.c: Fix flash initalization Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 44/56] board/svm_sc8xx/svm_sc8xx.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 45/56] board/svm_sc8xx/flash.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 46/56] board/svm_sc8xx/flash.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:15   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 47/56] TQM850M: Add missing CONFIG_BOOTDELAY Wolfgang Denk
2011-11-07 21:16   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 48/56] mpc8260/speed.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-07 21:16   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 49/56] board/tqc/tqm8272/tqm8272.c: Fix GCC 4.6 build warning: Wolfgang Denk
2011-11-07 21:16   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 50/56] common/cmd_i2c.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:16   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 51/56] mpc8220/fec.c: " Wolfgang Denk
2011-11-07 21:16   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 52/56] mpc8260/i2c.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:17   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 53/56] mpc8260/i2c.c: replace PRINTD() by debug() Wolfgang Denk
2011-11-07 21:17   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 54/56] mpc8220/i2c.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:17   ` Wolfgang Denk
2011-11-05  1:55 ` [U-Boot] [PATCH 55/56] arch/powerpc/lib/bat_rw.c: " Wolfgang Denk
2011-11-07 21:17   ` Wolfgang Denk
2011-11-05  1:56 ` [U-Boot] [PATCH 56/56] board/alaska/flash.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-07 21:17   ` Wolfgang Denk
2011-11-05 15:12 ` [U-Boot] [PATCH 00/22] Fix GCC 4.6 build warnings (Part 2) Wolfgang Denk
2011-11-05 15:12   ` [U-Boot] [PATCH 01/22] drivers/net/pcnet.c: Change debug code to fix build warning Wolfgang Denk
2011-11-07 21:18     ` Wolfgang Denk
2011-11-05 15:12   ` [U-Boot] [PATCH 02/22] board/etin/kvme080/multiverse.c: Fix GCC 4.6 " Wolfgang Denk
2011-11-07 21:18     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 03/22] board/mousse/flash.c: Fix GCC 4.6 buil warnings Wolfgang Denk
2011-11-07 21:18     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 04/22] drivers/net/rtl8169.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:18     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 05/22] board/linkstation/ide.c: Fix GCC 4.6 build warnings Wolfgang Denk
2011-11-06 17:46     ` Guennadi Liakhovetski
2011-11-07 21:47       ` Wolfgang Denk
2011-11-07 21:47     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 06/22] drivers/net/rtl8139.c: " Wolfgang Denk
2011-11-07 21:47     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 07/22] common/cmd_flash.c: " Wolfgang Denk
2011-11-07 21:47     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 08/22] arch/powerpc/cpu/mpc8260/spi.c: " Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 09/22] board/cogent/flash.c: Fix GCC 4.6 buiild warning Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 10/22] board/gw8260/flash.c: minimal CodingStyle cleanup Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 11/22] board/gw8260/flash.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 12/22] board/ep82xxm/ep82xxm.c: " Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 13/22] board/hymod/input.c: " Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 14/22] board/ids8247/ids8247.c: " Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 15/22] board/rpxsuper/flash.c: minimal CodingStyle cleanup Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 16/22] board/rpxsuper/flash.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:48     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 17/22] board/sacsng/sacsng.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:49     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 18/22] board/sacsng/sacsng.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:49     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 19/22] board/freescale/mpc8266ads/mpc8266ads.c: CodingStyle cleanup Wolfgang Denk
2011-11-07 21:49     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 20/22] board/freescale/mpc8266ads/mpc8266ads.c: Fix GCC 4.6 build warning Wolfgang Denk
2011-11-07 21:49     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 21/22] board/funkwerk/vovpn-gw/vovpn-gw.c: " Wolfgang Denk
2011-11-07 21:49     ` Wolfgang Denk
2011-11-05 15:13   ` [U-Boot] [PATCH 22/22] board/siemens/SCM/scm.c: " Wolfgang Denk
2011-11-07 21:50     ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1320458160-23136-41-git-send-email-wd@denx.de \
    --to=wd@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.