[U-Boot] [PATCH 0/4] ath79: fix some minor defects

Wills Wang wills.wang at live.com
Sat May 7 01:26:34 CEST 2016



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



More information about the U-Boot mailing list