Re: [PATCH v2] mei: deduplicate X86 && PCI dependencies, make them apply to all subentries

From: Greg Kroah-Hartman
Date: Tue Apr 25 2023 - 12:17:54 EST


On Tue, Apr 25, 2023 at 05:43:14PM +0200, kilobyte@xxxxxxxxxx wrote:
> From: Adam Borowski <kilobyte@xxxxxxxxxx>
>
> The Kconfig "depends on X86 && PCI" line was repeated for 4 out of 6 config
> symbols here -- which was both unnecessarily repetitive, and caused a
> dormant problem for the two remaining symbols lacking the dependency.
>
> Signed-off-by: Adam Borowski <kilobyte@xxxxxxxxxx>
> ---
> v2: instead of adding new depends, wrap all of mei in "if..endif"
>
> drivers/misc/mei/Kconfig | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/misc/mei/Kconfig b/drivers/misc/mei/Kconfig
> index d21486d69df2..f2ab2696fa78 100644
> --- a/drivers/misc/mei/Kconfig
> +++ b/drivers/misc/mei/Kconfig
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: GPL-2.0
> # Copyright (c) 2003-2019, Intel Corporation. All rights reserved.
> +if X86 && PCI

Please no, that's not a normal Kconfig thing to do at all, sorry.

greg k-h