diff mbox series

[v3,2/2] staging: media: sdis: move open braces to the previous line

Message ID 20240730212854.474708-2-santiagorr@riseup.net (mailing list archive)
State New
Headers
Series [v3,1/2] staging: media: sdis: move open brace to a new line |

Commit Message

Santiago Ruano Rincón July 30, 2024, 9:28 p.m. UTC 
  Fix checkpatch error "ERROR: that open brace { should be on the previous
line" in ia_css_sdis.host.c:253 and :258.

Signed-off-by: Santiago Ruano Rincón <santiagorr@riseup.net>

---
This is one of my first patches. Could you please if is there anything
wrong with it? Thank you

V3: Insert the change history (including for V2)

V2: Remove spurious [PATCH] header from the Subject, inserted by mistake
---
 .../pci/isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c        | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Dan Carpenter July 30, 2024, 9:41 p.m. UTC | #1 
On Wed, Jul 31, 2024 at 06:28:54AM +0900, Santiago Ruano Rincón wrote:
> Fix checkpatch error "ERROR: that open brace { should be on the previous
> line" in ia_css_sdis.host.c:253 and :258.
> 
> Signed-off-by: Santiago Ruano Rincón <santiagorr@riseup.net>
> 
> ---
> This is one of my first patches. Could you please if is there anything
> wrong with it? Thank you
> 
> V3: Insert the change history (including for V2)
> 
> V2: Remove spurious [PATCH] header from the Subject, inserted by mistake
> ---

Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>

regards,
dan carpenter
diff mbox series

Patch

diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c
index cabacfc84..467572d84 100644
--- a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c
+++ b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sdis.host.c
@@ -250,13 +250,11 @@  ia_css_get_dvs_statistics(
 	assert(isp_stats);
 
 	map = ia_css_isp_dvs_statistics_map_allocate(isp_stats, NULL);
-	if (map)
-	{
+	if (map) {
 		hmm_load(isp_stats->data_ptr, map->data_ptr, isp_stats->size);
 		ia_css_translate_dvs_statistics(host_stats, map);
 		ia_css_isp_dvs_statistics_map_free(map);
-	} else
-	{
+	} else {
 		IA_CSS_ERROR("out of memory");
 		ret = -ENOMEM;
 	}