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=-9.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED,USER_AGENT_NEOMUTT autolearn=ham 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 776CBC10F13 for ; Thu, 11 Apr 2019 08:10:20 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 44FDB217D4 for ; Thu, 11 Apr 2019 08:10:20 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="ip7OjabO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 44FDB217D4 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.20170209; h=Sender: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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=166n+YPsk5Ke5VHl8WinbJLbs3UCSdvGGLEXvVcishI=; b=ip7OjabOHP4GX9v6jGYRldFH2 4BhB9Rrdop+UvSHgwFhZvXwJCa5hvB+1im/bAzi4aJReAHNFwVtVGUy9l+tj7DAmM0D3+PSqxZnLD m+7JOgm+VvV2Iwis0gAc4xNQe/r1gdhJpvvacBn6ZYJhOkNIpDJG6fPXDIbyIyyRTx+Agw+Qtan9o TpfMJEPSpcwhj3+cNEU6XMBC6h8BZKPUsSaEfS6LlL79HBI9765VrRqNA+jrrtVGrTMdm5GxHqh8V OVbxmYg3PssZ66JlWwoWx9aQzm44hcWRg2BeYD9d8g+7crdYMTjUbT4k+B5VLVfkeXVQjkjRNqib/ ROQ97WyrA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEUmj-0006Kr-UN; Thu, 11 Apr 2019 08:10:13 +0000 Received: from relay7-d.mail.gandi.net ([217.70.183.200]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEUmg-0005rj-1e for linux-arm-kernel@lists.infradead.org; Thu, 11 Apr 2019 08:10:12 +0000 X-Originating-IP: 90.88.18.121 Received: from localhost (aaubervilliers-681-1-63-121.w90-88.abo.wanadoo.fr [90.88.18.121]) (Authenticated sender: maxime.ripard@bootlin.com) by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id 9E98720007; Thu, 11 Apr 2019 08:09:55 +0000 (UTC) Date: Thu, 11 Apr 2019 10:09:55 +0200 From: Maxime Ripard To: Yangtao Li Subject: Re: [PATCH v3 1/2] cpufreq: Add sunxi nvmem based CPU scaling driver Message-ID: <20190411080955.ysp3wreifwqrpn3n@flea> References: <20190410174139.20012-1-tiny.windzz@gmail.com> <20190410174139.20012-2-tiny.windzz@gmail.com> MIME-Version: 1.0 In-Reply-To: <20190410174139.20012-2-tiny.windzz@gmail.com> User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190411_011010_613633_BFCC3E37 X-CRM114-Status: GOOD ( 15.65 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: nm@ti.com, mark.rutland@arm.com, linux-pm@vger.kernel.org, sboyd@kernel.org, vireshk@kernel.org, rjw@rjwysocki.net, linux-kernel@vger.kernel.org, wens@csie.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org, gregkh@linuxfoundation.org, mchehab+samsung@kernel.org, davem@davemloft.net, devicetree@vger.kernel.org Content-Type: multipart/mixed; boundary="===============5086484481759665573==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5086484481759665573== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ljbikfewuqsnslmi" Content-Disposition: inline --ljbikfewuqsnslmi Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi, On Wed, Apr 10, 2019 at 01:41:38PM -0400, Yangtao Li wrote: > For some SoCs, the CPU frequency subset and voltage value of each OPP > varies based on the silicon variant in use. The sunxi-cpufreq-nvmem > driver reads the efuse value from the SoC to provide the OPP framework > with required information. > > Signed-off-by: Yangtao Li The driver has a bunch of checkpatch warnings, please fix them (running checkpatch with --strict). > --- > MAINTAINERS | 7 + > drivers/cpufreq/Kconfig.arm | 10 ++ > drivers/cpufreq/Makefile | 1 + > drivers/cpufreq/cpufreq-dt-platdev.c | 2 + > drivers/cpufreq/sunxi-cpufreq-nvmem.c | 232 ++++++++++++++++++++++++++ > 5 files changed, 252 insertions(+) > create mode 100644 drivers/cpufreq/sunxi-cpufreq-nvmem.c > > diff --git a/MAINTAINERS b/MAINTAINERS > index 391405091c6b..bfd18ba6aa1a 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -667,6 +667,13 @@ S: Maintained > F: Documentation/i2c/busses/i2c-ali1563 > F: drivers/i2c/busses/i2c-ali1563.c > > +ALLWINNER CPUFREQ DRIVER > +M: Yangtao Li > +L: linux-pm@vger.kernel.org > +S: Maintained > +F: Documentation/devicetree/bindings/opp/sunxi-nvmem-cpufreq.txt > +F: drivers/cpufreq/sunxi-cpufreq-nvmem.c > + Please use a less generic name for the driver as well. It's called sunxi, but applies to a single SoC (for now). > +struct sunxi_cpufreq_soc_data { > + u32 (*efuse_xlate)(const struct sunxi_cpufreq_soc_data *soc_data, > + u32 efuse); > + u32 nvmem_mask; > + u32 nvmem_shift; > +}; What cases do you have? You're mentionning that it might apply to more SoCs, which ones? How is the data stored on those SoCs (and why do you need this particular abstraction)? Thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --ljbikfewuqsnslmi Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXK72UwAKCRDj7w1vZxhR xddWAP4/ozszJEsxyLl3PcSuUQNUmGNPTI5/EZoFHMnPIMM1+gEA26Mdnj9YBQVs HKxva/PTNG3RNdsvgtKweCHQ+Qrllww= =diSg -----END PGP SIGNATURE----- --ljbikfewuqsnslmi-- --===============5086484481759665573== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============5086484481759665573==--