Skip to content

Commit 37fcf57

Browse files
committed
cmds: remove nvme_fw_download_seq
nvme-cli implements firmware download with additional checks, making it more robust and safer to use. There are no users of this function, so remove it. Signed-off-by: Daniel Wagner <[email protected]>
1 parent a2a03ed commit 37fcf57

3 files changed

Lines changed: 0 additions & 44 deletions

File tree

libnvme/src/libnvme.ld

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ LIBNVME_3 {
4242
nvme_free_host;
4343
nvme_free_ns;
4444
nvme_free_subsystem;
45-
nvme_fw_download_seq;
4645
nvme_gen_dhchap_key;
4746
nvme_generate_tls_key_identity;
4847
nvme_generate_tls_key_identity_compat;

libnvme/src/nvme/cmds.c

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -15,34 +15,6 @@
1515
#include "cleanup.h"
1616
#include "private.h"
1717

18-
int nvme_fw_download_seq(struct nvme_transport_handle *hdl, bool ish,
19-
__u32 size, __u32 xfer, __u32 offset, void *buf)
20-
{
21-
struct nvme_passthru_cmd cmd;
22-
void *data = buf;
23-
int err = 0;
24-
25-
if (ish && nvme_transport_handle_is_mi(hdl))
26-
nvme_init_mi_cmd_flags(&cmd, ish);
27-
28-
while (size > 0) {
29-
__u32 chunk = min(xfer, size);
30-
31-
err = nvme_init_fw_download(&cmd, data, chunk, offset);
32-
if (err)
33-
break;
34-
err = nvme_submit_admin_passthru(hdl, &cmd);
35-
if (err)
36-
break;
37-
38-
data += chunk;
39-
size -= chunk;
40-
offset += chunk;
41-
}
42-
43-
return err;
44-
}
45-
4618
int nvme_set_etdas(struct nvme_transport_handle *hdl, bool *changed)
4719
{
4820
struct nvme_feat_host_behavior da4;

libnvme/src/nvme/cmds.h

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -5227,21 +5227,6 @@ nvme_init_mi_cmd_flags(struct nvme_passthru_cmd *cmd, bool ish)
52275227
NVME_MI_ADMIN_CFLAGS_ISH_MASK);
52285228
}
52295229

5230-
/**
5231-
* nvme_fw_download_seq() - Firmware download sequence
5232-
* @hdl: Transport handle
5233-
* @ish: Ignore Shutdown (for NVMe-MI command)
5234-
* @size: Total size of the firmware image to transfer
5235-
* @xfer: Maximum size to send with each partial transfer
5236-
* @offset: Starting offset to send with this firmware download
5237-
* @buf: Address of buffer containing all or part of the firmware image.
5238-
*
5239-
* Return: 0 on success, the nvme command status if a response was
5240-
* received (see &enum nvme_status_field) or a negative error otherwise.
5241-
*/
5242-
int nvme_fw_download_seq(struct nvme_transport_handle *hdl, bool ish,
5243-
__u32 size, __u32 xfer, __u32 offset, void *buf);
5244-
52455230
/**
52465231
* nvme_set_etdas() - Set the Extended Telemetry Data Area 4 Supported bit
52475232
* @hdl: Transport handle

0 commit comments

Comments
 (0)