mbox series

[net,v3,0/4] net: ipa: minor bug fixes

Message ID 20230316145136.1795469-1-elder@linaro.org
Headers show
Series net: ipa: minor bug fixes | expand

Message

Alex Elder March 16, 2023, 2:51 p.m. UTC 
The four patches in this series fix some errors, though none of them
cause any compile or runtime problems.

The first changes the files included by "drivers/net/ipa/reg.h" to
ensure everything it requires is included with the file.  It also
stops unnecessarily including another file.  The prerequisites are
apparently satisfied other ways, currently.

The second adds two struct declarations to "gsi_reg.h", to ensure
they're declared before they're used later in the file.  Again, it
seems these declarations are currently resolved wherever this file
is included.

The third removes register definitions that were added for IPA v5.0
that are not needed.  And the last updates some validity checks for
IPA v5.0 registers.  No IPA v5.0 platforms are yet supported, so the
issues resolved here were never harmful.

Versions 2 and 3 of this series change the "Fixes" tags in patches
so they supply legitimate commit hashes.

					-Alex

Alex Elder (4):
  net: ipa: reg: include <linux/bug.h>
  net: ipa: add two missing declarations
  net: ipa: kill FILT_ROUT_CACHE_CFG IPA register
  net: ipa: fix some register validity checks

 drivers/net/ipa/gsi_reg.c |  9 ++++++++-
 drivers/net/ipa/gsi_reg.h |  4 ++++
 drivers/net/ipa/ipa_reg.c | 28 ++++++++++++++++++----------
 drivers/net/ipa/ipa_reg.h | 21 ++++++---------------
 drivers/net/ipa/reg.h     |  3 ++-
 5 files changed, 38 insertions(+), 27 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org March 17, 2023, 4:40 a.m. UTC | #1 
Hello:

This series was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu, 16 Mar 2023 09:51:32 -0500 you wrote:
> The four patches in this series fix some errors, though none of them
> cause any compile or runtime problems.
> 
> The first changes the files included by "drivers/net/ipa/reg.h" to
> ensure everything it requires is included with the file.  It also
> stops unnecessarily including another file.  The prerequisites are
> apparently satisfied other ways, currently.
> 
> [...]

Here is the summary with links:
  - [net,v3,1/4] net: ipa: reg: include <linux/bug.h>
    https://git.kernel.org/netdev/net/c/dd172d0c2cea
  - [net,v3,2/4] net: ipa: add two missing declarations
    https://git.kernel.org/netdev/net/c/55c49e5c9441
  - [net,v3,3/4] net: ipa: kill FILT_ROUT_CACHE_CFG IPA register
    https://git.kernel.org/netdev/net/c/786bbe50e1d5
  - [net,v3,4/4] net: ipa: fix some register validity checks
    https://git.kernel.org/netdev/net/c/21e8aaca401c

You are awesome, thank you!