diff mbox series

[net-next] net: fec: fix build error for ARCH m68k

Message ID 20210809042921.28931-1-qiangqing.zhang@nxp.com (mailing list archive)
State Accepted
Commit e08d6d42b6f9aaf627df2e0a4c8b1e27ce70fe87
Delegated to: Netdev Maintainers
Headers show
Series [net-next] net: fec: fix build error for ARCH m68k | expand

Checks

Context Check Description
netdev/cover_letter success Link
netdev/fixes_present success Link
netdev/patch_count success Link
netdev/tree_selection success Clearly marked for net-next
netdev/subject_prefix success Link
netdev/cc_maintainers fail 1 blamed authors not CCed: fugang.duan@nxp.com; 1 maintainers not CCed: fugang.duan@nxp.com
netdev/source_inline success Was 0 now: 0
netdev/verify_signedoff success Link
netdev/module_param success Was 0 now: 0
netdev/build_32bit success Errors and warnings before: 0 this patch: 0
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/verify_fixes success Link
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 8 lines checked
netdev/build_allmodconfig_warn success Errors and warnings before: 0 this patch: 0
netdev/header_inline success Link

Commit Message

Joakim Zhang Aug. 9, 2021, 4:29 a.m. UTC 
reproduce:
	wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
	chmod +x ~/bin/make.cross
	make.cross ARCH=m68k  m5272c3_defconfig
	make.cross ARCH=m68k

   drivers/net/ethernet/freescale/fec_main.c: In function 'fec_enet_eee_mode_set':
>> drivers/net/ethernet/freescale/fec_main.c:2758:33: error: 'FEC_LPI_SLEEP' undeclared (first use in this function); did you mean 'FEC_ECR_SLEEP'?
    2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
         |                                 ^~~~~~~~~~~~~
   arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
      25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
         |                                                                  ^~~~
   drivers/net/ethernet/freescale/fec_main.c:2758:2: note: in expansion of macro 'writel'
    2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
         |  ^~~~~~
   drivers/net/ethernet/freescale/fec_main.c:2758:33: note: each undeclared identifier is reported only once for each function it appears in
    2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
         |                                 ^~~~~~~~~~~~~
   arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
      25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
         |                                                                  ^~~~
   drivers/net/ethernet/freescale/fec_main.c:2758:2: note: in expansion of macro 'writel'
    2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
         |  ^~~~~~
>> drivers/net/ethernet/freescale/fec_main.c:2759:32: error: 'FEC_LPI_WAKE' undeclared (first use in this function)
    2759 |  writel(wake_cycle, fep->hwp + FEC_LPI_WAKE);
         |                                ^~~~~~~~~~~~
   arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
      25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
         |                                                                  ^~~~
   drivers/net/ethernet/freescale/fec_main.c:2759:2: note: in expansion of macro 'writel'
    2759 |  writel(wake_cycle, fep->hwp + FEC_LPI_WAKE);
         |  ^~~~~~

This patch adds register definition for M5272 platform to pass build.

Fixes: b82f8c3f1409 ("net: fec: add eee mode tx lpi support")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>
---
 drivers/net/ethernet/freescale/fec.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

patchwork-bot+netdevbpf@kernel.org Aug. 9, 2021, 10:10 a.m. UTC | #1 
Hello:

This patch was applied to netdev/net-next.git (refs/heads/master):

On Mon,  9 Aug 2021 12:29:21 +0800 you wrote:
> reproduce:
> 	wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> 	chmod +x ~/bin/make.cross
> 	make.cross ARCH=m68k  m5272c3_defconfig
> 	make.cross ARCH=m68k
> 
>    drivers/net/ethernet/freescale/fec_main.c: In function 'fec_enet_eee_mode_set':
> >> drivers/net/ethernet/freescale/fec_main.c:2758:33: error: 'FEC_LPI_SLEEP' undeclared (first use in this function); did you mean 'FEC_ECR_SLEEP'?
>     2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
>          |                                 ^~~~~~~~~~~~~
>    arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
>       25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
>          |                                                                  ^~~~
>    drivers/net/ethernet/freescale/fec_main.c:2758:2: note: in expansion of macro 'writel'
>     2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
>          |  ^~~~~~
>    drivers/net/ethernet/freescale/fec_main.c:2758:33: note: each undeclared identifier is reported only once for each function it appears in
>     2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
>          |                                 ^~~~~~~~~~~~~
>    arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
>       25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
>          |                                                                  ^~~~
>    drivers/net/ethernet/freescale/fec_main.c:2758:2: note: in expansion of macro 'writel'
>     2758 |  writel(sleep_cycle, fep->hwp + FEC_LPI_SLEEP);
>          |  ^~~~~~
> >> drivers/net/ethernet/freescale/fec_main.c:2759:32: error: 'FEC_LPI_WAKE' undeclared (first use in this function)
>     2759 |  writel(wake_cycle, fep->hwp + FEC_LPI_WAKE);
>          |                                ^~~~~~~~~~~~
>    arch/m68k/include/asm/io_no.h:25:66: note: in definition of macro '__raw_writel'
>       25 | #define __raw_writel(b, addr) (void)((*(__force volatile u32 *) (addr)) = (b))
>          |                                                                  ^~~~
>    drivers/net/ethernet/freescale/fec_main.c:2759:2: note: in expansion of macro 'writel'
>     2759 |  writel(wake_cycle, fep->hwp + FEC_LPI_WAKE);
>          |  ^~~~~~
> 
> [...]

Here is the summary with links:
  - [net-next] net: fec: fix build error for ARCH m68k
    https://git.kernel.org/netdev/net-next/c/e08d6d42b6f9

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/drivers/net/ethernet/freescale/fec.h b/drivers/net/ethernet/freescale/fec.h
index fe4dfe2d25ea..9d0f5854288a 100644
--- a/drivers/net/ethernet/freescale/fec.h
+++ b/drivers/net/ethernet/freescale/fec.h
@@ -189,6 +189,8 @@ 
 #define FEC_RXIC0		0xfff
 #define FEC_RXIC1		0xfff
 #define FEC_RXIC2		0xfff
+#define FEC_LPI_SLEEP		0xfff
+#define FEC_LPI_WAKE		0xfff
 #endif /* CONFIG_M5272 */