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

Subject: media: atomisp: Fix sh_css_config_input_network_2400() coding style
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Tue Apr 16 10:19:52 2024 +0100

Fix the if (stream->config.mode == IA_CSS_INPUT_MODE_PRBS) {} block
coding style. Specifically fix the weird declaration of variables split
over multiple lines.

While at it also split the multi-line width calculation at a more
logical place.

Reported-by: Andy Shevchenko <a...@kernel.org>
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.c | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/sh_css.c 
b/drivers/staging/media/atomisp/pci/sh_css.c
index 6520a35b6e47..42a69b26db01 100644
--- a/drivers/staging/media/atomisp/pci/sh_css.c
+++ b/drivers/staging/media/atomisp/pci/sh_css.c
@@ -498,17 +498,15 @@ sh_css_config_input_network_2400(struct ia_css_stream 
*stream)
        }
 
        if (stream->config.mode == IA_CSS_INPUT_MODE_PRBS) {
-               unsigned int hblank_cycles = 100,
-               vblank_lines = 6,
-               width,
-               height,
-               vblank_cycles;
-               width  = (stream->config.input_config.input_res.width) / (1 +
-                       (stream->config.pixels_per_clock == 2));
+               unsigned int width, height, vblank_cycles;
+               const unsigned int hblank_cycles = 100;
+               const unsigned int vblank_lines = 6;
+
+               width = (stream->config.input_config.input_res.width) /
+                       (1 + (stream->config.pixels_per_clock == 2));
                height = stream->config.input_config.input_res.height;
                vblank_cycles = vblank_lines * (width + hblank_cycles);
-               sh_css_sp_configure_sync_gen(width, height, hblank_cycles,
-                                            vblank_cycles);
+               sh_css_sp_configure_sync_gen(width, height, hblank_cycles, 
vblank_cycles);
        }
        ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE,
                            "sh_css_config_input_network() leave:\n");

Reply via email to