From nobody Thu Apr 2 14:09:47 2026 Received: from s.wfbtzhsw.outbound-mail.sendgrid.net (s.wfbtzhsw.outbound-mail.sendgrid.net [159.183.224.105]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7F45C340D84 for ; Tue, 10 Feb 2026 09:24:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=159.183.224.105 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715443; cv=none; b=Dov6p/+YEM2kOAoyaMfWRizUw20wxBp2VJSIYYaUYG4IRFiVFCrNhDFfZFN1sjvdjV4Sn5Et0Bbv5UDN7lyi+u4ZlbnDIdKgevi8r6sOx2DJE3iv0tqcU1R+CdOS1Ls6lOCbzeVgftVDo30Rrza4Gb7o6T3tjaInJ46vhPV/Ulg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715443; c=relaxed/simple; bh=EoOhV0iYPulK75C7UcACuYDCZnijisqgn5heFNvjJz8=; h=From:Subject:Date:Message-ID:In-Reply-To:References:MIME-Version: To:Cc:Content-Type; b=DiQVsM3/CT8wWhBRvBFxRpQL6g7p03Tu4MZq3ilm+4K9mbgu0swF5CEkJuxK2CcNAJuvWtHV9lGrs/pSG7qdA6iDAfWrgoDriGjjrEW/4FYV4m0jN4BET0pptA+USYLCWwtXnW0gcST8CE3QuM64rkHzjFNFpKuoSetzpEJQBlk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net; spf=pass smtp.mailfrom=em431.mattwardle.net; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b=VyxaJuME; arc=none smtp.client-ip=159.183.224.105 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=em431.mattwardle.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b="VyxaJuME" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mattwardle.net; h=from:subject:date:in-reply-to:references:mime-version:to:cc: content-transfer-encoding:content-type:cc:content-type:date:from:subject:to; s=s1; bh=1tDRLPS3O2Gfhhtx6r9Mg16xLFDUmUN68AAwnbWhoZs=; b=VyxaJuMExKg3WIH4i+v2KNYPkpjOp39lzWjrrbeZGE5QYGTwT+WdGPbVLNkr8g1M48/y HDAG4Ner/C+tNeghUJeyBZoWjtUapQ9DHePGBmx8HHAtWK5DLWCNPvcSfuApNQnj8GECZm BRnBlQ0MeXLrYbHftdgDOqA2Xfh4qXMpccYheJkr1obDmUAO4KrvvalM3oNhhExyTlBQv+ ZdfKRrUoZxRdKQwiD6sUrnCQiY2KcBQq5tpr7oBTEK3CiWfGCkSjpxaR2D/8ksNMzN/GEa 4VGgvxqhPz9nkIUBS5J4xKWLOp4YtypNhHbjWEFUlHUIbwLYNiTC9gCycpgzZlNg== Received: by recvd-5fb7fdbd94-fm764 with SMTP id recvd-5fb7fdbd94-fm764-1-698AF92E-11 2026-02-10 09:23:58.410211918 +0000 UTC m=+55687.289692281 Received: from mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (unknown) by geopod-ismtpd-3 (SG) with ESMTP id Oeutz_SbQC2N7qLj0qAZAA Tue, 10 Feb 2026 09:23:58.213 +0000 (UTC) Authentication-Results: mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net; dmarc=fail (p=reject dis=none) header.from=mattwardle.net Received: from thinkpad (unknown [161.142.117.49]) by mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (Postfix) with ESMTPSA id 0C05A40E37; Tue, 10 Feb 2026 09:23:58 +0000 (UTC) From: Matt Wardle Subject: [PATCH 1/3] staging: media: atomisp: Remove braces for single statement blocks Date: Tue, 10 Feb 2026 09:23:58 +0000 (UTC) Message-ID: <20260210092243.294732-2-matt@mattwardle.net> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260210092243.294732-1-matt@mattwardle.net> References: <20260210092243.294732-1-matt@mattwardle.net> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-SG-EID: =?us-ascii?Q?u001=2EFlbIg1=2FBrpD1HWcEBlj1tgh=2F0VDwewqbkmeZ1SFD9tLmzlDepFYnLmfnO?= =?us-ascii?Q?moWHy7n5HdVZTdyWWeMojZeck4brX7CTABEAGrD?= =?us-ascii?Q?ids5tm+WpzFU5gquPeWu0oAWODQtMi6sOSX2XAd?= =?us-ascii?Q?Py7n2vdVe349t0V=2FVvl3BXyFepPl6wZvLVZ5d5A?= =?us-ascii?Q?3+rKdLcRglKd0HRHWalssBk5ijP9K+=2FRWwtKdVe?= =?us-ascii?Q?sO+REYDbfph9YjxNndsgpBfFp+F5lICOjfdFFRp?= =?us-ascii?Q?17P=2F?= To: Hans de Goede , Mauro Carvalho Chehab , Andy Shevchenko Cc: Sakari Ailus , Greg Kroah-Hartman , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Matt Wardle X-Entity-ID: u001.OAl4HwJfzvoR6h5DRlrySA== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Fix checkpatch.pl warnings: WARNING: braces {} are not necessary for single statement blocks WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: Matt Wardle --- .../base/circbuf/interface/ia_css_circbuf.h | 3 +- .../circbuf/interface/ia_css_circbuf_desc.h | 3 +- .../atomisp/pci/base/refcount/src/refcount.c | 3 +- .../pci/camera/pipe/src/pipe_stagedesc.c | 9 ++---- .../atomisp/pci/camera/pipe/src/pipe_util.c | 3 +- .../host/isys_stream2mmio_private.h | 3 +- .../host/event_fifo_private.h | 3 +- .../host/input_formatter.c | 3 +- .../hive_isp_css_common/host/input_system.c | 14 ++++----- .../pci/hive_isp_css_common/host/irq.c | 30 ++++++++----------- .../pci/hive_isp_css_common/host/mmu.c | 3 +- .../pci/hive_isp_css_common/host/vmem.c | 4 +-- .../isp/kernels/anr/anr_2/ia_css_anr2.host.c | 3 +- .../pci/isp/kernels/bnlm/ia_css_bnlm.host.c | 6 ++-- .../isp/kernels/ctc/ctc2/ia_css_ctc2.host.c | 7 ++--- .../isp/kernels/eed1_8/ia_css_eed1_8.host.c | 13 ++++---- .../isp/kernels/s3a/s3a_1.0/ia_css_s3a.host.c | 3 +- .../kernels/sdis/sdis_1.0/ia_css_sdis.host.c | 6 ++-- .../atomisp/pci/runtime/binary/src/binary.c | 3 +- .../pci/runtime/debug/src/ia_css_debug.c | 6 ++-- .../pci/runtime/isys/src/virtual_isys.c | 3 +- .../pci/runtime/pipeline/src/pipeline.c | 3 +- .../staging/media/atomisp/pci/sh_css_params.c | 12 +++----- 23 files changed, 53 insertions(+), 93 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_cs= s_circbuf.h b/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_c= ss_circbuf.h index 86300991d30e..148f9f6febfb 100644 --- a/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_css_circb= uf.h +++ b/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_css_circb= uf.h @@ -146,9 +146,8 @@ static inline uint8_t ia_css_circbuf_get_pos_at_offset( OP___assert(cb->desc->size > 0); =20 /* step 1: adjudst the offset */ - while (offset < 0) { + while (offset < 0) offset +=3D cb->desc->size; - } =20 /* step 2: shift and round by the upper limit */ dest =3D OP_std_modadd(base, offset, cb->desc->size); diff --git a/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_cs= s_circbuf_desc.h b/drivers/staging/media/atomisp/pci/base/circbuf/interface= /ia_css_circbuf_desc.h index 5645a7bf493c..3d89626a0733 100644 --- a/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_css_circb= uf_desc.h +++ b/drivers/staging/media/atomisp/pci/base/circbuf/interface/ia_css_circb= uf_desc.h @@ -84,9 +84,8 @@ static inline uint8_t ia_css_circbuf_desc_get_pos_at_offs= et( OP___assert(cb_desc->size > 0); =20 /* step 1: adjust the offset */ - while (offset < 0) { + while (offset < 0) offset +=3D cb_desc->size; - } =20 /* step 2: shift and round by the upper limit */ dest =3D OP_std_modadd(base, offset, cb_desc->size); diff --git a/drivers/staging/media/atomisp/pci/base/refcount/src/refcount.c= b/drivers/staging/media/atomisp/pci/base/refcount/src/refcount.c index 58e4e3173b40..4a8675d0129a 100644 --- a/drivers/staging/media/atomisp/pci/base/refcount/src/refcount.c +++ b/drivers/staging/media/atomisp/pci/base/refcount/src/refcount.c @@ -238,9 +238,8 @@ void ia_css_refcount_clear(s32 id, clear_func clear_fun= c_ptr) hmm_free(entry->data); } =20 - if (entry->count !=3D 0) { + if (entry->count !=3D 0) IA_CSS_WARNING("Ref count for entry %x is not zero!", entry->id); - } =20 assert(entry->count =3D=3D 0); =20 diff --git a/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_stagede= sc.c b/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_stagedesc.c index a9f736398f50..fa2f8ed5b053 100644 --- a/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_stagedesc.c +++ b/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_stagedesc.c @@ -32,9 +32,8 @@ void ia_css_pipe_get_generic_stage_desc( stage_desc->max_input_width =3D 0; stage_desc->mode =3D binary->info->sp.pipeline.mode; stage_desc->in_frame =3D in_frame; - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) stage_desc->out_frame[i] =3D out_frame[i]; - } stage_desc->vf_frame =3D vf_frame; ERR: IA_CSS_LEAVE_PRIVATE(""); @@ -59,9 +58,8 @@ void ia_css_pipe_get_firmwares_stage_desc( stage_desc->max_input_width =3D 0; stage_desc->mode =3D mode; stage_desc->in_frame =3D in_frame; - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) stage_desc->out_frame[i] =3D out_frame[i]; - } stage_desc->vf_frame =3D vf_frame; } =20 @@ -82,8 +80,7 @@ void ia_css_pipe_get_sp_func_stage_desc( stage_desc->mode =3D (unsigned int)-1; stage_desc->in_frame =3D NULL; stage_desc->out_frame[0] =3D out_frame; - for (i =3D 1; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { + for (i =3D 1; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) stage_desc->out_frame[i] =3D NULL; - } stage_desc->vf_frame =3D NULL; } diff --git a/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_util.c = b/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_util.c index c7c42b472cc7..6cb3ecbd7297 100644 --- a/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_util.c +++ b/drivers/staging/media/atomisp/pci/camera/pipe/src/pipe_util.c @@ -26,9 +26,8 @@ void ia_css_pipe_util_create_output_frames( unsigned int i; =20 assert(frames); - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) frames[i] =3D NULL; - } } =20 void ia_css_pipe_util_set_output_frames( diff --git a/drivers/staging/media/atomisp/pci/css_2401_system/host/isys_st= ream2mmio_private.h b/drivers/staging/media/atomisp/pci/css_2401_system/hos= t/isys_stream2mmio_private.h index 3210dd6bf9ca..8e295cd78129 100644 --- a/drivers/staging/media/atomisp/pci/css_2401_system/host/isys_stream2mm= io_private.h +++ b/drivers/staging/media/atomisp/pci/css_2401_system/host/isys_stream2mm= io_private.h @@ -41,9 +41,8 @@ STORAGE_CLASS_STREAM2MMIO_C void stream2mmio_get_state( * Get the values of the register-set per * stream2mmio-controller sids. */ - for (i =3D STREAM2MMIO_SID0_ID; i < N_STREAM2MMIO_SID_PROCS[ID]; i++) { + for (i =3D STREAM2MMIO_SID0_ID; i < N_STREAM2MMIO_SID_PROCS[ID]; i++) stream2mmio_get_sid_state(ID, i, &state->sid_state[i]); - } } =20 /** diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/eve= nt_fifo_private.h b/drivers/staging/media/atomisp/pci/hive_isp_css_common/h= ost/event_fifo_private.h index 439c69444942..e77d7cf61356 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/event_fifo= _private.h +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/event_fifo= _private.h @@ -26,9 +26,8 @@ STORAGE_CLASS_EVENT_C void event_wait_for(const event_ID_= t ID) STORAGE_CLASS_EVENT_C void cnd_event_wait_for(const event_ID_t ID, const bool cnd) { - if (cnd) { + if (cnd) event_wait_for(ID); - } } =20 STORAGE_CLASS_EVENT_C hrt_data event_receive_token(const event_ID_t ID) diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/inp= ut_formatter.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host= /input_formatter.c index 40b3f1e48c56..c353461dcdf2 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_form= atter.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_form= atter.c @@ -62,9 +62,8 @@ void input_formatter_rst( * WICH USES THE STREAM2MEMRY BLOCK. * MUST BE FIXED PROPERLY */ - if (!HIVE_IF_BIN_COPY[ID]) { + if (!HIVE_IF_BIN_COPY[ID]) input_formatter_reg_store(ID, addr, rst); - } =20 return; } diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/inp= ut_system.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/in= put_system.c index 9f1199c4761c..68b0ad27615d 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_syst= em.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_syst= em.c @@ -138,11 +138,10 @@ void receiver_port_enable( hrt_data reg =3D receiver_port_reg_load(ID, port_ID, _HRT_CSS_RECEIVER_DEVICE_READY_REG_IDX); =20 - if (cnd) { + if (cnd) reg |=3D 0x01; - } else { + else reg &=3D ~0x01; - } =20 receiver_port_reg_store(ID, port_ID, _HRT_CSS_RECEIVER_DEVICE_READY_REG_IDX, reg); @@ -194,9 +193,8 @@ static void receiver_rst( assert(ID < N_RX_ID); =20 // Disable all ports. - for (port_id =3D MIPI_PORT0_ID; port_id < N_MIPI_PORT_ID; port_id++) { + for (port_id =3D MIPI_PORT0_ID; port_id < N_MIPI_PORT_ID; port_id++) receiver_port_enable(ID, port_id, false); - } =20 // AM: Additional actions for stopping receiver? } @@ -830,15 +828,13 @@ input_system_err_t input_system_configuration_commit(= void) // The last configuration step is to configure the input buffer. input_system_err_t error =3D input_buffer_configuration(); =20 - if (error !=3D INPUT_SYSTEM_ERR_NO_ERROR) { + if (error !=3D INPUT_SYSTEM_ERR_NO_ERROR) return error; - } =20 // Translate the whole configuration into registers. error =3D configuration_to_registers(); - if (error !=3D INPUT_SYSTEM_ERR_NO_ERROR) { + if (error !=3D INPUT_SYSTEM_ERR_NO_ERROR) return error; - } =20 // Translate the whole configuration into ctrl commands etc. =20 diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/irq= .c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/irq.c index b66560bca625..61dea386361a 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/irq.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/irq.c @@ -55,9 +55,8 @@ void irq_clear_all( assert(ID < N_IRQ_ID); assert(IRQ_N_CHANNEL[ID] <=3D HRT_DATA_WIDTH); =20 - if (IRQ_N_CHANNEL[ID] < HRT_DATA_WIDTH) { + if (IRQ_N_CHANNEL[ID] < HRT_DATA_WIDTH) mask =3D ~((~(hrt_data)0) >> IRQ_N_CHANNEL[ID]); - } =20 irq_reg_store(ID, _HRT_IRQ_CONTROLLER_CLEAR_REG_IDX, mask); @@ -115,9 +114,9 @@ void irq_enable_pulse( { unsigned int edge_out =3D 0x0; =20 - if (pulse) { + if (pulse) edge_out =3D 0xffffffff; - } + /* output is given as edge, not pulse */ irq_reg_store(ID, _HRT_IRQ_CONTROLLER_EDGE_NOT_PULSE_REG_IDX, edge_out); @@ -259,9 +258,9 @@ void virq_clear_all(void) { irq_ID_t irq_id; =20 - for (irq_id =3D (irq_ID_t)0; irq_id < N_IRQ_ID; irq_id++) { + for (irq_id =3D (irq_ID_t)0; irq_id < N_IRQ_ID; irq_id++) irq_clear_all(irq_id); - } + return; } =20 @@ -301,9 +300,9 @@ void virq_clear_info(struct virq_info *irq_info) =20 assert(irq_info); =20 - for (ID =3D (irq_ID_t)0 ; ID < N_IRQ_ID; ID++) { + for (ID =3D (irq_ID_t)0 ; ID < N_IRQ_ID; ID++) irq_info->irq_status_reg[ID] =3D 0; - } + return; } =20 @@ -324,20 +323,17 @@ enum hrt_isp_css_irq_status virq_get_channel_id( break; } =20 - if (idx =3D=3D IRQ_N_CHANNEL[IRQ0_ID]) { + if (idx =3D=3D IRQ_N_CHANNEL[IRQ0_ID]) return hrt_isp_css_irq_status_error; - } =20 /* Check whether there are more bits set on device 0 */ - if (irq_status !=3D (1U << idx)) { + if (irq_status !=3D (1U << idx)) status =3D hrt_isp_css_irq_status_more_irqs; - } =20 /* Check whether we have an IRQ on one of the nested devices */ for (ID =3D N_IRQ_ID - 1 ; ID > (irq_ID_t)0; ID--) { - if (IRQ_NESTING_ID[ID] =3D=3D (enum virq_id)idx) { + if (IRQ_NESTING_ID[ID] =3D=3D (enum virq_id)idx) break; - } } =20 /* If we have a nested IRQ, load that state, discard the device 0 state */ @@ -350,9 +346,8 @@ enum hrt_isp_css_irq_status virq_get_channel_id( break; } =20 - if (idx =3D=3D IRQ_N_CHANNEL[ID]) { + if (idx =3D=3D IRQ_N_CHANNEL[ID]) return hrt_isp_css_irq_status_error; - } =20 /* Alternatively check whether there are more bits set on this device */ if (irq_status !=3D (1U << idx)) { @@ -408,9 +403,8 @@ static inline irq_ID_t virq_get_irq_id( assert(channel_ID); =20 for (ID =3D (irq_ID_t)0 ; ID < N_IRQ_ID; ID++) { - if (irq_ID < IRQ_N_ID_OFFSET[ID + 1]) { + if (irq_ID < IRQ_N_ID_OFFSET[ID + 1]) break; - } } =20 *channel_ID =3D (unsigned int)irq_ID - IRQ_N_ID_OFFSET[ID]; diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/mmu= .c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/mmu.c index 064e88a5e064..70d118fe1e70 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/mmu.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/mmu.c @@ -32,7 +32,6 @@ void mmu_invalidate_cache_all(void) { mmu_ID_t mmu_id; =20 - for (mmu_id =3D (mmu_ID_t)0; mmu_id < N_MMU_ID; mmu_id++) { + for (mmu_id =3D (mmu_ID_t)0; mmu_id < N_MMU_ID; mmu_id++) mmu_invalidate_cache(mmu_id); - } } diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/vme= m.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/vmem.c index 722b684fbc37..1c5c50406633 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/vmem.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/vmem.c @@ -168,9 +168,9 @@ static void store_vector( //load_vector (&v[1][0], &to[ISP_NWAY]); /* Fetch the next vector, since = it will be overwritten. */ hive_uedge *data =3D (hive_uedge *)v; =20 - for (i =3D 0; i < ISP_NWAY; i++) { + for (i =3D 0; i < ISP_NWAY; i++) hive_sim_wide_pack(data, (hive_wide)&from[i], ISP_VEC_ELEMBITS, i); - } + assert(ISP_BAMEM_BASE[ID] !=3D (hrt_address) - 1); #if !defined(HRT_MEMORY_ACCESS) ia_css_device_store(ISP_BAMEM_BASE[ID] + (unsigned long)to, &v, size); diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/anr/anr_2/ia_css= _anr2.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/anr/anr_2/ia_c= ss_anr2.host.c index 09599884bdae..08af8a848fbf 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/anr/anr_2/ia_css_anr2.h= ost.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/anr/anr_2/ia_css_anr2.h= ost.c @@ -22,9 +22,8 @@ ia_css_anr2_vmem_encode( for (i =3D 0; i < ANR_PARAM_SIZE; i++) { unsigned int j; =20 - for (j =3D 0; j < ISP_VEC_NELEMS; j++) { + for (j =3D 0; j < ISP_VEC_NELEMS; j++) to->data[i][j] =3D from->data[i * ISP_VEC_NELEMS + j]; - } } } =20 diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/bnlm/ia_css_bnlm= .host.c b/drivers/staging/media/atomisp/pci/isp/kernels/bnlm/ia_css_bnlm.ho= st.c index cd867937ee13..fcd7d7e2afe8 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/bnlm/ia_css_bnlm.host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/bnlm/ia_css_bnlm.host.c @@ -125,14 +125,12 @@ ia_css_bnlm_vmem_encode( bnlm_lut_encode(&to->div_lut, div_lut_nearests, div_lut_slopes, BNLM_DIV_LUT_SIZE); memset(to->div_lut_intercepts, 0, sizeof(to->div_lut_intercepts)); - for (i =3D 0; i < BNLM_DIV_LUT_SIZE; i++) { + for (i =3D 0; i < BNLM_DIV_LUT_SIZE; i++) to->div_lut_intercepts[0][i] =3D div_lut_intercepts[i]; - } =20 memset(to->power_of_2, 0, sizeof(to->power_of_2)); - for (i =3D 0; i < (ISP_VEC_ELEMBITS - 1); i++) { + for (i =3D 0; i < (ISP_VEC_ELEMBITS - 1); i++) to->power_of_2[0][i] =3D 1 << i; - } } =20 /* - Encodes BNLM public parameters into DMEM parameters */ diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/ctc/ctc2/ia_css_= ctc2.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/ctc/ctc2/ia_css= _ctc2.host.c index 38751b8e9e6a..177487b6b237 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/ctc/ctc2/ia_css_ctc2.ho= st.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/ctc/ctc2/ia_css_ctc2.ho= st.c @@ -52,13 +52,12 @@ static int ctc2_slope(int y1, int y0, int x1, int x0) /*the slope must lie within the range (-max_slope-1) >=3D (dydx) >=3D (max_slope) */ - if (slope <=3D -max_slope - 1) { + if (slope <=3D -max_slope - 1) dydx =3D -max_slope - 1; - } else if (slope >=3D max_slope) { + else if (slope >=3D max_slope) dydx =3D max_slope; - } else { + else dydx =3D slope; - } =20 return dydx; } diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_ee= d1_8.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_e= ed1_8.host.c index 8e4451fcc8e3..76d76ca4401a 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.ho= st.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.ho= st.c @@ -140,17 +140,14 @@ ia_css_eed1_8_vmem_encode( assert(tcinv_x[0] =3D=3D 0); assert(fcinv_x[0] =3D=3D 0); =20 - for (j =3D 1; j < NUMBER_OF_CHGRINV_POINTS; j++) { + for (j =3D 1; j < NUMBER_OF_CHGRINV_POINTS; j++) assert(chgrinv_x[j] > chgrinv_x[j - 1]); - } =20 - for (j =3D 1; j < NUMBER_OF_TCINV_POINTS; j++) { + for (j =3D 1; j < NUMBER_OF_TCINV_POINTS; j++) assert(tcinv_x[j] > tcinv_x[j - 1]); - } =20 - for (j =3D 1; j < NUMBER_OF_FCINV_POINTS; j++) { + for (j =3D 1; j < NUMBER_OF_FCINV_POINTS; j++) assert(fcinv_x[j] > fcinv_x[j - 1]); - } =20 /* The implementation of the calculating 1/x is based on the availability * of the OP_vec_shuffle16 operation. @@ -260,9 +257,9 @@ ia_css_eed1_8_encode( to->margin_neg_diff =3D (from->neg_margin1 - from->neg_margin0); =20 /* Encode DEWEnhance exp (e_dew_enh_asr) */ - for (i =3D 0; i < (IA_CSS_NUMBER_OF_DEW_ENHANCE_SEGMENTS - 1); i++) { + for (i =3D 0; i < (IA_CSS_NUMBER_OF_DEW_ENHANCE_SEGMENTS - 1); i++) min_exp =3D max(min_exp, from->dew_enhance_seg_exp[i]); - } + to->e_dew_enh_asr =3D 13 - clamp(min_exp, 0, 13); =20 to->dedgew_max =3D from->dedgew_max; diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/s3a/s3a_1.0/ia_c= ss_s3a.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/s3a/s3a_1.0/i= a_css_s3a.host.c index 13678138c48c..823a1b0c0991 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/s3a/s3a_1.0/ia_css_s3a.= host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/s3a/s3a_1.0/ia_css_s3a.= host.c @@ -240,9 +240,8 @@ ia_css_s3a_hmem_decode( /* Calculate sum of histogram of R, which should not be less than count_for_3a */ sum_r =3D 0; - for (i =3D 0; i < HMEM_UNIT_SIZE; i++) { + for (i =3D 0; i < HMEM_UNIT_SIZE; i++) sum_r +=3D out_ptr[i].r; - } if (sum_r < count_for_3a) { /* histogram is invalid */ return; 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 a1bea8bd1a39..f483e690a114 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sd= is.host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_1.0/ia_css_sd= is.host.c @@ -50,9 +50,8 @@ void ia_css_sdis_horicoef_vmem_encode( assert(size % (IA_CSS_DVS_NUM_COEF_TYPES * ISP_VEC_NELEMS * sizeof( short)) =3D=3D 0); =20 - for (type =3D 0; type < IA_CSS_DVS_NUM_COEF_TYPES; type++) { + for (type =3D 0; type < IA_CSS_DVS_NUM_COEF_TYPES; type++) fill_row(&private[type * stride], &public[type * width], width, padding); - } } =20 void ia_css_sdis_vertcoef_vmem_encode( @@ -77,9 +76,8 @@ void ia_css_sdis_vertcoef_vmem_encode( assert(size % (IA_CSS_DVS_NUM_COEF_TYPES * ISP_VEC_NELEMS * sizeof( short)) =3D=3D 0); =20 - for (type =3D 0; type < IA_CSS_DVS_NUM_COEF_TYPES; type++) { + for (type =3D 0; type < IA_CSS_DVS_NUM_COEF_TYPES; type++) fill_row(&private[type * stride], &public[type * height], height, paddin= g); - } } =20 void ia_css_sdis_horiproj_encode( diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c = b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c index af93ca96747c..60ae7bf5512b 100644 --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c @@ -670,9 +670,8 @@ ia_css_binary_fill_info(const struct ia_css_binary_xinf= o *xinfo, err =3D ia_css_isp_param_allocate_isp_parameters( &binary->mem_params, &binary->css_params, &info->mem_initializers); - if (err) { + if (err) return err; - } } for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { diff --git a/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_deb= ug.c b/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c index b411ca2f415e..58d99abe70aa 100644 --- a/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c +++ b/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c @@ -1123,9 +1123,8 @@ ia_css_debug_pipe_graph_dump_prologue(void) =20 void ia_css_debug_pipe_graph_dump_epilogue(void) { - if (strlen(ring_buffer) > 0) { + if (strlen(ring_buffer) > 0) dtrace_dot(ring_buffer); - } =20 if (pg_inst.stream_format !=3D N_ATOMISP_INPUT_FORMAT) { /* An input stream format has been set so assume we have @@ -1778,9 +1777,8 @@ static void debug_dump_one_trace(enum TRACE_CORE_ID p= roc_id) * When tid value is 111b, the data will be interpreted differently: * tid val is ignored, major field contains 2 bits (msb) for format type */ - if (tid_val =3D=3D FIELD_TID_SEL_FORMAT_PAT) { + if (tid_val =3D=3D FIELD_TID_SEL_FORMAT_PAT) dump_format =3D FIELD_FORMAT_UNPACK(trace_read_buf[j]); - } } switch (dump_format) { case TRACE_DUMP_FORMAT_POINT: diff --git a/drivers/staging/media/atomisp/pci/runtime/isys/src/virtual_isy= s.c b/drivers/staging/media/atomisp/pci/runtime/isys/src/virtual_isys.c index e6c11d5f77b4..8e8433f5b07a 100644 --- a/drivers/staging/media/atomisp/pci/runtime/isys/src/virtual_isys.c +++ b/drivers/staging/media/atomisp/pci/runtime/isys/src/virtual_isys.c @@ -295,9 +295,8 @@ static bool create_input_system_channel( if (!rc) return false; =20 - if (!acquire_sid(me->stream2mmio_id, &me->stream2mmio_sid_id)) { + if (!acquire_sid(me->stream2mmio_id, &me->stream2mmio_sid_id)) return false; - } =20 if (!acquire_ib_buffer( metadata ? cfg->metadata.bits_per_pixel : diff --git a/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipelin= e.c b/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c index cb8d652227a7..fb79ac07e270 100644 --- a/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c +++ b/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c @@ -575,9 +575,8 @@ static int pipeline_stage_create( binary =3D stage_desc->binary; firmware =3D stage_desc->firmware; vf_frame =3D stage_desc->vf_frame; - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) out_frame[i] =3D stage_desc->out_frame[i]; - } =20 stage =3D kvzalloc(sizeof(*stage), GFP_KERNEL); if (!stage) { diff --git a/drivers/staging/media/atomisp/pci/sh_css_params.c b/drivers/st= aging/media/atomisp/pci/sh_css_params.c index 11d62313c908..992acbf758cb 100644 --- a/drivers/staging/media/atomisp/pci/sh_css_params.c +++ b/drivers/staging/media/atomisp/pci/sh_css_params.c @@ -1928,9 +1928,8 @@ sh_css_set_per_frame_isp_config_on_pipe( params =3D stream->per_frame_isp_params_configs; =20 /* update new ISP params object with the new config */ - if (!sh_css_init_isp_params_from_global(stream, params, false, pipe)) { + if (!sh_css_init_isp_params_from_global(stream, params, false, pipe)) err1 =3D -EINVAL; - } =20 err2 =3D sh_css_init_isp_params_from_config(stream->pipes[0], params, con= fig, pipe); =20 @@ -2004,9 +2003,8 @@ sh_css_init_isp_params_from_config(struct ia_css_pipe= *pipe, * user. */ /* we do not exit from this point immediately to allow internal * firmware feature testing. */ - if (is_dp_10bpp) { + if (is_dp_10bpp) err =3D -EINVAL; - } } else { err =3D -EINVAL; goto exit; @@ -3034,9 +3032,8 @@ process_kernel_parameters(unsigned int pipe_id, ia_css_ob_configure(¶ms->stream_configs.ob, isp_pipe_version, raw_bit_depth); } - if (params->config_changed[IA_CSS_S3A_ID]) { + if (params->config_changed[IA_CSS_S3A_ID]) ia_css_s3a_configure(raw_bit_depth); - } /* Copy stage uds parameters to config, since they can differ per stage. */ params->crop_config.crop_pos =3D params->uds[stage->stage_num].crop_pos; @@ -3899,9 +3896,8 @@ sh_css_invalidate_params(struct ia_css_stream *stream) params->isp_params_changed =3D true; for (i =3D 0; i < IA_CSS_PIPE_ID_NUM; i++) { for (j =3D 0; j < SH_CSS_MAX_STAGES; j++) { - for (mem =3D 0; mem < N_IA_CSS_MEMORIES; mem++) { + for (mem =3D 0; mem < N_IA_CSS_MEMORIES; mem++) params->isp_mem_params_changed[i][j][mem] =3D true; - } } } =20 --=20 2.53.0 From nobody Thu Apr 2 14:09:47 2026 Received: from s.wrqvwxzv.outbound-mail.sendgrid.net (s.wrqvwxzv.outbound-mail.sendgrid.net [149.72.154.232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 09165340A47 for ; Tue, 10 Feb 2026 09:24:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=149.72.154.232 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715445; cv=none; b=Y69N5XPeMAH04NS62LgHz39BiVPAd+Fy5F8l3BVJc4inRdtWPvmwxbcgGBgERAHphMrpHDa6ECxUmj1b9ervr0TcN799oVn0yidJFViQgr9APali0Z+T4Kl52PmJLkyb5hoZMA5tsgk+111ZApYIj+kinK9djHbugmX3TNebhLg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715445; c=relaxed/simple; bh=9jJs7MmJoKsYj3XlZrD6IjmxfM3I0wNi4KUdZUQ/z0w=; h=From:Subject:Date:Message-ID:In-Reply-To:References:MIME-Version: To:Cc:Content-Type; b=Ix0i8KUIXGBUKVacU2FlPEUJh/iHJNEN+tBChS1kHsxR6hJLhUU3YzaEsdC8ON2V92Zm8J9dFg5iAUN3m80LpGI983am14FDMO9Fp5iyhj97EvD2XCEz0JLxPSQgLZS++HgfdQfgTBViMe0/XxXPLu4fKi77cDXNUQcjH/wIxtI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net; spf=pass smtp.mailfrom=em431.mattwardle.net; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b=RC/gdnGS; arc=none smtp.client-ip=149.72.154.232 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=em431.mattwardle.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b="RC/gdnGS" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mattwardle.net; h=from:subject:date:in-reply-to:references:mime-version:to:cc: content-transfer-encoding:content-type:cc:content-type:date:from:subject:to; s=s1; bh=dcfTklgjaCy1rPUq2cimFzYnzz4hXIpUGtmQWpURVCM=; b=RC/gdnGSpA/05ZE2HfkUBNzc3GfZXA0ykVegS6mpfVt//+p9E8fszP2Pq+pXgNiRv9cv WWLiKonqUktfNWT0uyFv0A06R3KrJX2SZ7d5D3m2ZPnBRwjRy0pCfKD1laCXTxsODQiJUA zEq6LMlG0QWanbJcvbJMoVP/N7PbX4ueNI5eKV7vQ09Duq8tlPNoZfTgQvrq4SEAillXda wzURGYKhNHTmJi+OhpuVVEADl1xqy7mvmKz3yg5v7F1E7U6KtoLbxKu1zxC2YbumZTE4sd UzzIMOeJwfBE9IWCw0pmby6/zrIAjRvBjHLBOY1TzbnOinwau68bSF9/gW/fByiQ== Received: by recvd-5fb7fdbd94-cwpcz with SMTP id recvd-5fb7fdbd94-cwpcz-1-698AF932-1 2026-02-10 09:24:02.014661469 +0000 UTC m=+55669.421760190 Received: from mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (unknown) by geopod-ismtpd-9 (SG) with ESMTP id Hm-93m4YR7qK3Sh1Thencg Tue, 10 Feb 2026 09:24:01.896 +0000 (UTC) Authentication-Results: mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net; dmarc=fail (p=reject dis=none) header.from=mattwardle.net Received: from thinkpad (unknown [161.142.117.49]) by mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (Postfix) with ESMTPSA id A8D9040E37; Tue, 10 Feb 2026 09:24:01 +0000 (UTC) From: Matt Wardle Subject: [PATCH 2/3] staging: media: atomisp: Fix function indentation and braces Date: Tue, 10 Feb 2026 09:24:02 +0000 (UTC) Message-ID: <20260210092243.294732-3-matt@mattwardle.net> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260210092243.294732-1-matt@mattwardle.net> References: <20260210092243.294732-1-matt@mattwardle.net> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-SG-EID: =?us-ascii?Q?u001=2EFlbIg1=2FBrpD1HWcEBlj1tgh=2F0VDwewqbkmeZ1SFD9tLmzlDepFYnLmfnO?= =?us-ascii?Q?moWHy7n5HdVZTdyWWeMojZeck4braJp6ggXS5Hg?= =?us-ascii?Q?cJ0Tx4HbZa5NjIBJq7+IQTJAc9yh0Nsf2VPaIe1?= =?us-ascii?Q?fp9P9VSCOImMpkkO6ksDynSXEEjUhrfpshoE10E?= =?us-ascii?Q?O100pz3xsbC3dofMlQ=2FYccprr1I+KW1NQ089kLI?= =?us-ascii?Q?Z37esaK9GizzkqjV4pWtb=2FRD3g5ANuf83353awU?= =?us-ascii?Q?58lY?= To: Hans de Goede , Mauro Carvalho Chehab , Andy Shevchenko Cc: Sakari Ailus , Greg Kroah-Hartman , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Matt Wardle X-Entity-ID: u001.OAl4HwJfzvoR6h5DRlrySA== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Fix parameter indentation for functions and move opening braces onto new line. Fix checkpatch.pl errors: ERROR: open brace '{' following function definitions go on the next line Signed-off-by: Matt Wardle --- .../isp/kernels/dvs/dvs_1.0/ia_css_dvs.host.c | 11 ++--- .../isp/kernels/raw/raw_1.0/ia_css_raw.host.c | 4 +- .../kernels/sdis/sdis_2/ia_css_sdis2.host.c | 7 ++- .../isp/kernels/vf/vf_1.0/ia_css_vf.host.c | 22 ++++----- .../atomisp/pci/runtime/binary/src/binary.c | 48 +++++++++---------- .../pci/runtime/isp_param/src/isp_param.c | 18 ++++--- 6 files changed, 52 insertions(+), 58 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_c= ss_dvs.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/i= a_css_dvs.host.c index e9d6dd0bbfe2..4c85b5a62224 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_css_dvs.= host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_css_dvs.= host.c @@ -258,12 +258,11 @@ convert_allocate_dvs_6axis_config( return me; } =20 -int -store_dvs_6axis_config( - const struct ia_css_dvs_6axis_config *dvs_6axis_config, - const struct ia_css_binary *binary, - const struct ia_css_frame_info *dvs_in_frame_info, - ia_css_ptr ddr_addr_y) { +int store_dvs_6axis_config(const struct ia_css_dvs_6axis_config *dvs_6axis= _config, + const struct ia_css_binary *binary, + const struct ia_css_frame_info *dvs_in_frame_info, + ia_css_ptr ddr_addr_y) +{ struct ia_css_host_data *me; =20 assert(dvs_6axis_config); diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_c= ss_raw.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/i= a_css_raw.host.c index a00f8d049a33..80fd64a8eb98 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_css_raw.= host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_css_raw.= host.c @@ -21,8 +21,8 @@ static const struct ia_css_raw_configuration default_conf= ig =3D { }; =20 /* MW: These areMIPI / ISYS properties, not camera function properties */ -static enum sh_stream_format -css2isp_stream_format(enum atomisp_input_format from) { +static enum sh_stream_format css2isp_stream_format(enum atomisp_input_form= at from) +{ switch (from) { case ATOMISP_INPUT_FORMAT_YUV420_8_LEGACY: diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_c= ss_sdis2.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2= /ia_css_sdis2.host.c index 027eae0ca69e..aaabb3388044 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_css_sdis= 2.host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_css_sdis= 2.host.c @@ -164,10 +164,9 @@ void ia_css_sdis2_clear_coefficients( dvs2_coefs->ver_coefs.even_imag =3D NULL; } =20 -int -ia_css_get_dvs2_statistics( - struct ia_css_dvs2_statistics *host_stats, - const struct ia_css_isp_dvs_statistics *isp_stats) { +int ia_css_get_dvs2_statistics(struct ia_css_dvs2_statistics *host_stats, + const struct ia_css_isp_dvs_statistics *isp_stats) +{ struct ia_css_isp_dvs_statistics_map *map; int ret =3D 0; =20 diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css= _vf.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css= _vf.host.c index 3c675063c4a7..f8870f7e790d 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.hos= t.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.hos= t.c @@ -45,11 +45,10 @@ int ia_css_vf_config(struct sh_css_isp_vf_isp_config = *to, * to the requested viewfinder resolution on the upper side. The output ca= nnot * be smaller than the requested viewfinder resolution. */ -int -sh_css_vf_downscale_log2( - const struct ia_css_frame_info *out_info, - const struct ia_css_frame_info *vf_info, - unsigned int *downscale_log2) { +int sh_css_vf_downscale_log2(const struct ia_css_frame_info *out_info, + const struct ia_css_frame_info *vf_info, + unsigned int *downscale_log2) +{ unsigned int ds_log2 =3D 0; unsigned int out_width; =20 @@ -80,13 +79,12 @@ sh_css_vf_downscale_log2( return 0; } =20 -static int -configure_kernel( - const struct ia_css_binary_info *info, - const struct ia_css_frame_info *out_info, - const struct ia_css_frame_info *vf_info, - unsigned int *downscale_log2, - struct ia_css_vf_configuration *config) { +static int configure_kernel(const struct ia_css_binary_info *info, + const struct ia_css_frame_info *out_info, + const struct ia_css_frame_info *vf_info, + unsigned int *downscale_log2, + struct ia_css_vf_configuration *config) +{ int err; unsigned int vf_log_ds =3D 0; =20 diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c = b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c index 60ae7bf5512b..c7962549e999 100644 --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c @@ -347,10 +347,10 @@ ia_css_binary_dvs_stat_grid_info( return; } =20 -int -ia_css_binary_3a_grid_info(const struct ia_css_binary *binary, - struct ia_css_grid_info *info, - struct ia_css_pipe *pipe) { +int ia_css_binary_3a_grid_info(const struct ia_css_binary *binary, + struct ia_css_grid_info *info, + struct ia_css_pipe *pipe) +{ struct ia_css_3a_grid_info *s3a_info; int err =3D 0; =20 @@ -439,9 +439,9 @@ supports_bds_factor(u32 supported_factors, return ((supported_factors & PACK_BDS_FACTOR(bds_factor)) !=3D 0); } =20 -static int -binary_init_info(struct ia_css_binary_xinfo *info, unsigned int i, - bool *binary_found) { +static int binary_init_info(struct ia_css_binary_xinfo *info, unsigned int= i, + bool *binary_found) +{ const unsigned char *blob =3D sh_css_blob_info[i].blob; unsigned int size =3D sh_css_blob_info[i].header.blob.size; =20 @@ -464,8 +464,8 @@ binary_init_info(struct ia_css_binary_xinfo *info, unsi= gned int i, /* When binaries are put at the beginning, they will only * be selected if no other primary matches. */ -int -ia_css_binary_init_infos(void) { +int ia_css_binary_init_infos(void) +{ unsigned int i; unsigned int num_of_isp_binaries =3D sh_css_num_binaries - NUM_OF_SPS - N= UM_OF_BLS; =20 @@ -497,8 +497,8 @@ ia_css_binary_init_infos(void) { return 0; } =20 -int -ia_css_binary_uninit(void) { +int ia_css_binary_uninit(void) +{ unsigned int i; struct ia_css_binary_xinfo *b; =20 @@ -625,19 +625,19 @@ binary_in_frame_padded_width(int in_frame_width, return rval; } =20 -int -ia_css_binary_fill_info(const struct ia_css_binary_xinfo *xinfo, - bool online, - bool two_ppc, - enum atomisp_input_format stream_format, - const struct ia_css_frame_info *in_info, /* can be NULL */ - const struct ia_css_frame_info *bds_out_info, /* can be NULL */ - const struct ia_css_frame_info *out_info[], /* can be NULL */ - const struct ia_css_frame_info *vf_info, /* can be NULL */ - struct ia_css_binary *binary, - struct ia_css_resolution *dvs_env, - int stream_config_left_padding, - bool accelerator) { +int ia_css_binary_fill_info(const struct ia_css_binary_xinfo *xinfo, + bool online, + bool two_ppc, + enum atomisp_input_format stream_format, + const struct ia_css_frame_info *in_info, /* can be NULL */ + const struct ia_css_frame_info *bds_out_info, /* can be NULL */ + const struct ia_css_frame_info *out_info[], /* can be NULL */ + const struct ia_css_frame_info *vf_info, /* can be NULL */ + struct ia_css_binary *binary, + struct ia_css_resolution *dvs_env, + int stream_config_left_padding, + bool accelerator) +{ const struct ia_css_binary_info *info =3D &xinfo->sp; unsigned int dvs_env_width =3D 0, dvs_env_height =3D 0, diff --git a/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_pa= ram.c b/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c index 251dd75a7613..354e5405fd60 100644 --- a/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c +++ b/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c @@ -93,11 +93,10 @@ ia_css_init_memory_interface( } } =20 -int -ia_css_isp_param_allocate_isp_parameters( - struct ia_css_isp_param_host_segments *mem_params, - struct ia_css_isp_param_css_segments *css_params, - const struct ia_css_isp_param_isp_segments *mem_initializers) { +int ia_css_isp_param_allocate_isp_parameters(struct ia_css_isp_param_host_= segments *mem_params, + struct ia_css_isp_param_css_segments *css_params, + const struct ia_css_isp_param_isp_segments *mem_initializers) +{ int err =3D 0; unsigned int mem, pclass; =20 @@ -171,11 +170,10 @@ ia_css_isp_param_load_fw_params( } } =20 -int -ia_css_isp_param_copy_isp_mem_if_to_ddr( - struct ia_css_isp_param_css_segments *ddr, - const struct ia_css_isp_param_host_segments *host, - enum ia_css_param_class pclass) { +int ia_css_isp_param_copy_isp_mem_if_to_ddr(struct ia_css_isp_param_css_se= gments *ddr, + const struct ia_css_isp_param_host_segments *host, + enum ia_css_param_class pclass) +{ unsigned int mem; =20 for (mem =3D 0; mem < N_IA_CSS_ISP_MEMORIES; mem++) --=20 2.53.0 From nobody Thu Apr 2 14:09:47 2026 Received: from s.wrqvtbkv.outbound-mail.sendgrid.net (s.wrqvtbkv.outbound-mail.sendgrid.net [149.72.123.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3B9AB3242B2 for ; Tue, 10 Feb 2026 09:24:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=149.72.123.24 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715457; cv=none; b=i5gcCWstyt9k7aZKyKzt+2CrAOkbrHuE8oX7iZWxMPA7qE0gdawHYS1pdI9nAXJ7kR5A3jwESBBEY4FvD8KmVv3MQeQ03Qe+6AvfGdpgLrmmSaGuAaYhSundDWUW7p1jqSaCSBqip+vewylkGgNueGONN8+S7Fak0nAC67X4xeY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770715457; c=relaxed/simple; bh=mdHV/OrvcoTOfYGXpBzBJ5qf8muzn3HLdO3CHr+Ip3Y=; h=From:Subject:Date:Message-ID:In-Reply-To:References:MIME-Version: To:Cc:Content-Type; b=rtx3zQ1zia2lkSxmaw40vKwU+l0MmiaHJiLRQu+3dQrrXi05hEvmxREkULLp7bBcfSNpvziBulTe5vO7c476Mg3LzTzFLwAPvRv9r4ty5jhUa8FXyZfUXuqsDRO4TUpnKMx+42S2/xLt8zP4F0MPRlllgZyYQ9E4lGrHiQgQKi0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net; spf=pass smtp.mailfrom=em431.mattwardle.net; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b=lp0SA8Iv; arc=none smtp.client-ip=149.72.123.24 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=mattwardle.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=em431.mattwardle.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=mattwardle.net header.i=@mattwardle.net header.b="lp0SA8Iv" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mattwardle.net; h=from:subject:date:in-reply-to:references:mime-version:to:cc: content-transfer-encoding:content-type:cc:content-type:date:from:subject:to; s=s1; bh=snQso672sutlEFlqaXhUA/V91FaIPJyTuuy2toBCCeM=; b=lp0SA8IvRapHLb7CAvDenq94EAZbFwKGcj2l1P/PRxN0Waj54CWf30MSl0d1Q6STusvs VKfnJzqG7LyaAcI54dJYPs1CZlCXyA9KHjYA1MGJkWFLCu26vCS59JlQF87jqjrnCHEdBh MjTn4BHpsVCXyOcu9H5ubTnik8ljwxMCcnaezH5t5354p1MOOdoKtzO8PADtgSVtuAIQL0 Vr+CPGAvjgshU7OLXATjS3DAcjK/dAnSJcKoRbgvpqNq2sej3EvwK3EiTZhMOBuc5SIeAB /NXyOrdPH0B2J44A8F5kZ1CL0dgYdeUwx3e1xnjdjGzupLrBGi83v5yr5rjSYoPw== Received: by recvd-5fb7fdbd94-cwpcz with SMTP id recvd-5fb7fdbd94-cwpcz-1-698AF93F-E 2026-02-10 09:24:15.304883989 +0000 UTC m=+55682.711982724 Received: from mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (unknown) by geopod-ismtpd-3 (SG) with ESMTP id 0-zhEk9CQjCSmtlsjGGwsg Tue, 10 Feb 2026 09:24:05.513 +0000 (UTC) Authentication-Results: mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net; dmarc=fail (p=reject dis=none) header.from=mattwardle.net Received: from thinkpad (unknown [161.142.117.49]) by mail.3n0xdopx5uce3ivigaxioahche.ix.internal.cloudapp.net (Postfix) with ESMTPSA id 57BDE40E37; Tue, 10 Feb 2026 09:24:05 +0000 (UTC) From: Matt Wardle Subject: [PATCH 3/3] staging: media: atomisp: Fix braces on incorrect lines Date: Tue, 10 Feb 2026 09:24:15 +0000 (UTC) Message-ID: <20260210092243.294732-4-matt@mattwardle.net> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260210092243.294732-1-matt@mattwardle.net> References: <20260210092243.294732-1-matt@mattwardle.net> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-SG-EID: =?us-ascii?Q?u001=2EFlbIg1=2FBrpD1HWcEBlj1tgh=2F0VDwewqbkmeZ1SFD9tLmzlDepFYnLmfnO?= =?us-ascii?Q?moWHy7n5HdVZTdyWWeMojZeck4brVYul=2FIqvcwM?= =?us-ascii?Q?d58FkwoGE+aZ=2FgGAunHiGqEHnhv=2FBXvNq3Sld2d?= =?us-ascii?Q?LlXs9vZ21PJ+CaCiAVfYXkHFKFIS+L7TS+2bQ8L?= =?us-ascii?Q?WgJ1EBT4r2+NDmY5YYxqHeO1KduwgyO3RJ8j5UT?= =?us-ascii?Q?RMRZ1FWIjMxRsWqIiV11zrXjWdF0DB53AB2Q2Pk?= =?us-ascii?Q?6klv?= To: Hans de Goede , Mauro Carvalho Chehab , Andy Shevchenko Cc: Sakari Ailus , Greg Kroah-Hartman , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Matt Wardle X-Entity-ID: u001.OAl4HwJfzvoR6h5DRlrySA== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Fix checkpatch.pl errors: ERROR: that open brace { should be on the previous line Signed-off-by: Matt Wardle --- .../isp/kernels/dvs/dvs_1.0/ia_css_dvs.host.c | 3 +- .../isp/kernels/raw/raw_1.0/ia_css_raw.host.c | 3 +- .../kernels/sdis/sdis_2/ia_css_sdis2.host.c | 6 +- .../isp/kernels/vf/vf_1.0/ia_css_vf.host.c | 3 +- .../atomisp/pci/runtime/binary/src/binary.c | 60 +++++++------------ .../pci/runtime/isp_param/src/isp_param.c | 6 +- drivers/staging/media/atomisp/pci/sh_css.c | 6 +- 7 files changed, 29 insertions(+), 58 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_c= ss_dvs.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/i= a_css_dvs.host.c index 4c85b5a62224..cf6c29155758 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_css_dvs.= host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/dvs/dvs_1.0/ia_css_dvs.= host.c @@ -273,8 +273,7 @@ int store_dvs_6axis_config(const struct ia_css_dvs_6axi= s_config *dvs_6axis_confi binary, dvs_in_frame_info); =20 - if (!me) - { + if (!me) { IA_CSS_LEAVE_ERR_PRIVATE(-ENOMEM); return -ENOMEM; } diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_c= ss_raw.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/i= a_css_raw.host.c index 80fd64a8eb98..fb0e2a88cadb 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_css_raw.= host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/raw/raw_1.0/ia_css_raw.= host.c @@ -23,8 +23,7 @@ static const struct ia_css_raw_configuration default_conf= ig =3D { /* MW: These areMIPI / ISYS properties, not camera function properties */ static enum sh_stream_format css2isp_stream_format(enum atomisp_input_form= at from) { - switch (from) - { + switch (from) { case ATOMISP_INPUT_FORMAT_YUV420_8_LEGACY: return sh_stream_format_yuv420_legacy; case ATOMISP_INPUT_FORMAT_YUV420_8: diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_c= ss_sdis2.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2= /ia_css_sdis2.host.c index aaabb3388044..6f92709f87eb 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_css_sdis= 2.host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/sdis/sdis_2/ia_css_sdis= 2.host.c @@ -176,13 +176,11 @@ int ia_css_get_dvs2_statistics(struct ia_css_dvs2_sta= tistics *host_stats, assert(isp_stats); =20 map =3D 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_dvs2_statistics(host_stats, map); ia_css_isp_dvs_statistics_map_free(map); - } else - { + } else { IA_CSS_ERROR("out of memory"); ret =3D -ENOMEM; } diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css= _vf.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css= _vf.host.c index f8870f7e790d..152faab2b169 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.hos= t.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.hos= t.c @@ -64,8 +64,7 @@ int sh_css_vf_downscale_log2(const struct ia_css_frame_in= fo *out_info, * test for the height since the vmem buffers only put restrictions on * the width of a line, not on the number of lines in a frame. */ - while (out_width >=3D vf_info->res.width) - { + while (out_width >=3D vf_info->res.width) { ds_log2++; out_width /=3D 2; } diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c = b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c index c7962549e999..39b37b557aff 100644 --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c @@ -477,8 +477,7 @@ int ia_css_binary_init_infos(void) if (!all_binaries) return -ENOMEM; =20 - for (i =3D 0; i < num_of_isp_binaries; i++) - { + for (i =3D 0; i < num_of_isp_binaries; i++) { int ret; struct ia_css_binary_xinfo *binary =3D &all_binaries[i]; bool binary_found; @@ -502,8 +501,7 @@ int ia_css_binary_uninit(void) unsigned int i; struct ia_css_binary_xinfo *b; =20 - for (i =3D 0; i < IA_CSS_BINARY_NUM_MODES; i++) - { + for (i =3D 0; i < IA_CSS_BINARY_NUM_MODES; i++) { for (b =3D binary_infos[i]; b; b =3D b->next) { if (b->xmem_addr) hmm_free(b->xmem_addr); @@ -664,8 +662,7 @@ int ia_css_binary_fill_info(const struct ia_css_binary_= xinfo *xinfo, assert(binary); =20 binary->info =3D xinfo; - if (!accelerator) - { + if (!accelerator) { /* binary->css_params has been filled by accelerator itself. */ err =3D ia_css_isp_param_allocate_isp_parameters( &binary->mem_params, &binary->css_params, @@ -673,15 +670,13 @@ int ia_css_binary_fill_info(const struct ia_css_binar= y_xinfo *xinfo, if (err) return err; } - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) - { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { if (out_info[i] && (out_info[i]->res.width !=3D 0)) { bin_out_info =3D out_info[i]; break; } } - if (in_info && bin_out_info) - { + if (in_info && bin_out_info) { need_scaling =3D (in_info->res.width !=3D bin_out_info->res.width) || (in_info->res.height !=3D bin_out_info->res.height); } @@ -712,8 +707,7 @@ int ia_css_binary_fill_info(const struct ia_css_binary_= xinfo *xinfo, binary->internal_frame_info.res.height =3D isp_internal_height; binary->internal_frame_info.raw_bit_depth =3D bits_per_pixel; =20 - if (in_info) - { + if (in_info) { binary->effective_in_frame_res.width =3D in_info->res.width; binary->effective_in_frame_res.height =3D in_info->res.height; =20 @@ -741,15 +735,13 @@ int ia_css_binary_fill_info(const struct ia_css_binar= y_xinfo *xinfo, binary->in_frame_info.crop_info =3D in_info->crop_info; } =20 - if (online) - { + if (online) { bits_per_pixel =3D ia_css_util_input_format_bpp( stream_format, two_ppc); } binary->in_frame_info.raw_bit_depth =3D bits_per_pixel; =20 - for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) - { + for (i =3D 0; i < IA_CSS_BINARY_MAX_OUTPUT_PORTS; i++) { if (out_info[i]) { binary->out_frame_info[i].res.width =3D out_info[i]->res.width; binary->out_frame_info[i].res.height =3D out_info[i]->res.height; @@ -768,8 +760,7 @@ int ia_css_binary_fill_info(const struct ia_css_binary_= xinfo *xinfo, } } =20 - if (vf_info && (vf_info->res.width !=3D 0)) - { + if (vf_info && (vf_info->res.width !=3D 0)) { err =3D ia_css_vf_configure(binary, bin_out_info, (struct ia_css_frame_info *)vf_info, &vf_log_ds); if (err) { @@ -787,8 +778,7 @@ int ia_css_binary_fill_info(const struct ia_css_binary_= xinfo *xinfo, binary->input_format =3D stream_format; =20 /* viewfinder output info */ - if ((vf_info) && (vf_info->res.width !=3D 0)) - { + if ((vf_info) && (vf_info->res.width !=3D 0)) { unsigned int vf_out_vecs, vf_out_width, vf_out_height; =20 binary->vf_frame_info.format =3D vf_info->format; @@ -820,23 +810,20 @@ int ia_css_binary_fill_info(const struct ia_css_binar= y_xinfo *xinfo, binary->vf_frame_info.padded_width =3D vf_out_width; binary->vf_frame_info.res.height =3D vf_out_height; } - } else - { + } else { binary->vf_frame_info.res.width =3D 0; binary->vf_frame_info.padded_width =3D 0; binary->vf_frame_info.res.height =3D 0; } =20 - if (info->enable.ca_gdc) - { + if (info->enable.ca_gdc) { binary->morph_tbl_width =3D _ISP_MORPH_TABLE_WIDTH(isp_internal_width); binary->morph_tbl_aligned_width =3D _ISP_MORPH_TABLE_ALIGNED_WIDTH(isp_internal_width); binary->morph_tbl_height =3D _ISP_MORPH_TABLE_HEIGHT(isp_internal_height); - } else - { + } else { binary->morph_tbl_width =3D 0; binary->morph_tbl_aligned_width =3D 0; binary->morph_tbl_height =3D 0; @@ -846,8 +833,7 @@ int ia_css_binary_fill_info(const struct ia_css_binary_= xinfo *xinfo, sc_3a_dis_padded_width =3D binary->in_frame_info.padded_width; sc_3a_dis_height =3D binary->in_frame_info.res.height; if (bds_out_info && in_info && - bds_out_info->res.width !=3D in_info->res.width) - { + bds_out_info->res.width !=3D in_info->res.width) { /* TODO: Next, "internal_frame_info" should be derived from * bds_out. So this part will change once it is in place! */ sc_3a_dis_width =3D bds_out_info->res.width + info->pipeline.left_croppi= ng; @@ -857,18 +843,15 @@ int ia_css_binary_fill_info(const struct ia_css_binar= y_xinfo *xinfo, =20 s3a_isp_width =3D _ISP_S3A_ELEMS_ISP_WIDTH(sc_3a_dis_padded_width, info->pipeline.left_cropping); - if (info->s3a.fixed_s3a_deci_log) - { + if (info->s3a.fixed_s3a_deci_log) { s3a_log_deci =3D info->s3a.fixed_s3a_deci_log; - } else - { + } else { s3a_log_deci =3D binary_grid_deci_factor_log2(s3a_isp_width, sc_3a_dis_height); } binary->deci_factor_log2 =3D s3a_log_deci; =20 - if (info->enable.s3a) - { + if (info->enable.s3a) { binary->s3atbl_width =3D _ISP_S3ATBL_WIDTH(sc_3a_dis_width, s3a_log_deci); @@ -881,21 +864,18 @@ int ia_css_binary_fill_info(const struct ia_css_binar= y_xinfo *xinfo, binary->s3atbl_isp_height =3D _ISP_S3ATBL_ISP_HEIGHT(sc_3a_dis_height, s3a_log_deci); - } else - { + } else { binary->s3atbl_width =3D 0; binary->s3atbl_height =3D 0; binary->s3atbl_isp_width =3D 0; binary->s3atbl_isp_height =3D 0; } =20 - if (info->enable.sc) - { + if (info->enable.sc) { binary->sctbl_width_per_color =3D _ISP_SCTBL_WIDTH_PER_COLOR(sc_3a_dis_p= added_width, s3a_log_deci); binary->sctbl_aligned_width_per_color =3D SH_CSS_MAX_SCTBL_ALIGNED_WIDTH= _PER_COLOR; binary->sctbl_height =3D _ISP_SCTBL_HEIGHT(sc_3a_dis_height, s3a_log_dec= i); - } else - { + } else { binary->sctbl_width_per_color =3D 0; binary->sctbl_aligned_width_per_color =3D 0; binary->sctbl_height =3D 0; diff --git a/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_pa= ram.c b/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c index 354e5405fd60..1d20eb650757 100644 --- a/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c +++ b/drivers/staging/media/atomisp/pci/runtime/isp_param/src/isp_param.c @@ -101,8 +101,7 @@ int ia_css_isp_param_allocate_isp_parameters(struct ia_= css_isp_param_host_segmen unsigned int mem, pclass; =20 pclass =3D IA_CSS_PARAM_CLASS_PARAM; - for (mem =3D 0; mem < IA_CSS_NUM_MEMORIES; mem++) - { + for (mem =3D 0; mem < IA_CSS_NUM_MEMORIES; mem++) { for (pclass =3D 0; pclass < IA_CSS_NUM_PARAM_CLASSES; pclass++) { u32 size =3D 0; =20 @@ -176,8 +175,7 @@ int ia_css_isp_param_copy_isp_mem_if_to_ddr(struct ia_c= ss_isp_param_css_segments { unsigned int mem; =20 - for (mem =3D 0; mem < N_IA_CSS_ISP_MEMORIES; mem++) - { + for (mem =3D 0; mem < N_IA_CSS_ISP_MEMORIES; mem++) { size_t size =3D host->params[pclass][mem].size; ia_css_ptr ddr_mem_ptr =3D ddr->params[pclass][mem].address; char *host_mem_ptr =3D host->params[pclass][mem].address; diff --git a/drivers/staging/media/atomisp/pci/sh_css.c b/drivers/staging/m= edia/atomisp/pci/sh_css.c index 73bd87f43a8c..e7f2443696d1 100644 --- a/drivers/staging/media/atomisp/pci/sh_css.c +++ b/drivers/staging/media/atomisp/pci/sh_css.c @@ -2256,8 +2256,7 @@ alloc_continuous_frames(struct ia_css_pipe *pipe, boo= l init_time) ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "alloc_continuous_frames() IA_CSS_FRAME_FORMAT_RAW_PACKED\n"); ref_info.format =3D IA_CSS_FRAME_FORMAT_RAW_PACKED; - } else - { + } else { ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE, "alloc_continuous_frames() IA_CSS_FRAME_FORMAT_RAW\n"); ref_info.format =3D IA_CSS_FRAME_FORMAT_RAW; @@ -7862,8 +7861,7 @@ ia_css_stream_create(const struct ia_css_stream_confi= g *stream_config, =20 /* check if mipi size specified */ if (stream_config->mode =3D=3D IA_CSS_INPUT_MODE_BUFFERED_SENSOR) - if (!IS_ISP2401 || !stream_config->online) - { + if (!IS_ISP2401 || !stream_config->online) { unsigned int port =3D (unsigned int)stream_config->source.port.port; =20 if (port >=3D N_MIPI_PORT_ID) { --=20 2.53.0