Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1591433 > unrolled thread

[PATCH 5/7] staging: media: Remove multiple assignments

Started bysimran singhal <singhalsimran0@gmail.com>
First post2017-03-02 21:00 +0100
Last post2017-03-02 21:00 +0100
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 5/7] staging: media: Remove multiple assignments simran singhal <singhalsimran0@gmail.com> - 2017-03-02 21:00 +0100

#1591433 — [PATCH 5/7] staging: media: Remove multiple assignments

Fromsimran singhal <singhalsimran0@gmail.com>
Date2017-03-02 21:00 +0100
Subject[PATCH 5/7] staging: media: Remove multiple assignments
Message-ID<tgF3k-72m-29@gated-at.bofh.it>
Remove multiple assignments by factorizing them. Problem found using
checkpatch.pl

CHECK: multiple assignments should be avoided

Signed-off-by: simran singhal <singhalsimran0@gmail.com>
---
 drivers/staging/media/atomisp/i2c/gc2235.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/media/atomisp/i2c/gc2235.c b/drivers/staging/media/atomisp/i2c/gc2235.c
index 198df22..165dcb3 100644
--- a/drivers/staging/media/atomisp/i2c/gc2235.c
+++ b/drivers/staging/media/atomisp/i2c/gc2235.c
@@ -259,7 +259,8 @@ static int gc2235_get_intg_factor(struct i2c_client *client,
 		return -EINVAL;
 
 	/* pixel clock calculattion */
-	buf->vt_pix_clk_freq_mhz = dev->vt_pix_clk_freq_mhz = 30000000;
+	buf->vt_pix_clk_freq_mhz = 30000000;
+	dev->vt_pix_clk_freq_mhz = 30000000;
 
 	/* get integration time */
 	buf->coarse_integration_time_min = GC2235_COARSE_INTG_TIME_MIN;
-- 
2.7.4

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web