[PATCH] clk: renesas: rzv2h: Add missing include file

[PATCH] clk: renesas: rzv2h: Add missing include file

[Fabrizio Castro]

File `rzv2h-cpg.h` makes use of data types defined in `types.h`,
but it's not including it, which could translate to build errors.

Include `types.h` to fix this problem.

Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
---
 drivers/clk/renesas/rzv2h-cpg.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/renesas/rzv2h-cpg.h b/drivers/clk/renesas/rzv2h-cpg.h
index 7321b085f937..25e0a9e5761b 100644
--- a/drivers/clk/renesas/rzv2h-cpg.h
+++ b/drivers/clk/renesas/rzv2h-cpg.h
@@ -9,6 +9,7 @@
 #define __RENESAS_RZV2H_CPG_H__
 
 #include <linux/bitfield.h>
+#include <linux/types.h>
 
 /**
  * struct pll - Structure for PLL configuration
-- 
2.34.1

Re: [PATCH] clk: renesas: rzv2h: Add missing include file

[Geert Uytterhoeven]

Hi Fabrizio,

Thanks for your patch!

On Tue, 24 Jun 2025 at 21:28, Fabrizio Castro
<fabrizio.castro.jz@renesas.com> wrote:
> File `rzv2h-cpg.h` makes use of data types defined in `types.h`,
> but it's not including it, which could translate to build errors.
>
> Include `types.h` to fix this problem.

linux/types.h.

> Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-clk for v6.17, with the above fixed.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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

RE: [PATCH] clk: renesas: rzv2h: Add missing include file

[Fabrizio Castro]

Hi Geert,

> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 26 June 2025 13:45
> To: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> Subject: Re: [PATCH] clk: renesas: rzv2h: Add missing include file
> 
> Hi Fabrizio,
> 
> Thanks for your patch!
> 
> On Tue, 24 Jun 2025 at 21:28, Fabrizio Castro
> <fabrizio.castro.jz@renesas.com> wrote:
> > File `rzv2h-cpg.h` makes use of data types defined in `types.h`,
> > but it's not including it, which could translate to build errors.
> >
> > Include `types.h` to fix this problem.
> 
> linux/types.h.
> 
> > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> i.e. will queue in renesas-clk for v6.17, with the above fixed.

Thanks for fixing.

Cheers,
Fab

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> 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