From patchwork Tue May 16 00:19:29 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 94409 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp82120vqo; Mon, 15 May 2023 17:54:06 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6O/WL2tYabCd3ulVEcsBuOEGdufDePOo9DJpvNmoEriBvbFNxGGhNuh/svQIbWblpZdmW3 X-Received: by 2002:a05:6a20:3956:b0:106:30c5:3d02 with SMTP id r22-20020a056a20395600b0010630c53d02mr6254869pzg.40.1684198445929; Mon, 15 May 2023 17:54:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684198445; cv=none; d=google.com; s=arc-20160816; b=aKmM72S8JzDEwo5YURdJK570sZ3rwwvfq/gDVb5ZHRu4QcbNUaHqDIWRTa5rV24FmF maEvi1YRv/2+dUSGPIjJAkAYDzHzoegtE4ej26040Kj4wMuWlI9Zox+9V8Z18SJ5RsBo sb9s6Ya2eWjZAZDJvEx+5XHAks8VNsoZZWCgpC6P+MTotoun9L3oRzI4h6J+KAO+o34S A6oGZ24yeey5oqga29hCbPhKgoEnaJhVYl0sbvXummovdFsg6md9i68ivZkqusbm2yVO x0rBjxGHhOgXCiaYuZqSZlgqrMS9sF203s1PFRclvIwdkJtDe7KLc3gw8eukb1+QyBzr EhzA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=8NTpr8qNiWGggQwGnuWCOmOJ8isCIuCvJD5KAIya3ZY=; b=nuELxSg+v++7/MLkugFX6A6QhOkR/rLcEq+siv25QkUH8LqfFY98qSlckHNym5KuAq y4GK53dT2jM2+sAE1Z3Pw/LZbLYuVKP6DRL8BptrSbowcFKTDpcbs7c5o1UaQs7s4rmT at1ykRPAVjwmFa093rcW6a2x4GpKNTCnuHEL/dx8SzlWsqEkvPW3soCVAKsBNDYJSt8K W54YaazJnhy1toCgTpO+2nHHa/LKFd0jk1d6387BiLxA8o5XrxWq6bC1cP0keye2yiqs XYAtsQhi2AkJAO1NW7fAA4ElMw3lOZQKpJusY0v/O6OnOtcd0G8IBkJNfcL0kTsp5iv2 stdQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=OorFhxs4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id q4-20020a638c44000000b0050e60e36393si16742555pgn.776.2023.05.15.17.53.53; Mon, 15 May 2023 17:54:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=OorFhxs4; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245548AbjEPAU1 (ORCPT + 99 others); Mon, 15 May 2023 20:20:27 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52090 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245312AbjEPAUZ (ORCPT ); Mon, 15 May 2023 20:20:25 -0400 Received: from mail-pf1-x432.google.com (mail-pf1-x432.google.com [IPv6:2607:f8b0:4864:20::432]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A43297289 for ; Mon, 15 May 2023 17:20:23 -0700 (PDT) Received: by mail-pf1-x432.google.com with SMTP id d2e1a72fcca58-644d9bf05b7so7730194b3a.3 for ; Mon, 15 May 2023 17:20:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1684196423; x=1686788423; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=8NTpr8qNiWGggQwGnuWCOmOJ8isCIuCvJD5KAIya3ZY=; b=OorFhxs4iWBxONb5AwKmj083FRlcsY+o5/PpHIbSffcqkmzv6lJdBMeys9uniTMsRV m+Ja/w4bXKQvaPT9M//WJFUuL1mNpRoXRgJZwEzIZ9LqlejUObkwVI1gmQ2vMBlgWoSV 0mcJEmNrw2Lccl7iTgFbC9C4uRUJLxpZ1pl4s= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684196423; x=1686788423; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=8NTpr8qNiWGggQwGnuWCOmOJ8isCIuCvJD5KAIya3ZY=; b=Og4ZclwOBUUqxBbrx3qdcYLt8FkC9+jZKNcV1dTV2Gl+v1ylEYm/iVQqWwzgTek2UK AXom88RQrSLok02zuG0k1oBnKEwKKgEbtRBxhppRdPJHaBj9c5A/BcAhjYFgAmqq0nHf 3sXgEoqY6yPkIw+88wXzIdDaTdDigFRq3Clsx4JQjtf1C/DcSSzbga39NBCaHenL0ogW LKWcZVFtK6jEBDwzwdP26jAkAXtMJc42OLdsuwNoKJp2QyUz+83ypG2TOdbzTGZo2TaN QKt5xUvGZ34JSuOQEBlAHjUwrgYe6yQpq9Uei7e8BuwgHNg2fkUg9KjX4EHHxo/aWIVz spwA== X-Gm-Message-State: AC+VfDwiTdUO+BGc0E/toZdz3oSJvo6YURw6JNGWw19LCVnVwM5ocbXL uDUdUzdkjeY0zdihDsx53XGIfg== X-Received: by 2002:a05:6a00:2313:b0:64a:fa71:a98f with SMTP id h19-20020a056a00231300b0064afa71a98fmr13472377pfh.13.1684196423149; Mon, 15 May 2023 17:20:23 -0700 (PDT) Received: from tictac2.mtv.corp.google.com ([2620:15c:9d:2:9f33:9d98:277a:d4cd]) by smtp.gmail.com with ESMTPSA id t9-20020a62ea09000000b0064559b58eb8sm12286103pfh.154.2023.05.15.17.20.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 May 2023 17:20:22 -0700 (PDT) From: Douglas Anderson To: Bjorn Andersson Cc: Douglas Anderson , Andy Gross , Conor Dooley , Konrad Dybcio , Krzysztof Kozlowski , Rob Herring , Sibi Sankar , cros-qcom-dts-watchers@chromium.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] arm64: dts: qcom: sc7180-lite: Fix SDRAM freq for misidentified sc7180-lite boards Date: Mon, 15 May 2023 17:19:29 -0700 Message-ID: <20230515171929.1.Ic8dee2cb79ce39ffc04eab2a344dde47b2f9459f@changeid> X-Mailer: git-send-email 2.40.1.606.ga4b1b128d6-goog MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1766009883114910525?= X-GMAIL-MSGID: =?utf-8?q?1766010069436160406?= In general, the three SKUs of sc7180 (lite, normal, and pro) are handled dynamically. The cpufreq table in sc7180.dtsi includes the superset of all CPU frequencies. The "qcom-cpufreq-hw" driver in Linux shows that we can dynamically detect which frequencies are actually available on the currently running CPU and then we can just enable those ones. The GPU is similarly dynamic. The nvmem has a fuse in it (see "gpu_speed_bin" in sc7180.dtsi) that the GPU driver can use to figure out which frequencies to enable. There is one part, however, that is not so dynamic. The way SDRAM frequency works in sc7180 is that it's tied to cpufreq. At the busiest cpufreq operating points we'll pick the top supported SDRAM frequency. They ramp down together. For the "pro" SKU of sc7180, we only enable one extra cpufreq step. That extra cpufreq step runs SDRAM at the same speed as the step below. Thus, for normal and pro things are OK. There is no sc7180-pro device tree snippet. For the "lite" SKU if sc7180, however, things aren't so easy. The "lite" SKU drops 3 cpufreq entries but can still run SDRAM at max frequency. That messed things up with the whole scheme. This is why we added the "sc7180-lite" fragment in commit 8fd01e01fd6f ("arm64: dts: qcom: sc7180-lite: Tweak DDR/L3 scaling on SC7180-lite"). When the lite scheme came about, it was agreed that the WiFi SKUs of lazor would _always_ be "lite" and would, in fact, be the only "lite" devices. Unfortunately, this decision changed and folks didn't realize that it would be a problem. Specifically, some later lazor WiFi-only devices were built with "pro" CPUs. Building WiFi-only lazor with "pro" CPUs isn't the end of the world. The SDRAM will ramp up a little sooner than it otherwise would, but aside from a small power hit things work OK. One problem, though, is that the SDRAM scaling becomes a bit quirky. Specifically, with the current tables we'll max out SDRAM frequency at 2.1GHz but then _lower_ it at 2.2GHz / 2.3GHz only to raise it back to max for 2.4GHz and 2.55GHz. Let's at least fix this so that the SDRAM frequency doesn't go down in that quirky way. On true "lite" SKUs this change will be a no-op because the operating points we're touching are disabled. This change is only useful when a board that thinks it has a "lite" CPU actually has a "normal" or "pro" one stuffed. Fixes: 8fd01e01fd6f ("arm64: dts: qcom: sc7180-lite: Tweak DDR/L3 scaling on SC7180-lite") Signed-off-by: Douglas Anderson --- arch/arm64/boot/dts/qcom/sc7180-lite.dtsi | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sc7180-lite.dtsi b/arch/arm64/boot/dts/qcom/sc7180-lite.dtsi index d8ed1d7b4ec7..4b306a59d9be 100644 --- a/arch/arm64/boot/dts/qcom/sc7180-lite.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7180-lite.dtsi @@ -16,3 +16,11 @@ &cpu6_opp11 { &cpu6_opp12 { opp-peak-kBps = <8532000 23347200>; }; + +&cpu6_opp13 { + opp-peak-kBps = <8532000 23347200>; +}; + +&cpu6_opp14 { + opp-peak-kBps = <8532000 23347200>; +};