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

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@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