[U-Boot] [PATCH][v2] armv8: lx2160ardb : Add support for LX2160ARDB platform

From: Priyanka Jain <priyanka.jain@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH][v2] armv8: lx2160ardb : Add support for LX2160ARDB platform
Date: Tue, 20 Nov 2018 05:21:49 +0000	[thread overview]
Message-ID: <AM0PR04MB4931FE68324D8243CCFB5998E6D90@AM0PR04MB4931.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <AM0PR04MB3971175D293A57FAC9B702199AD80@AM0PR04MB3971.eurprd04.prod.outlook.com>

>-----Original Message-----
>From: York Sun
>Sent: Tuesday, November 20, 2018 2:17 AM
>To: Priyanka Jain <priyanka.jain@nxp.com>; u-boot at lists.denx.de
>Cc: Wasim Khan <wasim.khan@nxp.com>; Yogesh Narayan Gaur
><yogeshnarayan.gaur@nxp.com>; Meenakshi Aggarwal
><meenakshi.aggarwal@nxp.com>; Vabhav Sharma
><vabhav.sharma@nxp.com>; Sriram Dash <sriram.dash@nxp.com>; Rajesh
>Bhagat <rajesh.bhagat@nxp.com>; Pankit Garg <pankit.garg@nxp.com>
>Subject: Re: [PATCH][v2] armv8: lx2160ardb : Add support for LX2160ARDB
>platform
>
>On 10/29/18 2:30 AM, Priyanka Jain wrote:
>> LX2160ARDB is an evaluation board that supports LX2160A family SoCs.
>> This patch add base support for this board.
>>
>> Signed-off-by: Wasim Khan <wasim.khan@nxp.com>
>> Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@nxp.com>
>> Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
>> Signed-off-by: Vabhav Sharma <vabhav.sharma@nxp.com>
>> Signed-off-by: Sriram Dash <sriram.dash@nxp.com>
>> Signed-off-by: Rajesh Bhagat <rajesh.bhagat@nxp.com>
>> Signed-off-by: Pankit Garg <pankit.garg@nxp.com>
>> Signed-off-by: Priyanka Jain <priyanka.jain@nxp.com>
>> ---
>> Changes for v2:
>>  Rebased on top of test_qoriq branch of u-boot-fsl-qoriq.git
>>
>>  Corrected line
>> 	seria01.clock = get_serial_clock ->  serial1.clock =
>> get_serial_clock();
>>
>>  Corrected CONFIG_ENV_OFFSET to 0x500000 [Thanks to Ashish K for
>> pointing this]
>>
>
>Priyanka,
>
>I am seeing compiling error
>
>../board/freescale/lx2160a/eth_lx2160ardb.c:71:3: error: too few arguments
>to function ?wriop_set_phy_address?
>
Compilation error can be due to recent ethernet patch from Pankaj which changes API format.
LX2RDB patch needs to be rebased on top of this patch then
Please confirm your working branch. Is it  test_qoriq?
I will send another version after rebasing and testing on this branch.

>Please also check the patches I collected here
>http://patchwork.ozlabs.org/bundle/yorksun/lx2/ and confirm the
>dependency and order.
>
>York

The order of LX2160ARDB related dependent patches should be:
[U-Boot,v2] armv8: lx2160ardb : Add support for LX2160ARDB platform
[U-Boot] drivers/ddr/fsl: Update fsl_ddr_board_options as weak function 
[U-Boot,v3] armv8: lx2160a: Add LX2160A SoC Support
[U-Boot,v2] armv8:fsl-layerscape: Add support for Chassis 3.2
[U-Boot] board/freescale/vid: Add vdd table for NXP LX2160A SoC
[U-Boot] board/freescale/vid: Add correction for ltc3882 read error.
[U-Boot,v2] armv8: lsch3: Add support of serdes3 module

lx2160a-eSDHC related patches from Yingbo Zhu should be applied on top of the lx2160ardb patches.

Priyanka