| Message ID | 20180531104634.23215-1-ard.biesheuvel@linaro.org |
|---|---|
| Headers | show |
| Series | enable FPDT table for SynQuacer | expand |
On Thu, May 31, 2018 at 12:46:32PM +0200, Ard Biesheuvel wrote: > Enable some of the status code reporting infrastructure in patch #1 so > we can include the FPDT DXE driver in patch #2 which produces the FPDT > diagnostic table which tells us how much time was spent in the firmware, > in the OS loader, and in ExitBootServices(). > > Ard Biesheuvel (2): > Platform/DeveloperBox: wire up status code reporting for DXE phase > Platform/DeveloperBox: enable FPDT table For the series: Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> > Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 12 ++++++++++++ > Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 2 ++ > Silicon/Socionext/SynQuacer/Acpi.dsc.inc | 6 ++++++ > 3 files changed, 20 insertions(+) > > -- > 2.17.0 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
On Thu, May 31, 2018 at 10:16:08PM +0100, Leif Lindholm wrote: > On Thu, May 31, 2018 at 12:46:32PM +0200, Ard Biesheuvel wrote: > > Enable some of the status code reporting infrastructure in patch #1 so > > we can include the FPDT DXE driver in patch #2 which produces the FPDT > > diagnostic table which tells us how much time was spent in the firmware, > > in the OS loader, and in ExitBootServices(). > > > > Ard Biesheuvel (2): > > Platform/DeveloperBox: wire up status code reporting for DXE phase > > Platform/DeveloperBox: enable FPDT table > > For the series: > Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org> Hmm, after a side discussion with Ard, I have asked him _not_ to push this set for now. The reason for this is that its use under Linux is dependent on the fundamentally broken concept of directly mapping physical address regions into non-privileged processes using /dev/mem. I want to get a sane kernel interface in place for accessing this data before we start encouraging its use. If this holds anyone back for other operating systems, please speak up (off-list if needs be). The Reviewed-by still applies, and the patches are ready to go. / Leif > > Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 12 ++++++++++++ > > Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 2 ++ > > Silicon/Socionext/SynQuacer/Acpi.dsc.inc | 6 ++++++ > > 3 files changed, 20 insertions(+) > > > > -- > > 2.17.0 > > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel