From: Igor Russkikh <Igor.Russkikh@xxxxxxxxxxxx> Date: Fri, 5 Oct 2018 10:24:55 +0000 > + switch (aqc111_data->link_speed) { > + case AQ_INT_SPEED_5G: > + { > + link_speed = 5000; > + reg8 = 0x05; > + reg16 = 0x001F; > + break; > + } Please do not put curly braces around switch cases. You aren't declaring local scope variables in these case statements so the curly braces are completely unnecessary and look ugly.