On 25.08.2023 16:29, Andrew Lunn wrote:

quoted

quoted

+static bool nxp_c45_rx_sc_valid(struct nxp_c45_secy *phy_secy,
+				struct macsec_rx_sc *rx_sc)
+{
+	u16 port =  (__force u64)rx_sc->sci >> (ETH_ALEN * 8);

u64 sci = be64_to_cpu((__force __be64)rx_sc->sci);

why is the __force needed? What happens with a normal cast?

Sparse will print warnings if __force is missing.

     Andrew

-- 
Radu P.