mbox series

[net-next,v2,0/3] net: ethernet: ti: Address some warnings

Message ID 20241010-ti-warn-v2-0-9c8304af5544@kernel.org (mailing list archive)
Headers show
Series net: ethernet: ti: Address some warnings | expand

Message

Simon Horman Oct. 10, 2024, 11:04 a.m. UTC
Hi,

This patchset addresses some warnings flagged by Sparse, and clang-18 in
TI Ethernet drivers.

Although these changes do not alter the functionality of the code, by
addressing them real problems introduced in future which are flagged by
tooling will stand out more readily.

Compile tested only.

---
Changes in v2:
- Dropped patch to directly address __percpu Sparse warnings and, instead
- Add patch to use tstats
- Added tags
- Thanks to all for the review of v1
- Link to v1: https://lore.kernel.org/r/20240910-ti-warn-v1-0-afd1e404abbe@kernel.org

---
Simon Horman (3):
      net: ethernet: ti: am65-cpsw: Use __be64 type for id_temp
      net: ethernet: ti: am65-cpsw: Use tstats instead of open coded version
      net: ethernet: ti: cpsw_ale: Remove unused accessor functions

 drivers/net/ethernet/ti/am65-cpsw-nuss.c | 94 +++-----------------------------
 drivers/net/ethernet/ti/am65-cpsw-nuss.h |  9 ---
 drivers/net/ethernet/ti/cpsw_ale.c       | 30 +++++++---
 3 files changed, 30 insertions(+), 103 deletions(-)

base-commit: 09cf85ef183a5603db49d542264ddbece3258e55

Comments

patchwork-bot+netdevbpf@kernel.org Oct. 14, 2024, 12:30 p.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <davem@davemloft.net>:

On Thu, 10 Oct 2024 12:04:09 +0100 you wrote:
> Hi,
> 
> This patchset addresses some warnings flagged by Sparse, and clang-18 in
> TI Ethernet drivers.
> 
> Although these changes do not alter the functionality of the code, by
> addressing them real problems introduced in future which are flagged by
> tooling will stand out more readily.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/3] net: ethernet: ti: am65-cpsw: Use __be64 type for id_temp
    https://git.kernel.org/netdev/net-next/c/5c16e118b796
  - [net-next,v2,2/3] net: ethernet: ti: am65-cpsw: Use tstats instead of open coded version
    https://git.kernel.org/netdev/net-next/c/4a7b2ba94a59
  - [net-next,v2,3/3] net: ethernet: ti: cpsw_ale: Remove unused accessor functions
    https://git.kernel.org/netdev/net-next/c/2c9eacbb56de

You are awesome, thank you!