Reviewed-by: Ruiyu Ni <ruiyu...@intel.com> > -----Original Message----- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Friday, July 1, 2016 9:11 AM > To: edk2-devel-01 <edk2-de...@ml01.01.org> > Cc: Johnson, Brian J. <bjohn...@sgi.com>; Alex Williamson > <alex.william...@redhat.com>; Andrew Fish <af...@apple.com>; Tian, Feng > <feng.t...@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com>; Gao, > Liming <liming....@intel.com>; Marcel Apfelbaum <mar...@redhat.com>; > Kinney, Michael D <michael.d.kin...@intel.com>; Ni, Ruiyu > <ruiyu...@intel.com>; Zeng, Star <star.z...@intel.com> > Subject: [PATCH 1/4] MdePkg/IndustryStandard: introduce > EFI_PCI_CAPABILITY_ID_SHPC > > The "Pci22.h" header file defines the macro > EFI_PCI_CAPABILITY_ID_HOTPLUG with value 0x06. According to all of: > - later parts of the same header file, > - Appendix H ("Capability IDs") of the PCI Local Bus Specification > Revision 2.3, > - and Chapter 2 ("Capability IDs") of the PCI Code and ID Assignment > Specification Revision 0.9, > > 0x06 means "CompactPCI Hot Swap". It does not mean "PCI Hot-Plug": that > capability is described by ID 0x0C: > > 0Ch PCI Hot-Plug -- This Capability ID indicates that the associated > device conforms to the Standard Hot-Plug Controller model. > > Therefore EFI_PCI_CAPABILITY_ID_HOTPLUG is arguably a misnomer. > PciBusDxe (mis-)uses EFI_PCI_CAPABILITY_ID_HOTPLUG in the IsSHPC() > helper function to identify PCI Hot-Plug capability. > > In order to preserve compatibility with existent code, leave > EFI_PCI_CAPABILITY_ID_HOTPLUG alone, and introduce > EFI_PCI_CAPABILITY_ID_SHPC with the right ID value. > > Cc: "Johnson, Brian J." <bjohn...@sgi.com> > Cc: Alex Williamson <alex.william...@redhat.com> > Cc: Andrew Fish <af...@apple.com> > Cc: Feng Tian <feng.t...@intel.com> > Cc: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Liming Gao <liming....@intel.com> > Cc: Marcel Apfelbaum <mar...@redhat.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Ruiyu Ni <ruiyu...@intel.com> > Cc: Star Zeng <star.z...@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Laszlo Ersek <ler...@redhat.com> > --- > MdePkg/Include/IndustryStandard/Pci22.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MdePkg/Include/IndustryStandard/Pci22.h > b/MdePkg/Include/IndustryStandard/Pci22.h > index db24a153e15e..4cf8389c699b 100644 > --- a/MdePkg/Include/IndustryStandard/Pci22.h > +++ b/MdePkg/Include/IndustryStandard/Pci22.h > @@ -635,6 +635,7 @@ typedef union { > #define EFI_PCI_CAPABILITY_ID_SLOTID 0x04 > #define EFI_PCI_CAPABILITY_ID_MSI 0x05 > #define EFI_PCI_CAPABILITY_ID_HOTPLUG 0x06 > +#define EFI_PCI_CAPABILITY_ID_SHPC 0x0C > > /// > /// Capabilities List Header > -- > 1.8.3.1 >
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel