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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 12, 2021 at 04:21:45PM +0100, Jules Maselbas wrote:
> Signed-off-by: Jules Maselbas <jmaselbas@xxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxxxxxxxx
> 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 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux