Merge branch 'devel'

Author: kelvin-cao

nvme.c

@@ -153,26 +153,24 @@ int open_global_device(char *dev)
 	int err, fd;
 	char device_str[64];
 	char pdfid_str[64];
-	uint16_t pdfid;
-	uint32_t ns_id;
+	long pdfid;
+	char *end;
 	struct pax_nvme_device *pax;
 	struct rc_nvme_device *rc;
-	int n;
 
 	devicename = basename(dev);
 
 	if (sscanf(dev, "%2049[^@]@%s", pdfid_str, device_str) == 2) {
-		n = sscanf(pdfid_str, "%hxn%d", &pdfid, &ns_id);
-		if (!n) {
-			return -1;
+		errno = 0;
+		pdfid = strtol(pdfid_str, &end, 0);
+		if(*end || errno || (pdfid < 0 || pdfid > 0xffff)) {
+			fprintf(stderr, "Invalid device %s\n", dev);
+			return -ENODEV;
 		}
+
 		pax = malloc(sizeof(struct pax_nvme_device));
 		pax->pdfid = pdfid;
 		pax->device.ops = &pax_ops;
-		if (n == 2) {
-			pax->is_blk = 1;
-			pax->ns_id = ns_id;
-		}
 
 		pax->dev = switchtec_open(device_str);
 		if (!pax->dev) {

plugins/microchip/switchtec-nvme-device.c

@@ -221,18 +221,13 @@ int pax_nvme_submit_io_passthru(int fd, struct nvme_passthru_cmd *cmd)
 
 int pax_is_blk(void)
 {
-	struct pax_nvme_device *pax;
-	pax = to_pax_nvme_device(global_device);
-
-	return pax->is_blk;
+	return 0;
 }
 
 int pax_nvme_get_nsid(int fd)
 {
-	struct pax_nvme_device *pax;
-	pax = to_pax_nvme_device(global_device);
-
-	return pax->ns_id;
+	errno = -ENOTBLK;
+	return -1;
 }
 
 struct nvme_device_ops pax_ops = {

plugins/microchip/switchtec-nvme-device.h

@@ -33,7 +33,6 @@
 struct pax_nvme_device {
 	struct switchtec_dev *dev;
 	uint16_t pdfid;
-	int is_blk;
 	uint32_t ns_id;
 	uint32_t channel_status;
 	struct nvme_device device;