This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: staging: media: atomisp: Ensure trailing statements are on a 
newline and remove spurious whitespaces
Author:  Jonathan Bergh <bergh.jonat...@gmail.com>
Date:    Mon Feb 26 19:40:20 2024 +0000

This patch fixes the following minor code style issues:
 * Ensure trailing statements are located on a newline
 * Removes an instance of a spurious whitespace following a conditional
   statement

Link: https://lore.kernel.org/r/20240226194023.69070-4-bergh.jonat...@gmail.com

Signed-off-by: Jonathan Bergh <bergh.jonat...@gmail.com>
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/sh_css_sp.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/sh_css_sp.c 
b/drivers/staging/media/atomisp/pci/sh_css_sp.c
index 05b9c9ea0a21..273955f5128d 100644
--- a/drivers/staging/media/atomisp/pci/sh_css_sp.c
+++ b/drivers/staging/media/atomisp/pci/sh_css_sp.c
@@ -732,7 +732,8 @@ sh_css_sp_write_frame_pointers(const struct 
sh_css_binary_args *args)
 
        /* we don't pass this error back to the upper layer, so we add a assert 
here
           because we actually hit the error here but it still works by 
accident... */
-       if (err) assert(false);
+       if (err)
+               assert(false);
        return err;
 }
 
@@ -747,7 +748,8 @@ sh_css_sp_init_group(bool two_ppc,
 
        sh_css_sp_group.config.no_isp_sync = (uint8_t)no_isp_sync;
        /* decide whether the frame is processed online or offline */
-       if (if_config_index == SH_CSS_IF_CONFIG_NOT_NEEDED) return;
+       if (if_config_index == SH_CSS_IF_CONFIG_NOT_NEEDED)
+               return;
 
        if (!IS_ISP2401) {
                assert(if_config_index < SH_CSS_MAX_IF_CONFIGS);

Reply via email to