Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753131AbdF2NGS (ORCPT ); Thu, 29 Jun 2017 09:06:18 -0400 Received: from mail-pg0-f68.google.com ([74.125.83.68]:34470 "EHLO mail-pg0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753082AbdF2NGK (ORCPT ); Thu, 29 Jun 2017 09:06:10 -0400 From: Jaya Durga To: gregkh@linuxfoundation.org Cc: sergio.paracuellos@gmail.com, juliana.orod@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Jaya Durga , Jaya Durga Subject: [PATCH v2 6/6] Staging: wlan-ng: hfa384x.h:Fix use of volatile is usually wrong Date: Thu, 29 Jun 2017 18:31:00 +0530 Message-Id: <1498741260-23676-7-git-send-email-rjdurga@gmail.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 898 Lines: 27 Fix checkpatch.pl issue WARNING: Use of volatile is usually wrong: see Documentation/process/volatile-considered-harmful.rst The variables in the private data that are marked volatile don't need to be. Remove the volatile. Signed-off-by: Jaya Durga --- drivers/staging/wlan-ng/hfa384x.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h index 310e2c4..015945f 100644 --- a/drivers/staging/wlan-ng/hfa384x.h +++ b/drivers/staging/wlan-ng/hfa384x.h @@ -1175,7 +1175,7 @@ struct hfa384x_usbctlx { enum ctlx_state state; /* Tracks running state */ struct completion done; - volatile int reapable; /* Food for the reaper task */ + int reapable; /* Food for the reaper task */ ctlx_cmdcb_t cmdcb; /* Async command callback */ ctlx_usercb_t usercb; /* Async user callback, */ -- 1.9.1