diff mbox series

ram: mediatek: mt7629: include <linux/sizes.h>

Message ID 4bd66fd5b69eda41b4320fd6f8db50a7b7fa7bf7.1690828424.git.daniel@makrotopia.org
State Deferred
Delegated to: Tom Rini
Headers show
Series ram: mediatek: mt7629: include <linux/sizes.h> | expand

Commit Message

Daniel Golle July 31, 2023, 6:34 p.m. UTC
Something between U-Boot 2023.04 and 2023.07.02 resulted in no longer
implicitely including <linux/sizes.h> in the DDR3 RAM driver for the
MT7929 SoC. The result is a build failure:
drivers/ram/mediatek/ddr3-mt7629.c: In function 'mtk_ddr3_get_info':
drivers/ram/mediatek/ddr3-mt7629.c:734:30: error: 'SZ_128M' undeclared (first use in this function)
  734 |                 info->size = SZ_128M;
      |                              ^~~~~~~
drivers/ram/mediatek/ddr3-mt7629.c:734:30: note: each undeclared identifier is reported only once for each function it appears in
drivers/ram/mediatek/ddr3-mt7629.c:737:30: error: 'SZ_256M' undeclared (first use in this function)
  737 |                 info->size = SZ_256M;
      |                              ^~~~~~~
drivers/ram/mediatek/ddr3-mt7629.c:740:30: error: 'SZ_512M' undeclared (first use in this function)
  740 |                 info->size = SZ_512M;
      |                              ^~~~~~~
drivers/ram/mediatek/ddr3-mt7629.c:743:30: error: 'SZ_1G' undeclared (first use in this function)
  743 |                 info->size = SZ_1G;
      |                              ^~~~~

Include <linux/sizes.h> so SZ_* is defined.

Reported-by: Tianling Shen <cnsztl@immortalwrt.org>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
---
 drivers/ram/mediatek/ddr3-mt7629.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Weijie Gao (高惟杰) Aug. 4, 2023, 1:09 a.m. UTC | #1
On Mon, 2023-07-31 at 19:34 +0100, Daniel Golle wrote:
> Something between U-Boot 2023.04 and 2023.07.02 resulted in no
> longer
> implicitely including <linux/sizes.h> in the DDR3 RAM driver for the
> MT7929 SoC. The result is a build failure:
> drivers/ram/mediatek/ddr3-mt7629.c: In function 'mtk_ddr3_get_info':
> drivers/ram/mediatek/ddr3-mt7629.c:734:30: error: 'SZ_128M'
> undeclared (first use in this function)
>   734 |                 info->size = SZ_128M;
>       |                              ^~~~~~~
> drivers/ram/mediatek/ddr3-mt7629.c:734:30: note: each undeclared
> identifier is reported only once for each function it appears in
> drivers/ram/mediatek/ddr3-mt7629.c:737:30: error: 'SZ_256M'
> undeclared (first use in this function)
>   737 |                 info->size = SZ_256M;
>       |                              ^~~~~~~
> drivers/ram/mediatek/ddr3-mt7629.c:740:30: error: 'SZ_512M'
> undeclared (first use in this function)
>   740 |                 info->size = SZ_512M;
>       |                              ^~~~~~~
> drivers/ram/mediatek/ddr3-mt7629.c:743:30: error: 'SZ_1G' undeclared
> (first use in this function)
>   743 |                 info->size = SZ_1G;
>       |                              ^~~~~
> 
> Include <linux/sizes.h> so SZ_* is defined.
> 
> Reported-by: Tianling Shen <cnsztl@immortalwrt.org>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> ---
> 

Reviewed-by: Weijie Gao <weijie.gao@mediatek.com>
diff mbox series

Patch

diff --git a/drivers/ram/mediatek/ddr3-mt7629.c b/drivers/ram/mediatek/ddr3-mt7629.c
index 1737fdac970..f65fcf179cf 100644
--- a/drivers/ram/mediatek/ddr3-mt7629.c
+++ b/drivers/ram/mediatek/ddr3-mt7629.c
@@ -14,6 +14,7 @@ 
 #include <asm/io.h>
 #include <linux/bitops.h>
 #include <linux/delay.h>
+#include <linux/sizes.h>
 
 /* EMI */
 #define EMI_CONA			0x000