Re: [PATCH] arm64: dts: sdm845: Fixup OPP table for all qup devices

[Amit Pundir]

On Wed, 12 Aug 2020 at 15:52, Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> wrote:
>
> This OPP table was based on the clock VDD-FMAX tables seen in
> downstream code, however it turns out the downstream clock
> driver does update these tables based on later/production
> rev of the chip and whats seen in the tables belongs to an
> early engineering rev of the SoC.
> Fix up the OPP tables such that it now matches with the
> production rev of sdm845 SoC.
>
> Fixes: 13cadb34e593 ("arm64: dts: sdm845: Add OPP table for all qup
> devices")

Fixes BT on Xiaomi Poco F1 phone running AOSP.

Tested-by: Amit Pundir <amit.pundir@xxxxxxxxxx>

> Reported-by: John Stultz <john.stultz@xxxxxxxxxx>
> Signed-off-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/sdm845.dtsi | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index 2884577..eca81cf 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -1093,8 +1093,8 @@
>                 qup_opp_table: qup-opp-table {
>                         compatible = "operating-points-v2";
>
> -                       opp-19200000 {
> -                               opp-hz = /bits/ 64 <19200000>;
> +                       opp-50000000 {
> +                               opp-hz = /bits/ 64 <50000000>;
>                                 required-opps = <&rpmhpd_opp_min_svs>;
>                         };
>
> @@ -1107,6 +1107,11 @@
>                                 opp-hz = /bits/ 64 <100000000>;
>                                 required-opps = <&rpmhpd_opp_svs>;
>                         };
> +
> +                       opp-128000000 {
> +                               opp-hz = /bits/ 64 <128000000>;
> +                               required-opps = <&rpmhpd_opp_nom>;
> +                       };
>                 };
>
>                 qupv3_id_0: geniqup@8c0000 {
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>