Skip to content

Commit d1a2859

Browse files
davejiangvinodkoul
authored andcommitted
dmaengine: idxd: make idxd_wq_enable() return 0 if wq is already enabled
When calling idxd_wq_enable() and wq is already enabled, code should return 0 and indicate function is successful instead of return error code and fail. This should also put idxd_wq_enable() in sync with idxd_wq_disable() where it returns 0 if wq is already disabled. Signed-off-by: Dave Jiang <dave.jiang@intel.com> Link: https://lore.kernel.org/r/165090980906.1378449.1939401700832432886.stgit@djiang5-desk3.ch.intel.com Signed-off-by: Vinod Koul <vkoul@kernel.org>
1 parent 8292a15 commit d1a2859

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/dma/idxd/device.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ int idxd_wq_enable(struct idxd_wq *wq)
184184

185185
if (wq->state == IDXD_WQ_ENABLED) {
186186
dev_dbg(dev, "WQ %d already enabled\n", wq->id);
187-
return -ENXIO;
187+
return 0;
188188
}
189189

190190
idxd_cmd_exec(idxd, IDXD_CMD_ENABLE_WQ, wq->id, &status);

0 commit comments

Comments
 (0)