On 11/09/2020 02:23, Jesse Brandeburg wrote: > As part of the W=1 compliation series, these lines all created > warnings about unused variables that were assigned a value. Most > of them are from register reads, but some are just picking up > a return value from a function and never doing anything with it. > > The register reads should be OK, because the current > implementation of readl and friends will always execute even > without an lvalue. > > When it makes sense, just remove the lvalue assignment and the > local. Other times, just remove the offending code, and > occasionally, just mark the variable as maybe unused since it > could be used in an ifdef or debug scenario. > > Only compile tested with W=1 and an allyesconfig with all the > network drivers turned into modules (to try to test all options). > > Signed-off-by: Jesse Brandeburg <jesse.brandeb...@intel.com> For the sfc/falcon part: Acked-by: Edward Cree <ec...@solarflare.com> (it's not the same as the solution I went with for thevery similar code in the Siena driver (../farch.c), but it looks reasonable enough).
- [RFC PATCH net-next v1 00/11] make drivers/net/ethern... Jesse Brandeburg
- [RFC PATCH net-next v1 08/11] drivers/net/ethern... Jesse Brandeburg
- Re: [RFC PATCH net-next v1 08/11] drivers/ne... Jacob Keller
- [RFC PATCH net-next v1 07/11] drivers/net/ethern... Jesse Brandeburg
- [RFC PATCH net-next v1 02/11] i40e: clean up W=1... Jesse Brandeburg
- [RFC PATCH net-next v1 09/11] drivers/net/ethern... Jesse Brandeburg
- [RFC PATCH net-next v1 01/11] i40e: prepare flas... Jesse Brandeburg
- [RFC PATCH net-next v1 06/11] drivers/net/ethern... Jesse Brandeburg
- Re: [RFC PATCH net-next v1 06/11] drivers/ne... Edward Cree
- [RFC PATCH net-next v1 04/11] ixgbe: clean up W=... Jesse Brandeburg
- [RFC PATCH net-next v1 03/11] iavf: clean up W=1... Jesse Brandeburg
- [RFC PATCH net-next v1 10/11] drivers/net/ethern... Jesse Brandeburg
- [RFC PATCH net-next v1 05/11] intel-ethernet: ma... Jesse Brandeburg
- Re: [Intel-wired-lan] [RFC PATCH net-next v1... Vinicius Costa Gomes
- Re: [Intel-wired-lan] [RFC PATCH net-nex... Jesse Brandeburg
- Re: [RFC PATCH net-next v1 00/11] make drivers/n... Jakub Kicinski
- Re: [RFC PATCH net-next v1 00/11] make drive... Jesse Brandeburg
- Re: [RFC PATCH net-next v1 00/11] make d... Jakub Kicinski
- Re: [RFC PATCH net-next v1 00/11] ma... Jesse Brandeburg