[RFC PATCH] reset: Remove WARN_ON when CONFIG_RESET_CONTROLLER=n

[Sascha Hauer]

On Fri, Mar 12, 2021 at 04:21:45PM +0100, Jules Maselbas wrote:
> Signed-off-by: Jules Maselbas <jmaselbas at kalray.eu>
> ---
> I am using dwc2 but my platform doesn't has CONFIG_RESET_CONTROLLER enabled,
> when booting barebox I got this kind of message:
> WARNING: at barebox/include/linux/reset.h:54/device_reset_us()!
> 
> Not sure if removing all the WARN_ON is a good idea... asking for insight.

I looked at the kernel and it has no warnings in the stubs. For that
reason I applied this patch.

Having no warnings has its downside as well of course. If you really
need resets for your device to function then you won't notice that reset
support is missing.

Sascha


> ---
>  include/linux/reset.h | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/include/linux/reset.h b/include/linux/reset.h
> index 4a92a177b..a166fe1cf 100644
> --- a/include/linux/reset.h
> +++ b/include/linux/reset.h
> @@ -21,43 +21,36 @@ int __must_check device_reset_us(struct device_d *dev, int us);
>  
>  static inline int reset_control_reset(struct reset_control *rstc)
>  {
> -	WARN_ON(1);
>  	return 0;
>  }
>  
>  static inline int reset_control_assert(struct reset_control *rstc)
>  {
> -	WARN_ON(1);
>  	return 0;
>  }
>  
>  static inline int reset_control_deassert(struct reset_control *rstc)
>  {
> -	WARN_ON(1);
>  	return 0;
>  }
>  
>  static inline struct reset_control *
>  reset_control_get(struct device_d *dev, const char *id)
>  {
> -	WARN_ON(1);
>  	return NULL;
>  }
>  
>  static inline void reset_control_put(struct reset_control *rstc)
>  {
> -	WARN_ON(1);
>  }
>  
>  static inline int device_reset_us(struct device_d *dev, int us)
>  {
> -	WARN_ON(1);
>  	return 0;
>  }
>  
>  static inline int device_reset(struct device_d *dev)
>  {
> -	WARN_ON(1);
>  	return 0;
>  }
>  
> -- 
> 2.17.1
> 
> 
> 
> _______________________________________________
> barebox mailing list
> barebox at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |