> On Nov 12, 2024, at 13:09, Bart Van Assche <bvanassche@xxxxxxx> wrote: > > On 11/8/24 8:45 PM, himanshu.madhani@xxxxxxxxxx wrote: >> + switch(sdev->mpath_state) { >> + case SCSI_MPATH_OPTIMAL: >> + if (distance < found_distance) { >> + found_distance = distance; >> + sdev_found = sdev; >> + } > > Please follow the Linux kernel coding style. As an example, I have never > seen anyone else indenting statements under a case label in the kernel > with four spaces. You may want to run the entire patch series through > clang-format. > Sure will do that. > Thanks, > > Bart.