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.