Received: by 2002:a25:683:0:0:0:0:0 with SMTP id 125csp1095657ybg; Thu, 11 Jun 2020 00:24:41 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxN9chb30x6N8m/tSiKdNJjnvyXA24Ih74DanTMxsLmVLBtnfIUVHbaj32OmijY9ZjQ/i5h X-Received: by 2002:a50:bf03:: with SMTP id f3mr6029015edk.368.1591860281731; Thu, 11 Jun 2020 00:24:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1591860281; cv=none; d=google.com; s=arc-20160816; b=QZ5W3LIzlZElS1RPiB1Iap0dy0KFIjsujznCSUocXgntl4XPMsRicG74je7GmAevZ+ wpTOLTNtjSJQQ/9JEem3E6IZ9P6IPWaRvkf66LojKuzYSjeSKauGEMfabE8N8Q7DwYk2 UkqGoSCOIw0GBGE87VEIr+E7uK2KVGriX7sUh/hVOAOBUZ9d3TMlk8Aw3fUaS3saMcsN ZacYxWe7scM4Wspjb+GqhXEIO72DxKHsDW12w+G3fVGlE6VWOAIwsbs54acVysJPZgB8 LbDx/egNu7hlDjvDul6WkloMcOt2oBQlGaAhSzdGiVPMz4eDEW5/JylOaQMt22/MUf1R SgNw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=qo3jKj2TlYH3YqUl34KzJeMJH4fTcTcBHRWbQNMVHbQ=; b=t5i1cAm5BjV09vnx5dYmLojhwAH1WigGCseKQaV+pjp2Qsti0ZRdlZe/nyEtDQivdr rRiTzZ7pr247LL2a/vI/cSgWTMlZQuTE1F/8KTsMgZ3plGEOa2iPkkXciU6UxkhDwMjV hKT49or7FqcHfRGkBZXFxwjAjj2EChdoIO7hcoKv9RVGJKsLDm2ZOmM1qb2RXVVbu3Tk PslDA6Sj5P2OyKfvd2dRi67UAvfUBUEDJ0WWS2UqqaYgTR5K+qtUnQHiAFeZrDDNOZ30 W9mOEGdE35REfDb5QS75D7qISUjs4aRPqPgEnS/gfpRiGmqTCVFDIvToI6S0iohV9avp TPpg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=GgeT2KTH; 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=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id by8si1134834edb.354.2020.06.11.00.24.18; Thu, 11 Jun 2020 00:24:41 -0700 (PDT) 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; dkim=pass header.i=@chromium.org header.s=google header.b=GgeT2KTH; 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=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726856AbgFKHVU (ORCPT + 99 others); Thu, 11 Jun 2020 03:21:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49964 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726799AbgFKHVU (ORCPT ); Thu, 11 Jun 2020 03:21:20 -0400 Received: from mail-ej1-x642.google.com (mail-ej1-x642.google.com [IPv6:2a00:1450:4864:20::642]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E904BC08C5C1 for ; Thu, 11 Jun 2020 00:21:19 -0700 (PDT) Received: by mail-ej1-x642.google.com with SMTP id o15so5355814ejm.12 for ; Thu, 11 Jun 2020 00:21:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=qo3jKj2TlYH3YqUl34KzJeMJH4fTcTcBHRWbQNMVHbQ=; b=GgeT2KTHyRGE5UGaccyQn5lsdHLbqVcnDr4VLIeEDsBho6U8+B/+UBJN0Y8tVUTH6I 2e9omFEj/RKp6ewd/zoLOGO3BmN2qrJbo8tNra570xh8sI2t12UAb0dCRzmBMniJpcYF BhngUS6HMMTuaSfokudf7udJpEApEvTRNjrzU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=qo3jKj2TlYH3YqUl34KzJeMJH4fTcTcBHRWbQNMVHbQ=; b=nY0pjyaE/qjST4OR8KVjf/mQXwrKjDEjvy1c86rJ6L1lVBbsBUvis94KgBfy+xseXR E4fJWcYidA/8LRsv7fwIMc1Hd2bclOYAkQoKNmksvBl/nlq19bbrsAP2UQm5W1sOUzHy twKz8DMvZLsfmAMpgOqkEM8wgsxMI8psU6dTo8FNXO6v4DTq7GDUyQBdQ7FzDH/5Tfz/ 9nZIgulynRXIDEknGVQeUrpzzS3bwRu5tU040NlyoaDeJCNtl90isiu5cNzlPb+jJNgH AxcxYaOdz+mb1u1N03EZ6+aCwVnd76UK6QoTxpoeUhjx74g2NIP92qkA2o+gU0xq1X6D BzIg== X-Gm-Message-State: AOAM533G3FKHoLCWISzB1dioJQhXlhoIyK0RnDnvEI++fZocV1XwCUJ9 GFtACTSPv3N5LMdWaRF1TbUyI7A2BVnBL5jdzlVTJw== X-Received: by 2002:a17:906:e115:: with SMTP id gj21mr6795315ejb.528.1591860078482; Thu, 11 Jun 2020 00:21:18 -0700 (PDT) MIME-Version: 1.0 References: <20200610090748.45908-1-pihsun@chromium.org> <20200610090748.45908-3-pihsun@chromium.org> <3776237e-a6d5-ccda-79e4-39545b818e34@collabora.com> In-Reply-To: <3776237e-a6d5-ccda-79e4-39545b818e34@collabora.com> From: Pi-Hsun Shih Date: Thu, 11 Jun 2020 15:20:42 +0800 Message-ID: Subject: Re: [PATCH v3 2/2] regulator: Add driver for cros-ec-regulator To: Enric Balletbo i Serra Cc: Nicolas Boichat , Liam Girdwood , Mark Brown , Benson Leung , Guenter Roeck , Tzung-Bi Shih , Lee Jones , Yicheng Li , open list Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Thanks for the review, would address most of them in v4. An inline reply as below: On Thu, Jun 11, 2020 at 12:47 AM Enric Balletbo i Serra wrote: > > Hi Pi-Hsun, > > Thank you for your patch. > > On 10/6/20 11:07, Pi-Hsun Shih wrote: > > +/*****************************************************************************/ > > +/* Voltage regulator controls */ > > + > > +/* > > + * Get basic info of voltage regulator for given index. > > + * > > + * Returns the regulator name and supported voltage list in mV. > > + */ > > +#define EC_CMD_REGULATOR_GET_INFO 0x012B > > This introduces a new EC command, while you are here, please also add the > command in drivers/platform/chrome/cros_ec_trace.c, so we can trace properly the > command. Also can you point me to the commit that introduces this command in the > EC firmware? The commit that introduces this in EC firmware is at https://crrev.com/c/2100327