From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH v1] input: make use of the input_set_capability helper Date: Tue, 6 Jan 2015 23:57:03 -0800 Message-ID: <20150107075703.GE5256@dtor-ws> References: <1420615866-3528-1-git-send-email-oliver+list@schinagl.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1420615866-3528-1-git-send-email-oliver+list@schinagl.nl> Sender: linux-kernel-owner@vger.kernel.org To: Olliver Schinagl Cc: Wolfram Sang , Paul Gortmaker , Jingoo Han , "David S. Miller" , Sam Ravnborg , Olliver Schinagl , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-input@vger.kernel.org Hi Olliver, On Wed, Jan 07, 2015 at 08:31:06AM +0100, Olliver Schinagl wrote: > From: Olliver Schinagl > > Almost all of the speaker drivers under input manipulate the ev bits > directly, which is not needed, as there is a helper available. > > This patch makes use of the helper for the speaker drivers. > > Signed-off-by: Olliver Schinagl > --- > drivers/input/misc/cm109.c | 4 ++-- > drivers/input/misc/ixp4xx-beeper.c | 5 ++--- > drivers/input/misc/m68kspkr.c | 5 ++--- > drivers/input/misc/pcspkr.c | 5 ++--- > drivers/input/misc/pwm-beeper.c | 5 +---- > drivers/input/misc/sparcspkr.c | 6 ++---- > 6 files changed, 11 insertions(+), 19 deletions(-) > > diff --git a/drivers/input/misc/cm109.c b/drivers/input/misc/cm109.c > index 9365535..8e41070 100644 > --- a/drivers/input/misc/cm109.c > +++ b/drivers/input/misc/cm109.c > @@ -767,10 +767,10 @@ static int cm109_usb_probe(struct usb_interface *intf, > input_dev->keycodesize = sizeof(unsigned char); > input_dev->keycodemax = ARRAY_SIZE(dev->keymap); > > - input_dev->evbit[0] = BIT_MASK(EV_KEY) | BIT_MASK(EV_SND); > - input_dev->sndbit[0] = BIT_MASK(SND_BELL) | BIT_MASK(SND_TONE); > + input_set_capability(input_dev, EV_SND, SND_BELL | SND_TONE); No, input_set_capability() takes single event code, not bitmask. The fact that it works for these 2 values of SND events is pure coincidence (the old code wasn't much better though). > > /* register available key events */ > + input_dev->evbit[0] = BIT_MASK(EV_KEY); Would prefer __set_bit(EV_KEY, input_dev->evbit); here instead. Thanks. -- Dmitry