[net-next,0/6] net: ipa: a mix of cleanups

Message

Alex Elder Sept. 10, 2022, 1:11 a.m. UTC

This series contains a set of cleanups done in preparation for a
more substantitive upcoming series that reworks how IPA registers
and their fields are defined.

The first eliminates about half of the possible GSI register
constant symbols by removing offset definitions that are not
currently required.

The next two mainly rearrange code for some common enumerated types.

The next one fixes two spots that reuse local variable names in
inner scopes when defining offsets.

The next adds some additional restrictions on the value held in a
register.

And the last one just fixes two field mask symbol names so they
adhere to the common naming convention.

					-Alex

Alex Elder (6):
  net: ipa: don't define unneeded GSI register offsets
  net: ipa: move the definition of gsi_ee_id
  net: ipa: move and redefine ipa_version_valid()
  net: ipa: don't reuse variable names
  net: ipa: update sequencer definition constraints
  net: ipa: fix two symbol names

 drivers/net/ipa/gsi.h          |   8 --
 drivers/net/ipa/gsi_reg.h      | 208 +++++++++------------------------
 drivers/net/ipa/ipa_endpoint.c |  44 ++++---
 drivers/net/ipa/ipa_main.c     |  25 +---
 drivers/net/ipa/ipa_reg.h      |   5 +-
 drivers/net/ipa/ipa_version.h  |  28 ++++-
 6 files changed, 110 insertions(+), 208 deletions(-)

Comments

[e-mail] Sept. 20, 2022, 2:50 p.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <[e-mail]>:

On Fri,  9 Sep 2022 20:11:25 -0500 you wrote:
> This series contains a set of cleanups done in preparation for a
> more substantitive upcoming series that reworks how IPA registers
> and their fields are defined.
> 
> The first eliminates about half of the possible GSI register
> constant symbols by removing offset definitions that are not
> currently required.
> 
> [...]

Here is the summary with links:
  - [net-next,1/6] net: ipa: don't define unneeded GSI register offsets
    https://git.kernel.org/netdev/net-next/c/5ea4285829de
  - [net-next,2/6] net: ipa: move the definition of gsi_ee_id
    https://git.kernel.org/netdev/net-next/c/bb788de30a74
  - [net-next,3/6] net: ipa: move and redefine ipa_version_valid()
    https://git.kernel.org/netdev/net-next/c/8b3cb084b23e
  - [net-next,4/6] net: ipa: don't reuse variable names
    https://git.kernel.org/netdev/net-next/c/9eefd2fb966d
  - [net-next,5/6] net: ipa: update sequencer definition constraints
    https://git.kernel.org/netdev/net-next/c/a14d593724c4
  - [net-next,6/6] net: ipa: fix two symbol names
    https://git.kernel.org/netdev/net-next/c/dae4af6bf232

You are awesome, thank you!