Received: by 2002:ac0:a582:0:0:0:0:0 with SMTP id m2-v6csp3468378imm; Fri, 19 Oct 2018 11:04:32 -0700 (PDT) X-Google-Smtp-Source: ACcGV62dNxk/2DX0IninvOdW25L3eexwxybgFD7gytE05w3YFOTyzogLPnkPOAto2BkbbzIZD5bw X-Received: by 2002:a17:902:1c3:: with SMTP id b61-v6mr22176150plb.65.1539972272492; Fri, 19 Oct 2018 11:04:32 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1539972272; cv=none; d=google.com; s=arc-20160816; b=m0E0qP1bgXbIqOYgH9IqpA8quNHHhR3KmJUF3ezKKv+9w+7lBRzzxm2TX0yEdRHBS1 VTJIJBxf8sfqJrEUtkGI2+lKyqJBAIK1uX+Yt2j5fjx/S8UnfX94lamuwPDOznUCZ/Iy QRs9VJghts9ZxPt3VrMJGZPLKM9zSNgdV07c2PdLpDLoRG1lT72sFnEhVjsZyay5ndYy 4rcKGFnKBI4TXsfuxqOsAZtaRGaLthEW6D8PpehWBB67giyewkY+IlcKWx/SABVOZMZT Ytjju32Z9ZyK23+AIyjXFuus3kFscY9b+H/fATUCR8X6aflt/4+pcnF7q/oy2JundGyE Wsig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:subject:user-agent:message-id :references:cc:in-reply-to:from:to:content-transfer-encoding :mime-version:dkim-signature; bh=CsdbBOUb2ZyD11nLWypVQFqu+R+nr6so2l3cafKCodI=; b=RQj9yqIR/TC0u7YrQgyosfMYmaVYrH9CW8u3a67jBD0r7Tf8YvhftxALgyzWWYlFua p/xgx9oZvnOQjizk9uIQ02rZMaqTn60XJ8Gz7KItk2O8MvMvQjF4eA13tbTaUbVCuadM xcpHASnRxuK7f/ebxdILJulUcQFudE9ZJeI1YGmDgi5aOx0YFNOOuaLrPpOBrt3RJRpu YbBfNFGSEtlrlltGJI6pqBztqSbfXyiY6jY2eRlKVEvMps08if6IbJTz+avzFrqm3Ycb 0NKspaj843270IdUM0MFlCoSHNBbgitZ6p4YvVZqRVRiYpYo6HW6lGQ5U3Ddt8uTWabB 93KA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=ytrcf99P; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p25-v6si5930133pli.239.2018.10.19.11.04.16; Fri, 19 Oct 2018 11:04:32 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=ytrcf99P; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727570AbeJTCKj (ORCPT + 99 others); Fri, 19 Oct 2018 22:10:39 -0400 Received: from mail.kernel.org ([198.145.29.99]:43382 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727462AbeJTCKj (ORCPT ); Fri, 19 Oct 2018 22:10:39 -0400 Received: from localhost (unknown [104.132.0.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 41C6B21470; Fri, 19 Oct 2018 18:03:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1539972210; bh=nxD2AjDWzz8u9Kit+miopZ6TDj4oSNsKu2M+cah50rI=; h=To:From:In-Reply-To:Cc:References:Subject:Date:From; b=ytrcf99PA2XhsazV0CHJdYqMdVE0szbNaiBni00/r+aRxVBz1wO+L3EtB1uRqk7/C ysEx0ehP3p1CBpKky+KGFV+Fk4yLZIog7Vm7+s2mKgT5cP1Ke8PRkcNkc7xkGwE+xQ zLFPu9LALgaCmx20US1SGexk9OAkijBg7S5MtftE= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: Jerome Brunet , Jianxin Pan , Neil Armstrong From: Stephen Boyd In-Reply-To: <01d07c83-b17e-70b5-6e9b-8150ee3aedf2@amlogic.com> Cc: Yixun Lan , Kevin Hilman , Carlo Caione , Michael Turquette , Rob Herring , Miquel Raynal , Boris Brezillon , Martin Blumenstingl , Liang Yang , Jian Hu , Qiufang Dai , Hanjie Lin , Victor Wan , linux-clk@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <1539839245-13793-1-git-send-email-jianxin.pan@amlogic.com> <1539839245-13793-4-git-send-email-jianxin.pan@amlogic.com> <153988282130.5275.17528969137837015544@swboyd.mtv.corp.google.com> <01d07c83-b17e-70b5-6e9b-8150ee3aedf2@amlogic.com> Message-ID: <153997220960.53599.2059896905852359614@swboyd.mtv.corp.google.com> User-Agent: alot/0.7 Subject: Re: [PATCH v5 3/3] clk: meson: add sub MMC clock controller driver Date: Fri, 19 Oct 2018 11:03:29 -0700 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Jianxin Pan (2018-10-19 09:12:53) > On 2018/10/19 1:13, Stephen Boyd wrote: > > Quoting Jianxin Pan (2018-10-17 22:07:25) > >> diff --git a/drivers/clk/meson/clk-regmap.c b/drivers/clk/meson/clk-re= gmap.c > >> index 305ee30..f96314d 100644 > >> --- a/drivers/clk/meson/clk-regmap.c > >> +++ b/drivers/clk/meson/clk-regmap.c > >> @@ -113,8 +113,25 @@ static int clk_regmap_div_set_rate(struct clk_hw = *hw, unsigned long rate, > >> clk_div_mask(div->width) << div->shi= ft, val); > >> }; > >> = > >> -/* Would prefer clk_regmap_div_ro_ops but clashes with qcom */ > >> +static void clk_regmap_div_init(struct clk_hw *hw) > >> +{ > >> + struct clk_regmap *clk =3D to_clk_regmap(hw); > >> + struct clk_regmap_div_data *div =3D clk_get_regmap_div_data(cl= k); > >> + unsigned int val; > >> + int ret; > >> + > >> + ret =3D regmap_read(clk->map, div->offset, &val); > >> + if (ret) > >> + return; > >> = > >> + val &=3D (clk_div_mask(div->width) << div->shift); > >> + if (!val) > >> + regmap_update_bits(clk->map, div->offset, > >> + clk_div_mask(div->width) << div->sh= ift, > >> + clk_div_mask(div->width)); > >> +} > >> + > >> +/* Would prefer clk_regmap_div_ro_ops but clashes with qcom */ > > = > > We should add a patch to rename the symbol for qcom, i.e. > > qcom_clk_regmap_div_ro_ops, and then any symbols in this directory > > should be meson_clk_regmap_div_ro_ops. > "/* Would prefer clk_regmap_div_ro_ops but clashes with qcom */" > This comment is not introduced in this patch. > I followed the naming style in this file and add clk_regmap_divider_with_= init_ops. > = > @Jerome=EF=BC=8C What's your suggestion about this=EF=BC=9F Yes you don't need to fix anything in this series. Just saying that in the future we should work on cleaning this up.