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=-5.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no 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 D7DAFC55179 for ; Fri, 30 Oct 2020 09:53:25 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5B9D42072C for ; Fri, 30 Oct 2020 09:53:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="dfwvDqJa"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="aJO1l8mh" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5B9D42072C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=CiwiGunVKpU3MbUlXiOK+eTOHrE5faO9Nrmui0qTnto=; b=dfwvDqJaVZJiL+Au27/sHM4vK +L6xGrpqjzm9i3mD+BJXjZlXNyd3k9RIAf2utzlHXp/w0YTRUOHvLpoyf0Ex2+0NgOz7P7uj4MfuA Vyxir3ea8JgvhKjLpVd2N8N0I7Ae6LQqG51cQoo4QKi/I2ugLegefClOGSXd4cOf2jJcIT8xD8uMJ 28oyN0pBN8g7LjudB76G3OyRL6K7I0BYbmaeB6MXe0KOjU9bLmeVNL8aXC6P296PaE4XkjR5wC/0i SGczAF1gVMdPg9dLbW8QgXhKKz6kw6APuuZfGlj9xXUZ5jVMpzRqSvYt1yxqHn1Ji4Nj8HXDVEqdm 1o9dZ/46Q==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kYR5d-0005Qj-Ma; Fri, 30 Oct 2020 09:52:57 +0000 Received: from mail-pg1-x543.google.com ([2607:f8b0:4864:20::543]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kYR5a-0005P3-KG for linux-arm-kernel@lists.infradead.org; Fri, 30 Oct 2020 09:52:55 +0000 Received: by mail-pg1-x543.google.com with SMTP id z24so4766831pgk.3 for ; Fri, 30 Oct 2020 02:52:53 -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=LouD1D0I3seOHVdhKfAMAsty+PIB9yldQNSOZd2AunQ=; b=aJO1l8mh4uw/GnTl5HY0NA+dwk++sWXQuGfLTVtl533+jm1Ua8w+Wey6I5HA7kFbT7 /OrX0mM9SkI3hgIomS6aGj1VoU84VFIs/KngRRZZlv0eqFfA7QHc/dPusglaVUaZEHeS bCc9UMq+1p3c5eBbePMJ1pnm9gmaKjIDe4AC9KLBvDw5vX1+/4iEjastJbhBYdAuq8NU d07nCQ+GQrNbl7sOdhngJ4LJvFcudsJLAECxVwtud5DIg1uzO4jtHeIZh9B9LwUghSZU FLeKjsbeoTK+tXDKFmhuXDgzbY/mAj5g/zqtQEJDat2mO1fqz9eBhdPk7zGyQs1YRdhL 4Xwg== 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=LouD1D0I3seOHVdhKfAMAsty+PIB9yldQNSOZd2AunQ=; b=Si9j+vZid/Wo2XigMARNKut7RPGzWRRsFdDlQBuFv+CIes4wGcTstNcDGpu60siXA+ bK84f1x2XgGI7JcQj2XEF5SqBLEj8CGObmVi1MMaDhR0VfSn1dtdeo+i1rWZuhRgKNYi RtMyyVwSwKHD8IbYXUBN+jLr9ZsnHJND8yC3WBAYkG7CKQ28UOO6XnxAQKeolwemStJz EjffOxvCBqdkxZCIlw0E45QijutfcItG7sLcQ5xArFS/rocX15X29GUjOartiHtCSZZO +Dok5JU/OXu4kj785K59Quq8YUjx09aDU/S4ZxCRG5fxN3ynAhcAtQyUzKTZQ7gK6XpN 1jLQ== X-Gm-Message-State: AOAM5305xjZ8LPL8MmF84Hhg7+JDWJYDsvtvktubcQfawlq0Ccg0VQ2q r9j79c9nxNTgK+zMnKIumF4m2w== X-Google-Smtp-Source: ABdhPJwg8INqFRnK0JxdmUemZAqhRWMZQOBYvn0HloepQQgK4PC8ob7yeNm/niQ1Z7nrQd17NwIeSg== X-Received: by 2002:a17:90a:2ec1:: with SMTP id h1mr1894047pjs.51.1604051571650; Fri, 30 Oct 2020 02:52:51 -0700 (PDT) Received: from localhost ([122.181.54.133]) by smtp.gmail.com with ESMTPSA id z10sm5422539pff.218.2020.10.30.02.52.50 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 30 Oct 2020 02:52:50 -0700 (PDT) Date: Fri, 30 Oct 2020 15:22:48 +0530 From: Viresh Kumar To: Lukasz Luba Subject: Re: [PATCH 0/4] Add sustainable OPP concept Message-ID: <20201030095248.abej6h5wphud2ihb@vireshk-i7> References: <20201028140847.1018-1-lukasz.luba@arm.com> <20201029074057.6ugmwyzna52x3oli@vireshk-i7> <20201029075356.rruej6jlerhfa4oy@vireshk-i7> <228fa1b3-bbd3-6941-fd4b-06581016d839@arm.com> <20201030082937.xgjmko2ohwhkt6f5@vireshk-i7> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716-391-311a52 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201030_055254_769060_1333479C X-CRM114-Status: GOOD ( 17.87 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: nm@ti.com, devicetree@vger.kernel.org, daniel.lezcano@linaro.org, vincent.guittot@linaro.org, linux-pm@vger.kernel.org, sboyd@kernel.org, vireshk@kernel.org, rafael@kernel.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, sudeep.holla@arm.com, Dietmar.Eggemann@arm.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 30-10-20, 09:19, Lukasz Luba wrote: > How about dropping the DT binding, but just adding this new field into > dev_pm_opp? There will be no DT parsing code, just the get/set > functions, which will be used in SCMI patch 4/4 and in IPA? > That would not require to change any DT bindings. > I see. Just for your information SCMI supports 'Sustained Performance' > expressed in kHz. Even that doesn't sound great (but then I don't have any background of why that was added there). The problem is not about how do we get this data into the kernel (from DT or firmware), but why is it even required. I really feel that software can find the sustainable OPP by itself (which can keep changing). About moving it into the OPP core, I am open to getting something added there if it is really useful and if the OPP core is the best suited place to keep such data. Though I am not sure of that for this field right now. Is it ever going to be used by anyone else apart from IPA ? If not, what about adding a helper in IPA to set sustainable-freq for a device ? So only SCMI based platforms will be able to use this stuff ? That's very limited, isn't it ? I think we should still try to make it better for everyone by making the software smarter. It has so much data, the OPPs, the power it will consume (based on microvolt property?), the heat we produce from that (from thermal framework), etc. Perhaps building this information continuously at runtime based on when and how we hit the trip points ? So we know which is the right frequency where we can refrain from hitting the trip points. But may be I am asking too much :( -- viresh _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel