On Wed, Mar 06, 2024 at 09:58:53PM +0800, Xin Zeng wrote: > @@ -258,6 +259,20 @@ struct adf_dc_ops { > void (*build_deflate_ctx)(void *ctx); > }; > > +struct qat_migdev_ops { > + int (*init)(struct qat_mig_dev *mdev); > + void (*cleanup)(struct qat_mig_dev *mdev); > + void (*reset)(struct qat_mig_dev *mdev); > + int (*open)(struct qat_mig_dev *mdev); > + void (*close)(struct qat_mig_dev *mdev); > + int (*suspend)(struct qat_mig_dev *mdev); > + int (*resume)(struct qat_mig_dev *mdev); > + int (*save_state)(struct qat_mig_dev *mdev); > + int (*save_setup)(struct qat_mig_dev *mdev); > + int (*load_state)(struct qat_mig_dev *mdev); > + int (*load_setup)(struct qat_mig_dev *mdev, int size); > +}; Why do we still have these ops? There is only one implementation Jason