Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Wed, 9 Jun 2021 12:52:12 +0300 you wrote: > The "hi" variable is a u64 but the qca8k_read() writes to the top 32 > bits of it. That will work on little endian systems but it's a bit > subtle. It's cleaner to make declare "hi" as a u32. We will still need > to cast it when we shift it later on in the function but that's fine. > > Fixes: 7c9896e37807 ("net: dsa: qca8k: check return value of read functions correctly") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > [...] Here is the summary with links: - [1/2,net-next] net: dsa: qca8k: fix an endian bug in qca8k_get_ethtool_stats() https://git.kernel.org/netdev/net-next/c/aa3d020b22cb - [2/2,net-next] net: dsa: qca8k: check the correct variable in qca8k_set_mac_eee() https://git.kernel.org/netdev/net-next/c/3d0167f2a627 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html