| Message ID | 20181115023353.20159-5-ard.biesheuvel@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | IntelUndiPkg/XGigUndiDxe: fix GCC / ARM build issues | expand |
Reviewed-by: Ryszard Knop <ryszard.knop@linux.intel.com> On Wed, 2018-11-14 at 18:33 -0800, ard.biesheuvela wrote: > Move the extern declaration of mBrandingTable[] after the definition > of > the type. This solves a build issue with GCC. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel <ard.biesheuvel at linaro.org> > --- > IntelUndiPkg/XGigUndiDxe/DeviceSupport.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h > b/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h > index e997983ba3e7..d501d9bb1371 100644 > --- a/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h > +++ b/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h > @@ -33,9 +33,6 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE > POSSIBILITY OF SUCH DAMAGE. > > typedef struct BRAND_STRUCT_S BRAND_STRUCT; > > -extern BRAND_STRUCT mBrandingTable[]; > -extern UINTN mBrandingTableSize; > - > /* Defines */ > #define INVALID_VENDOR_ID 0xFFFF > #define INVALID_SUBVENDOR_ID 0xFFFF > @@ -53,6 +50,9 @@ struct BRAND_STRUCT_S { > CHAR16 *BrandString; > }; > > +extern BRAND_STRUCT mBrandingTable[]; > +extern UINTN mBrandingTableSize; > + > /* Function declarations */ > > /** Returns pointer to current device's branding string (looks for > best match) _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
diff --git a/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h b/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h index e997983ba3e7..d501d9bb1371 100644 --- a/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h +++ b/IntelUndiPkg/XGigUndiDxe/DeviceSupport.h @@ -33,9 +33,6 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. typedef struct BRAND_STRUCT_S BRAND_STRUCT; -extern BRAND_STRUCT mBrandingTable[]; -extern UINTN mBrandingTableSize; - /* Defines */ #define INVALID_VENDOR_ID 0xFFFF #define INVALID_SUBVENDOR_ID 0xFFFF @@ -53,6 +50,9 @@ struct BRAND_STRUCT_S { CHAR16 *BrandString; }; +extern BRAND_STRUCT mBrandingTable[]; +extern UINTN mBrandingTableSize; + /* Function declarations */ /** Returns pointer to current device's branding string (looks for best match)
Move the extern declaration of mBrandingTable[] after the definition of the type. This solves a build issue with GCC. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> --- IntelUndiPkg/XGigUndiDxe/DeviceSupport.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) -- 2.17.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel