Thread (18 messages) 18 messages, 3 authors, 2018-08-30

Re: [PATCH v3 6/6] mmc: tmio: refactor CLK_CTL bit calculation

From: Wolfram Sang <hidden>
Date: 2018-08-29 08:49:17
Also in: linux-renesas-soc, lkml

+	if (divisor <= 1) {
+		clk_sel = 1;
+		clk = 0;
+	} else {
+		clk_sel = 0;
+		/* bit7 set: 1/512, ... bit0 set:1/4, all bits clear: 1/2 */
+		clk = roundup_pow_of_two(divisor) >> 2;
+	}
What about

	clk_sel = (divisor <= 1);
	clk = clk_sel ? 0 : (roundup_pow_of_two(divisor) >> 2)

More concise, but I think still readable. I don't mind super much,
though.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help