Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/59070 )
Change subject: soc/amd/cezanne/fsp_m_parameters: add curly braces around else block ......................................................................
soc/amd/cezanne/fsp_m_parameters: add curly braces around else block
Since the if block contains multiple statements, it uses curly braces around them, so also add curly braces around the else block even though it only contains one statement.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: Ia8d6b45ec16916ff77078446414de259cffa1475 Reviewed-on: https://review.coreboot.org/c/coreboot/+/59070 Reviewed-by: Raul Rangel rrangel@chromium.org Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/amd/cezanne/fsp_m_params.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Raul Rangel: Looks good to me, approved
diff --git a/src/soc/amd/cezanne/fsp_m_params.c b/src/soc/amd/cezanne/fsp_m_params.c index 048b428..51e04e1 100644 --- a/src/soc/amd/cezanne/fsp_m_params.c +++ b/src/soc/amd/cezanne/fsp_m_params.c @@ -153,9 +153,9 @@ mcfg->usb_phy->Version_Major = 0xd; mcfg->usb_phy->Version_Minor = 0x6; mcfg->usb_phy->TableLength = 100; - } - else + } else { mcfg->usb_phy = NULL; + }
fsp_fill_pcie_ddi_descriptors(mcfg); fsp_assign_ioapic_upds(mcfg);