On Sun, Aug 02, 2015 at 12:06:31AM +0200, Hermann Kraus wrote:

> +     u32 speed_hz = tfr->speed_hz ? : spi->max_speed_hz;

You don't need to do this anyway since the framework will always ensure
that speed_hz is set so you can just use it unconditionally but in
general please don't use the ternery operator unless it's adding
something, it's not a legibility triumph.

Attachment: signature.asc
Description: Digital signature

Reply via email to