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=unavailable 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 D8AB2C468BC 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 B2558208C0 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 S1730630AbfFGTeL (ORCPT ); Fri, 7 Jun 2019 15:34:11 -0400 Received: from mail-pl1-f196.google.com ([209.85.214.196]:46051 "EHLO mail-pl1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729081AbfFGTeL (ORCPT ); Fri, 7 Jun 2019 15:34:11 -0400 Received: by mail-pl1-f196.google.com with SMTP id bi6so793463plb.12 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=P2PbUeRa3+VXiQLncwlbTUm0ZtSf9PYVYgULr1nrANHcTmSFwT7Fzwf4IguQs2MH5r IdL1LBGRNWpuGrpDM40yaX1XvWIXTpb8Dx1kVWozY01bBkLKqgfEev7SEjMeVOXwh6Oc G69NOUT5SrINwhNnaq6Q/kgTTdgFiRzRby/Yy/8f6xX8OVP/7RW/2ifmbxKYRomDrmTX +iCPg4NPmUzw0IA8kOk0pgj3FOBdbF/lPGVLoP4go+P47CUmOAU1vkWqhVcD9ceEidCF dKkHDfDkoklB5Zl2SK6X0+DlHwrWjyS/tXPenfhRZCvolj5C9ypOf1z16TeplSfrMeBZ 3pRQ== X-Gm-Message-State: APjAAAUq7HgBt4phKXFkuOMCyle4XRZ7/NGVhu+rTnWjTOjsq4gyvim8 I7nrrXxWqHVEgtNQILN/1uuhuAGrv9w= 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-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@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