* [PATCH trivial] PCI: convert struct pci_sriov comments to kernel-doc
@ 2014-07-31 14:56 Edward Cree
2014-09-04 4:32 ` Bjorn Helgaas
0 siblings, 1 reply; 2+ messages in thread
From: Edward Cree @ 2014-07-31 14:56 UTC (permalink / raw)
To: linux-pci; +Cc: Bjorn Helgaas
No changes to the field descriptions, just move them.
Signed-off-by: Edward Cree <ecree@solarflare.com>
---
drivers/pci/pci.h | 50 ++++++++++++++++++++++++++++++++++----------------
1 file changed, 34 insertions(+), 16 deletions(-)
diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h
index 0601890..28fcce8 100644
--- a/drivers/pci/pci.h
+++ b/drivers/pci/pci.h
@@ -221,23 +221,41 @@ static inline int pci_ari_enabled(struct pci_bus *bus)
void pci_reassigndev_resource_alignment(struct pci_dev *dev);
void pci_disable_bridge_window(struct pci_dev *dev);
-/* Single Root I/O Virtualization */
+/**
+ * struct pci_sriov - Single Root I/O Virtualization
+ *
+ * @pos: capability position
+ * @nres: number of resources
+ * @cap: SR-IOV Capabilities
+ * @ctrl: SR-IOV Control
+ * @total_VFs: total VFs associated with the PF
+ * @initial_VFs: initial VFs associated with the PF
+ * @num_vfs: number of VFs available
+ * @offset: first VF Routing ID offset
+ * @stride: following VF stride
+ * @pgsz: page size for BAR alignment
+ * @link: Function Dependency Link
+ * @driver_max_VFs:max num VFs driver supports
+ * @dev: lowest numbered PF
+ * @self: this PF
+ * @lock: lock for VF bus
+ */
struct pci_sriov {
- int pos; /* capability position */
- int nres; /* number of resources */
- u32 cap; /* SR-IOV Capabilities */
- u16 ctrl; /* SR-IOV Control */
- u16 total_VFs; /* total VFs associated with the PF */
- u16 initial_VFs; /* initial VFs associated with the PF */
- u16 num_VFs; /* number of VFs available */
- u16 offset; /* first VF Routing ID offset */
- u16 stride; /* following VF stride */
- u32 pgsz; /* page size for BAR alignment */
- u8 link; /* Function Dependency Link */
- u16 driver_max_VFs; /* max num VFs driver supports */
- struct pci_dev *dev; /* lowest numbered PF */
- struct pci_dev *self; /* this PF */
- struct mutex lock; /* lock for VF bus */
+ int pos;
+ int nres;
+ u32 cap;
+ u16 ctrl;
+ u16 total_VFs;
+ u16 initial_VFs;
+ u16 num_VFs;
+ u16 offset;
+ u16 stride;
+ u32 pgsz;
+ u8 link;
+ u16 driver_max_VFs;
+ struct pci_dev *dev;
+ struct pci_dev *self;
+ struct mutex lock;
};
#ifdef CONFIG_PCI_ATS
--
1.7.11.7
The information contained in this message is confidential and is intended for the addressee(s) only. If you have received this message in error, please notify the sender immediately and delete the message. Unless you are an addressee (or authorized to receive for an addressee), you may not use, copy or disclose to anyone this message or any information contained in this message. The unauthorized use, disclosure, copying or alteration of this message is strictly prohibited.
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH trivial] PCI: convert struct pci_sriov comments to kernel-doc
2014-07-31 14:56 [PATCH trivial] PCI: convert struct pci_sriov comments to kernel-doc Edward Cree
@ 2014-09-04 4:32 ` Bjorn Helgaas
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2014-09-04 4:32 UTC (permalink / raw)
To: Edward Cree; +Cc: linux-pci
On Thu, Jul 31, 2014 at 03:56:19PM +0100, Edward Cree wrote:
> No changes to the field descriptions, just move them.
>
> Signed-off-by: Edward Cree <ecree@solarflare.com>
> ---
> drivers/pci/pci.h | 50 ++++++++++++++++++++++++++++++++++----------------
> 1 file changed, 34 insertions(+), 16 deletions(-)
I don't think this is an improvement. It adds 18 lines of code and moves
the description away from the actual field. Does this make a major
improvement in kernel-doc somehow? I have to admit that I only read the
code and never the kernel-doc.
>
> diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h
> index 0601890..28fcce8 100644
> --- a/drivers/pci/pci.h
> +++ b/drivers/pci/pci.h
> @@ -221,23 +221,41 @@ static inline int pci_ari_enabled(struct pci_bus *bus)
> void pci_reassigndev_resource_alignment(struct pci_dev *dev);
> void pci_disable_bridge_window(struct pci_dev *dev);
>
> -/* Single Root I/O Virtualization */
> +/**
> + * struct pci_sriov - Single Root I/O Virtualization
> + *
> + * @pos: capability position
> + * @nres: number of resources
> + * @cap: SR-IOV Capabilities
> + * @ctrl: SR-IOV Control
> + * @total_VFs: total VFs associated with the PF
> + * @initial_VFs: initial VFs associated with the PF
> + * @num_vfs: number of VFs available
> + * @offset: first VF Routing ID offset
> + * @stride: following VF stride
> + * @pgsz: page size for BAR alignment
> + * @link: Function Dependency Link
> + * @driver_max_VFs:max num VFs driver supports
> + * @dev: lowest numbered PF
> + * @self: this PF
> + * @lock: lock for VF bus
> + */
> struct pci_sriov {
> - int pos; /* capability position */
> - int nres; /* number of resources */
> - u32 cap; /* SR-IOV Capabilities */
> - u16 ctrl; /* SR-IOV Control */
> - u16 total_VFs; /* total VFs associated with the PF */
> - u16 initial_VFs; /* initial VFs associated with the PF */
> - u16 num_VFs; /* number of VFs available */
> - u16 offset; /* first VF Routing ID offset */
> - u16 stride; /* following VF stride */
> - u32 pgsz; /* page size for BAR alignment */
> - u8 link; /* Function Dependency Link */
> - u16 driver_max_VFs; /* max num VFs driver supports */
> - struct pci_dev *dev; /* lowest numbered PF */
> - struct pci_dev *self; /* this PF */
> - struct mutex lock; /* lock for VF bus */
> + int pos;
> + int nres;
> + u32 cap;
> + u16 ctrl;
> + u16 total_VFs;
> + u16 initial_VFs;
> + u16 num_VFs;
> + u16 offset;
> + u16 stride;
> + u32 pgsz;
> + u8 link;
> + u16 driver_max_VFs;
> + struct pci_dev *dev;
> + struct pci_dev *self;
> + struct mutex lock;
> };
>
> #ifdef CONFIG_PCI_ATS
> --
> 1.7.11.7
> The information contained in this message is confidential and is intended for the addressee(s) only. If you have received this message in error, please notify the sender immediately and delete the message. Unless you are an addressee (or authorized to receive for an addressee), you may not use, copy or disclose to anyone this message or any information contained in this message. The unauthorized use, disclosure, copying or alteration of this message is strictly prohibited.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-09-04 4:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-31 14:56 [PATCH trivial] PCI: convert struct pci_sriov comments to kernel-doc Edward Cree
2014-09-04 4:32 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).