Skip to content

Commit

Permalink
block/sed-opal: tone down not supported warnings
Browse files Browse the repository at this point in the history
Not having OPAL or a sub-feature supported is an entirely normal
condition for many drives, so don't warn about it.  Keep the messages,
but tone them down to debug only.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Scott Bauer <scott.bauer@intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
  • Loading branch information
Christoph Hellwig authored and axboe committed Feb 17, 2017
1 parent 6732c74 commit f5b37b7
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions block/sed-opal.c
Original file line number Diff line number Diff line change
Expand Up @@ -387,16 +387,16 @@ static int opal_discovery0_end(struct opal_dev *dev)
}

if (!supported) {
pr_err("This device is not Opal enabled. Not Supported!\n");
pr_debug("This device is not Opal enabled. Not Supported!\n");
return -EOPNOTSUPP;
}

if (!single_user)
pr_warn("Device doesn't support single user mode\n");
pr_debug("Device doesn't support single user mode\n");


if (!found_com_id) {
pr_warn("Could not find OPAL comid for device. Returning early\n");
pr_debug("Could not find OPAL comid for device. Returning early\n");
return -EOPNOTSUPP;;
}

Expand Down Expand Up @@ -1951,7 +1951,7 @@ void init_opal_dev(struct opal_dev *opal_dev, sec_send_recv *send_recv)
mutex_init(&opal_dev->dev_lock);
opal_dev->send_recv = send_recv;
if (check_opal_support(opal_dev) < 0)
pr_warn("Opal is not supported on this device\n");
pr_debug("Opal is not supported on this device\n");
opal_dev->initialized = true;
}
EXPORT_SYMBOL(init_opal_dev);
Expand Down

0 comments on commit f5b37b7

Please sign in to comment.