From patchwork Fri Oct 21 15:10:12 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 6743 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:4242:0:0:0:0:0 with SMTP id s2csp753399wrr; Fri, 21 Oct 2022 08:15:27 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4ArxR2RE4BnPyyCsA6PMBYTiMG5AzUH73zzj9uTq76lryBRX3oSi7AgU7cnAm/1OYZoVhs X-Received: by 2002:a17:907:2708:b0:78d:d22b:c8a9 with SMTP id w8-20020a170907270800b0078dd22bc8a9mr16100734ejk.640.1666365327300; Fri, 21 Oct 2022 08:15:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666365327; cv=none; d=google.com; s=arc-20160816; b=VXY1QgXepHGuBh4GXeTKVxSClftMtjxpfg0/mhlImbjXZNNiYVL0+BmJACVEvQjWzx XBjS1KR3P0ustM9qQMdnhJPrQ2K/OmqtCiZKdHmSGSR8cmxTl22pb5ZcALWfMjZ9TFmM 5P/d6Iufq9GgYi7akuRREl6ma2v/MwxelDt8miQjC4fL63Qb9TObOrmcOobFrWZiCrMU fBq04NRd/ITwkNj5wGIo+rbDVCm7utL6St46HZQaZahhHDuIg+RywaG4OEOCuptkvYaG 5ZZJX+vqoOW+1qWsWmSaikX5lJWkbIaBk8dDRSukYSW+1Ix/VHf/GhxVZZ/RY9FQHRPK psIQ== 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=TF6XEvFWOEgGd45xR42b2Ik4eFRVeWARvUbr71hDMcE=; b=cLkg+AtVevxzlYcpo1pC39xol1Gr8LvXozLwKbXpBTcM0WEncjIMQDmAJM7AgyiihN Rbm9F0j3zqCTzz7Sul/eO3GCMzmouakTJNqiz18Cdua1Ve/9JChrBxpfpuWsc2KyuQFp EqL9/jJFuF7CcewvNZpzEvO4YqFby+RnVp3J6ZlC5KnsWNupgEf0ZE/0uEtHcK/wUtiw bMWqA0XcAvlPaBI0va19u7foC2i0zsJq05XnNLO7PhHrDGd5zSgtUwmWZpZ04ayJKpJ/ JVqhke+4qKPQzKwkgxpHJ9XljbJGhSw/NdZo+/HVMjt6WU3pWNa1usDdbaFIjrbxuKXW hwcw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=KRmXFvyH; 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=linaro.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id di15-20020a170906730f00b00791a7e441a3si8447601ejc.459.2022.10.21.08.15.02; Fri, 21 Oct 2022 08:15:27 -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=@linaro.org header.s=google header.b=KRmXFvyH; 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=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230157AbiJUPKw (ORCPT + 99 others); Fri, 21 Oct 2022 11:10:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49066 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230367AbiJUPK2 (ORCPT ); Fri, 21 Oct 2022 11:10:28 -0400 Received: from mail-lf1-x136.google.com (mail-lf1-x136.google.com [IPv6:2a00:1450:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AC49794130 for ; Fri, 21 Oct 2022 08:10:23 -0700 (PDT) Received: by mail-lf1-x136.google.com with SMTP id a29so5689415lfo.1 for ; Fri, 21 Oct 2022 08:10:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=TF6XEvFWOEgGd45xR42b2Ik4eFRVeWARvUbr71hDMcE=; b=KRmXFvyHUTg36Lgc4C6cPZqipnlxJ3e5OcYFJ4U0hCUoBED3qsJB6IxneioiSPfARF Rwbds9KpGW2mFOw0qbHD33d/ya85qVIKVGAq05kp6OyMIvtgII6O+tJeW9Aip9iy3RVF xjzZnET80g30lvQpHQDr6O34ocKDoy+aKAYLo9EZ2L8jsUdHfhZhlC659RFyJamBwgo6 1qs3muh+x4kkc+r6awAAXP4gp9EaaP6KeN00AlLJy7JaBd2O/hD0qZruhkU2SZWvmFI6 zJLSWvnEXbHqnNHfws7AGmVl6x4/1A/+VCIirmG9DFDkOn/qsxnER+YRYA6TWbScsxPz PCPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=TF6XEvFWOEgGd45xR42b2Ik4eFRVeWARvUbr71hDMcE=; b=1b2FBTNXQwLHJam4vaRn8Oos64Cznd1U5DwEkiWvGNy+hyUcPzgusRQd00WfNDUrN8 /A7D5DG7qxH9xxt3WngV8kdLA3qAWqH2MFoB5Fv5TGr72g6J6FXNt5MF3S0rm70UwhiX swu8giTzdrDkWbYPm1UB4QDr9C7h8ZzBZMlAcdL1SnXWANGufNHGjTxeNdjQ7upPl5Ri niveiRelUaosRZMVPr3Cfo8NM5g+sRkgQMQppNgz0xbfpFmPgNG3/NrhlNVVzUk5SyTO bA8LAC2M+2IsKV/8ydJQ0IUCbaBAPhQMO59p65R/FchUARNLtslXuhV0vbe9Z/Xpj6GK eiSA== X-Gm-Message-State: ACrzQf1q4/ArUbghz4SrgVndfQFDnYO6Y5OYNebtlyiBqRn4FlhWzY5Q s9AncUeuKDe7W3j9/ABfWcSnYw== X-Received: by 2002:ac2:5e9d:0:b0:4a0:5828:6cf8 with SMTP id b29-20020ac25e9d000000b004a058286cf8mr6450268lfq.567.1666365021804; Fri, 21 Oct 2022 08:10:21 -0700 (PDT) Received: from uffe-XPS13.. (h-94-254-63-18.NA.cust.bahnhof.se. [94.254.63.18]) by smtp.gmail.com with ESMTPSA id s10-20020a056512202a00b0049476667228sm3199764lfs.65.2022.10.21.08.10.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Oct 2022 08:10:20 -0700 (PDT) From: Ulf Hansson To: "Rafael J . Wysocki" , Daniel Lezcano , linux-pm@vger.kernel.org Cc: Sudeep Holla , Lorenzo Pieralisi , Catalin Marinas , Ulf Hansson , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/2] cpuidle: dt: Return the correct numbers of parsed idle states Date: Fri, 21 Oct 2022 17:10:12 +0200 Message-Id: <20221021151013.148457-1-ulf.hansson@linaro.org> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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?1747310689274276867?= X-GMAIL-MSGID: =?utf-8?q?1747310689274276867?= While we correctly skips to initialize an idle state from a disabled idle state node in DT, the returned value from dt_init_idle_driver() don't get adjusted accordingly. Instead the number of found idle state nodes are returned, while the callers are expecting the number of successfully initialized idle states from DT. This leads to cpuidle drivers unnecessarily continues to initialize their idle state specific data. Moreover, in the case when all idle states have been disabled in DT, we would end up registering a cpuidle driver, rather than relying on the default arch specific idle call. Fixes: 9f14da345599 ("drivers: cpuidle: implement DT based idle states infrastructure") Signed-off-by: Ulf Hansson Reviewed-by: Sudeep Holla --- drivers/cpuidle/dt_idle_states.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpuidle/dt_idle_states.c b/drivers/cpuidle/dt_idle_states.c index 252f2a9686a6..448bc796b0b4 100644 --- a/drivers/cpuidle/dt_idle_states.c +++ b/drivers/cpuidle/dt_idle_states.c @@ -223,6 +223,6 @@ int dt_init_idle_driver(struct cpuidle_driver *drv, * also be 0 on platforms with missing DT idle states or legacy DT * configuration predating the DT idle states bindings. */ - return i; + return state_idx - start_idx; } EXPORT_SYMBOL_GPL(dt_init_idle_driver);