Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50932 )
Change subject: soc/intel/{cnl,icl}: Use matching type cast ......................................................................
soc/intel/{cnl,icl}: Use matching type cast
Change-Id: Ie534a05f8d3945492ab5b817522486cdcd3c4cab Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/50932 Reviewed-by: Michael Niewöhner foss@mniewoehner.de Reviewed-by: Arthur Heymans arthur@aheymans.xyz Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/soc/intel/cannonlake/pmutil.c M src/soc/intel/icelake/pmutil.c 2 files changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved Michael Niewöhner: Looks good to me, approved
diff --git a/src/soc/intel/cannonlake/pmutil.c b/src/soc/intel/cannonlake/pmutil.c index bf4e34f..d6ecd8b 100644 --- a/src/soc/intel/cannonlake/pmutil.c +++ b/src/soc/intel/cannonlake/pmutil.c @@ -117,7 +117,7 @@ /* Set the DISB after DRAM init */ uint8_t disb_val; /* Only care about bits [23:16] of register GEN_PMCON_A */ - uint8_t *addr = (void *)(pmc_mmio_regs() + GEN_PMCON_A + 2); + uint8_t *addr = (uint8_t *)(pmc_mmio_regs() + GEN_PMCON_A + 2);
disb_val = read8(addr); disb_val |= (DISB >> 16); diff --git a/src/soc/intel/icelake/pmutil.c b/src/soc/intel/icelake/pmutil.c index 9cc6f02..894894f 100644 --- a/src/soc/intel/icelake/pmutil.c +++ b/src/soc/intel/icelake/pmutil.c @@ -116,7 +116,7 @@ /* Set the DISB after DRAM init */ uint8_t disb_val; /* Only care about bits [23:16] of register GEN_PMCON_A */ - uint8_t *addr = (void *)(pmc_mmio_regs() + GEN_PMCON_A + 2); + uint8_t *addr = (uint8_t *)(pmc_mmio_regs() + GEN_PMCON_A + 2);
disb_val = read8(addr); disb_val |= (DISB >> 16);