Browse Source

fix(msm8916): drop unneeded initialization of CNTACR

Normal world software is responsible to initialize CNTACR as needed.
There is no existing software for msm8916 that depends on having this
initialization in BL31 so drop it before anything starts to rely on it.

Related issue: https://github.com/ARM-software/tf-issues/issues/170

Change-Id: I9d037ab218c0c1c8a5d5523722013eba531f4728
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
pull/1982/merge
Stephan Gerhold 2 years ago
committed by Manish V Badarkhe
parent
commit
d833af3ab5
  1. 6
      plat/qti/msm8916/msm8916_bl31_setup.c

6
plat/qti/msm8916/msm8916_bl31_setup.c

@ -1,5 +1,5 @@
/*
* Copyright (c) 2021, Stephan Gerhold <stephan@gerhold.net>
* Copyright (c) 2021-2023, Stephan Gerhold <stephan@gerhold.net>
*
* SPDX-License-Identifier: BSD-3-Clause
*/
@ -121,10 +121,6 @@ static void msm8916_configure_timer(void)
/* Make frame 0 available to non-secure world */
mmio_write_32(APCS_QTMR + CNTNSAR, BIT_32(CNTNSAR_NS_SHIFT(0)));
mmio_write_32(APCS_QTMR + CNTACR_BASE(0),
BIT_32(CNTACR_RPCT_SHIFT) | BIT_32(CNTACR_RVCT_SHIFT) |
BIT_32(CNTACR_RFRQ_SHIFT) | BIT_32(CNTACR_RVOFF_SHIFT) |
BIT_32(CNTACR_RWVT_SHIFT) | BIT_32(CNTACR_RWPT_SHIFT));
}
/*

Loading…
Cancel
Save