On Wed, Jul 17, 2024 at 02:29:01PM +0200, Philip Oberfichtner wrote:

> This patch series implements the dwc_eth_qos glue driver for Intel SOCs.
> Before doing that, a few general adaptions to the dwc_eth_qos.c main
> driver are required. Most notably, the preparation for PCI based driver
> instances, which do not necessarily use a device tree.
> 
> This patch series depends on: "net: dwc_eth_qos: mdio: Implement clause 45":
> https://patchwork.ozlabs.org/project/uboot/patch/20240507094237.168238-1-...@denx.de/
> 
> Changes in V3:
>       - Replace mfence() with mb()
>       - Clean-up eqos_get_base_addr()
>       - Several style fixes for dwc_eth_qos_intel

This adds a failure to build on stm32mp15_dhcor_basic and all of the
other stm32mp15 platforms that use this driver:
+(stm32mp15_dhcor_basic) drivers/net/dwc_eth_qos.c: In function 
'eqos_get_base_addr_common':
+(stm32mp15_dhcor_basic) drivers/net/dwc_eth_qos.c:1415:30: error: format 
'%llx' expects argument of type 'long long unsigned int', but argument 2 has 
type 'fdt_addr_t' {aka 'unsigned int'} [-Werror=format=]
+(stm32mp15_dhcor_basic)  1415 |                 dev_err(dev, "addr=0x%llx is 
invalid.\n", addr);
+(stm32mp15_dhcor_basic)       |                              
^~~~~~~~~~~~~~~~~~~~~~~~~~~  ~~~~
+(stm32mp15_dhcor_basic)       |                                                
           |
+(stm32mp15_dhcor_basic)       |                                                
           fdt_addr_t {aka unsigned int}
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:51:24: note: in definition 
of macro 'dev_printk_emit'
+(stm32mp15_dhcor_basic)    51 |                 printf(fmt, ##__VA_ARGS__); \
+(stm32mp15_dhcor_basic)       |                        ^~~
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:110:9: note: in expansion 
of macro '__dev_printk'
+(stm32mp15_dhcor_basic)   110 |         __dev_printk(LOGL_ERR, dev, fmt, 
##__VA_ARGS__)
+(stm32mp15_dhcor_basic)       |         ^~~~~~~~~~~~
+(stm32mp15_dhcor_basic) drivers/net/dwc_eth_qos.c:1415:17: note: in expansion 
of macro 'dev_err'
+(stm32mp15_dhcor_basic)       |                 ^~~~~~~
+(stm32mp15_dhcor_basic) drivers/net/dwc_eth_qos.c:1415:41: note: format string 
is defined here
+(stm32mp15_dhcor_basic)       |                                      ~~~^
+(stm32mp15_dhcor_basic)       |                                         |
+(stm32mp15_dhcor_basic)       |                                         long 
long unsigned int
+(stm32mp15_dhcor_basic)       |                                      %x
+(stm32mp15_dhcor_basic) In file included from include/linux/printk.h:4,
+(stm32mp15_dhcor_basic)                  from include/linux/kernel.h:5,
+(stm32mp15_dhcor_basic)                  from include/linux/libfdt_env.h:13,
+(stm32mp15_dhcor_basic)                  from include/linux/libfdt.h:6,
+(stm32mp15_dhcor_basic)                  from include/fdtdec.h:17,
+(stm32mp15_dhcor_basic)                  from include/dm/ofnode.h:11,
+(stm32mp15_dhcor_basic)                  from include/clk.h:11,
+(stm32mp15_dhcor_basic)                  from drivers/net/dwc_eth_qos.c:32:
+(stm32mp15_dhcor_basic) include/log.h:224:24: note: in definition of macro 
'log'
+(stm32mp15_dhcor_basic)   224 |                 printf(_fmt, ##_args); \
+(stm32mp15_dhcor_basic)       |                        ^~~~
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:86:17: note: in expansion 
of macro 'dev_printk_emit'
+(stm32mp15_dhcor_basic)    86 |                 dev_printk_emit(LOG_CATEGORY, 
level, fmt, ##__VA_ARGS__); \
+(stm32mp15_dhcor_basic)       |                 ^~~~~~~~~~~~~~~
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:55:24: note: in definition 
of macro 'dev_printk_emit'
+(stm32mp15_dhcor_basic)    55 |                 printf(fmt, ##__VA_ARGS__); \
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:92:41: error: format '%llx' 
expects argument of type 'long long unsigned int', but argument 4 has type 
'fdt_addr_t' {aka 'unsigned int'} [-Werror=format=]
+(stm32mp15_dhcor_basic)    92 |                                         "%s 
%s: " fmt, \
+(stm32mp15_dhcor_basic)       |                                         
^~~~~~~~~
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:91:25: note: in expansion 
of macro 'dev_printk_emit'
+(stm32mp15_dhcor_basic)    91 |                         
dev_printk_emit(__dev->driver->id, level, \
+(stm32mp15_dhcor_basic)       |                         ^~~~~~~~~~~~~~~
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:97:41: error: format '%llx' 
expects argument of type 'long long unsigned int', but argument 4 has type 
'fdt_addr_t' {aka 'unsigned int'} [-Werror=format=]
+(stm32mp15_dhcor_basic)    97 |                                         "%s 
%s: " fmt, \
+(stm32mp15_dhcor_basic) include/dm/device_compat.h:96:25: note: in expansion 
of macro 'dev_printk_emit'
+(stm32mp15_dhcor_basic)    96 |                         
dev_printk_emit(LOG_CATEGORY, level, \
+(stm32mp15_dhcor_basic) cc1: all warnings being treated as errors
+(stm32mp15_dhcor_basic) make[3]: *** [scripts/Makefile.build:256: 
drivers/net/dwc_eth_qos.o] Error 1
+(stm32mp15_dhcor_basic) make[2]: *** [scripts/Makefile.build:397: drivers/net] 
Error 2
+(stm32mp15_dhcor_basic) make[1]: *** [Makefile:1895: drivers] Error 2
+(stm32mp15_dhcor_basic) make: *** [Makefile:177: sub-make] Error 2

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to