[U-Boot] [GIT] Pull request: u-boot-dfu
B, Ravi
ravibabu at ti.com
Wed Aug 17 09:40:04 CEST 2016
Hi Heiko
>>>
>>> is that for master or next ?
>>This patch _was_ supposed to go to "master"
>>> Was this build tested ?
>>Unfortunately, not so thoroughly as I thought.
>>Moving dfu code to SPL causes following error on some boards:
>> arm: + smartweb
>>+In file included from ../include/dfu.h:18:0,
>>+ from ../common/dfu.c:16:
>>+../include/linux/usb/composite.h:331:9: error: requested alignment is
>>+not an integer constant
>>+ struct usb_device_descriptor __aligned(CONFIG_SYS_CACHELINE_SIZE) desc;
>>+ ^
>>+make[3]: *** [spl/common/dfu.o] Error 1
>>+make[2]: *** [spl/common] Error 2
>+make[1]: *** [spl/u-boot-spl] Error 2
>>+make: *** [sub-make] Error 2
The CONFIG_SYS_CACHELINE_SIZE is not defined for smartweb platform which is causing build error.
By defining this error goes away. What would be value for cache line size for smartweb platform? (32/64/..?)
>>Ravi, to reproduce you need to fetch u-boot-dfu/test
>>and run buildman:
>>This is only a small subset of affected boards, so please test also "arm" (which might take long time).
>>./tools/buildman/buildman.py --branch=HEAD siemens --detail --verbose
>>--show_errors --force-build --count=5 --output-dir=./BUILD/
>>Thanks Marek, for pointing out.
>I have test build for all ti-platforms, not checked for rest. My bad ..!
>I could reproduce this error, for some platform CONFIG_SYS_CACHELINE_SIZE is not defined. (eg. configs/smartweb_defconfig).
>I will check with "arm" as well, let you know.
Regards
Ravi
More information about the U-Boot
mailing list