diff options
author | John Baldwin <jhb@FreeBSD.org> | 2023-08-01 22:20:53 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2023-08-01 22:20:53 +0000 |
commit | 32734eb14c5cc3aa4ad576ba746830c73fb8eb54 (patch) | |
tree | 553fc789b8e85ac3476f2491569797529e0b33b8 /sys/cam/mmc/mmc_xpt.c | |
parent | 76b2e3907cede6456897d2f950911d45ae627d8b (diff) | |
download | src-32734eb14c5cc3aa4ad576ba746830c73fb8eb54.tar.gz src-32734eb14c5cc3aa4ad576ba746830c73fb8eb54.zip |
mmc_xpt: Remove dubious end of mmc_print_ident
The end of this function finishes the passed in sbuf, calls printf
manually on the contents, and then clears it. The caller then tries
to print the resulting sbuf. This works currently but will not work
for future callers that pass in an external sbuf to be appended to.
Reviewed by: mav
Sponsored by: Chelsio Communications
Differential Revision: https://reviews.freebsd.org/D41260
Diffstat (limited to 'sys/cam/mmc/mmc_xpt.c')
-rw-r--r-- | sys/cam/mmc/mmc_xpt.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sys/cam/mmc/mmc_xpt.c b/sys/cam/mmc/mmc_xpt.c index 30b1b7f867e7..8fee7b4cad58 100644 --- a/sys/cam/mmc/mmc_xpt.c +++ b/sys/cam/mmc/mmc_xpt.c @@ -466,10 +466,6 @@ mmc_print_ident(struct mmc_params *ident_data, struct sbuf *sb) sbuf_printf(sb, "Number of functions: %u\n", ident_data->sdio_func_count); } - - sbuf_finish(sb); - printf("%s", sbuf_data(sb)); - sbuf_clear(sb); } static void |