If you change the return statement here then it will also affect RK_FBIOSET_YUV_ADDR, RK_FBIOSET_ENABLE and other cases that don't include a return statement. I think it might be better to make the change in the default case, after checking what dev_drv->ioctl() returns.