From: rd.dunlab@gmail.com
To: linux-rdma@vger.kernel.org
Cc: Doug Ledford <dledford@redhat.com>,
Jason Gunthorpe <jgg@mellanox.com>,
linux-doc@vger.kernel.org
Cc: Randy Dunlap <rdunlap@infradead.org>
Subject: [PATCH 05/12] infiniband: fix ulp/opa_vnic/opa_vnic_encap.h kernel-doc notation
Date: Wed, 09 Oct 2019 20:52:44 -0700 [thread overview]
Message-ID: <20191010035239.890311169@gmail.com> (raw)
In-Reply-To: 20191010035239.532908118@gmail.com
[-- Attachment #1: 010-iband-opa-vnic-encaphdr.patch --]
[-- Type: text/plain, Size: 5332 bytes --]
Make reserved struct fields "private:" so that they don't need to
be added to the kernel-doc notation. This removes 24 warnings.
Remove "[]" in one struct field description to that kernel-doc
won't be confused.
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:148: warning: Function parameter or member 'rsvd0' not described in 'opa_vesw_info' [5 rsvd members]
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:205: warning: Function parameter or member 'rsvd0' not described in 'opa_per_veswport_info' [4 rsvd members]
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:342: warning: Function parameter or member 'reserved' not described in 'opa_veswport_summary_counters'
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:394: warning: Function parameter or member 'rsvd0' not described in 'opa_veswport_error_counters' [10 rsvd members]
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:460: warning: Function parameter or member 'reserved' not described in 'opa_vnic_vema_mad'
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:485: warning: Function parameter or member 'reserved' not described in 'opa_vnic_notice_attr'
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:500: warning: Function parameter or member 'reserved' not described in 'opa_vnic_vema_mad_trap'
../drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h:263: warning: Function parameter or member 'tbl_entries' not described in 'opa_veswport_mactable'
Signed-off-by: Randy Dunlap <rd.dunlab@gmail.com>
Cc: Randy Dunlap <rdunlap@infradead.org>
Cc: Doug Ledford <dledford@redhat.com>
Cc: Jason Gunthorpe <jgg@mellanox.com>
Cc: linux-doc@vger.kernel.org
---
drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h | 42 ++++++++++++-
1 file changed, 41 insertions(+), 1 deletion(-)
--- linux-next-20191009.orig/drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h
+++ linux-next-20191009/drivers/infiniband/ulp/opa_vnic/opa_vnic_encap.h
@@ -129,21 +129,31 @@ struct opa_vesw_info {
__be16 fabric_id;
__be16 vesw_id;
+ /* private: */
u8 rsvd0[6];
+ /* public: */
__be16 def_port_mask;
+ /* private: */
u8 rsvd1[2];
+ /* public: */
__be16 pkey;
+ /* private: */
u8 rsvd2[4];
+ /* public: */
__be32 u_mcast_dlid;
__be32 u_ucast_dlid[OPA_VESW_MAX_NUM_DEF_PORT];
__be32 rc;
+ /* private: */
u8 rsvd3[56];
+ /* public: */
__be16 eth_mtu;
+ /* private: */
u8 rsvd4[2];
+ /* public: */
} __packed;
/**
@@ -172,7 +182,9 @@ struct opa_per_veswport_info {
__be32 port_num;
u8 eth_link_status;
+ /* private: */
u8 rsvd0[3];
+ /* public: */
u8 base_mac_addr[ETH_ALEN];
u8 config_state;
@@ -181,7 +193,9 @@ struct opa_per_veswport_info {
__be16 max_mac_tbl_ent;
__be16 max_smac_ent;
__be32 mac_tbl_digest;
+ /* private: */
u8 rsvd1[4];
+ /* public: */
__be32 encap_slid;
@@ -195,12 +209,16 @@ struct opa_per_veswport_info {
u8 non_vlan_sc_mc;
u8 non_vlan_vl_mc;
+ /* private: */
u8 rsvd2[48];
+ /* public: */
__be16 uc_macs_gen_count;
__be16 mc_macs_gen_count;
+ /* private: */
u8 rsvd3[8];
+ /* public: */
} __packed;
/**
@@ -239,7 +257,7 @@ struct opa_veswport_mactable_entry {
* @offset: mac table starting offset
* @num_entries: Number of entries to get or set
* @mac_tbl_digest: mac table digest
- * @tbl_entries[]: Array of table entries
+ * @tbl_entries: Array of table entries
*
* The EM sends down this structure in a MAD indicating
* the starting offset in the forwarding table that this
@@ -337,7 +355,9 @@ struct opa_veswport_summary_counters {
__be64 rx_1024_1518;
__be64 rx_1519_max;
+ /* private: */
__be64 reserved[16];
+ /* public: */
} __packed;
/**
@@ -368,28 +388,42 @@ struct opa_veswport_error_counters {
__be64 tx_errors;
__be64 rx_errors;
+ /* private: */
__be64 rsvd0;
+ /* public: */
__be64 tx_smac_filt;
+ /* private: */
__be64 rsvd1;
__be64 rsvd2;
__be64 rsvd3;
+ /* public: */
__be64 tx_dlid_zero;
+ /* private: */
__be64 rsvd4;
+ /* public: */
__be64 tx_logic;
+ /* private: */
__be64 rsvd5;
+ /* public: */
__be64 tx_drop_state;
__be64 rx_bad_veswid;
+ /* private: */
__be64 rsvd6;
+ /* public: */
__be64 rx_runt;
__be64 rx_oversize;
+ /* private: */
__be64 rsvd7;
+ /* public: */
__be64 rx_eth_down;
__be64 rx_drop_state;
__be64 rx_logic;
+ /* private: */
__be64 rsvd8;
__be64 rsvd9[16];
+ /* public: */
} __packed;
/**
@@ -453,7 +487,9 @@ struct opa_veswport_iface_macs {
struct opa_vnic_vema_mad {
struct ib_mad_hdr mad_hdr;
struct ib_rmpp_hdr rmpp_hdr;
+ /* private: */
u8 reserved;
+ /* public: */
u8 oui[3];
u8 data[OPA_VNIC_EMA_DATA];
};
@@ -478,7 +514,9 @@ struct opa_vnic_notice_attr {
__be16 trap_num;
__be16 toggle_count;
__be32 issuer_lid;
+ /* private: */
__be32 reserved;
+ /* public: */
u8 issuer_gid[16];
u8 raw_data[64];
} __packed;
@@ -493,7 +531,9 @@ struct opa_vnic_notice_attr {
struct opa_vnic_vema_mad_trap {
struct ib_mad_hdr mad_hdr;
struct ib_rmpp_hdr rmpp_hdr;
+ /* private: */
u8 reserved;
+ /* public: */
u8 oui[3];
struct opa_vnic_notice_attr notice;
};
next prev parent reply other threads:[~2019-10-10 4:09 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-10 3:52 [PATCH 00/12] infiniband kernel-doc fixes & driver-api/ chapter rd.dunlab
2019-10-10 3:52 ` [PATCH 01/12] infiniband: fix ulp/iser/iscsi_iser.[hc] kernel-doc notation rd.dunlab
2019-10-10 3:52 ` [PATCH 02/12] infiniband: fix core/ipwm_util.h kernel-doc warnings rd.dunlab
2019-10-10 3:52 ` [PATCH 03/12] infiniband: fix ulp/iser/iscsi_iser.h " rd.dunlab
2019-10-10 3:52 ` [PATCH 04/12] infiniband: fix ulp/opa_vnic/opa_vnic_internal.h kernel-doc notation rd.dunlab
2019-10-10 3:52 ` rd.dunlab [this message]
2019-10-22 17:52 ` [PATCH 05/12] infiniband: fix ulp/opa_vnic/opa_vnic_encap.h " Jason Gunthorpe
2019-10-22 19:51 ` Randy Dunlap
2019-10-23 7:26 ` Jani Nikula
2019-10-23 12:03 ` Dennis Dalessandro
2019-10-10 3:52 ` [PATCH 06/12] infiniband: fix ulp/srpt/ib_srpt.h " rd.dunlab
2019-10-23 20:30 ` Bart Van Assche
2019-10-23 20:39 ` Randy Dunlap
2019-10-10 3:52 ` [PATCH 07/12] infiniband: fix core/verbs.c " rd.dunlab
2019-10-10 3:52 ` [PATCH 08/12] infiniband: fix ulp/iser/iser_verbs.c " rd.dunlab
2019-10-10 3:52 ` [PATCH 09/12] infiniband: fix ulp/iser/iser_initiator.c kernel-doc warnings rd.dunlab
2019-10-10 3:52 ` [PATCH 10/12] infiniband: fix core/ kernel-doc notation rd.dunlab
2019-10-10 3:52 ` [PATCH 11/12] infiniband: fix sw/rdmavt/ " rd.dunlab
2019-10-10 3:52 ` [PATCH 12/12] infiniband: add a Documentation driver-api chapter for Infiniband rd.dunlab
2019-10-10 15:45 ` kbuild test robot
2019-10-10 15:45 ` kbuild test robot
2019-10-21 17:07 ` Jason Gunthorpe
2019-10-21 21:57 ` Randy Dunlap
2019-10-21 21:57 ` Randy Dunlap
2019-10-22 18:41 ` [PATCH 00/12] infiniband kernel-doc fixes & driver-api/ chapter Jason Gunthorpe
2019-10-22 22:51 ` Max Gurtovoy
2019-10-23 11:54 ` Dennis Dalessandro
2019-10-23 14:39 ` Jason Gunthorpe
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=20191010035239.890311169@gmail.com \
--to=rd.dunlab@gmail.com \
--cc=dledford@redhat.com \
--cc=jgg@mellanox.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.