On 3/2/21 23:22, Kanchan Joshi wrote: > + switch (bcmd->ioctl_cmd) { > + case NVME_IOCTL_IO_CMD: > + ret = nvme_user_cmd(ns->ctrl, ns, argp, ioucmd); > + break; > + default: > + ret = -ENOTTY; > + } Switch for one case ? why not use if else ?