From patchwork Wed Dec 20 22:12:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Konrad Dybcio X-Patchwork-Id: 756538 Received: from mail-ej1-f49.google.com (mail-ej1-f49.google.com [209.85.218.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7FAD84A988 for ; Wed, 20 Dec 2023 22:12:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Hr4Onva/" Received: by mail-ej1-f49.google.com with SMTP id a640c23a62f3a-a235eb41251so14176766b.3 for ; Wed, 20 Dec 2023 14:12:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1703110377; x=1703715177; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=WbE+p+rcyyFP+eEtMwxqlwbKKm3YVPbvUnctIfiDxV4=; b=Hr4Onva/aMYa64TV02v8sEKnKOa9dU62ufHv3yjaq1jplcVcisLWqPTnod6NLBmyFf Oj2XWDERSnbqEhbSHqPfRL+xOo0q4I8ZxNh4PksJmQ7akJXXsFjZJ6H269HHJKBhRZ3/ rSqRYkF0PHYQLSfwsHb8O5J1vcFEyhzJHTAjthYue2WHfZQE93GV21d9G19rMnsWnre/ +8kIGjaiGETubllpaBv2Z1E2IVgOfhaTDJO3L/47jiNzxlF1z1ODi/WuTDshXNx8EsX8 1R36bcOCsw9trvSTscRUDP9VnL0o9rKC9fZyQmmm62IPKdmS/2bdlUba3ugcWnuf7D0R LDTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703110377; x=1703715177; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=WbE+p+rcyyFP+eEtMwxqlwbKKm3YVPbvUnctIfiDxV4=; b=xFIZfn/L19I/rti5p3u/A1G0vWEpkiXNewACskYOru1j/mqJYr6I2FnkFZrwTs6LYZ bdzRj9wRQ3S62myJqa3Jc68yjAxUQ9nsmOoQgt2YHY0xrevqYGN9g3vdc9IfzYTutfPP 6URnaTdZcP6BshJbE6Hp0x0KNroWsF2/5kLFx3RYsyQ/m2QPNduuFPPMIJvQNyrO91sE Eccz0/mzh96Sc8yjgozUZqkmpgU6LAmM4g+vFfYguSoDQNJegv0aBnDasUE/cGIlSSLK wJxVyULi3DJfKL6QNF8Ynh5VkiNDwIzzl19EOAv23vFhnRiVv94vU8o+sjTpsTkfahDh P5NQ== X-Gm-Message-State: AOJu0YyUlrq4IpY1YXn4CVkCDv/A70/mhMFBrMWJMLvp87gO282aDW8z p/DWif1pb9s3v1eafONvWuUmtg== X-Google-Smtp-Source: AGHT+IHJw/FE2ewpobj61P+2P9GS8ZpVhAHis9tAoOMOkVKIcEuDOaRMsr1NH5TfHUZOtlI0W92iCQ== X-Received: by 2002:a17:906:65d8:b0:a19:a19b:4251 with SMTP id z24-20020a17090665d800b00a19a19b4251mr5876749ejn.188.1703110376744; Wed, 20 Dec 2023 14:12:56 -0800 (PST) Received: from [10.167.154.1] (178235179206.dynamic-4-waw-k-1-3-0.vectranet.pl. [178.235.179.206]) by smtp.gmail.com with ESMTPSA id x22-20020a170906135600b00a235f3b8259sm254654ejb.186.2023.12.20.14.12.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 14:12:56 -0800 (PST) From: Konrad Dybcio Subject: [PATCH RFT v2 0/3] Fix up SC8280XP idle states Date: Wed, 20 Dec 2023 23:12:53 +0100 Message-Id: <20230619-topic-sc8280xp-idle-v2-0-cde50bf02f3c@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAOVmg2UC/32NQQ6CMBBFr0Jmbc20gBZXrjyAcWdYlFJgkqZtW iQYwt1tOIDL919+3gbJRDIJbsUG0SyUyLsM4lSAnpQbDaM+MwgUJV54w2YfSLOkpZC4hmytYYi DRryKstIS8rNTybAuKqen/HUfa/MYohloPVJveD5e0OZxojT7+D3yCz/U39LCGbKyVrJrZFXzH u+WnIr+7OMI7b7vP1oDiMfRAAAA To: Andy Gross , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Johan Hovold Cc: Marijn Suijten , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio , Krzysztof Kozlowski , Konrad Dybcio X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1703110375; l=1554; i=konrad.dybcio@linaro.org; s=20230215; h=from:subject:message-id; bh=ARUY1xHpXtNbYJC73kXlyu2iubFsquAhjet7Dqjyorw=; b=jEr+FiShAw83HyJ/2AgXX7eCo9PSxs+fd6VLsvmwSxLTtAFYHteCzA7izJM+8hpXXahv2STz/ KP9ClVfkcC1BKCyOMLzdt6YpWdfFOmse7l+aVQMz3Ri4ySQHP+ZzN9u X-Developer-Key: i=konrad.dybcio@linaro.org; a=ed25519; pk=iclgkYvtl2w05SSXO5EjjSYlhFKsJ+5OSZBjOkQuEms= Comparing the data available in the downstream sources with what's there upstream, it was easy to spot some differences. This series aligns what we have upstream with what is there on the vendor kernel. The big asterisk there is that the downstream sources for SC8280XP can't always be trusted. A simple test shows that the lower idle states that were previously missing are implemented in the firmware (Linux reports no errors and enters them). HOWEVER The only cluster idle state that's been present until now (the deepest one) is now barely used if at all, as the scheduler seems to deem it inefficient or so. Hence, a request for testing and comments, especially from those who use the X13s daily or have reliable setup to measure the power usage. Signed-off-by: Konrad Dybcio --- Changes in v2: - Rename the idle states - Drop RFC, confirmed with Qualcomm - Rebase - Link to v1: https://lore.kernel.org/r/20230619-topic-sc8280xp-idle-v1-0-35a8b98451d0@linaro.org --- Konrad Dybcio (3): arm64: dts: qcom: sc8280xp: Add lower cluster idle states arm64: dts: qcom: sc8280xp: Add missing CPU idle states arm64: dts: qcom: sc8280xp: Fix up idle state periods arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 72 +++++++++++++++++++++++++--------- 1 file changed, 54 insertions(+), 18 deletions(-) --- base-commit: 20d857259d7d10cd0d5e8b60608455986167cfad change-id: 20230619-topic-sc8280xp-idle-00fc007234c8 Best regards,