Re: [PATCH] regmap: Move documentation to regmap.h

2015-08-11 Thread Mark Brown
On Tue, Aug 11, 2015 at 11:00:45AM +0800, Nicolas Boichat wrote:

> First, thanks for the applying the series above.

> This documentation patch is still valid, and now applies cleanly on
> regmap/for-next.

Please send any patches you think should be applied using the normal
submission process in SubmittingPatches.


signature.asc
Description: Digital signature


Re: [PATCH] regmap: Move documentation to regmap.h

2015-08-11 Thread Mark Brown
On Tue, Aug 11, 2015 at 11:00:45AM +0800, Nicolas Boichat wrote:

 First, thanks for the applying the series above.

 This documentation patch is still valid, and now applies cleanly on
 regmap/for-next.

Please send any patches you think should be applied using the normal
submission process in SubmittingPatches.


signature.asc
Description: Digital signature


Re: [PATCH] regmap: Move documentation to regmap.h

2015-08-10 Thread Nicolas Boichat
Hi Mark,

On Wed, Jul 8, 2015 at 6:32 PM, Nicolas Boichat  wrote:
> Init functions defined in regmap*.c files are now prefixed with
> __, take lockdep key and class parameters, and should not be
> called directly: move the documentation to regmap.h, where the
> macros are defined.
>
> Signed-off-by: Nicolas Boichat 
> ---
> Applies on top of this series:
> [PATCHv2 0/3] regmap: Use different lockdep class for each regmap init call
> sent earlier today.

First, thanks for the applying the series above.

This documentation patch is still valid, and now applies cleanly on
regmap/for-next.

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


Re: [PATCH] regmap: Move documentation to regmap.h

2015-08-10 Thread Nicolas Boichat
Hi Mark,

On Wed, Jul 8, 2015 at 6:32 PM, Nicolas Boichat drink...@chromium.org wrote:
 Init functions defined in regmap*.c files are now prefixed with
 __, take lockdep key and class parameters, and should not be
 called directly: move the documentation to regmap.h, where the
 macros are defined.

 Signed-off-by: Nicolas Boichat drink...@chromium.org
 ---
 Applies on top of this series:
 [PATCHv2 0/3] regmap: Use different lockdep class for each regmap init call
 sent earlier today.

First, thanks for the applying the series above.

This documentation patch is still valid, and now applies cleanly on
regmap/for-next.

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


[PATCH] regmap: Move documentation to regmap.h

2015-07-08 Thread Nicolas Boichat
Init functions defined in regmap*.c files are now prefixed with
__, take lockdep key and class parameters, and should not be
called directly: move the documentation to regmap.h, where the
macros are defined.

Signed-off-by: Nicolas Boichat 
---
Applies on top of this series:
[PATCHv2 0/3] regmap: Use different lockdep class for each regmap init call
sent earlier today.

 drivers/base/regmap/regmap-ac97.c |  19 -
 drivers/base/regmap/regmap-i2c.c  |  19 -
 drivers/base/regmap/regmap-mmio.c |  23 -
 drivers/base/regmap/regmap-spi.c  |  19 -
 drivers/base/regmap/regmap-spmi.c |  34 
 drivers/base/regmap/regmap.c  |  25 --
 include/linux/regmap.h| 173 +++---
 7 files changed, 162 insertions(+), 150 deletions(-)

diff --git a/drivers/base/regmap/regmap-ac97.c 
b/drivers/base/regmap/regmap-ac97.c
index aa631be..c03ebfd 100644
--- a/drivers/base/regmap/regmap-ac97.c
+++ b/drivers/base/regmap/regmap-ac97.c
@@ -78,15 +78,6 @@ static const struct regmap_bus ac97_regmap_bus = {
.reg_read = regmap_ac97_reg_read,
 };
 
-/**
- * regmap_init_ac97(): Initialise AC'97 register map
- *
- * @ac97: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_ac97(struct snd_ac97 *ac97,
  const struct regmap_config *config,
  struct lock_class_key *lock_key,
@@ -97,16 +88,6 @@ struct regmap *__regmap_init_ac97(struct snd_ac97 *ac97,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_ac97);
 
-/**
- * devm_regmap_init_ac97(): Initialise AC'97 register map
- *
- * @ac97: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
- * to a struct regmap.  The regmap will be automatically freed by the
- * device management code.
- */
 struct regmap *__devm_regmap_init_ac97(struct snd_ac97 *ac97,
   const struct regmap_config *config,
   struct lock_class_key *lock_key,
diff --git a/drivers/base/regmap/regmap-i2c.c b/drivers/base/regmap/regmap-i2c.c
index 3163b22..7007d6e 100644
--- a/drivers/base/regmap/regmap-i2c.c
+++ b/drivers/base/regmap/regmap-i2c.c
@@ -233,15 +233,6 @@ static const struct regmap_bus *regmap_get_i2c_bus(struct 
i2c_client *i2c,
return ERR_PTR(-ENOTSUPP);
 }
 
-/**
- * regmap_init_i2c(): Initialise register map
- *
- * @i2c: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_i2c(struct i2c_client *i2c,
 const struct regmap_config *config,
 struct lock_class_key *lock_key,
@@ -257,16 +248,6 @@ struct regmap *__regmap_init_i2c(struct i2c_client *i2c,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_i2c);
 
-/**
- * devm_regmap_init_i2c(): Initialise managed register map
- *
- * @i2c: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
- * to a struct regmap.  The regmap will be automatically freed by the
- * device management code.
- */
 struct regmap *__devm_regmap_init_i2c(struct i2c_client *i2c,
  const struct regmap_config *config,
  struct lock_class_key *lock_key,
diff --git a/drivers/base/regmap/regmap-mmio.c 
b/drivers/base/regmap/regmap-mmio.c
index a1b2b27..426a57e 100644
--- a/drivers/base/regmap/regmap-mmio.c
+++ b/drivers/base/regmap/regmap-mmio.c
@@ -296,17 +296,6 @@ err_free:
return ERR_PTR(ret);
 }
 
-/**
- * regmap_init_mmio_clk(): Initialise register map with register clock
- *
- * @dev: Device that will be interacted with
- * @clk_id: register clock consumer ID
- * @regs: Pointer to memory-mapped IO region
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_mmio_clk(struct device *dev, const char *clk_id,
  void __iomem *regs,
  const struct regmap_config *config,
@@ -324,18 +313,6 @@ struct regmap *__regmap_init_mmio_clk(struct device *dev, 
const char *clk_id,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_mmio_clk);
 
-/**
- * devm_regmap_init_mmio_clk(): Initialise managed register map with clock
- *
- * @dev: Device that will be interacted with
- * @clk_id: register clock consumer ID
- * @regs: Pointer to memory-mapped IO region
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
- * to a struct regmap. 

[PATCH] regmap: Move documentation to regmap.h

2015-07-08 Thread Nicolas Boichat
Init functions defined in regmap*.c files are now prefixed with
__, take lockdep key and class parameters, and should not be
called directly: move the documentation to regmap.h, where the
macros are defined.

Signed-off-by: Nicolas Boichat drink...@chromium.org
---
Applies on top of this series:
[PATCHv2 0/3] regmap: Use different lockdep class for each regmap init call
sent earlier today.

 drivers/base/regmap/regmap-ac97.c |  19 -
 drivers/base/regmap/regmap-i2c.c  |  19 -
 drivers/base/regmap/regmap-mmio.c |  23 -
 drivers/base/regmap/regmap-spi.c  |  19 -
 drivers/base/regmap/regmap-spmi.c |  34 
 drivers/base/regmap/regmap.c  |  25 --
 include/linux/regmap.h| 173 +++---
 7 files changed, 162 insertions(+), 150 deletions(-)

diff --git a/drivers/base/regmap/regmap-ac97.c 
b/drivers/base/regmap/regmap-ac97.c
index aa631be..c03ebfd 100644
--- a/drivers/base/regmap/regmap-ac97.c
+++ b/drivers/base/regmap/regmap-ac97.c
@@ -78,15 +78,6 @@ static const struct regmap_bus ac97_regmap_bus = {
.reg_read = regmap_ac97_reg_read,
 };
 
-/**
- * regmap_init_ac97(): Initialise AC'97 register map
- *
- * @ac97: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_ac97(struct snd_ac97 *ac97,
  const struct regmap_config *config,
  struct lock_class_key *lock_key,
@@ -97,16 +88,6 @@ struct regmap *__regmap_init_ac97(struct snd_ac97 *ac97,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_ac97);
 
-/**
- * devm_regmap_init_ac97(): Initialise AC'97 register map
- *
- * @ac97: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
- * to a struct regmap.  The regmap will be automatically freed by the
- * device management code.
- */
 struct regmap *__devm_regmap_init_ac97(struct snd_ac97 *ac97,
   const struct regmap_config *config,
   struct lock_class_key *lock_key,
diff --git a/drivers/base/regmap/regmap-i2c.c b/drivers/base/regmap/regmap-i2c.c
index 3163b22..7007d6e 100644
--- a/drivers/base/regmap/regmap-i2c.c
+++ b/drivers/base/regmap/regmap-i2c.c
@@ -233,15 +233,6 @@ static const struct regmap_bus *regmap_get_i2c_bus(struct 
i2c_client *i2c,
return ERR_PTR(-ENOTSUPP);
 }
 
-/**
- * regmap_init_i2c(): Initialise register map
- *
- * @i2c: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_i2c(struct i2c_client *i2c,
 const struct regmap_config *config,
 struct lock_class_key *lock_key,
@@ -257,16 +248,6 @@ struct regmap *__regmap_init_i2c(struct i2c_client *i2c,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_i2c);
 
-/**
- * devm_regmap_init_i2c(): Initialise managed register map
- *
- * @i2c: Device that will be interacted with
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
- * to a struct regmap.  The regmap will be automatically freed by the
- * device management code.
- */
 struct regmap *__devm_regmap_init_i2c(struct i2c_client *i2c,
  const struct regmap_config *config,
  struct lock_class_key *lock_key,
diff --git a/drivers/base/regmap/regmap-mmio.c 
b/drivers/base/regmap/regmap-mmio.c
index a1b2b27..426a57e 100644
--- a/drivers/base/regmap/regmap-mmio.c
+++ b/drivers/base/regmap/regmap-mmio.c
@@ -296,17 +296,6 @@ err_free:
return ERR_PTR(ret);
 }
 
-/**
- * regmap_init_mmio_clk(): Initialise register map with register clock
- *
- * @dev: Device that will be interacted with
- * @clk_id: register clock consumer ID
- * @regs: Pointer to memory-mapped IO region
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer to
- * a struct regmap.
- */
 struct regmap *__regmap_init_mmio_clk(struct device *dev, const char *clk_id,
  void __iomem *regs,
  const struct regmap_config *config,
@@ -324,18 +313,6 @@ struct regmap *__regmap_init_mmio_clk(struct device *dev, 
const char *clk_id,
 }
 EXPORT_SYMBOL_GPL(__regmap_init_mmio_clk);
 
-/**
- * devm_regmap_init_mmio_clk(): Initialise managed register map with clock
- *
- * @dev: Device that will be interacted with
- * @clk_id: register clock consumer ID
- * @regs: Pointer to memory-mapped IO region
- * @config: Configuration for register map
- *
- * The return value will be an ERR_PTR() on error or a valid pointer
-