[U-Boot] [PATCH] board: freescale: ls2080a: Intergrate and enable PPA on LS2080A

[york sun]

On 12/29/2016 10:01 PM, Prabhakar Kushwaha wrote:
>
>> -----Original Message-----
>> From: U-Boot [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Abhimanyu
>> Saini
>> Sent: Thursday, December 29, 2016 9:52 AM
>> To: york sun <york.sun at nxp.com>; u-boot at lists.denx.de
>> Cc: Priyanka Jain <priyanka.jain at nxp.com>
>> Subject: [U-Boot] [PATCH] board: freescale: ls2080a: Intergrate and enable PPA
>> on LS2080A
>>
>> Integrate and enable PPA on LS2080ARDB, LS2080AQDS, LS2088ARDB
>> and LS2088AQDS boards.
>>
>> Signed-off-by: Abhimanyu Saini <abhimanyu.saini at nxp.com>
>> ---
>>  board/freescale/ls2080aqds/ls2080aqds.c |  6 ++++++
>>  board/freescale/ls2080ardb/ls2080ardb.c |  6 ++++++
>>  include/configs/ls2080aqds.h            | 10 ++++++++++
>>  include/configs/ls2080ardb.h            | 10 ++++++++++
>>  4 files changed, 32 insertions(+)
>>
>> diff --git a/board/freescale/ls2080aqds/ls2080aqds.c
>> b/board/freescale/ls2080aqds/ls2080aqds.c
>> index 73a61fd..811b9a6 100644
>> --- a/board/freescale/ls2080aqds/ls2080aqds.c
>> +++ b/board/freescale/ls2080aqds/ls2080aqds.c
>> @@ -19,6 +19,9 @@
>>  #include <asm/arch/soc.h>
>>  #include <hwconfig.h>
>>  #include <fsl_sec.h>
>> +#ifdef CONFIG_FSL_LS_PPA
>> +#include <asm/arch/ppa.h>
>> +#endif
>>
>>  #include "../common/qixis.h"
>>  #include "ls2080aqds_qixis.h"
>> @@ -223,6 +226,9 @@ int board_init(void)
>>  #endif
>>  	select_i2c_ch_pca9547(I2C_MUX_CH_DEFAULT);
>>  	rtc_enable_32khz_output();
>> +#ifdef CONFIG_FSL_LS_PPA
>> +	ppa_init();
>> +#endif
>>
>>  	return 0;
>>  }
>> diff --git a/board/freescale/ls2080ardb/ls2080ardb.c
>> b/board/freescale/ls2080ardb/ls2080ardb.c
>> index 02954ef..339fa78 100644
>> --- a/board/freescale/ls2080ardb/ls2080ardb.c
>> +++ b/board/freescale/ls2080ardb/ls2080ardb.c
>> @@ -19,6 +19,9 @@
>>  #include <i2c.h>
>>  #include <asm/arch/soc.h>
>>  #include <fsl_sec.h>
>> +#ifdef CONFIG_FSL_LS_PPA
>> +#include <asm/arch/ppa.h>
>> +#endif
>>
>>  #include "../common/qixis.h"
>>  #include "ls2080ardb_qixis.h"
>> @@ -184,6 +187,9 @@ int board_init(void)
>>  	/* invert AQR405 IRQ pins polarity */
>>  	out_le32(irq_ccsr + IRQCR_OFFSET / 4, AQR405_IRQ_MASK);
>>  #endif
>> +#ifdef CONFIG_FSL_LS_PPA
>> +	ppa_init();
>> +#endif
>>
>>  	return 0;
>>  }
>> diff --git a/include/configs/ls2080aqds.h b/include/configs/ls2080aqds.h
>> index 838568f..5573e94 100644
>> --- a/include/configs/ls2080aqds.h
>> +++ b/include/configs/ls2080aqds.h
>> @@ -14,6 +14,16 @@ unsigned long get_board_sys_clk(void);
>>  unsigned long get_board_ddr_clk(void);
>>  #endif
>>
>> +#define CONFIG_FSL_LS_PPA
>
> I am not finding CONFIG_FSL_LS_PPA being defined in board header file.

Since this config option has been moved into Kconfig, we don't use board 
header file to define it any more. Actually, existing board don't enable 
this by default either. We can make it default after the PPA image is 
readily available to public.

> Please refer arch/arm/cpu/armv8/fsl-layerscape/Kconfig. Looks like its enablement is govern from here.

This patch lacks modification of 
arch/arm/cpu/armv8/fsl-layerscape/Kconfig to enable FSL_LS_PPA for 
LS2080A. There are some floating patches to change the 
CONFIG_SYS_LS_PPA_FW_IN_XIP. Please work with Zhiqiang on this.

York