Stefano, Le 30/08/2011 10:16, Albert ARIBAUD a écrit : > Le 30/08/2011 09:16, Stefano Babic a écrit : > >> On 08/30/2011 08:47 AM, Albert ARIBAUD wrote: >>> Hi Stefano, >> >> Hi Albert, >> >>> Building all versatile boards here with ELDK4.2 on a Natty Ubuntu, I get >>> warnings of two sorts: >>> >>> 1) in the QEMU build, you needlessly cast a pointer as 'volatile', >> >> This is wrong... >> >>> which >>> I missed on review but which my compiler does not like -- I did a >>> general removal patch for code in U-Boot that calls get_ram_size() with >>> a volatile-cast argument. >> >> I saw the same issue rebuilding all IMX board for the scb9328 - because >> this was the only board with this warning I found, I sent a patch for it >> and I have already applied to u-boot-imx. I sent to you a "pull >> request", but let me know if you prefer I drop my patch from u-boot-imx >> before you merge into u-boot-arm. > > I'll pick it with the IMX pull req, and I will apply Wolfgang's patch > set afterward, minus versatile if the board is completely fixed.
Sorry, I realize there might have been a misunderstanding. The global fix patch I did was applied some time ago and any spurious volatile left must be fixed by new patches, so your patch is welcome anyway. Note that you also need to provide a fix for versatile.c. Amicalement, -- Albert. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
