firmware loader: remove unnecessary wmb()
The wmb() inside fw_load_abort is not necessary, since complete() and wait_on_completion() has implied one pair of memory barrier. Also wmb() isn't a correct usage, so just remove it. Signed-off-by: Ming Lei <ming.lei@canonical.com> Cc: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
28eefa750b
commit
0c25a850f7
|
@ -112,7 +112,6 @@ static struct firmware_priv *to_firmware_priv(struct device *dev)
|
|||
static void fw_load_abort(struct firmware_priv *fw_priv)
|
||||
{
|
||||
set_bit(FW_STATUS_ABORT, &fw_priv->status);
|
||||
wmb();
|
||||
complete(&fw_priv->completion);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue