Re: [PATCH v2 38/40] s390: scsi: use 16-bits defined sense codes

On 9/7/26 22:20, Hannes Reinecke wrote:

On 9/3/26 5:41 AM, Damien Le Moal wrote:

quoted

Use the 16-bits sense codes and replace all hard-coded additional sense
codes and additional sense code qualifiers with the enum values defined
in include/scsi/scsi_sense.h. This helps with code clarity as the sense
codes being processed are easier to test and self-documented.

No functional change intended.

Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
---
  drivers/s390/scsi/zfcp_scsi.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

Reviewed-by: Hannes Reinecke <hare@kernel.org>

Hannes,

Thanks a lot for the review. But I posted a v5 of the series yesterday... There
are a few minor tweaks in v3, v4 and v5 compared to v2, mostly to address
sashiko review comments. If that is OK with you, I will carry your review tags
to v6 (need one more correction), unless you want to review that version.

-- 
Damien Le Moal
Western Digital Research