On Friday, May 06, 2016 06:43 PM, Daniel Schwierzeck wrote:
Hi Wills,

I'm getting a warning with gcc-5.3:

   CC      arch/mips/mach-ath79/ar933x/ddr.o
arch/mips/mach-ath79/ar933x/ddr.c: In function ‘ddr_tap_tuning’:
arch/mips/mach-ath79/ar933x/ddr.c:329:15: warning: ‘upper’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
   val = (upper + lower) / 2;
                ^

Could you please look at it? I don't know which default values make
sense for 'upper' and 'lower'.
Ok, compiler might need explicit initializer.
I will add a patch for this later.

Am 15.04.2016 um 13:19 schrieb Daniel Schwierzeck:

Am 12.04.2016 um 05:09 schrieb Wills Wang:
These series of patch based on top of mips/next, it fix some defects on
the previous patch series "add support for atheros ath79 based SOCs".


Wills Wang (4):
   ath79: spi: Remove the explicit pinctrl setting
   ar933x: serial: Remove the explicit pinctrl setting
   ath79: ar933x: use BIT macro for bit shift operation
   ath79: add readonly attribute for ath79_soc_desc

  arch/mips/mach-ath79/ar933x/ddr.c | 14 +++++++-------
  arch/mips/mach-ath79/cpu.c        |  8 ++++----
  drivers/serial/serial_ar933x.c    | 16 ++--------------
  drivers/spi/ath79_spi.c           | 12 ------------
  4 files changed, 13 insertions(+), 37 deletions(-)

all four patches applied to u-boot-mips/next, thanks!


--
Best Regards
Wills

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to