summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeLines
* Define uintptr_t as long int to simplify printf() format stringsSimon Glass2011-11-22-9/+3
| | | | | | | | | | If uintptr_t can be either an unsigned int or an unsigned long int, it is tricky to use it in a printf() format string. This changes it to unsigned long int consistently. This should do the right thing on both 32-bit and 64-bit architectures. Signed-off-by: Simon Glass <sjg@chromium.org> Acked-by: Mike Frysinger <vapier@gentoo.org>
* new tool mkenvimage: generates an env image from an arbitrary config fileDavid Wagner2011-11-22-0/+275
| | | | | | | | | | | This tool takes a key=value configuration file (same as would a `printenv' show) and generates the corresponding environment image, ready to be flashed. use case: flash the environment with an external tool Signed-off-by: David Wagner <david.wagner@free-electrons.com> Acked-by; Mike Frysinger <vapier@gentoo.org> Tested-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
* smsc95xx: Debug message cleanupWolfgang Grandegger2011-11-22-7/+3
| | | | | | | | Cc: Simon Glass <sjg@chromium.org> Cc: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Wolfgang Grandegger <wg@denx.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* smsc95xx: Fix MAC address programmingWolfgang Grandegger2011-11-22-3/+3
| | | | | | | | | | | | | Commit 79ad54400932d6484178a372fb3b659e3437473b broke MAC address programming in the SMSC95xx register set. Cc: Marek Vasut <marek.vasut@gmail.com> Cc: Wolfgang Denk <wd@denx.de> Cc: Simon Glass <sjg@chromium.org> Cc: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Wolfgang Grandegger <wg@denx.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Simon Glass <sjg@chromium.org>
* Merge branch 'sr@denx.de' of git://git.denx.de/u-boot-stagingWolfgang Denk2011-11-21-26/+71
|\ | | | | | | | | | | * 'sr@denx.de' of git://git.denx.de/u-boot-staging: Makefile: Add the missing dependency for spl target gpio: Adapt PCA9698 to standard GPIO API
| * Makefile: Add the missing dependency for spl targetAneesh V2011-11-18-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wolfgang's patch for build time improvement is bringing out issues due to missing dependencies in the top-level Makefile. I get errors such as the below while building with many threads. make[1]: /home/a0393566local/u-boot-denx/tools/mkimage: Command not found This also allows one to do: $ make spl/u-boot-spl.bin if you want to build only spl Cc: Wolfgang Denk <wd@denx.de> Signed-off-by: Aneesh V <aneesh@ti.com>
| * gpio: Adapt PCA9698 to standard GPIO APIDirk Eibach2011-11-18-25/+70
| | | | | | | | | | | | Signed-off-by: Dirk Eibach <eibach@gdsys.de> Acked-by: Mike Frysinger <vapier@gentoo.org> Acked-by: Stefan Roese <sr@denx.de>
* | Merge branch 'master' of git://git.denx.de/u-boot-blackfinWolfgang Denk2011-11-21-7/+12
|\ \ | | | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-blackfin: Blackfin: cache result of cpp check Blackfin: traps: fix up printf warnings from debug Blackfin: drop now unused local variable
| * | Blackfin: cache result of cpp checkMike Frysinger2011-11-18-1/+2
| | | | | | | | | | | | | | | | | | Avoid overhead of computing this value multiple times. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
| * | Blackfin: traps: fix up printf warnings from debugMike Frysinger2011-11-18-5/+10
| | | | | | | | | | | | | | | | | | Cast uint32_t to void* for %p, and use %zu for size_t. Signed-off-by: Mike Frysinger <vapier@gentoo.org>
| * | Blackfin: drop now unused local variableMike Frysinger2011-11-18-1/+0
| | | | | | | | | | | | Signed-off-by: Mike Frysinger <vapier@gentoo.org>
* | | Merge branch 'master' of git://git.denx.de/u-boot-nds32Wolfgang Denk2011-11-21-3/+25
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-nds32: nds32/lib: add ide generic support nds32: enhance io.h for compatibility with periphals ftide020: fix incorrect information display format
| * | | nds32/lib: add ide generic supportMacpaul Lin2011-11-18-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | Add ide generic support. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
| * | | nds32: enhance io.h for compatibility with periphalsMacpaul Lin2011-11-18-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Enhance io.h for periphals origin design on x86 systems. For example, pci, ide, etc. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
| * | | ftide020: fix incorrect information display formatMacpaul Lin2011-11-18-3/+2
| |/ / | | | | | | | | | | | | | | | Fix incorrect information display format when probing the device. Signed-off-by: Macpaul Lin <macpaul@andestech.com>
* | | Merge branch 'master' of git://git.denx.de/u-boot-niosWolfgang Denk2011-11-21-74/+394
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | * 'master' of git://git.denx.de/u-boot-nios: nios2: Offer ft_board_setup() capability and call fdt_fixup_ethernet(). board/nios2-generic: Use altera_pio driver and remove board specific driver gpio: Add driver for Altera's PIO core nios2: Pseudo implement dcache_status/enable/disable()
| * | nios2: Offer ft_board_setup() capability and call fdt_fixup_ethernet().Joachim Foerster2011-10-30-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | The main motivation is, to have the local-mac-address property of the ethernet peripherals fixed/synced with U-Boot's environment settings. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | board/nios2-generic: Use altera_pio driver and remove board specific driverJoachim Foerster2011-10-28-72/+4
| | | | | | | | | | | | | | | Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | gpio: Add driver for Altera's PIO coreJoachim Foerster2011-10-28-2/+320
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This driver may handle multiple PIO cores and thus needs to be setup by calling the altera_pio_init() function within the early board setup routine. The driver comes with some extras, see below the copyleft header. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Tested-by: Thomas Chou <thomas@wytron.com.tw> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
| * | nios2: Pseudo implement dcache_status/enable/disable()Joachim Foerster2011-10-28-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AFAIK the NIOS2 architecture does not have any possibility to dynamically switch off the data cache. So _status() always reports 1 and _enable/_disable() just issue a flush of everything. For example, common/cmd_elf.c depends on these symbols. Signed-off-by: Joachim Foerster <joachim.foerster@missinglinkelectronics.com> Signed-off-by: Thomas Chou <thomas@wytron.com.tw>
* | | tools: checkpatch.pl from Linux added to toolsJoe Hershberger2011-11-17-0/+3351
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added from Linux - commit 30ecad51849ae132dc6ef6ddb62d499c7257515b Include config file to ignore common false-positives Signed-off-by: Joe Hershberger <joe.hershberger@ni.com> Cc: Joe Hershberger <joe.hershberger@gmail.com> Cc: Wolfgang Denk <wd@denx.de>
* | | e1000: fix unused variable waring for e1000 driverZang Roy-R619112011-11-17-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the following build warning in drivers/net/e1000.c e1000.c: In function 'e1000_reset_hw': e1000.c:1373:11: warning: variable 'icr' set but not used [-Wunused-but-set-variable] e1000.c: In function 'e1000_phy_init_script': e1000.c:4395:11: warning: variable 'ret_val' set but not used [-Wunused-but-set-variable] Signed-off-by: Roy Zang <tie-fei.zang@freescale.com> Cc: Wolfgang Denk <wd@denx.de> Cc: Kyle Moffett <Kyle.D.Moffett@boeing.com>
* | | e1000: remove duplicate macros in e1000.hTimur Tabi2011-11-17-11/+0
| | | | | | | | | | | | | | | | | | | | | Some of the EEPROM Word Offset macros, and a few others, are defined twice in e1000.h. Signed-off-by: Timur Tabi <timur@freescale.com>
* | | Orphan EVB64260_750CX boardWolfgang Denk2011-11-16-5/+1
| | | | | | | | | | | | | | | | | | The last maintainer (Eran Man) cannot be reached any more. Signed-off-by: Wolfgang Denk <wd@denx.de>
* | | Revert "sandbox: Change md command to use map_physmem"Kumar Gala2011-11-16-7/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 355a835747c6f7c5071ead295a7dfc489c73cb03. The original commit broke long standing assumption that md commands work on effective addresses. This normally isn't an issue for most systems that map 1:1, however on systems with a 36-bit address map it breaks. Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
* | | common: tsi148 - fix gcc 4.6 compiler warningKim Phillips2011-11-16-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for vme8349 board... cmd_tsi148.c: In function 'tsi148_init': cmd_tsi148.c:56:17: warning: variable 'lastError' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
* | | mpc83xx: km83xx_i2c - fix gcc 4.6 compiler warningKim Phillips2011-11-16-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Configuring for kmeter1 board... km83xx_i2c.c: In function 'i2c_make_abort': km83xx_i2c.c:44:8: warning: variable 'dummy' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com> Acked-by: Heiko Schocher <hs@denx.de>
* | | mpc83xx: mpc8360emds - fix gcc 4.6 compiler warningKim Phillips2011-11-16-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for MPC8360EMDS_66_HOST_33 - Board: MPC8360EMDS, Options: CLKIN_66MHZ,PCI,PCI_33M,PQ_MDS_PIB=1 mpc8360emds.c: In function 'board_eth_init': mpc8360emds.c:178:12: warning: array subscript is above array bounds [-Warray-bounds] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
* | | mpc83xx: spd_sdram - fix gcc 4.6 compiler warningKim Phillips2011-11-16-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Configuring for sbc8349 board... spd_sdram.c: In function 'spd_sdram': spd_sdram.c:152:41: warning: variable 'trfc_high' set but not used [-Wunused-but-set-variable] Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
* | | Fix warnings in yaffsSimon Glass2011-11-16-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sorry if this is already fixed somewhere - I could not find it. This fixes the warnings show below. yaffs_tagscompat.c: In function 'yaffs_TagsCompatabilityReadChunkWithTagsFromNAND': yaffs_tagscompat.c:151: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:150: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:149: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:148: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:147: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:146: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:145: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:144: warning: dereferencing pointer 'tu' does break strict-aliasing rules yaffs_tagscompat.c:141: note: initialized from here Signed-off-by: Simon Glass <sjg@chromium.org>
* | | drivers/rtc/s3c24x0_rtc.c: fix GCC 4.6 warningsAnatolij Gustschin2011-11-16-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: s3c24x0_rtc.c: In function 'rtc_get': s3c24x0_rtc.c:67:53: warning: variable 'a_armed' set but not used s3c24x0_rtc.c:67:45: warning: variable 'a_year' set but not used s3c24x0_rtc.c:67:38: warning: variable 'a_mon' set but not used s3c24x0_rtc.c:67:30: warning: variable 'a_date' set but not used s3c24x0_rtc.c:67:22: warning: variable 'a_hour' set but not used s3c24x0_rtc.c:67:15: warning: variable 'a_min' set but not used s3c24x0_rtc.c:67:8: warning: variable 'a_sec' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | fs/yaffs2/yaffs_guts.c: Fix some GCC 4.6 warningsAnatolij Gustschin2011-11-16-23/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: yaffs_guts.c: In function 'yaffs_CheckChunkErased': yaffs_guts.c:854:6: warning: variable 'result' set but not used yaffs_guts.c: In function 'yaffs_UpdateObjectHeader': yaffs_guts.c:3463:6: warning: variable 'result' set but not used yaffs_guts.c: In function 'yaffs_GrabChunkCache': yaffs_guts.c:3774:6: warning: variable 'pushout' set but not used yaffs_guts.c: In function 'yaffs_Scan': yaffs_guts.c:5237:6: warning: variable 'result' set but not used yaffs_guts.c: In function 'yaffs_CheckObjectDetailsLoaded': yaffs_guts.c:5748:6: warning: variable 'alloc_failed' set but not used yaffs_guts.c:5747:6: warning: variable 'result' set but not used yaffs_guts.c: In function 'yaffs_ScanBackwards': yaffs_guts.c:5808:6: warning: variable 'deleted' set but not used yaffs_guts.c:5806:6: warning: variable 'result' set but not used yaffs_guts.c: In function 'yaffs_GetObjectName': yaffs_guts.c:6657:7: warning: variable 'result' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | arch/arm/cpu/armv7/s5p-common/pwm.c: fix GCC 4.6 warningAnatolij Gustschin2011-11-16-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: pwm.c: In function 'pwm_config': pwm.c:85:16: warning: variable 'timer_rate_hz' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/video/bus_vcxk.c: Fix GCC 4.6 warningAnatolij Gustschin2011-11-16-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: bus_vcxk.c: In function 'vcxk_display_bitmap': bus_vcxk.c:396:16: warning: variable 'compression' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/video/da8xx-fb.c: Fix GCC 4.6 warningAnatolij Gustschin2011-11-16-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: da8xx-fb.c: In function 'video_hw_init': da8xx-fb.c:688:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | board/cm4008/flash.c: Fix GCC 4.6 warningsAnatolij Gustschin2011-11-16-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: flash.c: In function 'flash_erase': flash.c:209:6: warning: variable 'flag' set but not used [-Wunused-but-set-variable] flash.c: In function 'write_data': flash.c:373:6: warning: variable 'flag' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | board/cm41xx/flash.c: Fix GCC 4.6 warningsAnatolij Gustschin2011-11-16-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: flash.c: In function 'flash_erase': flash.c:209:6: warning: variable 'flag' set but not used [-Wunused-but-set-variable] flash.c: In function 'write_data': flash.c:373:6: warning: variable 'flag' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/net/enc28j60.c: Fix GCC 4.6 warningsAnatolij Gustschin2011-11-16-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: enc28j60.c: In function 'enc_receive': enc28j60.c:435:5: warning: variable 'eir_reg' set but not used [-Wunused-but-set-variable] enc28j60.c: In function 'enc_poll': enc28j60.c:503:5: warning: variable 'estat_reg' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/net/at91_emac.c: Fix GCC 4.6 warningAnatolij Gustschin2011-11-16-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: at91_emac.c: In function 'at91emac_write_hwaddr': at91_emac.c:477:15: warning: variable 'dev' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/mtd/dataflash.c: Fix GCC 4.6 warningAnatolij Gustschin2011-11-16-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: dataflash.c: In function 'AT91F_DataflashInit': dataflash.c:42:6: warning: variable 'last_part' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | drivers/spi/atmel_spi.c: Fix GCC 4.6 warningAnatolij Gustschin2011-11-16-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: atmel_spi.c: In function 'spi_xfer': atmel_spi.c:139:7: warning: variable 'ret' set but not used [-Wunused-but-set-variable] Signed-off-by: Anatolij Gustschin <agust@denx.de>
* | | pip405.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: pip405.c: In function 'board_early_init_f': pip405.c:192:16: warning: variable 'tctp_clocks' set but not used [-Wunused-but-set-variable] pip405.c:182:16: warning: variable 'dataout' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | mip405.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-7/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: mip405.c: In function 'init_sdram': mip405.c:250:4: warning: variable 'tctp_clocks' set but not used [-Wunused-but-set-variable] mip405.c: In function 'initdram': mip405.c:629:9: warning: variable 'ds' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | auto_update.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: ../common/auto_update.c: In function 'au_check_header_valid': ../common/auto_update.c:94:16: warning: variable 'checksum' set but not used [-Wunused-but-set-variable] ../common/auto_update.c: In function 'do_auto_update': ../common/auto_update.c:400:30: warning: variable 'got_ctrlc' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | iop480_uart.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Fix: iop480_uart.c: In function 'serial_init': iop480_uart.c:137:16: warning: variable 'val' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | cmd_dasa_sim.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-28/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: cmd_dasa_sim.c: In function 'updatePci9054': cmd_dasa_sim.c:135:6: warning: variable 'val' set but not used [-Wunused-but-set-variable] cmd_dasa_sim.c: In function 'clearPci9054': cmd_dasa_sim.c:189:6: warning: variable 'val' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | cmd_pci405.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | Fix: cmd_pci405.c: In function 'do_loadpci': cmd_pci405.c:45:6: warning: variable 'status' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | dlvision-10g.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | Fix: dlvision-10g.c: In function 'print_fpga_info': dlvision-10g.c:90:11: warning: variable 'feature_compression' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | nand_boot.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: nand_boot.c: In function 'nand_read_page': nand_boot.c:190:6: warning: variable 'stat' set but not used [-Wunused-but-set-variable] nand_boot.c: In function 'nand_boot': nand_boot.c:271:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>
* | | sata_dwc.c: Fix GCC 4.6 build warningsStefan Roese2011-11-16-32/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: sata_dwc.c: In function 'sata_dwc_softreset': sata_dwc.c:444:5: warning: variable 'status' set but not used [-Wunused-but-set-variable] sata_dwc.c:443:6: warning: variable 'serror' set but not used [-Wunused-but-set-variable] sata_dwc.c: In function 'scan_sata': sata_dwc.c:654:16: warning: variable 'lba_desc' set but not used [-Wunused-but-set-variable] sata_dwc.c:538:16: warning: variable 'xfer_mask' set but not used [-Wunused-but-set-variable] sata_dwc.c: In function 'ata_dev_read_id': sata_dwc.c:747:14: warning: variable 'reason' set but not used [-Wunused-but-set-variable] sata_dwc.c: In function 'ata_dev_read_sectors': sata_dwc.c:1810:6: warning: variable 'rc' set but not used [-Wunused-but-set-variable] sata_dwc.c: In function 'ata_dev_write_sectors': sata_dwc.c:1994:6: warning: variable 'rc' set but not used [-Wunused-but-set-variable] Signed-off-by: Stefan Roese <sr@denx.de>