Re: [PATCH v2 2/3] net: ftmac100: simplify priv->iobase casting

Tom Rini
2023-02-02T22:05:34Z

[-- Attachment #1: Type: text/plain, Size: 1069 bytes --]

On Thu, Jan 19, 2023 at 10:51:15PM +0300, Sergei Antonov wrote:

> Replace 'phys_addr_t iobase' with 'struct ftmac100 *ftmac100'
> in order to cast once on assignment and remove casting in a
> number of other places.
>
> Signed-off-by: Sergei Antonov <saproj@gmail.com>
> Reviewed-by: Ramon Fried <rfried.dev@gmail.com>

This introduces:
riscv32: + ae350_rv32
+(ae350_rv32) drivers/net/ftmac100.c: In function 'ftmac100_of_to_plat':
+(ae350_rv32) drivers/net/ftmac100.c:317:26: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
+(ae350_rv32) 317 | priv->ftmac100 = (struct ftmac100 *)pdata->iobase;
+(ae350_rv32) | ^
+(ae350_rv32) cc1: all warnings being treated as errors
+(ae350_rv32) make[3]: *** [scripts/Makefile.build:257: drivers/net/ftmac100.o] Error 1
+(ae350_rv32) make[2]: *** [scripts/Makefile.build:397: drivers/net] Error 2
+(ae350_rv32) make[1]: *** [Makefile:1857: drivers] Error 2
+(ae350_rv32) make: *** [Makefile:177: sub-make] Error 2

--
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]