Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp392069pxb; Thu, 5 Nov 2020 02:58:26 -0800 (PST) X-Google-Smtp-Source: ABdhPJw1dnhvR2ebE/xhP9mUvdlHDfG5ATl2UBy2ic912GD+pPF85Eu2mCMZN2q7webiI7TKxr0m X-Received: by 2002:a17:906:3795:: with SMTP id n21mr1626987ejc.502.1604573905940; Thu, 05 Nov 2020 02:58:25 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1604573905; cv=none; d=google.com; s=arc-20160816; b=o9zoaiW8HbVnmORwapEjmErMrEJcrGJaMKlBZM91KF4s07Um+5Wqj5C4xn7p8ZVf1z jJZqwchEDYBQnPSmelp642/nvS6VeCV+An0j2GKk/RRUyk9sOK3mxroS3ztfYH511fVL IhO//O9VkTbieHQzcJluytCgWpmufXK9ldUKFeMWnvm00dg93nJ8qcGdusIZ63PnUKq/ GCB1zm8wulhOVCk0oqGwxzedT47sUP9c7dNfm3FRXOwWpYQCfKJ9pA2PvUMapRlni8bV 36kKyr+cTXcMho7kkZFXZotW8cGAlfmskS2NMscst5CAZZRo0hJJdRWs/9nU1wa8XsbX L3pQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=20momlBJpFAUiN1NNijRI/n+v21BplUT9/1MOqakhJQ=; b=AhJp+pyadPs8XDuT4hUi+49Ab/m3tOx/NXA1rZNdHmlfPJa/pjz14YaF9nkII+FTGf hS+OQHtyIujkXY5lYMat81/N9jlUGwG+0ELh57pKIOJS3QNdTmJyeBX8o12V602Ue3b3 5bpGdCvS8d+zNX8cXoXFX5x3OU9uDg0H21Q/JCJEpaegGHmL+F8El5AzwyxAf+LZ+zFg 9fcLJPLOa/yxlezci4ToLRiJS9m5mPV6aAGCeEr25DzQFla+o0TDATIjOcs21C5Xa8zq mJFLs7ZdnzTd0pp9Tuo1JwT90k9iq73EJqFcRZj2LTVa4Igbwgm11SIpvW0yiwQRFILg GhVA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id h23si956048ejx.192.2020.11.05.02.58.02; Thu, 05 Nov 2020 02:58:25 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730001AbgKEK4j (ORCPT + 99 others); Thu, 5 Nov 2020 05:56:39 -0500 Received: from foss.arm.com ([217.140.110.172]:57398 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726400AbgKEK4j (ORCPT ); Thu, 5 Nov 2020 05:56:39 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D0C9F142F; Thu, 5 Nov 2020 02:56:38 -0800 (PST) Received: from e123083-lin (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 0A4363F66E; Thu, 5 Nov 2020 02:56:35 -0800 (PST) Date: Thu, 5 Nov 2020 11:56:33 +0100 From: Morten Rasmussen To: Lukasz Luba Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-doc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, robh+dt@kernel.org, amitk@kernel.org, corbet@lwn.net, daniel.lezcano@linaro.org, Dietmar.Eggemann@arm.com, qperret@google.com, dianders@chromium.org, mka@chromium.org, rnayak@codeaurora.org, rafael@kernel.org, sudeep.holla@arm.com, viresh.kumar@linaro.org, sboyd@kernel.org, nm@ti.com Subject: Re: [PATCH v4 1/4] PM / EM: Add a flag indicating units of power values in Energy Model Message-ID: <20201105105633.GB8237@e123083-lin> References: <20201103090600.29053-1-lukasz.luba@arm.com> <20201103090600.29053-2-lukasz.luba@arm.com> <20201105091759.GA8237@e123083-lin> <876ca521-1576-63c5-9af5-b000f52cc17b@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <876ca521-1576-63c5-9af5-b000f52cc17b@arm.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 05, 2020 at 10:09:05AM +0000, Lukasz Luba wrote: > > > On 11/5/20 9:18 AM, Morten Rasmussen wrote: > > On Tue, Nov 03, 2020 at 09:05:57AM +0000, Lukasz Luba wrote: > > > @@ -79,7 +82,8 @@ struct em_data_callback { > > > struct em_perf_domain *em_cpu_get(int cpu); > > > struct em_perf_domain *em_pd_get(struct device *dev); > > > int em_dev_register_perf_domain(struct device *dev, unsigned int nr_states, > > > - struct em_data_callback *cb, cpumask_t *span); > > > + struct em_data_callback *cb, cpumask_t *spani, > > > > "spani" looks like a typo? > > > > Good catch, yes, the vim 'i'. > > Thank you Morten. I will resend this patch when you don't > find other issues in the rest of patches. The rest of the series looks okay to me. Morten