linux-kernel - Re: [PATCH v1 2/4] platform/x86: intel_skl_int3472: Fix dependencies (drop CLKDEV

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]

Message-ID: <c3aec3b4-1ba1-6442-fbed-57a16febde68@gmail.com>
Date:   Thu, 17 Jun 2021 23:50:36 +0100
From:   Daniel Scally <djrscally@...il.com>
To:     Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Hans de Goede <hdegoede@...hat.com>,
        Mario Limonciello <mario.limonciello@....com>,
        Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
        linux-kernel@...r.kernel.org, platform-driver-x86@...r.kernel.org
Cc:     Mark Gross <mgross@...ux.intel.com>
Subject: Re: [PATCH v1 2/4] platform/x86: intel_skl_int3472: Fix dependencies
 (drop CLKDEV_LOOKUP)

Hi Andy

On 17/06/2021 19:30, Andy Shevchenko wrote:
> Besides the fact that COMMON_CLK selects CLKDEV_LOOKUP


So it does - thanks

> , the latter
> is going to be removed from clock framework.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>


Reviewed-by: Daniel Scally <djrscally@...il.com>

> ---
>  drivers/platform/x86/intel-int3472/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/intel-int3472/Kconfig b/drivers/platform/x86/intel-int3472/Kconfig
> index c112878e833b..62e5d4cf9ee5 100644
> --- a/drivers/platform/x86/intel-int3472/Kconfig
> +++ b/drivers/platform/x86/intel-int3472/Kconfig
> @@ -1,7 +1,7 @@
>  config INTEL_SKL_INT3472
>  	tristate "Intel SkyLake ACPI INT3472 Driver"
>  	depends on ACPI
> -	depends on COMMON_CLK && CLKDEV_LOOKUP
> +	depends on COMMON_CLK
>  	depends on I2C
>  	depends on GPIOLIB
>  	depends on REGULATOR