- 19 May, 2021 15 commits
-
-
Deepak R Varma authored
Balance braces around the if else blocks as per the code style guidelines. Add braces to branches where it is missing. Resolves checkpatch script CHECK / WARNING feedback messages. Signed-off-by:
Deepak R Varma <drv@mailo.com> Signed-off-by:
Hans Verkuil <hverkuil-cisco@xs4all.nl> Acked-by:
Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by:
Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
-
Martiros Shakhzadyan authored
Fix line split issues and resolve adjacent extra parens. Signed-off-by:
Martiros Shakhzadyan <vrzh@vrzh.net> Signed-off-by:
-
Martiros Shakhzadyan authored
Move the support check to the beginning of the function. Change the logic to avoid multiple nesting blocks. Move 'err' variable assignment outside of the if statement. Remove an unnecessary check whether 'j' is zero. Signed-off-by:
-
Martiros Shakhzadyan authored
Fix alignment style issues and adjacent line length issues in sh_css.c Signed-off-by:
-
Martiros Shakhzadyan authored
Use the ternary operator for conditional variable assignment in create_host_video_pipeline(). Signed-off-by:
-
Martiros Shakhzadyan authored
Remove a superfluous else clause in ia_css_pipe_check_format() Signed-off-by:
-
Martiros Shakhzadyan authored
Remove the remainder of assert() in places where the condition is already handled. Signed-off-by:
-
Martiros Shakhzadyan authored
Fix the remainder of brace coding style issues. Signed-off-by:
-
Martiros Shakhzadyan authored
Remove assert() in places where the condition is already handled. Signed-off-by:
-
Martiros Shakhzadyan authored
Fix brace coding style issues. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: tegra-video: Align line break to match with the open parenthesis in file vi.c Align line break to match with the open parenthesis. Issue detected by checkpatch.pl. Signed-off-by:
Aline Santana Cordeiro <alinesantanacordeiro@gmail.com> Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_v4l2.c Balance braces around conditional statements. Issue detected by checkpatch.pl. It happens in if-else statements where one of the commands uses braces around a block of code and the other command does not since it has just a single line of code. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_subdev.c Balance braces around conditional statements. Issue detected by checkpatch.pl. It happens in if-else statements where one of the commands uses braces around a block of code and the other command does not since it has just a single line of code. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_compat_css20.c Balance braces around conditional statements. Issue detected by checkpatch.pl. It happens in if-else statements where one of the commands uses braces around a block of code and the other command does not since it has just a single line of code. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_cmd.c Balance braces around conditional statements. Issue detected by checkpatch.pl. It happens in if-else statements where one of the commands uses braces around a block of code and the other command does not since it has just a single line of code. Signed-off-by:
-
- 18 May, 2021 1 commit
-
-
Mauro Carvalho Chehab authored
The right thing to do is to add a new object to the building system when a certain config option is selected, and *not* override them. So, fix obj-$(config) logic at sti makefiles, using "+=", instead of ":=". Signed-off-by:
-
- 11 May, 2021 1 commit
-
-
Mauro Carvalho Chehab authored
The usage of pm_runtime_resume_and_get() removed the need of a temporary integer. So, drop it. Fixes: 59f96244 ("media: exynos4-is: fix pm_runtime_get_sync() usage count") Reported-by:
Stephen Rothwell <sfr@canb.auug.org.au> Signed-off-by:
-
- 10 May, 2021 23 commits
-
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Format comments according to coding-style in file atomisp_cmd.h Format all comments according to the coding-style. Issue detected by checkpatch.pl. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Format comments according to coding-style in file atomisp_acc.c Format all comments according to the coding-style. Issue detected by checkpatch.pl. Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Correct identation in block of conditional statements in file atomisp_acc.c Correct identation in block of conditional statements. The conditional statement depends on the results of the macro function "list_for_each_entry()". Signed-off-by:
-
Aline Santana Cordeiro authored
media: staging: media: atomisp: pci: Correct identation in block of conditional statements in file atomisp_v4l2.c Correct identation in block of conditional statements. The function "v4l2_device_unregister_subdev()" depends on the results of the macro function "list_for_each_entry_safe()". Signed-off-by:
-
Aline Santana Cordeiro authored
Remove unused macro function "v4l2_dev_to_iss_device(dev)". Signed-off-by:
-
Martiros Shakhzadyan authored
Fixed line continuation and parenthesis alignment issues. Signed-off-by:
-
Mitali Borkar authored
Added spaces around '<<' operator to improve readability and meet linux kernel coding style. Reported by checkpatch Signed-off-by:
Mitali Borkar <mitaliborkar810@gmail.com> Signed-off-by:
-
Mitali Borkar authored
Fixed the issue of line should not end with '[' by moving argument from next line to line ending with '[' and made it under 80 characters. Reported by checkpatch. Signed-off-by:
-
Mitali Borkar authored
Removed unnecessary space before tabs to adhere to linux kernel coding style. Reported by checkpatch. Signed-off-by:
-
Mitali Borkar authored
Reduced length of line as it was exceeding 100 characters by removing comments from same line and adding it to previous line. This makes code neater, and meets linux kernel coding style. Reported by checkpatch. Signed-off-by:
-
Mitali Borkar authored
Removed an unnecessary blank line to meet linux kernel coding style. Reported by checkpatch.pl Signed-off-by:
-
Beatriz Martins de Carvalho authored
Aligns line break with the remaining function arguments to the open parenthesis. Issue found by checkpatch in file atomisp-gc2235.c Signed-off-by:
Beatriz Martins de Carvalho <martinsdecarvalhobeatriz@gmail.com> Acked-by:
-
Martiros Shakhzadyan authored
Fixed a coding style issue. Signed-off-by:
-
Aline Santana Cordeiro authored
Aligns line break with the remaining function arguments to the open parenthesis. Issue found by checkpatch. Signed-off-by:
-
Aline Santana Cordeiro authored
Aligns line break with the remaining function arguments to the open parenthesis. Issue found by checkpatch. Signed-off-by:
-
Aline Santana Cordeiro authored
Aligns line break with the remaining function arguments to the open parenthesis. Issue found by checkpatch. Signed-off-by:
-
Mauro Carvalho Chehab authored
The pm_runtime_get_sync() internally increments the dev->power.usage_count without decrementing it, even on errors. Replace it by the new pm_runtime_resume_and_get(), introduced by: commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") in order to properly decrement the usage counter, avoiding a potential PM usage counter leak. As a bonus, as pm_runtime_get_sync() always return 0 on success, the logic can be simplified. Reviewed-by:
Jonathan Cameron <Jonathan.Cameron@huawei.com> Reviewed-by:
Sylwester Nawrocki <s.nawrocki@samsung.com> Signed-off-by:
-
Mauro Carvalho Chehab authored
The pm_runtime_get_sync() internally increments the dev->power.usage_count without decrementing it, even on errors. On some places, this is ok, but on others the usage count ended being unbalanced on failures. Replace it by the new pm_runtime_resume_and_get(), introduced by: commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") in order to properly decrement the usage counter, avoiding a potential PM usage counter leak. As a bonus, such function always return zero on success. So, some code can be simplified. Reviewed-by:
-
Mauro Carvalho Chehab authored
The pm_runtime_get_sync() internally increments the dev->power.usage_count without decrementing it, even on errors. The bdisp_start_streaming() doesn't take it into account, which would unbalance PM usage counter at bdisp_stop_streaming(). The logic at bdisp_probe() is correct, but the best is to use the same call along the driver. So, replace it by the new pm_runtime_resume_and_get(), introduced by: commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") in order to properly decrement the usage counter, avoiding a potential PM usage counter leak. Reviewed-by:
-
Mauro Carvalho Chehab authored
The pm_runtime_get_sync() internally increments the dev->power.usage_count without decrementing it, even on errors. Replace it by the new pm_runtime_resume_and_get(), introduced by: commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") in order to properly decrement the usage counter, avoiding a potential PM usage counter leak. Reviewed-by:
-
Mauro Carvalho Chehab authored
Commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") added pm_runtime_resume_and_get() in order to automatically handle dev->power.usage_count decrement on errors. Use the new API, in order to cleanup the error check logic. Reviewed-by:
-
Mauro Carvalho Chehab authored
The pm_runtime_get_sync() internally increments the dev->power.usage_count without decrementing it, even on errors. Replace it by the new pm_runtime_resume_and_get(), introduced by: commit dd8088d5 ("PM: runtime: Add pm_runtime_resume_and_get to deal with usage counter") in order to properly decrement the usage counter, avoiding a potential PM usage counter leak. As a plus, pm_runtime_resume_and_get() doesn't return positive numbers, so the return code validation can be removed. Reviewed-by:
Andrzej Pietrasiewicz <andrzejtp2010@gmail.com> Signed-off-by:
-
Mauro Carvalho Chehab authored
Currently, the driver just assumes that PM runtime logic succeded resuming the device. That may not be the case, as pm_runtime_get_sync() can fail (but keeping the usage count incremented). Replace the code to use pm_runtime_resume_and_get(), and letting it return the error code. This way, if mtk_vcodec_dec_pw_on() fails, the logic under fops_vcodec_open() will do the right thing and return an error, instead of just assuming that the device is ready to be used. Reviewed-by:
-
