From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nestor Lopez Casado Subject: [PATCH 2/2] HID: hid-logitech-dj, querying_devices was never set Date: Thu, 18 Jul 2013 06:21:31 -0700 Message-ID: <1374153691-25100-2-git-send-email-nlopezcasad@logitech.com> References: <1374153691-25100-1-git-send-email-nlopezcasad@logitech.com> Return-path: In-Reply-To: <1374153691-25100-1-git-send-email-nlopezcasad@logitech.com> Sender: linux-kernel-owner@vger.kernel.org To: jkosina@suse.cz Cc: benjamin.tissoires@gmail.com, adlr@chromium.org, joseph.salisbury@canonical.com, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Nestor Lopez Casado List-Id: linux-input@vger.kernel.org Set querying_devices flag to true when we start the enumeration process. This was missing from the original patch. It never produced undesirable effects as it is highly improbable to have a second enumeration triggered while a first one was still in progress. Signed-off-by: Nestor Lopez Casado --- drivers/hid/hid-logitech-dj.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-logitech-dj.c index 0d13389..d4657a5 100644 --- a/drivers/hid/hid-logitech-dj.c +++ b/drivers/hid/hid-logitech-dj.c @@ -488,6 +488,8 @@ static int logi_dj_recv_query_paired_devices(struct dj_receiver_dev *djrcv_dev) if (djrcv_dev->querying_devices) return 0; + djrcv_dev->querying_devices = true; + dj_report = kzalloc(sizeof(struct dj_report), GFP_KERNEL); if (!dj_report) return -ENOMEM; -- 1.7.9.5