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(-)
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
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
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
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
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
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
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
-
Leif Lindholm -
Ming Huang