linux-kernel - Re: [PATCH] THERMAL must not select HWMON

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] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 14 Mar 2008 13:43:01 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Bastian Blank <bastian@...di.eu.org>
Cc:	rdunlap@...otime.net, linux-kbuild@...r.kernel.org,
	linux-kernel@...r.kernel.org, rui.zhang@...el.com
Subject: Re: [PATCH] THERMAL must not select HWMON

On Fri, 14 Mar 2008 19:37:58 +0100
Bastian Blank <bastian@...di.eu.org> wrote:

> On Fri, Mar 14, 2008 at 11:21:12AM -0700, Randy Dunlap wrote:
> 
> [ select forces HWMON to yes even if deps are never applicable ]
> 
> > Yes, that's a known & longstanding problem (that select does not
> > follow/honor dependencies).
> 
> Okay.
> 
> So THERMAL can't select HWMON, instead it needs to do a proper depends.
> 
> Bastian
> 
> Signed-off-by: Bastian Blank <waldi@...ian.org>
> Cc: Zhang Rui <rui.zhang@...el.com>
> 
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 3ab313e..394912c 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -4,7 +4,7 @@
>  
>  menuconfig THERMAL
>  	bool "Generic Thermal sysfs driver"
> -	select HWMON
> +	depends on HWMON
>  	default y
>  	help
>  	  Generic Thermal Sysfs driver offers a generic mechanism for

Sorry, but we have insufficient information here.

Presumably it fixes some build error, but the changelog should tell us (at
least) what that error was, why it occurs and how the patch fixes it.

Often (for bugfixes) I'll go off and find this stuff out from the mailing
list and put together a proper description.  I also need to do this so I
can work out whether the patch is needed in 2.6.25 or -stable.  But I can't
even find any reference to the original problem in the lkml archives.

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