linux-kernel - Re: [PATCH] reset: Add missing function stub for device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 04 Apr 2016 18:19:19 +0200
From:	Philipp Zabel <p.zabel@...gutronix.de>
To:	Daniel Lezcano <daniel.lezcano@...aro.org>
Cc:	matthias.bgg@...il.com, open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] reset: Add missing function stub for device_reset

Hi Daniel,

Am Freitag, den 01.04.2016, 21:38 +0200 schrieb Daniel Lezcano:
> The Mediatek's thermal driver fails to compile when the RESET_CONTROLLER
> option is not set. Logically, as the driver depends on this option to compile,
> the Kconfig should select it but actually that is not correct because the
> Kconfig provides also the COMPILE_TEST to increase the compile test coverage.
> 
> By providing the missing 'device_reset' stub for the driver in reset.h, that
> let the kernel to compile on different platforms with the Mediatek thermal
> driver enabled with the COMPILE_TEST option.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@...aro.org>
> ---
>  include/linux/reset.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/include/linux/reset.h b/include/linux/reset.h
> index c4c097d..db8cd7b 100644
> --- a/include/linux/reset.h
> +++ b/include/linux/reset.h
> @@ -72,6 +72,11 @@ static inline void reset_control_put(struct reset_control *rstc)
>  	WARN_ON(1);
>  }
>  
> +static inline int __must_check device_reset(struct device *dev)
> +{

+	WARN_ON(1);

> +	return -ENOTSUPP;
> +}
> +
>  static inline int device_reset_optional(struct device *dev)
>  {
>  	return -ENOTSUPP;

Thank you, I've added the warning above and applied the patch.

best regards
Philipp