On Mon, Apr 11, 2016 at 01:59:59AM +0200, Marek Vasut wrote: > > This looks more like silencing the warning by a forced cast. > The pointer should most likely be sanity-checked to make sure it's in > 4GiB address space at least. I am worried such forced casts will bite > us in the long run.
Yes, this patch assumes that addresses of buffers and descriptors are in the first 4GiB of address space. AFAIU this should be a safe premise for current SoCs, since the physical memory is always mapped within the 32bit addressable range; other drivers (see for example sunxi_mmc) do a similar assumption. So, right now I don't think a check is strictly needed, but to be future-proof probably it wouldn't hurt either. Beniamino _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot