diff mbox series

[iproute2-next] devlink: Extend man page for port function set command

Message ID 20201130164712.571540-1-parav@nvidia.com
State Superseded
Headers show
Series [iproute2-next] devlink: Extend man page for port function set command | expand

Commit Message

Parav Pandit Nov. 30, 2020, 4:47 p.m. UTC
Extended devlink-port man page for synopsis, description and
example for setting devlink port function attribute.

Signed-off-by: Parav Pandit <parav@nvidia.com>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
---
 man/man8/devlink-port.8 | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

David Ahern Dec. 3, 2020, 3:41 a.m. UTC | #1
On 11/30/20 9:47 AM, Parav Pandit wrote:
> Extended devlink-port man page for synopsis, description and
> example for setting devlink port function attribute.
> 
> Signed-off-by: Parav Pandit <parav@nvidia.com>
> Reviewed-by: Jiri Pirko <jiri@nvidia.com>
> ---
>  man/man8/devlink-port.8 | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/man/man8/devlink-port.8 b/man/man8/devlink-port.8
> index 966faae6..7e2dc110 100644
> --- a/man/man8/devlink-port.8
> +++ b/man/man8/devlink-port.8
> @@ -46,6 +46,12 @@ devlink-port \- devlink port configuration
>  .ti -8
>  .B devlink port help
>  
> +.ti -8
> +.BR "devlink port function set "

The command exists in iproute2 main branch correct?
Parav Pandit Dec. 3, 2020, 6:33 a.m. UTC | #2
> From: David Ahern <dsahern@gmail.com>
> Sent: Thursday, December 3, 2020 9:12 AM
> 
> On 11/30/20 9:47 AM, Parav Pandit wrote:
> > Extended devlink-port man page for synopsis, description and example
> > for setting devlink port function attribute.
> >
> > Signed-off-by: Parav Pandit <parav@nvidia.com>
> > Reviewed-by: Jiri Pirko <jiri@nvidia.com>
> > ---
> >  man/man8/devlink-port.8 | 21 +++++++++++++++++++++
> >  1 file changed, 21 insertions(+)
> >
> > diff --git a/man/man8/devlink-port.8 b/man/man8/devlink-port.8 index
> > 966faae6..7e2dc110 100644
> > --- a/man/man8/devlink-port.8
> > +++ b/man/man8/devlink-port.8
> > @@ -46,6 +46,12 @@ devlink-port \- devlink port configuration  .ti -8
> > .B devlink port help
> >
> > +.ti -8
> > +.BR "devlink port function set "
> 
> The command exists in iproute2 main branch correct?
Yes, you are correct.
diff mbox series

Patch

diff --git a/man/man8/devlink-port.8 b/man/man8/devlink-port.8
index 966faae6..7e2dc110 100644
--- a/man/man8/devlink-port.8
+++ b/man/man8/devlink-port.8
@@ -46,6 +46,12 @@  devlink-port \- devlink port configuration
 .ti -8
 .B devlink port help
 
+.ti -8
+.BR "devlink port function set "
+.RI "[ "
+.BR "hw_addr "
+.RI "ADDR ]"
+
 .SH "DESCRIPTION"
 .SS devlink port set - change devlink port attributes
 
@@ -99,6 +105,16 @@  If this argument is omitted all ports are listed.
 Is an alias for
 .BR devlink-health (8).
 
+.SS devlink port function set - Set the port function attribute.
+
+.PP
+.B "DEV/PORT_INDEX"
+- specifies the devlink port to operate on.
+
+.PP
+.BI hw_addr " ADDR"
+- hardware address of the function to set.
+
 .SH "EXAMPLES"
 .PP
 devlink port show
@@ -135,6 +151,11 @@  devlink port health show pci/0000:01:00.0/1 reporter tx
 .RS 4
 Shows status and configuration of tx reporter registered on pci/0000:01:00.0/1 devlink port.
 .RE
+.PP
+devlink port function set pci/0000:01:00.0/1 hw_addr 00:00:00:11:22:33
+.RS 4
+Configure hardware address of the PCI function represented by devlink port.
+.RE
 
 .SH SEE ALSO
 .BR devlink (8),