From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Qiming Yang <qiming.yang@intel.com>
Cc: dev@dpdk.org, ferruh.yigit@intel.com, remy.horton@intel.com
Subject: Re: [PATCH v8 1/5] ethdev: add firmware version get
Date: Sun, 15 Jan 2017 21:56:27 +0100 [thread overview]
Message-ID: <2137227.KlaXuaYADV@xps13> (raw)
In-Reply-To: <1484202716-41669-2-git-send-email-qiming.yang@intel.com>
2017-01-12 14:31, Qiming Yang:
> + * @param fw_size
> + * The size of the array pointed by fw_version, which should be
> + * large enough to store firmware version of the device.
> + * @return
> + * - (0) if successful.
> + * - (-ENOTSUP) if operation is not supported.
> + * - (-ENODEV) if *port_id* invalid.
> + * - (-EINVAL) if *fw_size* is not enough to store firmware version.
> + */
> +int rte_eth_dev_fw_version_get(uint8_t port_id,
> + char *fw_version, size_t fw_size);
> +
How to know the required size?
You should return the size of the non truncated string as snprintf does.
Note: Andrew Rybchenko already commented on it and you didn't reply.
next prev parent reply other threads:[~2017-01-15 20:56 UTC|newest]
Thread overview: 128+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-17 9:42 [PATCH] e1000: add firmware version get Qiming Yang
2016-11-17 9:42 ` [PATCH 1/5] ethdev: " Qiming Yang
2016-11-17 13:36 ` Thomas Monjalon
2016-11-18 2:10 ` Yang, Qiming
2016-11-18 1:09 ` Remy Horton
2016-11-18 2:18 ` Yang, Qiming
2016-12-06 7:16 ` [PATCH v2 0/5] example/ethtool: add bus info and fw " Qiming Yang
2016-12-06 7:16 ` [PATCH v2 1/5] ethdev: add firmware " Qiming Yang
2016-12-08 11:07 ` Ferruh Yigit
2016-12-12 1:28 ` Yang, Qiming
2016-12-06 7:16 ` [PATCH v2 2/5] net/e1000: " Qiming Yang
2016-12-07 1:16 ` Lu, Wenzhuo
2016-12-06 7:16 ` [PATCH v2 3/5] net/ixgbe: " Qiming Yang
2016-12-06 7:16 ` [PATCH v2 4/5] net/i40e: " Qiming Yang
2016-12-06 7:16 ` [PATCH v2 5/5] ethtool: dispaly bus info and firmware version Qiming Yang
2016-12-23 12:50 ` Ferruh Yigit
2016-12-27 13:06 ` [PATCH v3] " Qiming Yang
2017-01-04 7:51 ` Wu, Jingjing
2017-01-04 12:18 ` [PATCH v4] ethtool: dispaly bus information Qiming Yang
2017-01-04 14:49 ` Mcnamara, John
2017-01-05 1:51 ` Yang, Qiming
2017-02-09 21:32 ` Thomas Monjalon
2016-12-08 8:34 ` [PATCH v2 0/5] example/ethtool: add bus info and fw version get Remy Horton
2016-12-12 1:43 ` Yang, Qiming
2016-12-22 11:07 ` Thomas Monjalon
2016-12-22 14:36 ` Ferruh Yigit
2016-12-22 14:47 ` Thomas Monjalon
2016-12-22 15:05 ` Ferruh Yigit
2016-12-22 15:31 ` Thomas Monjalon
2016-12-23 12:48 ` Ferruh Yigit
2017-01-05 3:04 ` Zhang, Helin
2016-12-27 12:30 ` [PATCH v3 0/4] new API 'rte_eth_dev_fw_info_get' Qiming Yang
2016-12-27 12:30 ` [PATCH v3 1/4] ethdev: add firmware information get Qiming Yang
2017-01-02 15:38 ` Thomas Monjalon
[not found] ` <F5DF4F0E3AFEF648ADC1C3C33AD4DBF16EDC9515@SHSMSX101.ccr.corp.intel.com>
[not found] ` <10603884.vrshqR2O82@xps13>
2017-01-03 9:05 ` Yang, Qiming
2017-01-03 14:49 ` Ferruh Yigit
2017-01-04 3:33 ` Yang, Qiming
2017-01-04 7:48 ` Wu, Jingjing
2017-01-04 8:43 ` Ferruh Yigit
2017-01-05 1:04 ` Wu, Jingjing
2017-01-03 14:58 ` Ferruh Yigit
2016-12-27 12:30 ` [PATCH v3 2/4] net/e1000: add firmware version get Qiming Yang
2017-01-03 15:02 ` Ferruh Yigit
2017-01-04 3:14 ` Yang, Qiming
2017-01-04 8:47 ` Ferruh Yigit
2016-12-27 12:30 ` [PATCH v3 3/4] net/ixgbe: " Qiming Yang
2017-01-03 15:04 ` Ferruh Yigit
2017-01-04 2:44 ` Yang, Qiming
2017-01-04 9:06 ` Ferruh Yigit
2017-01-04 9:48 ` Yang, Qiming
2017-01-04 12:01 ` Ferruh Yigit
2016-12-27 12:30 ` [PATCH v3 4/4] net/i40e: " Qiming Yang
2017-01-04 12:03 ` [PATCH v4 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-04 12:03 ` [PATCH v4 1/5] ethdev: add firmware version get Qiming Yang
2017-01-05 13:44 ` Thomas Monjalon
2017-01-08 3:09 ` Yang, Qiming
2017-01-04 12:03 ` [PATCH v4 2/5] net/e1000: " Qiming Yang
2017-01-04 13:59 ` Ferruh Yigit
2017-01-05 1:50 ` Yang, Qiming
2017-01-04 12:03 ` [PATCH v4 3/5] net/ixgbe: " Qiming Yang
2017-01-04 12:03 ` [PATCH v4 4/5] net/i40e: " Qiming Yang
2017-01-04 14:00 ` Ferruh Yigit
2017-01-04 12:03 ` [PATCH v4 5/5] ethtool: dispaly firmware version Qiming Yang
2017-01-04 14:00 ` Ferruh Yigit
2017-01-05 1:31 ` Yang, Qiming
2017-01-06 15:55 ` Remy Horton
2017-01-08 4:11 ` [PATCH v5 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-08 4:11 ` [PATCH v5 1/5] ethdev: add firmware version get Qiming Yang
2017-01-08 6:38 ` Andrew Rybchenko
2017-01-08 23:05 ` Stephen Hemminger
2017-01-09 7:16 ` Yang, Qiming
2017-01-09 10:01 ` Remy Horton
2017-01-09 17:23 ` Stephen Hemminger
2017-01-08 4:11 ` [PATCH v5 2/5] net/e1000: " Qiming Yang
2017-01-08 23:03 ` Stephen Hemminger
2017-01-09 1:48 ` Yang, Qiming
2017-01-08 4:11 ` [PATCH v5 3/5] net/ixgbe: " Qiming Yang
2017-01-08 4:11 ` [PATCH v5 4/5] net/i40e: " Qiming Yang
2017-01-08 23:08 ` Stephen Hemminger
2017-01-08 4:11 ` [PATCH v5 5/5] ethtool: display firmware version Qiming Yang
2017-01-08 23:11 ` Stephen Hemminger
2017-01-10 9:00 ` [DPDK 1/5] ethdev: add firmware version get Qiming Yang
2017-01-10 9:00 ` [DPDK 2/5] net/e1000: " Qiming Yang
2017-01-10 9:00 ` [DPDK 3/5] net/ixgbe: " Qiming Yang
2017-01-10 9:08 ` [PATCH v6 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-10 9:08 ` [PATCH v6 1/5] ethdev: add firmware version get Qiming Yang
2017-01-11 6:41 ` [PATCH v7 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-11 6:41 ` [PATCH v7 1/5] ethdev: add firmware version get Qiming Yang
2017-01-11 6:41 ` [PATCH v7 2/5] net/e1000: " Qiming Yang
2017-01-11 15:45 ` Remy Horton
2017-01-12 1:25 ` Yang, Qiming
2017-01-11 6:41 ` [PATCH v7 3/5] net/ixgbe: " Qiming Yang
2017-01-11 6:41 ` [PATCH v7 4/5] net/i40e: " Qiming Yang
2017-01-11 6:41 ` [PATCH v7 5/5] ethtool: display firmware version Qiming Yang
2017-01-12 6:31 ` [PATCH v8 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-12 6:31 ` [PATCH v8 1/5] ethdev: add firmware version get Qiming Yang
2017-01-15 20:56 ` Thomas Monjalon [this message]
2017-01-16 5:44 ` [PATCH v9 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-16 5:44 ` [PATCH v9 1/5] ethdev: add firmware version get Qiming Yang
2017-01-16 7:05 ` Andrew Rybchenko
2017-01-16 8:51 ` Yang, Qiming
2017-01-16 5:44 ` [PATCH v9 2/5] net/e1000: " Qiming Yang
2017-01-16 5:44 ` [PATCH v9 3/5] net/ixgbe: " Qiming Yang
2017-01-16 5:44 ` [PATCH v9 4/5] net/i40e: " Qiming Yang
2017-01-16 5:44 ` [PATCH v9 5/5] ethtool: display firmware version Qiming Yang
2017-01-16 10:48 ` [PATCH v10 0/5] new API 'rte_eth_dev_fw_version_get' Qiming Yang
2017-01-16 10:48 ` [PATCH v10 1/5] ethdev: add firmware version get Qiming Yang
2017-01-16 10:48 ` [PATCH v10 2/5] net/e1000: " Qiming Yang
2017-01-16 10:48 ` [PATCH v10 3/5] net/ixgbe: " Qiming Yang
2017-01-16 10:48 ` [PATCH v10 4/5] net/i40e: " Qiming Yang
2017-01-16 10:48 ` [PATCH v10 5/5] ethtool: display firmware version Qiming Yang
2017-01-17 21:35 ` [PATCH v10 0/5] new API 'rte_eth_dev_fw_version_get' Thomas Monjalon
2017-01-12 6:31 ` [PATCH v8 2/5] net/e1000: add firmware version get Qiming Yang
2017-01-12 6:31 ` [PATCH v8 3/5] net/ixgbe: " Qiming Yang
2017-01-12 6:31 ` [PATCH v8 4/5] net/i40e: " Qiming Yang
2017-01-12 6:31 ` [PATCH v8 5/5] ethtool: display firmware version Qiming Yang
2017-01-10 9:08 ` [PATCH v6 2/5] net/e1000: add firmware version get Qiming Yang
2017-01-10 9:08 ` [PATCH v6 3/5] net/ixgbe: " Qiming Yang
2017-01-10 9:08 ` [PATCH v6 4/5] net/i40e: " Qiming Yang
2017-01-10 9:08 ` [PATCH v6 5/5] ethtool: display firmware version Qiming Yang
2016-11-17 9:42 ` [PATCH] ethtool: dispaly bus info and " Qiming Yang
2016-11-17 9:42 ` [PATCH] i40e: add firmware version get Qiming Yang
2016-11-17 9:42 ` [PATCH] ixgbe: " Qiming Yang
2016-11-17 9:42 ` [PATCH 2/5] e1000: " Qiming Yang
2016-11-17 9:42 ` [PATCH 3/5] ixgbe: " Qiming Yang
2016-11-17 9:42 ` [PATCH 4/5] i40e: " Qiming Yang
2016-11-17 9:42 ` [PATCH 5/5] ethtool: dispaly bus info and firmware version Qiming Yang
2016-11-18 1:10 ` Remy Horton
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=2137227.KlaXuaYADV@xps13 \
--to=thomas.monjalon@6wind.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=qiming.yang@intel.com \
--cc=remy.horton@intel.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 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.