* [PATCH] NVMe: removed unused nn var from nvme_dev_add
@ 2015-07-30 9:23 ` Matias Bjørling
0 siblings, 0 replies; 8+ messages in thread
From: Matias Bjørling @ 2015-07-30 9:23 UTC (permalink / raw)
The logic in nvme_dev_add to enumerate namespaces was moved to
nvme_dev_scan. When moved, the nn variable is no longer used. This patch
removes it.
Fixes: a5768aai ("NVMe: Automatic namespace rescan")
Signed-off-by: Matias Bj?rling <m at bjorling.me>
---
drivers/block/nvme-core.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c
index d844ec4..82a3241 100644
--- a/drivers/block/nvme-core.c
+++ b/drivers/block/nvme-core.c
@@ -2344,7 +2344,6 @@ static int nvme_dev_add(struct nvme_dev *dev)
{
struct pci_dev *pdev = to_pci_dev(dev->dev);
int res;
- unsigned nn;
struct nvme_id_ctrl *ctrl;
int shift = NVME_CAP_MPSMIN(readq(&dev->bar->cap)) + 12;
@@ -2354,7 +2353,6 @@ static int nvme_dev_add(struct nvme_dev *dev)
return -EIO;
}
- nn = le32_to_cpup(&ctrl->nn);
dev->oncs = le16_to_cpup(&ctrl->oncs);
dev->abort_limit = ctrl->acl + 1;
dev->vwc = ctrl->vwc;
--
2.1.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH] NVMe: removed unused nn var from nvme_dev_add
@ 2015-07-30 9:23 ` Matias Bjørling
0 siblings, 0 replies; 8+ messages in thread
From: Matias Bjørling @ 2015-07-30 9:23 UTC (permalink / raw)
To: linux-kernel, linux-nvme; +Cc: Matias Bjørling
The logic in nvme_dev_add to enumerate namespaces was moved to
nvme_dev_scan. When moved, the nn variable is no longer used. This patch
removes it.
Fixes: a5768aai ("NVMe: Automatic namespace rescan")
Signed-off-by: Matias Bjørling <m@bjorling.me>
---
drivers/block/nvme-core.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c
index d844ec4..82a3241 100644
--- a/drivers/block/nvme-core.c
+++ b/drivers/block/nvme-core.c
@@ -2344,7 +2344,6 @@ static int nvme_dev_add(struct nvme_dev *dev)
{
struct pci_dev *pdev = to_pci_dev(dev->dev);
int res;
- unsigned nn;
struct nvme_id_ctrl *ctrl;
int shift = NVME_CAP_MPSMIN(readq(&dev->bar->cap)) + 12;
@@ -2354,7 +2353,6 @@ static int nvme_dev_add(struct nvme_dev *dev)
return -EIO;
}
- nn = le32_to_cpup(&ctrl->nn);
dev->oncs = le16_to_cpup(&ctrl->oncs);
dev->abort_limit = ctrl->acl + 1;
dev->vwc = ctrl->vwc;
--
2.1.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH] NVMe: removed unused nn var from nvme_dev_add
2015-07-30 9:23 ` Matias Bjørling
@ 2015-08-04 11:28 ` Christoph Hellwig
-1 siblings, 0 replies; 8+ messages in thread
From: Christoph Hellwig @ 2015-08-04 11:28 UTC (permalink / raw)
Looks good,
Reviewed-by: Christoph Hellwig <hch at lst.de>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-08-18 16:14 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-30 9:23 [PATCH] NVMe: removed unused nn var from nvme_dev_add Matias Bjørling
2015-07-30 9:23 ` Matias Bjørling
2015-08-04 11:28 ` Christoph Hellwig
2015-08-04 11:28 ` Christoph Hellwig
2015-08-17 23:19 ` Matias Bjørling
2015-08-17 23:19 ` Matias Bjørling
2015-08-18 16:14 ` Jens Axboe
2015-08-18 16:14 ` Jens Axboe
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.