From patchwork Fri Feb 12 20:50:35 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lina Iyer X-Patchwork-Id: 8297391 Return-Path: X-Original-To: patchwork-linux-pm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 757409F1C5 for ; Fri, 12 Feb 2016 20:51:45 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 7327920459 for ; Fri, 12 Feb 2016 20:51:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id AAEDE20454 for ; Fri, 12 Feb 2016 20:51:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751149AbcBLUvk (ORCPT ); Fri, 12 Feb 2016 15:51:40 -0500 Received: from mail-pf0-f180.google.com ([209.85.192.180]:33016 "EHLO mail-pf0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751060AbcBLUvj (ORCPT ); Fri, 12 Feb 2016 15:51:39 -0500 Received: by mail-pf0-f180.google.com with SMTP id q63so52950876pfb.0 for ; Fri, 12 Feb 2016 12:51:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=uLapP8ck13277NtAt0HaY1XfEGxPdS4HiOCcnwpQJZc=; b=U30K74SMoaOZwcO5oucFatqb8QPM8u3c+nNKI1gHDk/4IQfL9yA9mGjNvvQLvVeKuJ k3lDRYkVcT0gBNBroZsxg4q7r47thA3HkxbKmA9Xdo8r91wmuvucMqhyGyrL2u0JTtWW 30lWfllwNnGOVWKeoFvHb1KG4dkac82KBJRrA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=uLapP8ck13277NtAt0HaY1XfEGxPdS4HiOCcnwpQJZc=; b=G6sxqSsp6wNVGtrPE4K+qi8bHcjQDkBi2uN5QE050cvRfd40bC9ANk7My2TSnl1mqS vgHlusd/9BmEimf/0AO1zQNvJxcCITUq+k+nZJntJG/idPv0BJb9tuU1D9BC1axeRZXp coqUFOHSav48olRcXKtg9u5N5oXyGv1BPD1Nh/t8+aZpLI6voneRidif9Q24Sb3A05H9 CJdSAUjmvC8/9bK7olQlBsgQuUZaEIHUSWZsgXIL+PcuW2TPy/VT4uzRi7inhVOdxV9u uTp4lvn+pCArbp91GjriEo9Yjv6tAU9wM0qBfdakzqpqx/u4s0FZUbfjXIRV+DrS0C1K 3m/Q== X-Gm-Message-State: AG10YOTwbuNYmeVOheNScgcdW9Kr8U7/wiiqGLlyjTOU4Tj+/LsqrQ1iGXDZzfcs5AIJeY+E X-Received: by 10.98.71.15 with SMTP id u15mr5058810pfa.161.1455310299316; Fri, 12 Feb 2016 12:51:39 -0800 (PST) Received: from ubuntu.localdomain ([172.56.8.98]) by smtp.gmail.com with ESMTPSA id x12sm21401070pfi.88.2016.02.12.12.51.36 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 12 Feb 2016 12:51:38 -0800 (PST) From: Lina Iyer To: ulf.hansson@linaro.org, khilman@kernel.org, rjw@rjwysocki.net, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: geert@linux-m68k.org, k.kozlowski@samsung.com, msivasub@codeaurora.org, agross@codeaurora.org, sboyd@codeaurora.org, linux-arm-msm@vger.kernel.org, lorenzo.pieralisi@arm.com, ahaslam@baylibre.com, mtitinger@baylibre.com, Lina Iyer , Mark Rutland Subject: [RFC v2 09/12] drivers: firmware: psci: Allow OS Initiated suspend mode Date: Fri, 12 Feb 2016 13:50:35 -0700 Message-Id: <1455310238-8963-10-git-send-email-lina.iyer@linaro.org> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1455310238-8963-1-git-send-email-lina.iyer@linaro.org> References: <1455310238-8963-1-git-send-email-lina.iyer@linaro.org> Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP PSCI firmware v1.0 onwards may support 2 different modes for CPU_SUSPEND. Platform coordinated mode is the default and every firmware should support it. The firmware shall also be the default mode. With the kernel capable of deciding the state for CPU cluster and coherency domains, the OS Initiated mode may now be used by the kernel, provided the firmware supports it. SET_SUSPEND_MODE is a PSCI function available on v1.0 onwards and can be used to set the mode in the firmware. Cc: Lorenzo Pieralisi Cc: Mark Rutland Signed-off-by: Lina Iyer --- drivers/firmware/psci.c | 45 ++++++++++++++++++++++++++++++++++++++++++++- include/linux/psci.h | 2 ++ include/uapi/linux/psci.h | 5 +++++ 3 files changed, 51 insertions(+), 1 deletion(-) diff --git a/drivers/firmware/psci.c b/drivers/firmware/psci.c index f25cd79..03c084e 100644 --- a/drivers/firmware/psci.c +++ b/drivers/firmware/psci.c @@ -49,12 +49,19 @@ * require cooperation with a Trusted OS driver. */ static int resident_cpu = -1; +static bool has_osi_pd; +static bool psci_suspend_mode_is_osi; bool psci_tos_resident_on(int cpu) { return cpu == resident_cpu; } +bool psci_has_osi_pd_support(void) +{ + return has_osi_pd; +} + struct psci_operations psci_ops; typedef unsigned long (psci_fn)(unsigned long, unsigned long, @@ -250,6 +257,26 @@ static int psci_system_suspend(unsigned long unused) virt_to_phys(cpu_resume), 0, 0); } +int psci_set_suspend_mode_osi(bool enable) +{ + int ret; + int mode; + + if (enable && !psci_has_osi_pd_support()) + return -ENODEV; + + if (enable == psci_suspend_mode_is_osi) + return 0; + + mode = enable ? PSCI_1_0_SUSPEND_MODE_OSI : PSCI_1_0_SUSPEND_MODE_PC; + ret = invoke_psci_fn(PSCI_FN_NATIVE(1_0, SET_SUSPEND_MODE), + mode, 0, 0); + if (!ret) + psci_suspend_mode_is_osi = enable; + + return psci_to_linux_errno(ret); +} + static int psci_system_suspend_enter(suspend_state_t state) { return cpu_suspend(0, psci_system_suspend); @@ -443,10 +470,26 @@ out_put_node: return err; } +static int __init psci_1_0_init(struct device_node *np) +{ + int ret; + + ret = psci_0_2_init(np); + if (ret) + return ret; + + /* Check if PSCI OSI mode is available */ + ret = psci_features(PSCI_FN_NATIVE(0_2, CPU_SUSPEND)); + if (ret & PSCI_1_0_OS_INITIATED) + has_osi_pd = true; + + return 0; +} + static const struct of_device_id const psci_of_match[] __initconst = { { .compatible = "arm,psci", .data = psci_0_1_init}, { .compatible = "arm,psci-0.2", .data = psci_0_2_init}, - { .compatible = "arm,psci-1.0", .data = psci_0_2_init}, + { .compatible = "arm,psci-1.0", .data = psci_1_0_init}, {}, }; diff --git a/include/linux/psci.h b/include/linux/psci.h index 12c4865..deae633 100644 --- a/include/linux/psci.h +++ b/include/linux/psci.h @@ -23,6 +23,8 @@ bool psci_tos_resident_on(int cpu); bool psci_power_state_loses_context(u32 state); bool psci_power_state_is_valid(u32 state); +bool psci_has_osi_pd_support(void); +int psci_set_suspend_mode_osi(bool enable); struct psci_operations { int (*cpu_suspend)(u32 state, unsigned long entry_point); diff --git a/include/uapi/linux/psci.h b/include/uapi/linux/psci.h index 3d7a0fc..eaab6e3 100644 --- a/include/uapi/linux/psci.h +++ b/include/uapi/linux/psci.h @@ -50,6 +50,7 @@ #define PSCI_1_0_FN_SYSTEM_SUSPEND PSCI_0_2_FN(14) #define PSCI_1_0_FN64_SYSTEM_SUSPEND PSCI_0_2_FN64(14) +#define PSCI_1_0_FN64_SET_SUSPEND_MODE PSCI_0_2_FN64(15) /* PSCI v0.2 power state encoding for CPU_SUSPEND function */ #define PSCI_0_2_POWER_STATE_ID_MASK 0xffff @@ -93,6 +94,10 @@ #define PSCI_1_0_FEATURES_CPU_SUSPEND_PF_MASK \ (0x1 << PSCI_1_0_FEATURES_CPU_SUSPEND_PF_SHIFT) +#define PSCI_1_0_OS_INITIATED BIT(0) +#define PSCI_1_0_SUSPEND_MODE_PC 0 +#define PSCI_1_0_SUSPEND_MODE_OSI 1 + /* PSCI return values (inclusive of all PSCI versions) */ #define PSCI_RET_SUCCESS 0 #define PSCI_RET_NOT_SUPPORTED -1