Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1210327 > unrolled thread

[PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus

Started byMarkus Pargmann <mpa@pengutronix.de>
First post2015-08-20 11:20 +0200
Last post2015-08-20 20:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus Markus Pargmann <mpa@pengutronix.de> - 2015-08-20 11:20 +0200
    Re: [PATCH v2 1/5] regmap: Add missing comments about struct  regmap_bus Mark Brown <broonie@kernel.org> - 2015-08-20 20:30 +0200

#1210327 — [PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus

FromMarkus Pargmann <mpa@pengutronix.de>
Date2015-08-20 11:20 +0200
Subject[PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus
Message-ID<pZuaR-wx-15@gated-at.bofh.it>
There are some fields of this struct undocumented or old. This patch
updates the missing comments.

Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
---
 include/linux/regmap.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/include/linux/regmap.h b/include/linux/regmap.h
index 59c55ea0f0b5..915454a1b54e 100644
--- a/include/linux/regmap.h
+++ b/include/linux/regmap.h
@@ -296,8 +296,11 @@ typedef void (*regmap_hw_free_context)(void *context);
  *                if not implemented  on a given device.
  * @async_write: Write operation which completes asynchronously, optional and
  *               must serialise with respect to non-async I/O.
+ * @reg_write: Write operation for a register. Writes value to register.
  * @read: Read operation.  Data is returned in the buffer used to transmit
  *         data.
+ * @reg_read: Read operation for a register. Reads a value from a register.
+ * @free_context: Free context.
  * @async_alloc: Allocate a regmap_async() structure.
  * @read_flag_mask: Mask to be set in the top byte of the register when doing
  *                  a read.
@@ -307,7 +310,6 @@ typedef void (*regmap_hw_free_context)(void *context);
  * @val_format_endian_default: Default endianness for formatted register
  *     values. Used when the regmap_config specifies DEFAULT. If this is
  *     DEFAULT, BIG is assumed.
- * @async_size: Size of struct used for async work.
  */
 struct regmap_bus {
 	bool fast_io;
-- 
2.4.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1210663 — Re: [PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus

FromMark Brown <broonie@kernel.org>
Date2015-08-20 20:30 +0200
SubjectRe: [PATCH v2 1/5] regmap: Add missing comments about struct regmap_bus
Message-ID<pZCL7-4sw-3@gated-at.bofh.it>
In reply to#1210327

[Multipart message — attachments visible in raw view] — view raw

On Thu, Aug 20, 2015 at 11:12:32AM +0200, Markus Pargmann wrote:

>   * @async_write: Write operation which completes asynchronously, optional and
>   *               must serialise with respect to non-async I/O.
> + * @reg_write: Write operation for a register. Writes value to register.

This seems like documenation for the sake of filling in the blank :(  At
least distinguish it from async_write() please.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web