diff options
| author | Stephan Gerhold <stephan@gerhold.net> | 2023-03-22 18:15:15 +0100 |
|---|---|---|
| committer | Manish V Badarkhe <manish.badarkhe@arm.com> | 2023-05-09 21:44:51 +0200 |
| commit | d833af3ab50cd2cfecb8868c3d5340df1572f042 (patch) | |
| tree | 3ee48d5984896cbb17463428a30d0a131167f8bf | |
| parent | 4a3e2cb355dfa3e243d2546b96c7e0d112030bc8 (diff) | |
| download | arm-trusted-firmware-d833af3ab50cd2cfecb8868c3d5340df1572f042.tar.gz | |
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>
| -rw-r--r-- | plat/qti/msm8916/msm8916_bl31_setup.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/plat/qti/msm8916/msm8916_bl31_setup.c b/plat/qti/msm8916/msm8916_bl31_setup.c index 638cd09d0..d24fb349b 100644 --- a/plat/qti/msm8916/msm8916_bl31_setup.c +++ b/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)); } /* |