[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/xeon_sp/chip_common.c: check SOC_INTEL_PCIE_64BIT_ALLOC

[Elyes Haouas (Code Review)]

Attention is currently required from: Jonathan Zhang, Johnny Lin, David Hendricks, Christian Walter, Arthur Heymans, Tim Chu.

Elyes Haouas has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72441 )

Change subject: soc/intel/xeon_sp/chip_common.c: check SOC_INTEL_PCIE_64BIT_ALLOC
......................................................................

Patch Set 6:

(2 comments)

Patchset:

PS6: 
Maybe:
static uint8_t is_pci64bit_alloc(void)
{
/*
 * For SPR-SP FSP which supports SOC_INTEL_PCIE_64BITS_ALLOC,
 * Pci64BitResourceAllocation field does not exist in IIO_UDS HOB.
 */
    if CONFIG(SOC_INTEL_PCIE_64BIT_ALLOC)
    	return 1;

const IIO_UDS *hob = get_iio_uds();
    return hob->PlatformData.Pci64BitResourceAllocation;
}

File src/soc/intel/xeon_sp/chip_common.c:

https://review.coreboot.org/c/coreboot/+/72441/comment/de9f735f_be4fc805 
PS6, Line 410: #if
why "#if"?
"else" is not needed after "return".