Skip to content
  • Ansuel Smith's avatar
    ipq806x: fix bug in L2 cache scaling · bc0ca20c
    Ansuel Smith authored
    It has been notice a buf in L2 cache scaling where the scaling is not
    done proprely if the frequency is set to the initial state before
    the new frequency.
    
    From: https://patchwork.kernel.org/patch/10565443/
    
    * The clocks are set to aux clock rate first to make sure the
    * secondary mux is not sourcing off of QSB. The rate is then set to
    * two different rates to force a HFPLL reinit under all
    * circumstances.
    
    In the initial stage of boot to force a new frequency to apply, is
    needed to first set the frequency back to the lowest one (aux_rate)
    and then to the target one. This force and make sure the controller
    actually switch the frequency to the right one. Apply the same
    mechanism to L2 frequency scaling. Before scaling to the target
    frequency, first set the frequency to the aux_rate to force the
    transition, then scale it to the target frequency. Doing the wrong way
    can produce unexpected results and could lock the scaling mechanism
    until a full reboot is done (Causing a full reset by the krait-cc driver)
    
    From: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=77612720a2362230af726baa4149c40ec7a7fb05
    
    
    
    When the Hfplls are reprogrammed during the rate change,
    the primary muxes which are sourced from the same hfpll
    for higher frequencies, needs to be switched to the 'safe
    secondary mux' as the parent for that small window. This
    is done by registering a clk notifier for the muxes and
    switching to the safe parent in the PRE_RATE_CHANGE notifier
    and back to the original parent in the POST_RATE_CHANGE notifier.
    
    This should apply also to L2 scaling... as we can't relly use
    the notifier, we manually do this on L2 scaling.
    
    Tested-By: Marc Benoit <marcb62185@gmail.com> [19.07: R7800]
    Tested-by: Stefan Lippers-Hollmann <s.l-h@gmx.de> [nbg6817/ipq8065]
    Signed-off-by: default avatarAnsuel Smith <ansuelsmth@gmail.com>
    (cherry picked from commit 5ab9c0b3)
    bc0ca20c