From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752302AbbGPMxd (ORCPT ); Thu, 16 Jul 2015 08:53:33 -0400 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:53896 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751914AbbGPMxb (ORCPT ); Thu, 16 Jul 2015 08:53:31 -0400 Date: Thu, 16 Jul 2015 13:52:54 +0100 From: Mark Brown To: Nariman Poushin Cc: gregkh@linuxfoundation.org, airlied@linux.ie, dmitry.torokhov@gmail.com, sameo@linux.intel.com, lee.jones@linaro.org, tony@atomide.com, lgirdwood@gmail.com, brian.austin@cirrus.com, Paul.Handrigan@cirrus.com, support.opensource@diasemi.com, bardliao@realtek.com, oder_chiou@realtek.com, laurent.pinchart+renesas@ideasonboard.com, lars@metafoo.de, dmurphy@ti.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, patches@opensource.wolfsonmicro.com, linux-omap@vger.kernel.org, alsa-devel@alsa-project.org, peter.ujfalusi@ti.com Message-ID: <20150716125254.GE4039@sirena.org.uk> References: <1436885152-19850-1-git-send-email-nariman@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="WK3l2KTTmXPVedZ6" Content-Disposition: inline In-Reply-To: <1436885152-19850-1-git-send-email-nariman@opensource.wolfsonmicro.com> X-Cookie: Eschew obfuscation. User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 87.113.81.128 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH 1/2] V4 regmap: Use reg_sequence for multi_reg_write / register_patch X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --WK3l2KTTmXPVedZ6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Jul 14, 2015 at 03:45:51PM +0100, Nariman Poushin wrote: Please submit patches in the format covered in SubmittingPatches, version information goes inside the []. > Add support for writing sequences of registers / patches with specified > delays (in microseconds). Logically separates the functionality using > sequences of register writes from the functions that take register > defaults, as adding a delay field on the reg_defaults can increase > memory usage substantially. This change doesn't do what the above changelog says. It introduces a new struct reg_sequence and updates the multi write and patch APIs to use that but it doesn't implement any delay functionality. Please resend with a clearer changelog that describes why the struct is being split out from the reg_defaults struct and makes it clear that this is just a rename. It's probably best to also defer the addition of the delay field until the second patch where this function is actually implemented. > +/** > + * Register / Value pairs for sequences of writes, incorporating an optional Register/value. > + * delay in microseconds. > + * > + * @reg: Register address. > + * @def: Register default value. > + * @delay_us: Delay in microseconds > + */ > + > +struct reg_sequence { No blank line between the kerneldoc and the struct (as is the style for other kernel code). --WK3l2KTTmXPVedZ6 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJVp6klAAoJECTWi3JdVIfQdeIH/0xR3Zsana0o8euXAzPqm3ok r9IlQGzEyF7pZWEuAaLuEL89f9F/OWOXBU/BpmW+bSatlq726j5a4TiMqrODwoVn YwFLn8RqnmtBjM6fBpYKRf+xwNeKvcWPPDp5WkFueV4XQhkLD7HVLM9NEfRSjPyM GTQboL9PRQYD8PXA/ACHYVtSQu7kSUlLTEP2LAIkmlQ65Bp4lEHDHezwIA4aMwVM 404hpcSnYhuHGg2K3UjoFJeRPJVxsJRQavm8BdKHA7rjGY2irwo2wUt3keQRAKpI 0iGWhWR/Va/3sLty64k6gJuxe2Wm2lpNVlVf/dSKfEU2NTL3faayq6YGEEBtPHI= =UO9S -----END PGP SIGNATURE----- --WK3l2KTTmXPVedZ6--