From: Aristo Chen <aristo.chen@canonical.com>
To: u-boot@lists.u-boot-project.org
Cc: trini@konsulko.com, Aristo Chen <aristo.chen@canonical.com>,
David Lechner <dlechner@baylibre.com>
Subject: [PATCH v1 2/6] scsi: drop the duplicate COMPARE and SEND DIAGNOSTIC opcodes
Date: Fri, 2 Oct 2026 07:31:52 +0000 [thread overview]
Message-ID: <20261002073357.382228-3-aristo.chen@canonical.com> (raw)
In-Reply-To: <20261002073357.382228-1-aristo.chen@canonical.com>
SCSI_COMPARE and SCSI_SEND_DIAG are defined in the table of commands
for all device types and again in the table of commands unique to
direct access devices. SCSI-2 (table 31) lists both as commands for all
device types, so drop the copies from the second table.
The two SCSI_SEND_DIAG copies also disagreed on the command type: the
first said optional, the second mandatory. SCSI-2 names SEND DIAGNOSTIC
as one of the four commands every target implements, together with
INQUIRY, REQUEST SENSE and TEST UNIT READY, so the remaining comment now
says mandatory, as the other three do.
No functional change.
Signed-off-by: Aristo Chen <aristo.chen@canonical.com>
---
include/scsi.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/include/scsi.h b/include/scsi.h
index 2520a8b8fe6..93561749b7f 100644
--- a/include/scsi.h
+++ b/include/scsi.h
@@ -144,13 +144,12 @@ struct scsi_cmd {
#define SCSI_MODE_SEN10 0x5A /* Mode Sense 10-byte (Device Specific) */
#define SCSI_READ_BUFF 0x3C /* Read Buffer (O) */
#define SCSI_REQ_SENSE 0x03 /* Request Sense (MANDATORY) */
-#define SCSI_SEND_DIAG 0x1D /* Send Diagnostic (O) */
+#define SCSI_SEND_DIAG 0x1D /* Send Diagnostic (MANDATORY) */
#define SCSI_TST_U_RDY 0x00 /* Test Unit Ready (MANDATORY) */
#define SCSI_WRITE_BUFF 0x3B /* Write Buffer (O) */
/***************************************************************************
* %%% Commands Unique to Direct Access Devices %%%
***************************************************************************/
-#define SCSI_COMPARE 0x39 /* Compare (O) */
#define SCSI_FORMAT 0x04 /* Format Unit (MANDATORY) */
#define SCSI_LCK_UN_CAC 0x36 /* Lock Unlock Cache (O) */
#define SCSI_PREFETCH 0x34 /* Prefetch (O) */
@@ -172,7 +171,6 @@ struct scsi_cmd {
#define SCSI_SRCH_DAT_L 0x32 /* Search Data Low (O) */
#define SCSI_SEEK6 0x0B /* Seek 6-Byte (O) */
#define SCSI_SEEK10 0x2B /* Seek 10-Byte (O) */
-#define SCSI_SEND_DIAG 0x1D /* Send Diagnostics (MANDATORY) */
#define SCSI_SET_LIMIT 0x33 /* Set Limits (O) */
#define SCSI_START_STP 0x1B /* Start/Stop Unit (O) */
#define SCSI_SYNC_CACHE 0x35 /* Synchronize Cache (O) */
--
2.43.0
next prev parent reply other threads:[~2026-10-02 12:26 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-02 7:31 [PATCH v1 0/6] Drop duplicate macro definitions Aristo Chen
2026-10-02 7:31 ` [PATCH v1 1/6] mtd: spi-nor: drop the duplicate EN4B/EX4B opcode definitions Aristo Chen
2026-10-02 7:43 ` Takahiro.Kuwano
2026-10-03 13:25 ` Simon Glass
2026-10-02 7:31 ` Aristo Chen [this message]
2026-10-03 13:25 ` [PATCH v1 2/6] scsi: drop the duplicate COMPARE and SEND DIAGNOSTIC opcodes Simon Glass
2026-10-02 7:31 ` [PATCH v1 3/6] acpi: drop the duplicate ACPI_PPTT_CACHE_TYPE_DATA definition Aristo Chen
2026-10-03 13:42 ` Simon Glass
2026-10-02 7:31 ` [PATCH v1 4/6] pci: gt64120: drop the duplicate GT_PCI1M1LD/HD_OFS definitions Aristo Chen
2026-10-03 13:40 ` Simon Glass
2026-10-02 7:31 ` [PATCH v1 5/6] dt-bindings: soc: imx8_pd: drop the duplicate PD_LVDS0_PWM definition Aristo Chen
2026-10-03 13:28 ` Simon Glass
2026-10-02 7:31 ` [PATCH v1 6/6] test: video: define WHITE and SCROLL_LINES once at file scope Aristo Chen
2026-10-03 5:40 ` Simon Glass
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=20261002073357.382228-3-aristo.chen@canonical.com \
--to=aristo.chen@canonical.com \
--cc=dlechner@baylibre.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.u-boot-project.org \
/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.