[U-Boot] [PATCH v3 2/2] armv8: layerscape: move ns_dev[] define from h to c file.

York Sun york.sun at nxp.com
Thu Aug 9 19:45:46 UTC 2018


On 08/03/2018 12:56 AM, Ran Wang wrote:
> Since more c files will include ns_access.h, this move will fix some
> compiling warnings and make it sense.
> 
> Signed-off-by: Ran Wang <ran.wang_1 at nxp.com>
> ---
> Change in v3:
> 	- New file
> 
>  .../include/asm/arch-fsl-layerscape/ns_access.h    | 80 ----------------------
>  board/freescale/common/ns_access.c                 | 80 ++++++++++++++++++++++
>  2 files changed, 80 insertions(+), 80 deletions(-)
> 

Two issues with this patch. First this patch should be in front of your
patch #1. Otherwise you patch #1 has this compiling warning

warning: ?ns_dev? defined but not used [-Wunused-variable]

Second issue is for ls1021a. You will see compiling failure.

York


More information about the U-Boot mailing list