On 6/2/22 23:55, Mike Christie wrote:
+static inline char block_pr_type_to_scsi(enum pr_type type)
+{
+       switch (type) {
+       case PR_WRITE_EXCLUSIVE:
+               return 0x01;
+       case PR_EXCLUSIVE_ACCESS:
+               return 0x03;
+       case PR_WRITE_EXCLUSIVE_REG_ONLY:
+               return 0x05;
+       case PR_EXCLUSIVE_ACCESS_REG_ONLY:
+               return 0x06;
+       case PR_WRITE_EXCLUSIVE_ALL_REGS:
+               return 0x07;
+       case PR_EXCLUSIVE_ACCESS_ALL_REGS:
+               return 0x08;
+       default:
+               return 0;
+       }
+};
+
+static inline enum pr_type scsi_pr_type_to_block(char type)
+{
+       switch (type) {
+       case 0x01:
+               return PR_WRITE_EXCLUSIVE;
+       case 0x03:
+               return PR_EXCLUSIVE_ACCESS;
+       case 0x05:
+               return PR_WRITE_EXCLUSIVE_REG_ONLY;
+       case 0x06:
+               return PR_EXCLUSIVE_ACCESS_REG_ONLY;
+       case 0x07:
+               return PR_WRITE_EXCLUSIVE_ALL_REGS;
+       case 0x08:
+               return PR_EXCLUSIVE_ACCESS_ALL_REGS;
+       default:
+               return 0;
+       }
+}

Since 'char' is used above to represent a single byte, please use u8 or uint8_t instead.

Thanks,

Bart.

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Reply via email to