GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/qualcomm/ipq806x: Remove unnecessary allocation 

The bus variable doesn't live outside the scope of this function, and is
only used as a convenient way for passing the pointers to all the
sub-functions, so it doesn't need to be allocated. Put it on the stack
instead.

Change-Id: I4370d77445952731d20f7d9a91803612f4d21aef
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: Coverity CID 1294801
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34499
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Jacob Garber 2019-07-22 16:20:36 -06:00 committed by Patrick Georgi
parent 877481cda2
commit 0f33d8c29a
1 changed files with 11 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/soc/qualcomm/ipq806x/lcc.c
View File

@ -287,29 +287,21 @@ static int lcc_enable_mi2s(Ipq806xLccClocks *bus)
int audio_clock_config(unsigned frequency)
{
Ipq806xLccClocks *bus = malloc(sizeof(*bus));
Ipq806xLccClocks bus = {
.gcc_apcs_regs = (void *)(MSM_GCC_BASE + GCC_PLL_APCS_REG),
.lcc_pll0_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_PLL0_MODE_REG),
.lcc_ahbix_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_AHBIX_NS_REG),
.lcc_mi2s_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_MI2S_NS_REG),
.lcc_pll_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_PLL_PCLK_REG),
};
if (!bus) {
printk(BIOS_ERR, "%s: failed to allocate bus structure\n",
__func__);
if (lcc_init_enable_pll0(&bus))
return 1;
}
bus->gcc_apcs_regs = (void *)(MSM_GCC_BASE + GCC_PLL_APCS_REG);
bus->lcc_pll0_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_PLL0_MODE_REG);
bus->lcc_ahbix_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_AHBIX_NS_REG);
bus->lcc_mi2s_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_MI2S_NS_REG);
bus->lcc_pll_regs = (void *)(MSM_LPASS_LCC_BASE + LCC_PLL_PCLK_REG);
if (lcc_init_enable_pll0(bus))
if (lcc_init_enable_ahbix(&bus))
return 1;
if (lcc_init_enable_ahbix(bus))
if (lcc_init_mi2s(&bus, frequency))
return 1;
if (lcc_init_mi2s(bus, frequency))
return 1;
if (lcc_enable_mi2s(bus))
if (lcc_enable_mi2s(&bus))
return 1;
return 0;