On Fri, 2018-03-30 at 15:07 +0530, Chaitra P B wrote: > + switch (event_data->ReasonCode) { > + > + case MPI25_EVENT_SAS_DISC_ERR_SMP_FAILED: > + pr_warn(MPT3SAS_FMT "SMP command sent to the expander" > + "(handle:0x%04x, sas_address:0x%016llx," > + "physical_port:0x%02x) has failed", > + ioc->name, le16_to_cpu(event_data->DevHandle), > + (unsigned long long)le64_to_cpu(event_data->SASAddress), > + event_data->PhysicalPort); > + break; > + > + case MPI25_EVENT_SAS_DISC_ERR_SMP_TIMEOUT: Please follow the style that is used elsewhere in the Linux kernel and do not leave blank lines after "switch (...) {" nor after "break". Thanks, Bart.