[PATCH] firmware-gsmi: Delete an error message for a failed memory allocation in gsmi_buf_alloc()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 22:22:02 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] firmware-gsmi: Delete an error message for a failed memory allocation in gsmi_buf_alloc()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 22:22:02 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/firmware/google/gsmi.c | 4 +--- 1 file changed, 1

[PATCH] gpio-intel-mid: Delete an error message for a failed memory allocation in intel_gpio_probe()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 21:56:42 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] gpio-intel-mid: Delete an error message for a failed memory allocation in intel_gpio_probe()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 21:56:42 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpio/gpio-intel-mid.c | 4 +--- 1 file changed, 1

[PATCH] gpio: merrifield: Delete an error message for a failed memory allocation in mrfld_gpio_probe()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 13:30:14 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] gpio: merrifield: Delete an error message for a failed memory allocation in mrfld_gpio_probe()

2018-02-11 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 11 Feb 2018 13:30:14 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpio/gpio-merrifield.c | 4 +--- 1 file changed, 1

[PATCH] gpio-ml-ioh: Delete an error message for a failed memory allocation in ioh_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 22:27:15 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] gpio-ml-ioh: Delete an error message for a failed memory allocation in ioh_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 22:27:15 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpio/gpio-ml-ioh.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH 2/2] gpio: omap: Improve a size determination in omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:49:22 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/2] gpio: omap: Improve a size determination in omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:49:22 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

[PATCH 1/2] gpio: omap: Delete an error message for a failed memory allocation in omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:46:30 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/2] gpio: omap: Delete an error message for a failed memory allocation in omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:46:30 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpio/gpio-omap.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 0/2] GPIO-OMAP: Adjustments for omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:56:54 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination

[PATCH 0/2] GPIO-OMAP: Adjustments for omap_gpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:56:54 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination drivers/gpio/gpio-omap.c | 6 ++ 1 file

[PATCH 2/2] gpio: timberdale: Improve a size determination in timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:13:28 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/2] gpio: timberdale: Improve a size determination in timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:13:28 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

[PATCH 1/2] gpio: timberdale: Delete an error message for a failed memory allocation in timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:09:08 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/2] gpio: timberdale: Delete an error message for a failed memory allocation in timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:09:08 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpio/gpio-timberdale.c | 5 ++--- 1 file changed, 2

[PATCH 0/2] GPIO-Timberdale: Adjustments for timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:28:29 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination

[PATCH 0/2] GPIO-Timberdale: Adjustments for timbgpio_probe()

2018-02-10 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 10 Feb 2018 21:28:29 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination drivers/gpio/gpio-timberdale.c | 7 +++ 1

Re: Software evolution around “checkpatch.pl”?

2018-02-10 Thread SF Markus Elfring
>> So I think checkpatch is striking the right balance here in >> how it warns. Obviously if it could assess the text >> and come to an informed decision that would be great but >> we are some way from that ;) > > The 'informed' bit is difficult as it is mostly a political problem. I find such

Re: Software evolution around “checkpatch.pl”?

2018-02-10 Thread SF Markus Elfring
>> So I think checkpatch is striking the right balance here in >> how it warns. Obviously if it could assess the text >> and come to an informed decision that would be great but >> we are some way from that ;) > > The 'informed' bit is difficult as it is mostly a political problem. I find such

Re: Adjustments for a lot of function implementations

2018-02-10 Thread SF Markus Elfring
>> Do any contributors get into the mood to take another look at software >> updates >> from my selection of change possibilities in a more constructive way? >> >> Do you need any additional development resources? > > One last time: either post per-driver patches with all the cleanups for a >

Re: Adjustments for a lot of function implementations

2018-02-10 Thread SF Markus Elfring
>> Do any contributors get into the mood to take another look at software >> updates >> from my selection of change possibilities in a more constructive way? >> >> Do you need any additional development resources? > > One last time: either post per-driver patches with all the cleanups for a >

Re: scripts/coccinelle/misc/shift.cocci: Refactoring?

2018-02-09 Thread SF Markus Elfring
> +| > +WARN_ON(x op@p y) > +| > +WARN_ON_ONCE(x op@p y) > +| Can it be nice to work with nested SmPL disjunctions so that a bit of duplicate SmPL code will be reduced? > +coccilib.org.print_todo(j0[0], msg) Will it be nicer to pass a string literal instead of a variable for the message as a

Re: scripts/coccinelle/misc/shift.cocci: Refactoring?

2018-02-09 Thread SF Markus Elfring
> +| > +WARN_ON(x op@p y) > +| > +WARN_ON_ONCE(x op@p y) > +| Can it be nice to work with nested SmPL disjunctions so that a bit of duplicate SmPL code will be reduced? > +coccilib.org.print_todo(j0[0], msg) Will it be nicer to pass a string literal instead of a variable for the message as a

[PATCH] drm/amdkfd: Delete an error message for a failed memory allocation in kfd_topology_init()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 22:23:57 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] drm/amdkfd: Delete an error message for a failed memory allocation in kfd_topology_init()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 22:23:57 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/amdkfd/kfd_topology.c | 1 - 1 file changed, 1

[PATCH 3/3] drm/amd/powerplay/hwmgr: Delete an unnecessary return statement in three functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:10:58 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected functions.

[PATCH 3/3] drm/amd/powerplay/hwmgr: Delete an unnecessary return statement in three functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:10:58 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected functions. Signed-off-by: Markus Elfring ---

[PATCH 2/3] drm/amd/powerplay/hwmgr: Adjust layout for source code from five if statements

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:01:24 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: Comparisons should place the constant on the right side of the test WARNING: else is not generally useful after a break or

[PATCH 2/3] drm/amd/powerplay/hwmgr: Adjust layout for source code from five if statements

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:01:24 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: Comparisons should place the constant on the right side of the test WARNING: else is not generally useful after a break or return Thus fix the affected

[PATCH 1/3] drm/amd/powerplay/hwmgr: Delete an error message for a failed memory allocation in three functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 20:32:39 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/3] drm/amd/powerplay/hwmgr: Delete an error message for a failed memory allocation in three functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 20:32:39 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c | 4 +---

[PATCH 0/3] drm/amd/powerplay/hwmgr: Adjustments for eight function implementations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:37:42 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in three functions Adjust layout for source

[PATCH 0/3] drm/amd/powerplay/hwmgr: Adjustments for eight function implementations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 21:37:42 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in three functions Adjust layout for source code from five if statements

[PATCH] drm_fb_cma_helper: Delete an error message for a failed memory allocation in drm_fbdev_cma_init_with_funcs()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 19:30:06 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] drm_fb_cma_helper: Delete an error message for a failed memory allocation in drm_fbdev_cma_init_with_funcs()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 19:30:06 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/drm_fb_cma_helper.c | 5 ++--- 1 file changed, 2

[PATCH] drm/exynos: g2d: Delete an error message for a failed memory allocation in two functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 18:42:51 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] drm/exynos: g2d: Delete an error message for a failed memory allocation in two functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 18:42:51 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/exynos/exynos_drm_g2d.c | 6 ++ 1 file changed,

[PATCH 3/3] drm/gma500: Delete an unnecessary return statement in oaktrail_crtc_hdmi_dpms()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:17:48 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function.

[PATCH 3/3] drm/gma500: Delete an unnecessary return statement in oaktrail_crtc_hdmi_dpms()

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:17:48 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function. Signed-off-by: Markus Elfring ---

[PATCH 2/3] drm/gma500: Improve four size determinations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:08:39 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/3] drm/gma500: Improve four size determinations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:08:39 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

[PATCH 1/3] drm/gma500: Delete an error message for a failed memory allocation in two functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 14:55:49 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/3] drm/gma500: Delete an error message for a failed memory allocation in two functions

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 14:55:49 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/gma500/framebuffer.c | 4 +---

[PATCH 0/3] GPU-DRM-GMA500: Adjustments for four function implementations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:27:38 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in two functions Improve four size

[PATCH 0/3] GPU-DRM-GMA500: Adjustments for four function implementations

2018-02-08 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 8 Feb 2018 15:27:38 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in two functions Improve four size determinations Delete an unnecessary

[PATCH 2/2] drm/msm/mdp4: Return directly after a failed kzalloc() in mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:34:45 +0100 * Return directly after a call of the function "kzalloc" failed at the beginning. * Delete an initialisation and a check (for the local variable "kms") which became unnecessary with this refactoring.

[PATCH 2/2] drm/msm/mdp4: Return directly after a failed kzalloc() in mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:34:45 +0100 * Return directly after a call of the function "kzalloc" failed at the beginning. * Delete an initialisation and a check (for the local variable "kms") which became unnecessary with this refactoring. Signed-off-by: Markus Elfring

[PATCH 1/2] drm/msm/mdp4: Delete an error message for a failed memory allocation in mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:22:28 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/2] drm/msm/mdp4: Delete an error message for a failed memory allocation in mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:22:28 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c | 1 - 1 file changed, 1

[PATCH 0/2] drm/msm/mdp4: Adjustments for mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:38:44 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Return directly after a failed kzalloc()

[PATCH 0/2] drm/msm/mdp4: Adjustments for mdp4_kms_init()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 22:38:44 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Return directly after a failed kzalloc()

[PATCH] drm/msm/mdp5: Delete an error message for a failed memory allocation in two functions

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 21:50:07 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] drm/msm/mdp5: Delete an error message for a failed memory allocation in two functions

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 21:50:07 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_ctl.c | 1 -

[PATCH] drm/tegra/fb: Delete an error message for a failed memory allocation in tegra_fbdev_create()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 21:17:17 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] drm/tegra/fb: Delete an error message for a failed memory allocation in tegra_fbdev_create()

2018-02-07 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 7 Feb 2018 21:17:17 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/tegra/fb.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 2/2] drm/tilcdc: panel: Use common error handling code in of_get_panel_info()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 22:10:11 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring ---

[PATCH 2/2] drm/tilcdc: panel: Use common error handling code in of_get_panel_info()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 22:10:11 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring --- drivers/gpu/drm/tilcdc/tilcdc_panel.c | 12 +--- 1 file changed, 5 insertions(+), 7

[PATCH 1/2] drm/tilcdc: Delete an error message for a failed memory allocation in seven functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 21:51:15 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/2] drm/tilcdc: Delete an error message for a failed memory allocation in seven functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 21:51:15 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 4 +---

[PATCH 0/2] GPU-DRM-TILCDC: Adjustments for seven function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 22:17:44 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation in seven functions Use common error handling

[PATCH 0/2] GPU-DRM-TILCDC: Adjustments for seven function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 22:17:44 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation in seven functions Use common error handling code in of_get_panel_info()

[PATCH] hid-a4tech: Delete an error message for a failed memory allocation in a4_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 20:48:04 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] hid-a4tech: Delete an error message for a failed memory allocation in a4_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 20:48:04 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-a4tech.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH] HID: apple: Delete an error message for a failed memory allocation in apple_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 20:06:53 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] HID: apple: Delete an error message for a failed memory allocation in apple_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 20:06:53 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-apple.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 2/2] HID: asus: Return an error code only as a constant in asus_start_multitouch()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:18:50 +0100 Return an error code without storing it in an intermediate variable. Signed-off-by: Markus Elfring --- drivers/hid/hid-asus.c | 6 ++ 1 file changed, 2 insertions(+), 4

[PATCH 2/2] HID: asus: Return an error code only as a constant in asus_start_multitouch()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:18:50 +0100 Return an error code without storing it in an intermediate variable. Signed-off-by: Markus Elfring --- drivers/hid/hid-asus.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/hid/hid-asus.c

[PATCH 1/2] HID: asus: Delete an error message for a failed memory allocation in two functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:03:19 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/2] HID: asus: Delete an error message for a failed memory allocation in two functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:03:19 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-asus.c | 5 + 1 file changed, 1 insertion(+), 4

[PATCH 0/2] HID-Asus: Adjustments for two function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:24:56 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation in two functions Return an error code only as

[PATCH 0/2] HID-Asus: Adjustments for two function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 18:24:56 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation in two functions Return an error code only as a constant in

[PATCH] HID: core: Delete an error message for a failed memory allocation in open_collection()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:40:51 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH] HID: core: Delete an error message for a failed memory allocation in open_collection()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:40:51 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-core.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH 3/3] HID: lenovo: Adjust four checks for null pointers

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:02:53 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus

[PATCH 3/3] HID: lenovo: Adjust four checks for null pointers

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:02:53 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code

[PATCH 2/3] HID: lenovo: Improve a size determination in lenovo_probe_tpkbd()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 16:52:58 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/3] HID: lenovo: Improve a size determination in lenovo_probe_tpkbd()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 16:52:58 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

[PATCH 1/3] HID: lenovo: Delete an error message for a failed memory allocation in two functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 16:48:52 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/3] HID: lenovo: Delete an error message for a failed memory allocation in two functions

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 16:48:52 +0100 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-lenovo.c | 6 ++ 1 file changed, 2

[PATCH 0/3] HID-Lenovo: Adjustments for three function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:12:34 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in two functions Improve a size

[PATCH 0/3] HID-Lenovo: Adjustments for three function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 17:12:34 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in two functions Improve a size determination in lenovo_probe_tpkbd()

[PATCH 3/3] hid-lg4ff: Delete an error message for a failed memory allocation in lg4ff_init()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 15:26:06 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 3/3] hid-lg4ff: Delete an error message for a failed memory allocation in lg4ff_init()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 15:26:06 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-lg4ff.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 2/3] hid-lg: Improve a size determination in lg_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 14:54:14 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/3] hid-lg: Improve a size determination in lg_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 14:54:14 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

[PATCH 1/3] hid-lg: Delete an error message for a failed memory allocation in lg_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 14:25:32 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 1/3] hid-lg: Delete an error message for a failed memory allocation in lg_probe()

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 14:25:32 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/hid/hid-lg.c | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH 0/3] HID-Logitech: Adjustments for two function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 15:38:30 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in lg_probe() Improve a size determination

[PATCH 0/3] HID-Logitech: Adjustments for two function implementations

2018-02-06 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 6 Feb 2018 15:38:30 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in lg_probe() Improve a size determination in lg_probe() Delete an error

Re: Software evolution around “checkpatch.pl”?

2018-02-06 Thread SF Markus Elfring
> Check patch is dumb and often gives false positives. Would you like to improve this software situation anyhow? Regards, Markus

Re: Software evolution around “checkpatch.pl”?

2018-02-06 Thread SF Markus Elfring
> Check patch is dumb and often gives false positives. Would you like to improve this software situation anyhow? Regards, Markus

[PATCH 4/4] HID: logitech-dj: Fix a typo in a comment line

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 22:14:49 +0100 Adjust a word in this description. Signed-off-by: Markus Elfring --- drivers/hid/hid-logitech-dj.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 4/4] HID: logitech-dj: Fix a typo in a comment line

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 22:14:49 +0100 Adjust a word in this description. Signed-off-by: Markus Elfring --- drivers/hid/hid-logitech-dj.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-logitech-dj.c index

[PATCH 3/4] HID: logitech-dj: Combine substrings for seven messages

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 22:06:20 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: quoted string split across lines Thus fix the affected source code places. Signed-off-by: Markus Elfring

[PATCH 3/4] HID: logitech-dj: Combine substrings for seven messages

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 22:06:20 +0100 The script "checkpatch.pl" pointed information out like the following. WARNING: quoted string split across lines Thus fix the affected source code places. Signed-off-by: Markus Elfring --- drivers/hid/hid-logitech-dj.c | 29

[PATCH 2/4] HID: logitech-dj: Improve a size determination in four functions

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 21:47:45 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 2/4] HID: logitech-dj: Improve a size determination in four functions

2018-02-05 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 5 Feb 2018 21:47:45 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was

<    1   2   3   4   5   6   7   8   9   10   >