[PATCH edk2-non-osi v2 0/4] Upload D06 binary modules for SBSA test

[PATCH edk2-non-osi v2 0/4] Upload D06 binary modules for SBSA test

[Ming Huang]

Main Change since v1:
1 Modify comments;

Code can also be found in github:
https://github.com/hisilicon/OpenPlatformPkg.git
branch: d06-acs-non-osi-v2


Ming Huang (4):
  Hisilicon/D06: Add cpu on/off feature in TrustedFirmware
  Hisilicon/D06: Fix SBSA test case 42 failed issues
  Hisilicon/D06: Fix set usb reg failed issue
  Hisilicon/D06: Fix SBSA PE-15 failed issue

 Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229216 -> 232832 bytes
 Platform/Hisilicon/D06/bl1.bin                         | Bin 12432 -> 12432 bytes
 Platform/Hisilicon/D06/fip.bin                         | Bin 113578 -> 113450 bytes
 3 files changed, 0 insertions(+), 0 deletions(-)

-- 
2.9.5

[PATCH edk2-non-osi v2 1/4] Hisilicon/D06: Add cpu on/off feature in TrustedFirmware

[Ming Huang]

Add cpu on/off feature to support SBSA-PE test. This patch
also fix bug 3996:
https://bugs.linaro.org/show_bug.cgi?id=3996

Build commit information:
TrustedFirmware:5888a78d43c

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ming Huang <ming.huang@linaro.org>
---
 Platform/Hisilicon/D06/bl1.bin | Bin 12432 -> 12432 bytes
 Platform/Hisilicon/D06/fip.bin | Bin 113578 -> 113450 bytes
 2 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/Platform/Hisilicon/D06/bl1.bin b/Platform/Hisilicon/D06/bl1.bin
index d291359..416535f 100644
Binary files a/Platform/Hisilicon/D06/bl1.bin and b/Platform/Hisilicon/D06/bl1.bin differ
diff --git a/Platform/Hisilicon/D06/fip.bin b/Platform/Hisilicon/D06/fip.bin
index a72bef8..c9b7ca0 100644
Binary files a/Platform/Hisilicon/D06/fip.bin and b/Platform/Hisilicon/D06/fip.bin differ
-- 
2.9.5

[PATCH edk2-non-osi v2 2/4] Hisilicon/D06: Fix SBSA test case 42 failed issues

[Ming Huang]

As SBSA uefi tool can't configuare interrupt following
WatchdogTimerFlags in GTDT, and watchdog interrupt in Hi1620
is edge-trigger, so modify watchdog interrupt type for SBSA
test case 42.

Build commit informations:
edk2:53caffc33b6
edk2-platforms:d4d7e39886a
HwPkg:bf0bdef14d5
TrustedFirmware:5888a78d43c

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ming Huang <ming.huang@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229216 -> 229248 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi
index afd3ebe..c9172ff 100644
Binary files a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi and b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi differ
-- 
2.9.5

[PATCH edk2-non-osi v2 3/4] Hisilicon/D06: Fix set usb reg failed issue

[Ming Huang]

The default link timeout value of USB 3.0 controller is a bit
short for some USB devices, and may cause it timeout in some
cases. We have modify the registers in IoInitDxe,but a bug let
the modifying not successful.

Build commit informations:
edk2:53caffc33b6
edk2-platforms:d4d7e39886a
HwPkg:2a7ee82855a
TrustedFirmware:5888a78d43c

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ming Huang <ming.huang@linaro.org>
---
 Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229248 -> 230784 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi
index c9172ff..8b6d740 100644
Binary files a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi and b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi differ
-- 
2.9.5

[PATCH edk2-non-osi v2 4/4] Hisilicon/D06: Fix SBSA PE-15 failed issue

[Ming Huang]

PE test case 15 flow:
Primary core(cacheable shareable) and slave cores(non-cacheable)
access the same memory area for communication.
For each slave core{
  1 Turn on slave core;
  2 run the payload function;
  3 Write result in memory to notify primary core and follow
                                clean and invalidate instruction;
  4 Slave core turn off itself;
}
The result in DDR may rewrite by cache data. The essence of
this problem is that primary core and slave core access the
same area with different cache attribute.
Configure L3T register to fix this issue;

Build commit informations:
edk2:53caffc33b6
edk2-platforms:d4d7e39886a
HwPkg:6e91ea20fda
TrustedFirmware:5888a78d43c

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ming Huang <ming.huang@linaro.org>
---
 Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 230784 -> 232832 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)

diff --git a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi
index 8b6d740..e32c056 100644
Binary files a/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi and b/Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi differ
-- 
2.9.5

Re: [PATCH edk2-non-osi v2 0/4] Upload D06 binary modules for SBSA test

[Leif Lindholm]

On Fri, Nov 16, 2018 at 04:13:02PM +0800, Ming Huang wrote:
> Main Change since v1:
> 1 Modify comments;
> 
> Code can also be found in github:
> https://github.com/hisilicon/OpenPlatformPkg.git
> branch: d06-acs-non-osi-v2

>From the black-box standpoint, this series look fine.
Are there any implications of the TB->TA boot that means I need to
synchronise pushing these with pushing that change for edk2-platforms?

If not, for the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
and I'll push when I have confirmation.
 
> Ming Huang (4):
>   Hisilicon/D06: Add cpu on/off feature in TrustedFirmware
>   Hisilicon/D06: Fix SBSA test case 42 failed issues
>   Hisilicon/D06: Fix set usb reg failed issue
>   Hisilicon/D06: Fix SBSA PE-15 failed issue
> 
>  Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229216 -> 232832 bytes
>  Platform/Hisilicon/D06/bl1.bin                         | Bin 12432 -> 12432 bytes
>  Platform/Hisilicon/D06/fip.bin                         | Bin 113578 -> 113450 bytes
>  3 files changed, 0 insertions(+), 0 deletions(-)
> 
> -- 
> 2.9.5
> 

Re: [PATCH edk2-non-osi v2 0/4] Upload D06 binary modules for SBSA test

[Ming Huang]

On 11/20/2018 2:46 AM, Leif Lindholm wrote:
> On Fri, Nov 16, 2018 at 04:13:02PM +0800, Ming Huang wrote:
>> Main Change since v1:
>> 1 Modify comments;
>>
>> Code can also be found in github:
>> https://github.com/hisilicon/OpenPlatformPkg.git
>> branch: d06-acs-non-osi-v2
> 
> From the black-box standpoint, this series look fine.
> Are there any implications of the TB->TA boot that means I need to
> synchronise pushing these with pushing that change for edk2-platforms?

This four binary patches is irrelevant about TB->TA boot. D06 has been
TA boot from last series (August).
Thanks.

> 
> If not, for the series:
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> and I'll push when I have confirmation.
>  
>> Ming Huang (4):
>>   Hisilicon/D06: Add cpu on/off feature in TrustedFirmware
>>   Hisilicon/D06: Fix SBSA test case 42 failed issues
>>   Hisilicon/D06: Fix set usb reg failed issue
>>   Hisilicon/D06: Fix SBSA PE-15 failed issue
>>
>>  Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229216 -> 232832 bytes
>>  Platform/Hisilicon/D06/bl1.bin                         | Bin 12432 -> 12432 bytes
>>  Platform/Hisilicon/D06/fip.bin                         | Bin 113578 -> 113450 bytes
>>  3 files changed, 0 insertions(+), 0 deletions(-)
>>
>> -- 
>> 2.9.5
>>

Re: [PATCH edk2-non-osi v2 0/4] Upload D06 binary modules for SBSA test

[Leif Lindholm]

On Tue, Nov 20, 2018 at 09:34:51AM +0800, Ming Huang wrote:
> 
> 
> On 11/20/2018 2:46 AM, Leif Lindholm wrote:
> > On Fri, Nov 16, 2018 at 04:13:02PM +0800, Ming Huang wrote:
> >> Main Change since v1:
> >> 1 Modify comments;
> >>
> >> Code can also be found in github:
> >> https://github.com/hisilicon/OpenPlatformPkg.git
> >> branch: d06-acs-non-osi-v2
> > 
> > From the black-box standpoint, this series look fine.
> > Are there any implications of the TB->TA boot that means I need to
> > synchronise pushing these with pushing that change for edk2-platforms?
> 
> This four binary patches is irrelevant about TB->TA boot. D06 has been
> TA boot from last series (August).

Gotcha.

For the series:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Pushed as 572f1053..047bb77d.

> 
> > 
> > If not, for the series:
> > Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> > and I'll push when I have confirmation.
> >  
> >> Ming Huang (4):
> >>   Hisilicon/D06: Add cpu on/off feature in TrustedFirmware
> >>   Hisilicon/D06: Fix SBSA test case 42 failed issues
> >>   Hisilicon/D06: Fix set usb reg failed issue
> >>   Hisilicon/D06: Fix SBSA PE-15 failed issue
> >>
> >>  Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.efi | Bin 229216 -> 232832 bytes
> >>  Platform/Hisilicon/D06/bl1.bin                         | Bin 12432 -> 12432 bytes
> >>  Platform/Hisilicon/D06/fip.bin                         | Bin 113578 -> 113450 bytes
> >>  3 files changed, 0 insertions(+), 0 deletions(-)
> >>
> >> -- 
> >> 2.9.5
> >>