All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] regmap: Remove warning on stubbed dev_get_regmap()
@ 2012-06-13 13:38 Mark Brown
  2012-06-13 14:20 ` Lee Jones
  2012-06-13 15:27 ` Fabio Estevam
  0 siblings, 2 replies; 3+ messages in thread
From: Mark Brown @ 2012-06-13 13:38 UTC (permalink / raw)
  To: linux-kernel; +Cc: Lee Jones, Fabio Estevam, Mark Brown

It's perfectly sensible to ask if there's a regmap for a device which
doesn't have one so the stubbed version shouldn't complain, the caller
should be prepared for this.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 include/linux/regmap.h |    1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/regmap.h b/include/linux/regmap.h
index 4c9e642..c3508d0 100644
--- a/include/linux/regmap.h
+++ b/include/linux/regmap.h
@@ -388,7 +388,6 @@ static inline int regmap_register_patch(struct regmap *map,
 static inline struct regmap *dev_get_regmap(struct device *dev,
 					    const char *name)
 {
-	WARN_ONCE(1, "regmap API is disabled");
 	return NULL;
 }
 
-- 
1.7.10


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] regmap: Remove warning on stubbed dev_get_regmap()
  2012-06-13 13:38 [PATCH] regmap: Remove warning on stubbed dev_get_regmap() Mark Brown
@ 2012-06-13 14:20 ` Lee Jones
  2012-06-13 15:27 ` Fabio Estevam
  1 sibling, 0 replies; 3+ messages in thread
From: Lee Jones @ 2012-06-13 14:20 UTC (permalink / raw)
  To: Mark Brown; +Cc: linux-kernel, Fabio Estevam

On 13/06/12 14:38, Mark Brown wrote:
> It's perfectly sensible to ask if there's a regmap for a device which
> doesn't have one so the stubbed version shouldn't complain, the caller
> should be prepared for this.
>
> Signed-off-by: Mark Brown<broonie@opensource.wolfsonmicro.com>
> ---
>   include/linux/regmap.h |    1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/include/linux/regmap.h b/include/linux/regmap.h
> index 4c9e642..c3508d0 100644
> --- a/include/linux/regmap.h
> +++ b/include/linux/regmap.h
> @@ -388,7 +388,6 @@ static inline int regmap_register_patch(struct regmap *map,
>   static inline struct regmap *dev_get_regmap(struct device *dev,
>   					    const char *name)
>   {
> -	WARN_ONCE(1, "regmap API is disabled");
>   	return NULL;
>   }

Thanks Mark.

Reported-by: Lee Jones <lee.jones@linaro.org>
Tested-by: Lee Jones <lee.jones@linaro.org>

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] regmap: Remove warning on stubbed dev_get_regmap()
  2012-06-13 13:38 [PATCH] regmap: Remove warning on stubbed dev_get_regmap() Mark Brown
  2012-06-13 14:20 ` Lee Jones
@ 2012-06-13 15:27 ` Fabio Estevam
  1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2012-06-13 15:27 UTC (permalink / raw)
  To: Mark Brown; +Cc: linux-kernel, Lee Jones

On Wed, Jun 13, 2012 at 10:38 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> It's perfectly sensible to ask if there's a regmap for a device which
> doesn't have one so the stubbed version shouldn't complain, the caller
> should be prepared for this.
>
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

Tested-by: Fabio Estevam <fabio.estevam@freescale.com>

Thanks,

Fabio Estevam

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-06-13 15:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-13 13:38 [PATCH] regmap: Remove warning on stubbed dev_get_regmap() Mark Brown
2012-06-13 14:20 ` Lee Jones
2012-06-13 15:27 ` Fabio Estevam

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.