[PATCH 00/34] [media] DaVinci-Video Processing: Fine-tuning for several function implementations

From: SF Markus Elfring
Date: Wed Oct 12 2016 - 10:26:36 EST


From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 12 Oct 2016 16:20:02 +0200

Several update suggestions were taken into account
from static source code analysis.

Markus Elfring (34):
Use kmalloc_array() in vpbe_initialize()
Delete two error messages for a failed memory allocation
Adjust 16 checks for null pointers
Combine substrings for four messages
Return an error code only as a constant in vpbe_probe()
Return an error code only by a single variable in vpbe_initialize()
Delete an unnecessary variable initialisation in vpbe_initialize()
Return the success indication only as a constant in vpbe_set_mode()
Reduce the scope for a variable in vpbe_set_default_output()
Check return value of a setup_if_config() call in vpbe_set_output()
Rename a jump label in vpbe_set_output()
Delete an unnecessary variable initialisation in vpbe_set_output()
Capture: Use kmalloc_array() in vpfe_probe()
Capture: Delete three error messages for a failed memory allocation
Capture: Improve another size determination in vpfe_probe()
Capture: Delete an unnecessary variable initialisation in vpfe_probe()
Capture: Improve another size determination in vpfe_enum_input()
Capture: Combine substrings for an error message in vpfe_enum_input()
Capture: Improve another size determination in vpfe_open()
Capture: Adjust 13 checks for null pointers
Capture: Delete an unnecessary variable initialisation in 11 functions
Capture: Move two assignments in vpfe_s_input()
Capture: Delete unnecessary braces in vpfe_isr()
Capture: Delete an unnecessary return statement in vpfe_unregister_ccdc_device()
Capture: Use kcalloc() in vpif_probe()
Capture: Delete an error message for a failed memory allocation
Capture: Adjust ten checks for null pointers
Capture: Delete an unnecessary variable initialisation in vpif_querystd()
Capture: Delete an unnecessary variable initialisation in vpif_channel_isr()
Display: Use kcalloc() in vpif_probe()
Display: Delete an error message for a failed memory allocation
Display: Adjust 11 checks for null pointers
Display: Delete an unnecessary variable initialisation in vpif_channel_isr()
Display: Delete an unnecessary variable initialisation in process_progressive_mode()

drivers/media/platform/davinci/vpbe.c | 93 ++++++++++++---------------
drivers/media/platform/davinci/vpfe_capture.c | 88 ++++++++++++-------------
drivers/media/platform/davinci/vpif_capture.c | 28 ++++----
drivers/media/platform/davinci/vpif_display.c | 30 ++++-----
4 files changed, 109 insertions(+), 130 deletions(-)

--
2.10.1