public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Leif Lindholm <leif.lindholm@linaro.org>
To: Ming Huang <heyi.guo@linaro.org>
Cc: linaro-uefi@lists.linaro.org, edk2-devel@lists.01.org,
	graeme.gregory@linaro.org, ard.biesheuvel@linaro.org,
	guoheyi@huawei.com, wanghuiqiang@huawei.com,
	huangming23@huawei.com, zhangjinsong2@huawei.com,
	mengfanrong@huawei.com, waip23@126.com
Subject: Re: [PATCH edk2-platforms v1 00/14] Improve D0x platforms and bug fix
Date: Tue, 23 Jan 2018 14:24:27 +0000	[thread overview]
Message-ID: <20180123142427.bk7rpushv3x7ltuz@bivouac.eciton.net> (raw)
In-Reply-To: <1516287703-35516-1-git-send-email-huangming23@huawei.com>

I have no further comments on this series beyond the patches I have
already responded to - in general it is in good shape and addresses
most if not all changes we had requested for the 18.02 cycle.

/
    Leif

On Thu, Jan 18, 2018 at 11:01:29PM +0800, Ming Huang wrote:
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> 
> *** BLURB HERE ***
> 
> Jason Zhang (4):
>   Hisilicon/D05: Add PPTT support
>   Hisilicon D03/D05: Add capsule upgrade support
>   Hisilicon D03/D05: Open SasPlatform source code
>   Hisilicon D03/D05: Open SnpPlatform source code
> 
> Ming Huang (9):
>   Hisilicon D03/D05:Switch to Generic BDS driver
>   Hisilicon D03/D05: Optimize the feature of BMC set boot option
>   Hisilicon/Smbios: modify type 4
>   Hisilicon/D05: Replace SP805Watchdog by WatchdogTimer driver.
>   Hisilicon/D03: Replace SP805Watchdog by WatchdogTimer driver.
>   Hisilicon/D05/ACPI: Add ITS PXM
>   Hisilicon/D05/ACPI: Add Pcie, HNS and SAS PXM
>   Hisilicon/Library: Add OsBootLib
>   Hisilicon D03/D05: Update firmware version to 18.02
> 
> Yan Zhang (1):
>   Hisilicon/PCIe: Disable PCIe ASPM
> 
>  Platform/Hisilicon/D03/Capsule/SystemFirmwareUpdateConfig/SystemFirmwareUpdateConfig.ini |  45 ++
>  Platform/Hisilicon/D03/D03.dsc                                                           |  51 +-
>  Platform/Hisilicon/D03/D03.fdf                                                           |  84 ++-
>  Platform/Hisilicon/D05/Capsule/SystemFirmwareUpdateConfig/SystemFirmwareUpdateConfig.ini |  45 ++
>  Platform/Hisilicon/D05/D05.dsc                                                           |  56 +-
>  Platform/Hisilicon/D05/D05.fdf                                                           |  85 ++-
>  Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.c                                      |  89 +++
>  Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.h                                      |  49 ++
>  Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.inf                                    |  61 ++
>  Silicon/Hisilicon/Drivers/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c                |   4 +-
>  Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.c                                      |  99 +++
>  Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.h                                      |  43 ++
>  Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.inf                                    |  60 ++
>  Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc         |  81 +++
>  Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf          |  50 ++
>  Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei.c         |  70 ++
>  Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitLib.c                              |  71 ++
>  Silicon/Hisilicon/Hi1610/Drivers/PcieInit1610/PcieInitLib.h                              |   2 +
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/D05Srat.aslc                                      |  10 +
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/Dsdt/D05Hns.asl                                   |   9 +
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/Dsdt/D05Pci.asl                                   |  34 +-
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/Dsdt/D05Sas.asl                                   |  19 +-
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/Hi1616Platform.h                                  |  27 +
>  Silicon/Hisilicon/Hi1616/D05AcpiTables/MadtHi1616.aslc                                   |  31 +-
>  Silicon/Hisilicon/Hi1616/Pptt/Pptt.c                                                     | 447 +++++++++++++
>  Silicon/Hisilicon/Hi1616/Pptt/Pptt.h                                                     | 142 ++++
>  Silicon/Hisilicon/Hi1616/Pptt/Pptt.inf                                                   |  55 ++
>  Silicon/Hisilicon/HisiPkg.dec                                                            |   3 +
>  Silicon/Hisilicon/Hisilicon.dsc.inc                                                      |  12 +-
>  Silicon/Hisilicon/Hisilicon.fdf.inc                                                      |   9 +
>  Silicon/Hisilicon/Include/Library/AcpiNextLib.h                                          |  10 +-
>  Silicon/Hisilicon/Include/Library/BmcConfigBootLib.h                                     |  31 +
>  Silicon/Hisilicon/Include/Library/OemDevicePath.h                                        |  54 ++
>  Silicon/Hisilicon/Include/Library/OsBootLib.h                                            |  47 ++
>  Silicon/Hisilicon/Include/Protocol/PlatformSasProtocol.h                                 |  11 +
>  Silicon/Hisilicon/Include/Protocol/SnpPlatformProtocol.h                                 |  32 +
>  Silicon/Hisilicon/Include/Regs/HisiPcieV1RegOffset.h                                     |   2 +
>  Silicon/Hisilicon/Library/BmcConfigBootLib/BmcConfigBootLib.c                            | 454 +++++++++++++
>  Silicon/Hisilicon/Library/BmcConfigBootLib/BmcConfigBootLib.inf                          |  51 ++
>  Silicon/Hisilicon/Library/OsBootLib/OsBoot.h                                             | 124 ++++
>  Silicon/Hisilicon/Library/OsBootLib/OsBootLib.c                                          | 217 +++++++
>  Silicon/Hisilicon/Library/OsBootLib/OsBootLib.inf                                        |  59 ++
>  Silicon/Hisilicon/Library/OsBootLib/OsBootLibMisc.c                                      | 514 +++++++++++++++
>  Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBm.c                            | 601 +++++++++++++++++
>  Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBm.h                            |  59 ++
>  Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf              |  91 +++
>  Silicon/Hisilicon/Library/PlatformBootManagerLib/QuietBoot.c                             | 681 ++++++++++++++++++++
>  Silicon/Hisilicon/Library/PlatformFlashAccessLib/PlatformFlashAccessLibDxe.c             | 106 +++
>  Silicon/Hisilicon/Library/PlatformFlashAccessLib/PlatformFlashAccessLibDxe.inf           |  51 ++
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/IntelBdsPlatform.c                         | 434 +------------
>  Silicon/Hisilicon/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf                    |   4 +-
>  51 files changed, 4987 insertions(+), 489 deletions(-)
>  create mode 100644 Platform/Hisilicon/D03/Capsule/SystemFirmwareUpdateConfig/SystemFirmwareUpdateConfig.ini
>  create mode 100644 Platform/Hisilicon/D05/Capsule/SystemFirmwareUpdateConfig/SystemFirmwareUpdateConfig.ini
>  create mode 100644 Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.c
>  create mode 100644 Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.h
>  create mode 100644 Silicon/Hisilicon/Drivers/SasPlatform/SasPlatform.inf
>  create mode 100644 Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.c
>  create mode 100644 Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.h
>  create mode 100644 Silicon/Hisilicon/Drivers/SnpPlatform/SnpPlatform.inf
>  create mode 100644 Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc
>  create mode 100644 Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf
>  create mode 100644 Silicon/Hisilicon/Drivers/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei.c
>  create mode 100644 Silicon/Hisilicon/Hi1616/Pptt/Pptt.c
>  create mode 100644 Silicon/Hisilicon/Hi1616/Pptt/Pptt.h
>  create mode 100644 Silicon/Hisilicon/Hi1616/Pptt/Pptt.inf
>  create mode 100644 Silicon/Hisilicon/Include/Library/BmcConfigBootLib.h
>  create mode 100644 Silicon/Hisilicon/Include/Library/OemDevicePath.h
>  create mode 100644 Silicon/Hisilicon/Include/Library/OsBootLib.h
>  create mode 100644 Silicon/Hisilicon/Include/Protocol/SnpPlatformProtocol.h
>  create mode 100644 Silicon/Hisilicon/Library/BmcConfigBootLib/BmcConfigBootLib.c
>  create mode 100644 Silicon/Hisilicon/Library/BmcConfigBootLib/BmcConfigBootLib.inf
>  create mode 100644 Silicon/Hisilicon/Library/OsBootLib/OsBoot.h
>  create mode 100644 Silicon/Hisilicon/Library/OsBootLib/OsBootLib.c
>  create mode 100644 Silicon/Hisilicon/Library/OsBootLib/OsBootLib.inf
>  create mode 100644 Silicon/Hisilicon/Library/OsBootLib/OsBootLibMisc.c
>  create mode 100644 Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBm.c
>  create mode 100644 Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBm.h
>  create mode 100644 Silicon/Hisilicon/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
>  create mode 100644 Silicon/Hisilicon/Library/PlatformBootManagerLib/QuietBoot.c
>  create mode 100644 Silicon/Hisilicon/Library/PlatformFlashAccessLib/PlatformFlashAccessLibDxe.c
>  create mode 100644 Silicon/Hisilicon/Library/PlatformFlashAccessLib/PlatformFlashAccessLibDxe.inf
> 
> -- 
> 1.9.1
> 


      parent reply	other threads:[~2018-01-23 14:19 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-18 15:01 [PATCH edk2-platforms v1 00/14] Improve D0x platforms and bug fix Ming Huang
2018-01-18 15:01 ` [PATCH edk2-platforms v1 01/14] Hisilicon/D05: Add PPTT support Ming Huang
2018-01-20 10:16   ` Ard Biesheuvel
2018-01-22  9:16     ` Huangming (Mark)
2018-01-23  6:00     ` Huangming (Mark)
2018-01-22 13:53   ` Leif Lindholm
2018-01-22 14:15     ` Leif Lindholm
2018-01-24 13:49     ` graeme.gregory
2018-01-23 21:29   ` Jeremy Linton
2018-01-24  7:57     ` Huangming (Mark)
2018-01-25  5:56     ` Huangming (Mark)
2018-01-25 15:27       ` Jeremy Linton
2018-01-18 15:01 ` [PATCH edk2-platforms v1 02/14] Hisilicon D03/D05:Switch to Generic BDS driver Ming Huang
2018-01-20 10:27   ` Ard Biesheuvel
2018-01-22 18:38   ` Leif Lindholm
2018-01-23  6:03     ` Huangming (Mark)
2018-01-18 15:01 ` [PATCH edk2-platforms v1 03/14] Hisilicon D03/D05: Optimize the feature of BMC set boot option Ming Huang
2018-01-20 10:41   ` Ard Biesheuvel
2018-01-23  8:28     ` Huangming (Mark)
2018-01-23 10:28   ` Leif Lindholm
2018-01-23 10:51     ` Huangming (Mark)
2018-01-18 15:01 ` [PATCH edk2-platforms v1 04/14] Hisilicon D03/D05: Add capsule upgrade support Ming Huang
2018-01-20 10:50   ` Ard Biesheuvel
2018-01-23  8:53     ` Huangming (Mark)
2018-01-23  9:33       ` Ard Biesheuvel
2018-01-24 11:10     ` Huangming (Mark)
2018-01-24 11:21       ` Ard Biesheuvel
2018-01-25  0:53         ` Huangming (Mark)
2018-01-23 14:06   ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 05/14] Hisilicon D03/D05: Open SasPlatform source code Ming Huang
2018-01-20 10:57   ` Ard Biesheuvel
2018-01-23 11:01     ` Huangming (Mark)
2018-01-23 14:04   ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 06/14] Hisilicon D03/D05: Open SnpPlatform " Ming Huang
2018-01-20 11:00   ` Ard Biesheuvel
2018-01-23 11:01     ` Huangming (Mark)
2018-01-23 14:07   ` Leif Lindholm
2018-01-24 12:31     ` Huangming (Mark)
2018-01-24 13:47       ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 07/14] Hisilicon/Smbios: modify type 4 Ming Huang
2018-01-20 11:01   ` Ard Biesheuvel
2018-01-23 14:15   ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 08/14] Hisilicon/PCIe: Disable PCIe ASPM Ming Huang
2018-01-20 11:04   ` Ard Biesheuvel
2018-01-18 15:01 ` [PATCH edk2-platforms v1 09/14] Hisilicon/D05: Replace SP805Watchdog by WatchdogTimer driver Ming Huang
2018-01-20 11:05   ` Ard Biesheuvel
2018-01-23 14:21   ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 10/14] Hisilicon/D03: " Ming Huang
2018-01-20 11:05   ` Ard Biesheuvel
2018-01-23 14:21   ` Leif Lindholm
2018-01-18 15:01 ` [PATCH edk2-platforms v1 11/14] Hisilicon/D05/ACPI: Add ITS PXM Ming Huang
2018-01-20 11:06   ` Ard Biesheuvel
2018-01-18 15:01 ` [PATCH edk2-platforms v1 12/14] Hisilicon/D05/ACPI: Add Pcie, HNS and SAS PXM Ming Huang
2018-01-20 11:08   ` Ard Biesheuvel
2018-01-18 15:01 ` [PATCH edk2-platforms v1 13/14] Hisilicon/Library: Add OsBootLib Ming Huang
2018-01-20 11:11   ` Ard Biesheuvel
2018-01-23 10:23   ` Leif Lindholm
2018-01-27  1:47     ` Huangming (Mark)
2018-01-27 10:37       ` Ard Biesheuvel
2018-01-29  8:55         ` Huangming (Mark)
2018-01-29 10:19           ` Ard Biesheuvel
2018-01-29 11:16       ` Leif Lindholm
2018-02-07 21:16         ` Peter Jones
2018-02-11  6:03           ` Huangming (Mark)
2018-02-26  1:12           ` Guo Heyi
2018-01-18 15:01 ` [PATCH edk2-platforms v1 14/14] Hisilicon D03/D05: Update firmware version to 18.02 Ming Huang
2018-01-20 11:11   ` Ard Biesheuvel
2018-01-23 10:18   ` Leif Lindholm
2018-01-24  1:17     ` Huangming (Mark)
2018-01-24  7:54       ` Leif Lindholm
2018-01-22 13:26 ` [PATCH edk2-platforms v1 00/14] Improve D0x platforms and bug fix Leif Lindholm
2018-01-23 14:24 ` Leif Lindholm [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180123142427.bk7rpushv3x7ltuz@bivouac.eciton.net \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox