scsi: ufs: core: Print error value as hex format in ufshcd_err_handler()
authorWonkon Kim <wkon.kim@samsung.com>
Mon, 12 May 2025 02:52:10 +0000 (11:52 +0900)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 13 May 2025 02:21:52 +0000 (22:21 -0400)
It is better to print saved_err and saved_uic_err in hex format.  Integer
format is hard to decode.

[ 1024.485428] [2: kworker/u20:13:28211] exynos-ufs 17100000.ufs: ufshcd_err_handler started; HBA state eh_fatal; powered 1; shutting down 0; saved_err = 131072; saved_uic_err = 0; force_reset = 0; link is broken

Signed-off-by: Wonkon Kim <wkon.kim@samsung.com>
Link: https://lore.kernel.org/r/20250512025210.5802-1-wkon.kim@samsung.com
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/ufs/core/ufshcd.c

index 1c53ccf5a616c096af7b69c81b40fc4ac25d12f0..de30346083c88848e816bcc70c9aacabffe97ea8 100644 (file)
@@ -6584,7 +6584,7 @@ static void ufshcd_err_handler(struct work_struct *work)
        hba = container_of(work, struct ufs_hba, eh_work);
 
        dev_info(hba->dev,
-                "%s started; HBA state %s; powered %d; shutting down %d; saved_err = %d; saved_uic_err = %d; force_reset = %d%s\n",
+                "%s started; HBA state %s; powered %d; shutting down %d; saved_err = 0x%x; saved_uic_err = 0x%x; force_reset = %d%s\n",
                 __func__, ufshcd_state_name[hba->ufshcd_state],
                 hba->is_powered, hba->shutting_down, hba->saved_err,
                 hba->saved_uic_err, hba->force_reset,