From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oliver Neukum Subject: Re: [PATCH] HID: usbhid: enable remote wakeup for mice Date: Fri, 14 Oct 2011 10:18:28 +0200 Message-ID: <201110141018.28446.oneukum@suse.de> References: <1318578888-7599-1-git-send-email-bleung@chromium.org> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Return-path: Received: from cantor2.suse.de ([195.135.220.15]:45935 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751731Ab1JNISL (ORCPT ); Fri, 14 Oct 2011 04:18:11 -0400 In-Reply-To: <1318578888-7599-1-git-send-email-bleung@chromium.org> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Benson Leung Cc: jkosina@suse.cz, stern@rowland.harvard.edu, linux-usb@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org Am Freitag, 14. Oktober 2011, 09:54:48 schrieb Benson Leung: > This patch enables remote wakeup by default on USB mouse > devices. This only covers USB mice that support > the boot protocol. See commit 3d61510f for the > equivalent patch for USB keyboard devices. This doesn't loo like a very good idea. Firstly your patch introduces gratious change in the hid driver. Secondly you are changing the usbmouse driver, which is to be used under special circumstances only, only. Thirdly a fundamental change like that needs a commit entry explaining the reason for the change. Regards Oliver