Re: [PATCH v1 0/6] Revert DynamicTablesPkg: Framework updates and fixes

[Laszlo Ersek <lersek@redhat.com>]

On 02/26/19 12:01, Laszlo Ersek wrote:
> On 02/26/19 09:44, Sami Mujawar wrote:
>> Reverting this patch series as Soft Feature Freeze for
>> edk2-stable201903 started on 22 Feb 2019.
>>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Cc: Alexei Fedorov <alexei.fedorov@arm.com>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
>>
>> The changes can be seen at https://github.com/samimujawar/edk2/tree/Revert_473_dynamic_tables_framework_v1
>>
>> Sami Mujawar (6):
>>   Revert "DynamicTablesPkg: Minor updates and fix typos"
>>   Revert "DynamicTablesPkg: Remove GIC Distributor Id field"
>>   Revert "DynamicTablesPkg: DGB2: Update DBG2_DEBUG_PORT_DDI"
>>   Revert "DynamicTablesPkg: Add OEM Info"
>>   Revert "DynamicTablesPkg: Rename enum used for ID Mapping"
>>   Revert "DynamicTablesPkg: Fix protocol section"
>>
>>  DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.inf |  7 +-
>>  DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.inf |  7 +-
>>  DynamicTablesPkg/Include/ArmNameSpaceObjects.h                             | 73 +++++---------------
>>  DynamicTablesPkg/Include/Library/TableHelperLib.h                          |  4 +-
>>  DynamicTablesPkg/Include/StandardNameSpaceObjects.h                        | 18 -----
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c           |  7 +-
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c           |  2 +-
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c           |  2 +-
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c           |  8 +--
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c           |  6 +-
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c           |  2 +-
>>  DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c           |  2 +-
>>  DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c               | 26 ++-----
>>  13 files changed, 48 insertions(+), 116 deletions(-)
>>
> 
> Thank you. Sorry about the inconvenience.
> 
> Acked-by: Laszlo Ersek <lersek@redhat.com>
> 
> Laszlo
> 

Note: if you have small individual patches that cleanly qualify as
bugfixes, especially for features introduced during this development
cycle (since the last table tag), those should be acceptable even during
the hard feature freeze.

https://github.com/tianocore/tianocore.github.io/wiki/HardFeatureFreeze

So if you have fixes like that (possibly a subset of the present patch
set), I certainly encourage you to repost those. I'm not familiar with
DynamicTablesPkg, and so I can't evaluate this question myself, on a
patch-by-patch basis. It's also possible that you'll have to split out
parts of larger patches (refactor them) so that only the strict-sense
fixes can be posted and applied.

Thanks!
Laszlo