On 9/25/21 6:10 PM, Xu Yilun wrote: >>>> >>>> + cdev_init(&imgld->cdev, &fpga_image_load_fops); >>>> + imgld->cdev.owner = parent->driver->owner; >>>> + imgld->cdev.kobj.parent = &imgld->dev.kobj; >>> Could be replaced by cdev_set_parent() > Sorry I ended last mail before this comment, and you may not notice it. Yes - I did miss that. Thanks! - Russ > > Thanks, > Yilun >