Re: [PATCH net-next 4/5] net: phy: broadcom: Add support for WAKE_FILTER

[Florian Fainelli]

On 10/26/23 14:56, Jakub Kicinski wrote:
> On Thu, 26 Oct 2023 10:55:10 -0700 Florian Fainelli wrote:
> > > > Also, did you run parse to check your endianness conversions?
> >
> > I did, though nothing came out with C=1 or C=2, I might have to check
> > that separately.
>
> FWIW
>
> drivers/net/phy/bcm-phy-lib.c:1128:42: warning: cast to restricted __be16
> drivers/net/phy/bcm-phy-lib.c:1128:40: warning: incorrect type in assignment (different base types)
> drivers/net/phy/bcm-phy-lib.c:1128:40:    expected restricted __be16 [usertype] h_proto
> drivers/net/phy/bcm-phy-lib.c:1128:40:    got unsigned short [usertype]
> drivers/net/phy/bcm-phy-lib.c:1188:17: warning: incorrect type in assignment (different base types)
> drivers/net/phy/bcm-phy-lib.c:1188:17:    expected restricted __be16 [usertype] h_proto
> drivers/net/phy/bcm-phy-lib.c:1188:17:    got unsigned short [usertype]
> drivers/net/phy/bcm-phy-lib.c:1190:25: warning: incorrect type in assignment (different base types)
> drivers/net/phy/bcm-phy-lib.c:1190:25:    expected restricted __be16 [usertype] h_proto
> drivers/net/phy/bcm-phy-lib.c:1190:25:    got int
> drivers/net/phy/bcm-phy-lib.c:1193:33: warning: incorrect type in argument 3 (different base types)
> drivers/net/phy/bcm-phy-lib.c:1193:33:    expected unsigned short [usertype] val
> drivers/net/phy/bcm-phy-lib.c:1193:33:    got restricted __be16 [usertype] h_proto

Yep, finally able to see that with an x86 build, not sure yet why sparse 
did not work with my aarch64 build, now fixed, thanks!
--
Florian

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature