drivers: can: common: terminate if ... else if ... constructs with else

henrikbrixandersen · kartben · commit d5f07eb8c9d0 · 2025-05-09T18:00:27.000+02:00
Terminate "if ... else if ..." constructs with an empty "else" clause to
indicate that consideration has been given regarding the behaviour when all
other conditions evaluate to false.

Signed-off-by: Henrik Brix Andersen &lt;hebad@vestas.com&gt;
diff --git a/drivers/can/can_common.c b/drivers/can/can_common.c
@@ -183,6 +183,8 @@ static int update_sample_pnt(uint32_t total_tq, uint32_t sample_pnt, struct can_
 		if (tseg2 < min->phase_seg2) {
 			return -ENOTSUP;
 		}
+	} else {
+		/* Sample point location within range */
 	}
 
 	res->phase_seg2 = tseg2;
@@ -199,6 +201,8 @@ static int update_sample_pnt(uint32_t total_tq, uint32_t sample_pnt, struct can_
 		/* Even tseg1 distribution not possible, increase phase_seg1 */
 		res->phase_seg1 = min->phase_seg1;
 		res->prop_seg = tseg1 - res->phase_seg1;
+	} else {
+		/* No redistribution necessary */
 	}
 
 	/* Calculate the resulting sample point */

Original file line number	Diff line number	Diff line change
`@@ -183,6 +183,8 @@ static int update_sample_pnt(uint32_t total_tq, uint32_t sample_pnt, struct can_`
`183`	`183`	`if (tseg2 < min->phase_seg2) {`
`184`	`184`	`return -ENOTSUP;`
`185`	`185`	`}`
	`186`	`+ } else {`
	`187`	`+ /* Sample point location within range */`
`186`	`188`	`}`
`187`	`189`
`188`	`190`	`res->phase_seg2 = tseg2;`
`@@ -199,6 +201,8 @@ static int update_sample_pnt(uint32_t total_tq, uint32_t sample_pnt, struct can_`
`199`	`201`	`/* Even tseg1 distribution not possible, increase phase_seg1 */`
`200`	`202`	`res->phase_seg1 = min->phase_seg1;`
`201`	`203`	`res->prop_seg = tseg1 - res->phase_seg1;`
	`204`	`+ } else {`
	`205`	`+ /* No redistribution necessary */`
`202`	`206`	`}`
`203`	`207`
`204`	`208`	`/* Calculate the resulting sample point */`