2020-05-06 12:37:07

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH 2/2] memory: tegra: Drop <linux/clk-provider.h>

The Tegra EMC scaling support code is not a clock provider, but merely a
clock consumer, and thus does not need to include
<linux/clk-provider.h>.

Fixes: ec37a9a17afbfad5 ("memory: tegra: Add EMC scaling support code for Tegra210")
Signed-off-by: Geert Uytterhoeven <[email protected]>
---
drivers/memory/tegra/tegra210-emc-core.c | 1 -
drivers/memory/tegra/tegra210-emc.h | 1 -
2 files changed, 2 deletions(-)

diff --git a/drivers/memory/tegra/tegra210-emc-core.c b/drivers/memory/tegra/tegra210-emc-core.c
index d64ffe27c2457c9e..cdd663ba4733d8ff 100644
--- a/drivers/memory/tegra/tegra210-emc-core.c
+++ b/drivers/memory/tegra/tegra210-emc-core.c
@@ -6,7 +6,6 @@
#include <linux/bitfield.h>
#include <linux/clk.h>
#include <linux/clk/tegra.h>
-#include <linux/clk-provider.h>
#include <linux/debugfs.h>
#include <linux/delay.h>
#include <linux/kernel.h>
diff --git a/drivers/memory/tegra/tegra210-emc.h b/drivers/memory/tegra/tegra210-emc.h
index 63b3d0a790f1a908..8988bcf1529072a7 100644
--- a/drivers/memory/tegra/tegra210-emc.h
+++ b/drivers/memory/tegra/tegra210-emc.h
@@ -7,7 +7,6 @@
#define TEGRA210_EMC_H

#include <linux/clk.h>
-#include <linux/clk-provider.h>
#include <linux/clk/tegra.h>
#include <linux/io.h>
#include <linux/platform_device.h>
--
2.17.1


2020-05-07 20:06:24

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH 2/2] memory: tegra: Drop <linux/clk-provider.h>

On Wed, May 06, 2020 at 02:32:36PM +0200, Geert Uytterhoeven wrote:
> The Tegra EMC scaling support code is not a clock provider, but merely a
> clock consumer, and thus does not need to include
> <linux/clk-provider.h>.
>
> Fixes: ec37a9a17afbfad5 ("memory: tegra: Add EMC scaling support code for Tegra210")
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---
> drivers/memory/tegra/tegra210-emc-core.c | 1 -
> drivers/memory/tegra/tegra210-emc.h | 1 -
> 2 files changed, 2 deletions(-)

Applied to for-5.8/memory, thanks.

Thierry


Attachments:
(No filename) (574.00 B)
signature.asc (849.00 B)
Download all attachments

2020-05-12 21:17:36

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH 2/2] memory: tegra: Drop <linux/clk-provider.h>

Quoting Geert Uytterhoeven (2020-05-06 05:32:36)
> The Tegra EMC scaling support code is not a clock provider, but merely a
> clock consumer, and thus does not need to include
> <linux/clk-provider.h>.
>
> Fixes: ec37a9a17afbfad5 ("memory: tegra: Add EMC scaling support code for Tegra210")
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---

Reviewed-by: Stephen Boyd <[email protected]>

2020-06-15 08:41:52

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH 2/2] memory: tegra: Drop <linux/clk-provider.h>

Hi Thierry,

On Thu, May 7, 2020 at 10:04 PM Thierry Reding <[email protected]> wrote:
> On Wed, May 06, 2020 at 02:32:36PM +0200, Geert Uytterhoeven wrote:
> > The Tegra EMC scaling support code is not a clock provider, but merely a
> > clock consumer, and thus does not need to include
> > <linux/clk-provider.h>.
> >
> > Fixes: ec37a9a17afbfad5 ("memory: tegra: Add EMC scaling support code for Tegra210")
> > Signed-off-by: Geert Uytterhoeven <[email protected]>
> > ---
> > drivers/memory/tegra/tegra210-emc-core.c | 1 -
> > drivers/memory/tegra/tegra210-emc.h | 1 -
> > 2 files changed, 2 deletions(-)
>
> Applied to for-5.8/memory, thanks.

Which hasn't made it into v5.8-rc1, and was rebased.
New fixes tag is
Fixes: 01218c59f9bcf067 ("memory: tegra: Add EMC scaling support code
for Tegra210")

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds