Re: [PATCH] mtd: nand: s3c2410: Register cpufreq notifier only on S3C24xx

From: Boris Brezillon
Date: Sun Aug 14 2016 - 11:51:46 EST


On Mon, 27 Jun 2016 14:51:38 +0200
Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> wrote:

> The driver registered for CPU frequency transitions to recalculate its
> clock when ARM clock frequency changes (ratio between frequencies of
> ARM's parent clock (fclk) and clock for peripherals remains fixed).
>
> This is needed only on S3C24xx platform when cpufreq driver is enabled
> so limit the ifdef to respective cpufreq Kconfig.
>

Applied.

Thanks,

Boris

> Suggested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
> ---
> drivers/mtd/nand/s3c2410.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
> index d9309cf0ce2e..8d440cc0f002 100644
> --- a/drivers/mtd/nand/s3c2410.c
> +++ b/drivers/mtd/nand/s3c2410.c
> @@ -180,7 +180,7 @@ struct s3c2410_nand_info {
>
> enum s3c_cpu_type cpu_type;
>
> -#ifdef CONFIG_CPU_FREQ
> +#ifdef CONFIG_ARM_S3C24XX_CPUFREQ
> struct notifier_block freq_transition;
> #endif
> };
> @@ -701,7 +701,7 @@ static void s3c2440_nand_write_buf(struct mtd_info *mtd, const u_char *buf,
>
> /* cpufreq driver support */
>
> -#ifdef CONFIG_CPU_FREQ
> +#ifdef CONFIG_ARM_S3C24XX_CPUFREQ
>
> static int s3c2410_nand_cpufreq_transition(struct notifier_block *nb,
> unsigned long val, void *data)