[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