On Tue, Sep 24, 2019 at 07:33:02AM +0000, Mehta, Sanju wrote: > +static const struct file_operations pt_debugfs_info_ops = { > + .owner = THIS_MODULE, > + .open = simple_open, > + .read = ptdma_debugfs_info_read, > + .write = NULL, > +}; > + > +static const struct file_operations pt_debugfs_queue_ops = { > + .owner = THIS_MODULE, > + .open = simple_open, > + .read = ptdma_debugfs_queue_read, > + .write = ptdma_debugfs_queue_write, > +}; > + > +static const struct file_operations pt_debugfs_stats_ops = { > + .owner = THIS_MODULE, > + .open = simple_open, > + .read = ptdma_debugfs_stats_read, > + .write = ptdma_debugfs_stats_write, > +}; Can you use DEFINE_SIMPLE_ATTRIBUTE() here intead of these? thanks, greg k-h