On 4/22/19 12:48 PM, Tom Rini wrote:
> On Mon, Apr 01, 2019 at 05:32:17PM -0500, Dinh Nguyen wrote:
> 
>> The cache UCLASS will be used for configure settings that can be found
>> in a CPU's L2 cache controller.
>>
>> Add a uclass and a test for cache.
>>
>> Reviewed-by: Simon Glass <s...@chromium.org>
>> Signed-off-by: Dinh Nguyen <dingu...@kernel.org>
>> ---
>> v4: re-order includes and add Reviewed-by:
>> v3: Add cache_get_info() to check for non-zero value
>>     Add comments to cache_info struct
>> v2: separate out uclass patch from driver and add test
> 
> NAK:
>    sandbox:  +   tools-only
> +(tools-only) In file included from drivers/cache/sandbox_cache.c:6:0:
> +(tools-only) include/cache.h:13:2: error: unknown type name 'phys_addr_t'
> +(tools-only)   phys_addr_t base; /* Base physical address of cache device. */
> +(tools-only)   ^~~~~~~~~~~
> +(tools-only)   int (*get_info)(struct udevice *dev, struct cache_info *info);
> +(tools-only)                          ^~~~~~~
> +(tools-only)  int cache_get_info(struct udevice *dev, struct cache_info 
> *info);
> +(tools-only)                            ^~~~~~~
> +(tools-only)   .get_info = sandbox_get_info,
> +(tools-only)               ^~~~~~~~~~~~~~~~
> +(tools-only) drivers/cache/sandbox_cache.c:21:14: note: (near initialization 
> for 'sandbox_cach
> e_ops.get_info')
> +(tools-only) make[3]: *** [drivers/cache/sandbox_cache.o] Error 1
> +(tools-only) make[2]: *** [drivers/cache] Error 2
> +(tools-only) make[1]: *** [drivers] Error 2
> +(tools-only) make: *** [sub-make] Error 2
> w+(tools-only) include/cache.h:24:25: warning: 'struct udevice' declared 
> inside parameter list will not be visible outside of this definition or 
> declaration
> w+(tools-only) include/cache.h:36:27: warning: 'struct udevice' declared 
> inside parameter list will not be visible outside of this definition or 
> declaration
> w+(tools-only) drivers/cache/sandbox_cache.c:21:14: warning: initialization 
> from incompatible pointer type [-Wincompatible-pointer-types]
> 

I apologize for that! V5 is enroute and should be error free.

Dinh

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to