From: Alex Elder <elder@linaro.org>
To: davem@davemloft.net, edumazet@google.com, kuba@kernel.org,
pabeni@redhat.com
Cc: mka@chromium.org, andersson@kernel.org,
quic_cpratapa@quicinc.com, quic_avuyyuru@quicinc.com,
quic_jponduru@quicinc.com, quic_subashab@quicinc.com,
elder@kernel.org, netdev@vger.kernel.org,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH net-next 0/7] net: ipa: header hygiene
Date: Tue, 16 Apr 2024 18:10:11 -0500 [thread overview]
Message-ID: <20240416231018.389520-1-elder@linaro.org> (raw)
The end result of this series is that the list of files included in
every IPA source file will be maintained in sorted order. This
imposes some consistency that was previously not possible.
If an IPA header file requires a symbol or type declared in another
header, that other header must be included. E.g., if bool or u32
type is used in a function declaration in an IPA header file, the
IPA header must include <linux/types.h>.
If a type used is just a struct or union *pointer* or enum type (and
no members within these types are needed), then these types only need
to be *declared* within the header that uses it.
This is sufficient, but in addition, this series removes includes of
files that aren't necessary, as well as unneeded type declarations.
-Alex
Alex Elder (7):
net: ipa: include some standard header files
net: ipa: remove unneeded standard includes
net: ipa: include "ipa_interrupt.h" where needed
net: ipa: add some needed struct declarations
net: ipa: eliminate unneeded struct declarations
net: ipa: more include file cleanup
net: ipa: sort all includes
drivers/net/ipa/data/ipa_data-v3.1.c | 5 +++--
drivers/net/ipa/data/ipa_data-v3.5.1.c | 5 +++--
drivers/net/ipa/data/ipa_data-v4.11.c | 5 +++--
drivers/net/ipa/data/ipa_data-v4.2.c | 5 +++--
drivers/net/ipa/data/ipa_data-v4.5.c | 5 +++--
drivers/net/ipa/data/ipa_data-v4.7.c | 5 +++--
drivers/net/ipa/data/ipa_data-v4.9.c | 5 +++--
drivers/net/ipa/data/ipa_data-v5.0.c | 5 +++--
drivers/net/ipa/data/ipa_data-v5.5.c | 5 +++--
drivers/net/ipa/gsi.c | 18 ++++++++---------
drivers/net/ipa/gsi.h | 10 +++-------
drivers/net/ipa/gsi_private.h | 7 ++++---
drivers/net/ipa/gsi_reg.c | 6 +++---
drivers/net/ipa/gsi_trans.c | 12 ++++++------
drivers/net/ipa/gsi_trans.h | 9 ++++-----
drivers/net/ipa/ipa.h | 15 +++++---------
drivers/net/ipa/ipa_cmd.c | 13 +++++++------
drivers/net/ipa/ipa_cmd.h | 10 +++-------
drivers/net/ipa/ipa_data.h | 4 ++--
drivers/net/ipa/ipa_endpoint.c | 19 ++++++++++--------
drivers/net/ipa/ipa_endpoint.h | 4 ++--
drivers/net/ipa/ipa_gsi.c | 7 ++++---
drivers/net/ipa/ipa_interrupt.c | 10 +++++-----
drivers/net/ipa/ipa_interrupt.h | 6 ++++--
drivers/net/ipa/ipa_main.c | 27 +++++++++++++-------------
drivers/net/ipa/ipa_mem.c | 15 +++++++-------
drivers/net/ipa/ipa_mem.h | 4 +++-
drivers/net/ipa/ipa_modem.c | 14 ++++++-------
drivers/net/ipa/ipa_modem.h | 5 +++--
drivers/net/ipa/ipa_power.c | 8 ++++----
drivers/net/ipa/ipa_power.h | 5 +++--
drivers/net/ipa/ipa_qmi.c | 10 +++-------
drivers/net/ipa/ipa_qmi.h | 4 +++-
drivers/net/ipa/ipa_qmi_msg.c | 3 ++-
drivers/net/ipa/ipa_qmi_msg.h | 3 ++-
drivers/net/ipa/ipa_reg.c | 4 ++--
drivers/net/ipa/ipa_reg.h | 6 +-----
drivers/net/ipa/ipa_resource.c | 3 +--
drivers/net/ipa/ipa_smp2p.c | 10 +++++-----
drivers/net/ipa/ipa_sysfs.c | 7 +++----
drivers/net/ipa/ipa_sysfs.h | 4 +---
drivers/net/ipa/ipa_table.c | 19 ++++++++----------
drivers/net/ipa/ipa_uc.c | 10 ++++++----
drivers/net/ipa/ipa_uc.h | 3 +--
drivers/net/ipa/ipa_version.h | 4 +++-
drivers/net/ipa/reg.h | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v3.1.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v3.5.1.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v4.0.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v4.11.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v4.5.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v4.9.c | 8 +++++---
drivers/net/ipa/reg/gsi_reg-v5.0.c | 8 +++++---
drivers/net/ipa/reg/ipa_reg-v3.1.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v3.5.1.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v4.11.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v4.2.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v4.5.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v4.7.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v4.9.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v5.0.c | 6 ++++--
drivers/net/ipa/reg/ipa_reg-v5.5.c | 6 +++---
62 files changed, 253 insertions(+), 228 deletions(-)
--
2.40.1
next reply other threads:[~2024-04-16 23:10 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-16 23:10 Alex Elder [this message]
2024-04-16 23:10 ` [PATCH net-next 1/7] net: ipa: include some standard header files Alex Elder
2024-04-16 23:10 ` [PATCH net-next 2/7] net: ipa: remove unneeded standard includes Alex Elder
2024-04-16 23:10 ` [PATCH net-next 3/7] net: ipa: include "ipa_interrupt.h" where needed Alex Elder
2024-04-16 23:10 ` [PATCH net-next 4/7] net: ipa: add some needed struct declarations Alex Elder
2024-04-16 23:10 ` [PATCH net-next 5/7] net: ipa: eliminate unneeded " Alex Elder
2024-04-16 23:10 ` [PATCH net-next 6/7] net: ipa: more include file cleanup Alex Elder
2024-04-16 23:10 ` [PATCH net-next 7/7] net: ipa: sort all includes Alex Elder
2024-04-18 11:10 ` [PATCH net-next 0/7] net: ipa: header hygiene patchwork-bot+netdevbpf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240416231018.389520-1-elder@linaro.org \
--to=elder@linaro.org \
--cc=andersson@kernel.org \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=elder@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mka@chromium.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=quic_avuyyuru@quicinc.com \
--cc=quic_cpratapa@quicinc.com \
--cc=quic_jponduru@quicinc.com \
--cc=quic_subashab@quicinc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox