From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B31DBC2BCA1 for ; Fri, 7 Jun 2019 19:34:12 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 7C543212F5 for ; Fri, 7 Jun 2019 19:34:12 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="tf63b50a" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730573AbfFGTeL (ORCPT ); Fri, 7 Jun 2019 15:34:11 -0400 Received: from mail-pl1-f194.google.com ([209.85.214.194]:43416 "EHLO mail-pl1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729450AbfFGTeL (ORCPT ); Fri, 7 Jun 2019 15:34:11 -0400 Received: by mail-pl1-f194.google.com with SMTP id cl9so1184793plb.10 for ; Fri, 07 Jun 2019 12:34:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=wpKUd2nT5fJdpxBIuy/BOsoGuLISr+yeuc18GcpDf6g=; b=tf63b50aos3gIs6Uh0x0ZPeVIy4dEEbpe9EQmwGyT5+A2Gw+N/fvymG2EdZPGIy8L9 5xNyeQKO8zT6AP0IKirJTEmIaiKb0OS22rDKMOn0xOxkquKtpLyow98OJ4nggW92g6My 9pdMTgF13mlwfqUP3g8TWHhyUpxL9SbblVrXJdKjTdqxqojrU5td0HmAQ0uovPfbnxVC drUELbnT0qtfXIoREosTwsWm+DTIQBTK17ZyUprcwTpWecwDd1KhtfUWf8qd3HewI/gj yccWS8LfVbTTNCkPA7VUiFxQPzMCcZkELxdnOr46rMOQzYY/LSG5VMTANT0WWgP6jyeo 1SaQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=wpKUd2nT5fJdpxBIuy/BOsoGuLISr+yeuc18GcpDf6g=; b=h5KjpoclIJQ6DZNPgZeHMaP+Ff+oQJWy5FwESF+dHNfl0DrCsHgfVBcQTriw1dxGoV NlNnpBQEiQhPRHytg7doM/41ce5Y9wRoX4vGDHiz2t17DCzhxbGup5uuGE6cCbPbSqQP vqtf0qPCwK/Y6Lk21HIuE9d7pqp+gcFxZbcA6Oyu0keqp+ThopQPn3RpPwabPCidULPm AGQv4Ryq8SCVTGL0SsjPWI6jhn4U7oVyLjdxOh0sDywfm9gf28TcEXT6a36voVpdMO5d ZKhdgpnWVcIU4msQhbJKWwU89eazugM4C0j/DBRKS9gehuZyay2pnK39JtjT9/VOA5Ng D4fw== X-Gm-Message-State: APjAAAVqZXq2gnsY7cIThk/e9QC2SrBmyzJC1wcpinCsunCXm3qdCjG3 Z548FPJKtZ8URw56BqIuwHUrUQ== X-Google-Smtp-Source: APXvYqxSC7rw05vLxn/PlSEW90GOUry7OdvPp1Qde6epr5HHI6UoO46GU4ItChia53NkMezfJO+8dA== X-Received: by 2002:a17:902:2883:: with SMTP id f3mr57841151plb.111.1559936050681; Fri, 07 Jun 2019 12:34:10 -0700 (PDT) Received: from builder (104-188-17-28.lightspeed.sndgca.sbcglobal.net. [104.188.17.28]) by smtp.gmail.com with ESMTPSA id o26sm2726497pgv.47.2019.06.07.12.34.09 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 07 Jun 2019 12:34:10 -0700 (PDT) Date: Fri, 7 Jun 2019 12:34:07 -0700 From: Bjorn Andersson To: Sudeep Holla Cc: Ulf Hansson , "Rafael J. Wysocki" , Lorenzo Pieralisi , Mark Rutland , Linux ARM , "Rafael J . Wysocki" , Daniel Lezcano , "Raju P . L . S . S . S . N" , Amit Kucheria , Stephen Boyd , Niklas Cassel , Tony Lindgren , Kevin Hilman , Lina Iyer , Viresh Kumar , Vincent Guittot , Geert Uytterhoeven , Souvik Chakravarty , Linux PM , linux-arm-msm , Linux Kernel Mailing List Subject: Re: [PATCH 00/18] ARM/ARM64: Support hierarchical CPU arrangement for PSCI Message-ID: <20190607193407.GB24059@builder> References: <20190513192300.653-1-ulf.hansson@linaro.org> <20190607154210.GJ15577@e107155-lin> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190607154210.GJ15577@e107155-lin> User-Agent: Mutt/1.10.0 (2018-05-17) Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Fri 07 Jun 08:42 PDT 2019, Sudeep Holla wrote: > On Tue, May 14, 2019 at 10:58:04AM +0200, Ulf Hansson wrote: > > On Tue, 14 May 2019 at 10:08, Rafael J. Wysocki wrote: > > > > > > On Mon, May 13, 2019 at 9:23 PM Ulf Hansson wrote: > > > > > > > > This series enables support for hierarchical CPU arrangement, managed by PSCI > > > > for ARM/ARM64. It's based on using the generic PM domain (genpd), which > > > > recently was extended to manage devices belonging to CPUs. > > > > > > ACK for the patches touching cpuidle in this series (from the > > > framework perspective), but I'm assuming it to be taken care of by > > > ARM/ARM64 maintainers. > > > > Thanks for the ack! Yes, this is for PSCI/ARM maintainers. > > > > BTW, apologize for sending this in the merge window, but wanted to > > take the opportunity for people to have a look before OSPM Pisa next > > week. > > > > I will start looking at this series. But I would request PSCI/other > maintainers to wait until we see some comparison data before we merge. What comparison are you asking for here? Do you want to see the improvement this series gives or are you hoping to compare it with some other mechanism? > If they are fine to merge w/o that, I am fine. As of now we have just > 1-2 platforms to test(that too not so simple to get started) and the > long term support for them are questionable. Why is the support for these platforms questionable? People are actively working on these platforms and the feature set constantly improving. > Also with SDM845 supporting PC, we have excellent opportunity to > compare and conclude the results found. That's correct, ATF exists for SDM845. But with the standard choice of firmware you will get OSI and I don't know of a board out there where you can switch between them and do a apple to apple comparison. Devices such as RB3 (96boards SDM845), Pixel3 and the Windows laptops are all OSI only. So landing this support is not a question of PC or OSI being the better choice, it's a question of do we want to be able to enter these lower power states - with the upstream kernel - on any past, present or future Qualcomm devices. Regards, Bjorn