Skip to contentSkip to conversation

da830evm build regression due to commit 11a6fb79745b06610b334bada7167e9e0b619851

U-Boot Albert ARIBAUD 2 participants 3 messages Sep 13, 2011

lore
Share & tools
lore
#0 lore raw

Hi all,

Commit 11a6fb79745b06610b334bada7167e9e0b619851 ("da830: modify the MEMTEST start and end address") in u-boot-arm (initially in u-boot-ti) causes a regression in building the da830evm board, by causing the following warning:

cmd_mem.c: In function 'do_mem_mtest':
cmd_mem.c:665: warning: suggest parentheses around + or - inside shift

This should not have functional consequences, but please fix.

Amicalement,

Signature · 1 line
Albert.
#1 lore raw
Quoted 6 lines

Hi all,

Quoted 8 lines

Commit 11a6fb79745b06610b334bada7167e9e0b619851 ("da830: modify the MEMTEST start and end address") in u-boot-arm (initially in u-boot-ti) causes a regression in building the da830evm board, by causing the following warning:

cmd_mem.c: In function 'do_mem_mtest':
cmd_mem.c:665: warning: suggest parentheses around + or - inside shift

I sent a patch.

Regards,
Sandeep

#2 lore raw

Hi Sandeep,

Le 13/09/2011 21:59, Paulraj, Sandeep a ?crit :

Quoted 9 lines
Quoted 9 lines

Hi all,

Commit 11a6fb79745b06610b334bada7167e9e0b619851 ("da830: modify the MEMTEST start and end address") in u-boot-arm (initially in u-boot-ti) causes a regression in building the da830evm board, by causing the following warning:

Quoted 6 lines
Quoted 3 lines

cmd_mem.c: In function 'do_mem_mtest':
cmd_mem.c:665: warning: suggest parentheses around + or - inside shift

I sent a patch.

Thanks! I will apply it as soon as my current regression run is done.

Quoted 2 lines

Regards,
Sandeep

Amicalement,

Signature · 1 line
Albert.