Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi Chenhui,

Do you have any update about how to fixup the related device node according to 
the PSCI and spin-table?

> -Original Message-
> From: york sun
> Sent: 2016年3月18日 5:09
> To: Zhiqiang Hou ; Bhupesh Sharma
> ; Scott Wood ; Zhiqiang Hou
> ; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder
> ; le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Rod Dorris ; Chenhui
> Zhao 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On 01/24/2016 07:22 PM, Zhiqiang Hou wrote:
> >
> > Chenhui is working on it.
> >
> 
> Guys,
> 
> Where are we on this set?
> 
> York
> 

Thanks,
Zhiqiang

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[york sun]

On 01/24/2016 07:22 PM, Zhiqiang Hou wrote:
> 
> Chenhui is working on it.
> 

Guys,

Where are we on this set?

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi Rod and Chenhui,

Can you help to answer the questions about PSCI?

> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: 2016年1月23日 4:09
> To: york sun ; Zhiqiang Hou ;
> u-boot@lists.denx.de; albert.u.b...@aribaud.net; mingkai...@freescale.com;
> Stuart Yoder ; le...@freescale.com;
> prabha...@freescale.com; bhupesh.sha...@freescale.com; Zhiqiang Hou
> 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > From: Hou Zhiqiang 
> > >
> > > The FSL Primary Protected Application (PPA) is a software component
> > > loaded during boot which runs in TrustZone and remains resident
> > > after boot.
> > >
> > > Signed-off-by: Hou Zhiqiang 
> > > ---
> > > Tested on LS1043A RDB board
> > >
> >
> > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > change with PPA? U-boot has to remain functional in the absence of PPA, 
> > right?
> >
> > Would it be possible to write a readme for PPA?
> 
> Also, how does the devicetree get updated to indicate the presence of PSCI?
> 

Thanks,
Zhiqiang
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi Rod and Chenhui,

Can you help to answer the questions about PSCI?

> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: 2016年1月23日 4:18
> To: Bhupesh Sharma ; york sun ;
> Zhiqiang Hou ; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder
> ; le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On Fri, 2016-01-22 at 20:14 +, Bhupesh Sharma wrote:
> >
> > > -Original Message-
> > > From: Scott Wood [mailto:o...@buserror.net]
> > > Sent: Saturday, January 23, 2016 1:39 AM
> > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > le...@freescale.com; prabha...@freescale.com;
> > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > >
> > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > From: Hou Zhiqiang 
> > > > >
> > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > component loaded during boot which runs in TrustZone and remains
> > > > > resident after boot.
> > > > >
> > > > > Signed-off-by: Hou Zhiqiang 
> > > > > ---
> > > > > Tested on LS1043A RDB board
> > > > >
> > > >
> > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > change with PPA? U-boot has to remain functional in the absence of
> > > > PPA,
> > > right?
> > > >
> > > > Would it be possible to write a readme for PPA?
> > >
> > > Also, how does the devicetree get updated to indicate the presence
> > > of PSCI?
> > >
> >
> > One needs to add PSCI node in the DTS for the same, claiming support
> > for PSCI specification.
> 
> So then we need a different dts based on whether PPA is loaded?  No thanks.
> 
> > The cpu.enable_method needs to be changed from spin-table to psci as well.
> 
> Right, that's part of updating the devicetree to indicate the presence of 
> PSCI.
> 
> > See
> > http://lxr.free-electrons.com/source/Documentation/devicetree/bindings
> > /a
> > rm/psci.txt for details.
> 
> I'm familiar with the binding.
> 
> -Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

+ Rod and Chenhui

> -Original Message-
> From: Zhiqiang Hou
> Sent: 2016年1月25日 11:16
> To: 'Scott Wood' ; york sun ; Bhupesh
> Sharma ; Zhiqiang Hou
> ; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder
> ; le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com
> Subject: RE: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> Hi Rod and Chenhui,
> 
> Can you help to answer the questions about PSCI and PPA?
> 
> > -Original Message-
> > From: Scott Wood [mailto:o...@buserror.net]
> > Sent: 2016年1月23日 4:22
> > To: york sun ; Bhupesh Sharma
> > ; Zhiqiang Hou ;
> > u-boot@lists.denx.de; albert.u.b...@aribaud.net;
> > mingkai...@freescale.com; Stuart Yoder ;
> > le...@freescale.com; prabha...@freescale.com;
> > bhupesh.sha...@freescale.com; Zhiqiang Hou 
> > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> >
> > On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> > > On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> > > >
> > > >
> > > > > -Original Message-
> > > > > From: Scott Wood [mailto:o...@buserror.net]
> > > > > Sent: Saturday, January 23, 2016 1:39 AM
> > > > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart
> > > > > Yoder; le...@freescale.com; prabha...@freescale.com;
> > > > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > > > >
> > > > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > > > From: Hou Zhiqiang 
> > > > > > >
> > > > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > > > component loaded during boot which runs in TrustZone and
> > > > > > > remains resident after boot.
> > > > > > >
> > > > > > > Signed-off-by: Hou Zhiqiang 
> > > > > > > ---
> > > > > > > Tested on LS1043A RDB board
> > > > > > >
> > > > > >
> > > > > > I presume PSCI is implemented in PPA. How does the SMP boot
> > > > > > flow change with PPA? U-boot has to remain functional in the
> > > > > > absence of PPA,
> > > > > right?
> > > > > >
> > > > > > Would it be possible to write a readme for PPA?
> > > > >
> > > > > Also, how does the devicetree get updated to indicate the
> > > > > presence of PSCI?
> > > > >
> > > >
> > > > One needs to add PSCI node in the DTS for the same, claiming
> > > > support for PSCI specification.
> > > > The cpu.enable_method needs to be changed from spin-table to psci as 
> > > > well.
> > > >
> > > > See
> > > > http://lxr.free-electrons.com/source/Documentation/devicetree/bind
> > > > in
> > > > gs
> > > > /arm/psci.txt for details.
> > > >
> > >
> > > Bhupesh,
> > >
> > > Before PPA is fully adopted, u-boot has to detect the existence of
> > > PPA and decide to use PSCI or spin table, right? So the device tree
> > > node should also be fixed accordingly.
> >
> > How would U-Boot know what version of PSCI the loaded PPA supports?
> > Is or could there be any metadata on the PPA image to convey such 
> > information?
> >
> 
> Thanks,
> Zhiqiang
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi Rod and Chenhui,

Can you help to answer the questions about PSCI and PPA?

> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: 2016年1月23日 4:22
> To: york sun ; Bhupesh Sharma ;
> Zhiqiang Hou ; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder
> ; le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> > On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> > >
> > >
> > > > -Original Message-
> > > > From: Scott Wood [mailto:o...@buserror.net]
> > > > Sent: Saturday, January 23, 2016 1:39 AM
> > > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > > le...@freescale.com; prabha...@freescale.com;
> > > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > > >
> > > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > > From: Hou Zhiqiang 
> > > > > >
> > > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > > component loaded during boot which runs in TrustZone and
> > > > > > remains resident after boot.
> > > > > >
> > > > > > Signed-off-by: Hou Zhiqiang 
> > > > > > ---
> > > > > > Tested on LS1043A RDB board
> > > > > >
> > > > >
> > > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > > change with PPA? U-boot has to remain functional in the absence
> > > > > of PPA,
> > > > right?
> > > > >
> > > > > Would it be possible to write a readme for PPA?
> > > >
> > > > Also, how does the devicetree get updated to indicate the presence
> > > > of PSCI?
> > > >
> > >
> > > One needs to add PSCI node in the DTS for the same, claiming support
> > > for PSCI specification.
> > > The cpu.enable_method needs to be changed from spin-table to psci as well.
> > >
> > > See
> > > http://lxr.free-electrons.com/source/Documentation/devicetree/bindin
> > > gs
> > > /arm/psci.txt for details.
> > >
> >
> > Bhupesh,
> >
> > Before PPA is fully adopted, u-boot has to detect the existence of PPA
> > and decide to use PSCI or spin table, right? So the device tree node
> > should also be fixed accordingly.
> 
> How would U-Boot know what version of PSCI the loaded PPA supports?  Is or
> could there be any metadata on the PPA image to convey such information?
> 

Thanks,
Zhiqiang
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi York,

Sorry, I didn't touch the PSCI in u-boot and what works the PPA is doing, I 
just work on
integrating PPA to u-boot.

Hi Rod and Chenhui,

Can you help to answer the questions about the PSCI and PPA?

> -Original Message-
> From: york sun
> Sent: 2016年1月23日 4:05
> To: Zhiqiang Hou ; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder
> ; le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou ; Scott
> Wood 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > From: Hou Zhiqiang 
> >
> > The FSL Primary Protected Application (PPA) is a software component
> > loaded during boot which runs in TrustZone and remains resident after
> > boot.
> >
> > Signed-off-by: Hou Zhiqiang 
> > ---
> > Tested on LS1043A RDB board
> >
> 
> I presume PSCI is implemented in PPA. How does the SMP boot flow change with
> PPA? U-boot has to remain functional in the absence of PPA, right?
> 
> Would it be possible to write a readme for PPA?
> 

Thanks,
Zhiqiang

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Zhiqiang Hou]

Hi Rod and Chenhui,

Can you help to answer the questions about PSCI and PPA?

> -Original Message-
> From: york sun
> Sent: 2016年1月23日 4:30
> To: Bhupesh Sharma ; Scott Wood
> ; Zhiqiang Hou ; u-
> b...@lists.denx.de; albert.u.b...@aribaud.net; mingkai...@freescale.com;
> Stuart Yoder ; le...@freescale.com;
> prabha...@freescale.com; bhupesh.sha...@freescale.com; Zhiqiang Hou
> 
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On 01/22/2016 12:27 PM, Bhupesh Sharma wrote:
> >
> >
> >> -Original Message-
> >> From: Scott Wood [mailto:o...@buserror.net]
> >> Sent: Saturday, January 23, 2016 1:52 AM
> >> To: york sun; Bhupesh Sharma; Zhiqiang Hou; u-boot@lists.denx.de;
> >> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> >> le...@freescale.com; prabha...@freescale.com;
> >> bhupesh.sha...@freescale.com; Zhiqiang Hou
> >> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> >>
> >> On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> >>> On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> 
> 
> > -Original Message-
> > From: Scott Wood [mailto:o...@buserror.net]
> > Sent: Saturday, January 23, 2016 1:39 AM
> > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > le...@freescale.com; prabha...@freescale.com;
> > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> >
> > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> >> On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> >>> From: Hou Zhiqiang 
> >>>
> >>> The FSL Primary Protected Application (PPA) is a software
> >>> component loaded during boot which runs in TrustZone and remains
> >>> resident after boot.
> >>>
> >>> Signed-off-by: Hou Zhiqiang 
> >>> ---
> >>> Tested on LS1043A RDB board
> >>>
> >>
> >> I presume PSCI is implemented in PPA. How does the SMP boot flow
> >> change with PPA? U-boot has to remain functional in the absence
> >> of PPA,
> > right?
> >>
> >> Would it be possible to write a readme for PPA?
> >
> > Also, how does the devicetree get updated to indicate the presence
> > of PSCI?
> >
> 
>  One needs to add PSCI node in the DTS for the same, claiming
>  support for PSCI specification.
>  The cpu.enable_method needs to be changed from spin-table to psci
>  as
> >> well.
> 
>  See
>  http://lxr.free-electrons.com/source/Documentation/devicetree/bindi
>  n
>  gs
>  /arm/psci.txt for details.
> 
> >>>
> >>> Bhupesh,
> >>>
> >>> Before PPA is fully adopted, u-boot has to detect the existence of
> >>> PPA and decide to use PSCI or spin table, right? So the device tree
> >>> node should also be fixed accordingly.
> >
> > A static CONFIG_ flag is used for enabling the PPA support currently in 
> > QorIQ-LS
> u-boot code.
> 
> Not enough. The driver parse PPA image and gives up if it is not a valid 
> image.
> Before we have a way to deploy PPA image to all our customers, we need to keep
> both options open.
> 

Chenhui is working on it.

Thanks,
Zhiqiang

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Scott Wood]

On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> > 
> > 
> > > -Original Message-
> > > From: Scott Wood [mailto:o...@buserror.net]
> > > Sent: Saturday, January 23, 2016 1:39 AM
> > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > le...@freescale.com; prabha...@freescale.com;
> > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > > 
> > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > From: Hou Zhiqiang 
> > > > > 
> > > > > The FSL Primary Protected Application (PPA) is a software component
> > > > > loaded during boot which runs in TrustZone and remains resident
> > > > > after boot.
> > > > > 
> > > > > Signed-off-by: Hou Zhiqiang 
> > > > > ---
> > > > > Tested on LS1043A RDB board
> > > > > 
> > > > 
> > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > change with PPA? U-boot has to remain functional in the absence of
> > > > PPA,
> > > right?
> > > > 
> > > > Would it be possible to write a readme for PPA?
> > > 
> > > Also, how does the devicetree get updated to indicate the presence of
> > > PSCI?
> > > 
> > 
> > One needs to add PSCI node in the DTS for the same, claiming support for
> > PSCI specification.
> > The cpu.enable_method needs to be changed from spin-table to psci as well.
> > 
> > See http://lxr.free-electrons.com/source/Documentation/devicetree/bindings
> > /arm/psci.txt for details.
> > 
> 
> Bhupesh,
> 
> Before PPA is fully adopted, u-boot has to detect the existence of PPA and
> decide to use PSCI or spin table, right? So the device tree node should also
> be
> fixed accordingly.

How would U-Boot know what version of PSCI the loaded PPA supports?  Is or
could there be any metadata on the PPA image to convey such information?

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[york sun]

On 01/22/2016 12:27 PM, Bhupesh Sharma wrote:
> 
> 
>> -Original Message-
>> From: Scott Wood [mailto:o...@buserror.net]
>> Sent: Saturday, January 23, 2016 1:52 AM
>> To: york sun; Bhupesh Sharma; Zhiqiang Hou; u-boot@lists.denx.de;
>> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
>> le...@freescale.com; prabha...@freescale.com;
>> bhupesh.sha...@freescale.com; Zhiqiang Hou
>> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
>>
>> On Fri, 2016-01-22 at 20:18 +, york sun wrote:
>>> On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:


> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: Saturday, January 23, 2016 1:39 AM
> To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
>
> On Fri, 2016-01-22 at 20:05 +, york sun wrote:
>> On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
>>> From: Hou Zhiqiang 
>>>
>>> The FSL Primary Protected Application (PPA) is a software
>>> component loaded during boot which runs in TrustZone and
>>> remains resident after boot.
>>>
>>> Signed-off-by: Hou Zhiqiang 
>>> ---
>>> Tested on LS1043A RDB board
>>>
>>
>> I presume PSCI is implemented in PPA. How does the SMP boot flow
>> change with PPA? U-boot has to remain functional in the absence
>> of PPA,
> right?
>>
>> Would it be possible to write a readme for PPA?
>
> Also, how does the devicetree get updated to indicate the presence
> of PSCI?
>

 One needs to add PSCI node in the DTS for the same, claiming support
 for PSCI specification.
 The cpu.enable_method needs to be changed from spin-table to psci as
>> well.

 See
 http://lxr.free-electrons.com/source/Documentation/devicetree/bindin
 gs
 /arm/psci.txt for details.

>>>
>>> Bhupesh,
>>>
>>> Before PPA is fully adopted, u-boot has to detect the existence of PPA
>>> and decide to use PSCI or spin table, right? So the device tree node
>>> should also be fixed accordingly.
> 
> A static CONFIG_ flag is used for enabling the PPA support currently in 
> QorIQ-LS u-boot code.

Not enough. The driver parse PPA image and gives up if it is not a valid image.
Before we have a way to deploy PPA image to all our customers, we need to keep
both options open.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Scott Wood]

On Fri, 2016-01-22 at 20:27 +, Bhupesh Sharma wrote:
> 
> > -Original Message-
> > From: Scott Wood [mailto:o...@buserror.net]
> > Sent: Saturday, January 23, 2016 1:52 AM
> > To: york sun; Bhupesh Sharma; Zhiqiang Hou; u-boot@lists.denx.de;
> > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > le...@freescale.com; prabha...@freescale.com;
> > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > 
> > On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> > > On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> > > > 
> > > > 
> > > > > -Original Message-
> > > > > From: Scott Wood [mailto:o...@buserror.net]
> > > > > Sent: Saturday, January 23, 2016 1:39 AM
> > > > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > > > le...@freescale.com; prabha...@freescale.com;
> > > > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > > > > 
> > > > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > > > From: Hou Zhiqiang 
> > > > > > > 
> > > > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > > > component loaded during boot which runs in TrustZone and
> > > > > > > remains resident after boot.
> > > > > > > 
> > > > > > > Signed-off-by: Hou Zhiqiang 
> > > > > > > ---
> > > > > > > Tested on LS1043A RDB board
> > > > > > > 
> > > > > > 
> > > > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > > > change with PPA? U-boot has to remain functional in the absence
> > > > > > of PPA,
> > > > > right?
> > > > > > 
> > > > > > Would it be possible to write a readme for PPA?
> > > > > 
> > > > > Also, how does the devicetree get updated to indicate the presence
> > > > > of PSCI?
> > > > > 
> > > > 
> > > > One needs to add PSCI node in the DTS for the same, claiming support
> > > > for PSCI specification.
> > > > The cpu.enable_method needs to be changed from spin-table to psci as
> > well.
> > > > 
> > > > See
> > > > http://lxr.free-electrons.com/source/Documentation/devicetree/bindin
> > > > gs
> > > > /arm/psci.txt for details.
> > > > 
> > > 
> > > Bhupesh,
> > > 
> > > Before PPA is fully adopted, u-boot has to detect the existence of PPA
> > > and decide to use PSCI or spin table, right? So the device tree node
> > > should also be fixed accordingly.
> 
> A static CONFIG_ flag is used for enabling the PPA support currently in
> QorIQ-LS u-boot code.
> 
> > 
> > How would U-Boot know what version of PSCI the loaded PPA supports?  Is
> > or could there be any metadata on the PPA image to convey such
> > information?
> 
> PSCI specifications support the PSCI_VERSION API which is implemented by PPA
> and it returns the PSCI_VERSION
> supported by PPA.

That does not answer the question unless we can assume that PPA always
implements PSCI >= 0.2.  And maybe there are other services worth advertising
in the device tree?  Some sort of metadata would be nice.

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[york sun]

On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> From: Hou Zhiqiang 
> 
> The FSL Primary Protected Application (PPA) is a software component
> loaded during boot which runs in TrustZone and remains resident
> after boot.
> 
> Signed-off-by: Hou Zhiqiang 
> ---
> Tested on LS1043A RDB board
> 

I presume PSCI is implemented in PPA. How does the SMP boot flow change with
PPA? U-boot has to remain functional in the absence of PPA, right?

Would it be possible to write a readme for PPA?

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Scott Wood]

On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > From: Hou Zhiqiang 
> > 
> > The FSL Primary Protected Application (PPA) is a software component
> > loaded during boot which runs in TrustZone and remains resident
> > after boot.
> > 
> > Signed-off-by: Hou Zhiqiang 
> > ---
> > Tested on LS1043A RDB board
> > 
> 
> I presume PSCI is implemented in PPA. How does the SMP boot flow change with
> PPA? U-boot has to remain functional in the absence of PPA, right?
> 
> Would it be possible to write a readme for PPA?

Also, how does the devicetree get updated to indicate the presence of PSCI?

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[york sun]

On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> 
> 
>> -Original Message-
>> From: Scott Wood [mailto:o...@buserror.net]
>> Sent: Saturday, January 23, 2016 1:39 AM
>> To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
>> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
>> le...@freescale.com; prabha...@freescale.com;
>> bhupesh.sha...@freescale.com; Zhiqiang Hou
>> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
>>
>> On Fri, 2016-01-22 at 20:05 +, york sun wrote:
>>> On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
 From: Hou Zhiqiang 

 The FSL Primary Protected Application (PPA) is a software component
 loaded during boot which runs in TrustZone and remains resident
 after boot.

 Signed-off-by: Hou Zhiqiang 
 ---
 Tested on LS1043A RDB board

>>>
>>> I presume PSCI is implemented in PPA. How does the SMP boot flow
>>> change with PPA? U-boot has to remain functional in the absence of PPA,
>> right?
>>>
>>> Would it be possible to write a readme for PPA?
>>
>> Also, how does the devicetree get updated to indicate the presence of
>> PSCI?
>>
> 
> One needs to add PSCI node in the DTS for the same, claiming support for PSCI 
> specification.
> The cpu.enable_method needs to be changed from spin-table to psci as well.
> 
> See 
> http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/arm/psci.txt
>  for details.
> 

Bhupesh,

Before PPA is fully adopted, u-boot has to detect the existence of PPA and
decide to use PSCI or spin table, right? So the device tree node should also be
fixed accordingly.

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Scott Wood]

On Fri, 2016-01-22 at 20:14 +, Bhupesh Sharma wrote:
> 
> > -Original Message-
> > From: Scott Wood [mailto:o...@buserror.net]
> > Sent: Saturday, January 23, 2016 1:39 AM
> > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > le...@freescale.com; prabha...@freescale.com;
> > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > 
> > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > From: Hou Zhiqiang 
> > > > 
> > > > The FSL Primary Protected Application (PPA) is a software component
> > > > loaded during boot which runs in TrustZone and remains resident
> > > > after boot.
> > > > 
> > > > Signed-off-by: Hou Zhiqiang 
> > > > ---
> > > > Tested on LS1043A RDB board
> > > > 
> > > 
> > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > change with PPA? U-boot has to remain functional in the absence of PPA,
> > right?
> > > 
> > > Would it be possible to write a readme for PPA?
> > 
> > Also, how does the devicetree get updated to indicate the presence of
> > PSCI?
> > 
> 
> One needs to add PSCI node in the DTS for the same, claiming support for
> PSCI specification.

So then we need a different dts based on whether PPA is loaded?  No thanks.

> The cpu.enable_method needs to be changed from spin-table to psci as well.

Right, that's part of updating the devicetree to indicate the presence of
PSCI.

> See http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/a
> rm/psci.txt for details.

I'm familiar with the binding.

-Scott

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Bhupesh Sharma]

> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: Saturday, January 23, 2016 1:52 AM
> To: york sun; Bhupesh Sharma; Zhiqiang Hou; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On Fri, 2016-01-22 at 20:18 +, york sun wrote:
> > On 01/22/2016 12:14 PM, Bhupesh Sharma wrote:
> > >
> > >
> > > > -Original Message-
> > > > From: Scott Wood [mailto:o...@buserror.net]
> > > > Sent: Saturday, January 23, 2016 1:39 AM
> > > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > > le...@freescale.com; prabha...@freescale.com;
> > > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > > >
> > > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > > From: Hou Zhiqiang 
> > > > > >
> > > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > > component loaded during boot which runs in TrustZone and
> > > > > > remains resident after boot.
> > > > > >
> > > > > > Signed-off-by: Hou Zhiqiang 
> > > > > > ---
> > > > > > Tested on LS1043A RDB board
> > > > > >
> > > > >
> > > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > > change with PPA? U-boot has to remain functional in the absence
> > > > > of PPA,
> > > > right?
> > > > >
> > > > > Would it be possible to write a readme for PPA?
> > > >
> > > > Also, how does the devicetree get updated to indicate the presence
> > > > of PSCI?
> > > >
> > >
> > > One needs to add PSCI node in the DTS for the same, claiming support
> > > for PSCI specification.
> > > The cpu.enable_method needs to be changed from spin-table to psci as
> well.
> > >
> > > See
> > > http://lxr.free-electrons.com/source/Documentation/devicetree/bindin
> > > gs
> > > /arm/psci.txt for details.
> > >
> >
> > Bhupesh,
> >
> > Before PPA is fully adopted, u-boot has to detect the existence of PPA
> > and decide to use PSCI or spin table, right? So the device tree node
> > should also be fixed accordingly.

A static CONFIG_ flag is used for enabling the PPA support currently in 
QorIQ-LS u-boot code.

> 
> How would U-Boot know what version of PSCI the loaded PPA supports?  Is
> or could there be any metadata on the PPA image to convey such
> information?

PSCI specifications support the PSCI_VERSION API which is implemented by PPA 
and it returns the PSCI_VERSION
supported by PPA.

Regards,
Bhupesh

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Bhupesh Sharma]

> -Original Message-
> From: Scott Wood [mailto:o...@buserror.net]
> Sent: Saturday, January 23, 2016 1:39 AM
> To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> le...@freescale.com; prabha...@freescale.com;
> bhupesh.sha...@freescale.com; Zhiqiang Hou
> Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> 
> On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > From: Hou Zhiqiang 
> > >
> > > The FSL Primary Protected Application (PPA) is a software component
> > > loaded during boot which runs in TrustZone and remains resident
> > > after boot.
> > >
> > > Signed-off-by: Hou Zhiqiang 
> > > ---
> > > Tested on LS1043A RDB board
> > >
> >
> > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > change with PPA? U-boot has to remain functional in the absence of PPA,
> right?
> >
> > Would it be possible to write a readme for PPA?
> 
> Also, how does the devicetree get updated to indicate the presence of
> PSCI?
> 

One needs to add PSCI node in the DTS for the same, claiming support for PSCI 
specification.
The cpu.enable_method needs to be changed from spin-table to psci as well.

See 
http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/arm/psci.txt
 for details.


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support

[Bhupesh Sharma]

> From: Scott Wood [mailto:o...@buserror.net]
> Sent: Saturday, January 23, 2016 1:48 AM
> On Fri, 2016-01-22 at 20:14 +, Bhupesh Sharma wrote:
> >
> > > -Original Message-
> > > From: Scott Wood [mailto:o...@buserror.net]
> > > Sent: Saturday, January 23, 2016 1:39 AM
> > > To: york sun; Zhiqiang Hou; u-boot@lists.denx.de;
> > > albert.u.b...@aribaud.net; mingkai...@freescale.com; Stuart Yoder;
> > > le...@freescale.com; prabha...@freescale.com;
> > > bhupesh.sha...@freescale.com; Zhiqiang Hou
> > > Subject: Re: [PATCHv2 2/3] ARMv8/layerscape: Add FSL PPA support
> > >
> > > On Fri, 2016-01-22 at 20:05 +, york sun wrote:
> > > > On 01/21/2016 01:53 AM, Zhiqiang Hou wrote:
> > > > > From: Hou Zhiqiang 
> > > > >
> > > > > The FSL Primary Protected Application (PPA) is a software
> > > > > component loaded during boot which runs in TrustZone and remains
> > > > > resident after boot.
> > > > >
> > > > > Signed-off-by: Hou Zhiqiang 
> > > > > ---
> > > > > Tested on LS1043A RDB board
> > > > >
> > > >
> > > > I presume PSCI is implemented in PPA. How does the SMP boot flow
> > > > change with PPA? U-boot has to remain functional in the absence of
> > > > PPA,
> > > right?
> > > >
> > > > Would it be possible to write a readme for PPA?
> > >
> > > Also, how does the devicetree get updated to indicate the presence
> > > of PSCI?
> > >
> >
> > One needs to add PSCI node in the DTS for the same, claiming support
> > for PSCI specification.
> 
> So then we need a different dts based on whether PPA is loaded?  No
> thanks.

For e.g the current LS2080A DTS has no separate node for PSCI, the same can be 
patched-up/added
by the Bootloader.

> 
> > The cpu.enable_method needs to be changed from spin-table to psci as
> well.
> 
> Right, that's part of updating the devicetree to indicate the presence of
> PSCI.

Similarly the cpu.enable-method is also empty in the LS2080A DTS to allow 
Bootloader to patch
it accordingly.
 
Please see: 
http://lxr.free-electrons.com/source/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi#L65
 

> > See
> > http://lxr.free-electrons.com/source/Documentation/devicetree/bindings
> > /a
> > rm/psci.txt for details.
> 
> I'm familiar with the binding.

Regards,
Bhupesh
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot