From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932197AbdBGThe (ORCPT ); Tue, 7 Feb 2017 14:37:34 -0500 Received: from mout.web.de ([212.227.15.3]:63505 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932081AbdBGTha (ORCPT ); Tue, 7 Feb 2017 14:37:30 -0500 To: linux-input@vger.kernel.org, Benjamin Tissoires , =?UTF-8?Q?Bruno_Pr=c3=a9mont?= , Jiri Kosina Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Subject: [PATCH 00/18] HID: Fine-tuning for several function implementations Message-ID: Date: Tue, 7 Feb 2017 20:36:52 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:b5QEu3TyOMx83aeNKGqeZ8lX2YdwddBkUeT46817kypqw3KLdA9 FOcNyNiqvfWgUTZg6nc8r/X3Uw8c+5FGzO3EWu1tIiAROcmOHsaKvPInDO+aFL0iDaYNxls SLPXIpi7ZOzDhtqLFBPDxqPwuDmnU8jaJz+Tv8N3dmCHUL+GDBogqmHK/CAZF4EHFY+oZPn NKkqvRV0LjrCjYb7qmuow== X-UI-Out-Filterresults: notjunk:1;V01:K0:8GvFrRJcs4M=:xOgKv6ihoPWCq6IJ4gEM09 rsCLWVU26L31ZZWJVjblGqUhssu54MFP5RUr7LfwNiPJYhH1oyg9r+LULbJ6XSOZfCSxe5/Bv G78+PrcnwD2Y/O5gPMkwXHG7HH3GSF1EAvZV1fOGPzoaE4/YiSD6t9dQlAmloQoJ2a8bOeF/M rEnrSIhBXS4Thr1PobqjTVLqsJVDeSxPDgqAyWAXSR3Mmfo98vjg7y2PvCUHChOuu2yyJZUy7 p3/f/ySfA7N8qlVUd4VvbSxatE6K0ANzpk3TDDRbLZugOR11bKdJmDQWZdjg7DKCeL6gakD/9 jZsN5vXId2eokceMEuwvAAOyLMNwO//NEWksAWdGZWqZsUPqTMGu5fwIDdwJZpF1dNcqXoITi b143gBMvwk4sdBDpO9xK3abuSSDpV0hH/2vsV4X1GyexHq+DtcO8I3qqTiJhj379zb622rtHS gCgLGY9htsbmdJ5TAi7bK5whhdHHerJUtxA6MgT4m3x8HDAu5S43EsAQiDC+KSEIFtpRlCfq8 cSNVC8jUpG93dmo6aZzmgqDkl9JcFCh6n9S6RcZ3EHsPFBQfYg1J/bmUsagBoOwKYKvUKK+H5 WxWeYB/3wNZgvtAu1QLwcPs7FMOJ//Bh/3WrtnDsshmumvqrvEp3rCTPjoUcoAsJCUaJS2XuP 3O4Et7jkqxAn1/mJ+9wHsrqfQzVls0ZggNUpqNWtYcRPhrRenZyu89OqYJ3sQM+RCm5G9GaUK uc0NbsXVf4UDp4NzHNS6UBfN0oO07wSmggwVAOzsUNDhPC4tb07+KWBCZajOdPn2JeTO4h1dt OW8OznD Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 7 Feb 2017 20:26:54 +0100 Several update suggestions were taken into account from static source code analysis. Markus Elfring (18): Use kmalloc_array() in hid_input_field() Delete an error message for a failed memory allocation in open_collection() Use kmalloc_array() in open_collection() Improve another size determination in open_collection() Improve two size determinations in hid_open_report() Improve another size determination in hid_scan_report() Improve another size determination in hid_register_report() Adjust five checks for null pointers Return an error code only as a constant in hid_allocate_device() debug: Replace five seq_printf() calls by seq_putc() debug: Delete an unnecessary seq_printf() call in hid_dump_device() debug: Replace 12 seq_printf() calls by seq_puts() debug: Combine 11 seq_printf() calls into one call in hid_dump_field() debug: Adjust two function calls together with a variable assignment debug: Return an error code only as a constant in hid_debug_events_open() debug: Add some spaces for better code readability debug: Delete an unnecessary variable initialisation in hid_resolv_usage() Use seq_puts() in picolcd_debug_reset_show() drivers/hid/hid-core.c | 39 ++++++------- drivers/hid/hid-debug.c | 115 +++++++++++++++++++------------------- drivers/hid/hid-picolcd_debugfs.c | 6 +- 3 files changed, 78 insertions(+), 82 deletions(-) -- 2.11.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Tue, 07 Feb 2017 19:36:52 +0000 Subject: [PATCH 00/18] HID: Fine-tuning for several function implementations Message-Id: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-input@vger.kernel.org, Benjamin Tissoires , =?UTF-8?Q?Bruno_Pr=c3=a9mont?= , Jiri Kosina Cc: LKML , kernel-janitors@vger.kernel.org From: Markus Elfring Date: Tue, 7 Feb 2017 20:26:54 +0100 Several update suggestions were taken into account from static source code analysis. Markus Elfring (18): Use kmalloc_array() in hid_input_field() Delete an error message for a failed memory allocation in open_collection() Use kmalloc_array() in open_collection() Improve another size determination in open_collection() Improve two size determinations in hid_open_report() Improve another size determination in hid_scan_report() Improve another size determination in hid_register_report() Adjust five checks for null pointers Return an error code only as a constant in hid_allocate_device() debug: Replace five seq_printf() calls by seq_putc() debug: Delete an unnecessary seq_printf() call in hid_dump_device() debug: Replace 12 seq_printf() calls by seq_puts() debug: Combine 11 seq_printf() calls into one call in hid_dump_field() debug: Adjust two function calls together with a variable assignment debug: Return an error code only as a constant in hid_debug_events_open() debug: Add some spaces for better code readability debug: Delete an unnecessary variable initialisation in hid_resolv_usage() Use seq_puts() in picolcd_debug_reset_show() drivers/hid/hid-core.c | 39 ++++++------- drivers/hid/hid-debug.c | 115 +++++++++++++++++++------------------- drivers/hid/hid-picolcd_debugfs.c | 6 +- 3 files changed, 78 insertions(+), 82 deletions(-) -- 2.11.1