Re: [PATCH] Remove logically superfluous comparisons from Kconfig files.

["Horst H. von Brand" <vonbrand@inf.utfsm.cl>]

Robert P. J. Day <rpjday@mindspring.com> wrote:
>   Remove Kconfig comparisons of the form FUBAR || FUBAR=n, since they
> appear to be superfluous.
> 
> Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
> 
> ---
> 
>   based on what i read in kconfig-language.txt, it would *appear* that
> those comparisons are redundant, but i'm willing to be convinced
> otherwise.  (unless the developer specifically wanted the case of
> "!=m", which i'm fairly sure is not the same thing, yes?)

Would be clearer written that way if so.

>  drivers/char/drm/Kconfig   |    2 +-
>  fs/dlm/Kconfig             |    1 -
>  net/ipv4/netfilter/Kconfig |    1 -
>  net/sctp/Kconfig           |    1 -
>  4 files changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/char/drm/Kconfig b/drivers/char/drm/Kconfig
> index ef833a1..d681e68 100644
> --- a/drivers/char/drm/Kconfig
> +++ b/drivers/char/drm/Kconfig
> @@ -6,7 +6,7 @@
>  #
>  config DRM
>  	tristate "Direct Rendering Manager (XFree86 4.1.0 and higher DRI support)"
> -	depends on (AGP || AGP=n) && PCI
> +	depends on && PCI
                   ^^ ???

>  	help
-- 
Dr. Horst H. von Brand                   User #22616 counter.li.org
Departamento de Informatica                    Fono: +56 32 2654431
Universidad Tecnica Federico Santa Maria             +56 32 2654239
Casilla 110-V, Valparaiso, Chile               Fax:  +56 32 2797513
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/