<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/Documentation/networking/devlink/ice.rst, branch linux-5.12.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.12.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.12.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2020-10-28T17:41:15Z</updated>
<entry>
<title>ice: docs fix a devlink info that broke a table</title>
<updated>2020-10-28T17:41:15Z</updated>
<author>
<name>Mauro Carvalho Chehab</name>
</author>
<published>2020-10-27T09:51:20Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=4b7560c5e4bda24fcbd54337c87056833819c73b'/>
<id>urn:sha1:4b7560c5e4bda24fcbd54337c87056833819c73b</id>
<content type='text'>
Changeset 410d06879c01 ("ice: add the DDP Track ID to devlink info")
added description for a new devlink field, but forgot to add
one of its columns, causing it to break:

	.../Documentation/networking/devlink/ice.rst:15: WARNING: Error parsing content block for the "list-table" directive: uniform two-level bullet list expected, but row 11 does not contain the same number of items as row 1 (3 vs 4).

	.. list-table:: devlink info versions implemented
	    :widths: 5 5 5 90
...
	    * - ``fw.app.bundle_id``
	      - 0xc0000001
	      - Unique identifier for the DDP package loaded in the device. Also
	        referred to as the DDP Track ID. Can be used to uniquely identify
	        the specific DDP package.

Add the type field to the ``fw.app.bundle_id`` row.

Fixes: 410d06879c01 ("ice: add the DDP Track ID to devlink info")
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+huawei@kernel.org&gt;
Reviewed-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Link: https://lore.kernel.org/r/84ae28bda1987284033966b7b56a4b27ae40713b.1603791716.git.mchehab+huawei@kernel.org
Signed-off-by: Jonathan Corbet &lt;corbet@lwn.net&gt;
</content>
</entry>
<entry>
<title>ice: add the DDP Track ID to devlink info</title>
<updated>2020-10-09T20:14:19Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-10-07T17:54:43Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=410d06879c01751e6fe6e9aebc2e0d9100a71d77'/>
<id>urn:sha1:410d06879c01751e6fe6e9aebc2e0d9100a71d77</id>
<content type='text'>
Add "fw.app.bundle_id" to display the DDP Track ID of the active DDP
package. This id is similar to "fw.bundle_id" and is a unique identifier
for the DDP package that is loaded in the device. Each new DDP has
a unique Track ID generated for it, and the ID can be used to identify
and track the DDP package.

Add documentation for the new devlink info version.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Tested-by: Aaron Brown &lt;aaron.f.brown@intel.com&gt;
Signed-off-by: Tony Nguyen &lt;anthony.l.nguyen@intel.com&gt;
Signed-off-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
</content>
</entry>
<entry>
<title>ice: add support for flash update overwrite mask</title>
<updated>2020-09-26T00:20:57Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-09-25T20:46:09Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=50db1bca55208d7f0528a1fa0619ce5d406a8d40'/>
<id>urn:sha1:50db1bca55208d7f0528a1fa0619ce5d406a8d40</id>
<content type='text'>
Support the recently added DEVLINK_ATTR_FLASH_UPDATE_OVERWRITE_MASK
parameter in the ice flash update handler. Convert the overwrite mask
bitfield into the appropriate preservation level used by the firmware
when updating.

Because there is no equivalent preservation level for overwriting only
identifiers, this combination is rejected by the driver as not supported
with an appropriate extended ACK message.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Reviewed-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ice: add documentation for device-caps region</title>
<updated>2020-07-07T19:41:01Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-07-06T21:53:41Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=14f5d8e3bb4cffce4590c04b5d9c6dc1fc397f85'/>
<id>urn:sha1:14f5d8e3bb4cffce4590c04b5d9c6dc1fc397f85</id>
<content type='text'>
The recent change by commit 8d7aab3515fa ("ice: implement snapshot for
device capabilities") to implement the device-caps region for the ice
driver forgot to document it.

Add documentation to the ice devlink documentation file describing the
new region and add some sample output to the shell commands provided as
an example.

Fixes: 8d7aab3515fa ("ice: implement snapshot for device capabilities")
Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ice: report netlist version in .info_get</title>
<updated>2020-05-22T05:10:03Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-05-05T22:55:37Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=f45a645fa6af37abb5484a8d8be779283a38fb53'/>
<id>urn:sha1:f45a645fa6af37abb5484a8d8be779283a38fb53</id>
<content type='text'>
The flash memory for the ice hardware contains a block of information
used for link management called the Netlist module.

As this essentially represents another section of firmware, add its
version information to the output of the driver's .info_get handler.

This includes both a version and the first few bytes of a hash of the
module contents.

  fw.netlist -&gt; the version information extracted from the netlist module
  fw.netlist.build-&gt; first 4 bytes of the hash of the contents, similar
                     to fw.mgmt.build

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>ice: cleanup language in ice.rst for fw.app</title>
<updated>2020-05-01T22:30:24Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-04-29T20:59:50Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=709e7158f0991ec26b2819ef944c276a19b910db'/>
<id>urn:sha1:709e7158f0991ec26b2819ef944c276a19b910db</id>
<content type='text'>
The documentation for the ice driver around "fw.app" has a spelling
mistake in variation. Additionally, the language of "shall have a unique
name" sounds like a requirement. Reword this to read more like
a description or property.

Reported-by: Benjamin Fisher &lt;benjamin.l.fisher@intel.com&gt;
Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Acked-by: Jakub Kicinski &lt;kubakici@wp.pl&gt;
Acked-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>devlink: don't wrap commands in rST shell blocks</title>
<updated>2020-03-30T05:08:45Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-03-27T20:55:36Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=9c11cc784963597eb2796b544bfd98bfb5612c2b'/>
<id>urn:sha1:9c11cc784963597eb2796b544bfd98bfb5612c2b</id>
<content type='text'>
The devlink-region.rst and ice-region.rst documentation files wrapped
some lines within shell code blocks due to being longer than 80 lines.

It was pointed out during review that wrapping these lines shouldn't be
done. Fix these two rST files and remove the line wrapping on these
shell command examples.

Reported-by: Jiri Pirko &lt;jiri@resnulli.us&gt;
Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ice: add a devlink region for dumping NVM contents</title>
<updated>2020-03-27T02:39:26Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-03-26T18:37:18Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=dce730f17825f4814649c1ba7970af5737415c78'/>
<id>urn:sha1:dce730f17825f4814649c1ba7970af5737415c78</id>
<content type='text'>
Add a devlink region for exposing the device's Non Volatime Memory flash
contents.

Support the recently added .snapshot operation, enabling userspace to
request a snapshot of the NVM contents via DEVLINK_CMD_REGION_NEW.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Reviewed-by: Jiri Pirko &lt;jiri@mellanox.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ice: add board identifier info to devlink .info_get</title>
<updated>2020-03-21T08:02:19Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-03-12T01:58:18Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=e961b679fb0b25004f5a57db0ccb34256391646b'/>
<id>urn:sha1:e961b679fb0b25004f5a57db0ccb34256391646b</id>
<content type='text'>
Export a unique board identifier using "board.id" for devlink's
.info_get command.

Obtain this by reading the NVM for the PBA identification string.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Reviewed-by: Jesse Brandeburg &lt;jesse.brandeburg@intel.com&gt;
Acked-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>ice: add basic handler for devlink .info_get</title>
<updated>2020-03-21T08:00:32Z</updated>
<author>
<name>Jacob Keller</name>
</author>
<published>2020-03-12T01:58:17Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ff2e5c700e085f4f9357e128c3fe9eb61e9df752'/>
<id>urn:sha1:ff2e5c700e085f4f9357e128c3fe9eb61e9df752</id>
<content type='text'>
The devlink .info_get callback allows the driver to report detailed
version information. The following devlink versions are reported with
this initial implementation:

 "fw.mgmt" -&gt; The version of the firmware that controls PHY, link, etc
 "fw.mgmt.api" -&gt; API version of interface exposed over the AdminQ
 "fw.mgmt.build" -&gt; Unique build id of the source for the management fw
 "fw.undi" -&gt; Version of the Option ROM containing the UEFI driver
 "fw.psid.api" -&gt; Version of the NVM image format.
 "fw.bundle_id" -&gt; Unique identifier for the combined flash image.
 "fw.app.name" -&gt; The name of the active DDP package.
 "fw.app" -&gt; The version of the active DDP package.

With this, devlink dev info can report at least as much information as
is reported by ETHTOOL_GDRVINFO.

Compare the output from ethtool vs from devlink:

  $ ethtool -i ens785s0
  driver: ice
  version: 0.8.1-k
  firmware-version: 0.80 0x80002ec0 1.2581.0
  expansion-rom-version:
  bus-info: 0000:3b:00.0
  supports-statistics: yes
  supports-test: yes
  supports-eeprom-access: yes
  supports-register-dump: yes
  supports-priv-flags: yes

  $ devlink dev info pci/0000:3b:00.0
  pci/0000:3b:00.0:
  driver ice
  serial number 00-01-ab-ff-ff-ca-05-68
  versions:
      running:
        fw.mgmt 2.1.7
        fw.mgmt.api 1.5
        fw.mgmt.build 0x305d955f
        fw.undi 1.2581.0
        fw.psid.api 0.80
        fw.bundle_id 0x80002ec0
        fw.app.name ICE OS Default Package
        fw.app 1.3.1.0

More pieces of information can be displayed, each version is kept
separate instead of munged together, and each version has an identifier
which comes with associated documentation.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Acked-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
</feed>
