On Thu, 17 Aug 2017 19:52:42 +0800 Aviad Krawczyk <aviad.krawc...@huawei.com> wrote:
> + > +/** > + * init_pfhwdev - Initialize the extended components of PF > + * @pfhwdev: the HW device for PF > + * > + * Return 0 - success, negative - failure > + **/ > +static int init_pfhwdev(struct hinic_pfhwdev *pfhwdev) > +{ > + /* Initialize PF HW device extended components */ > + return 0; > +} > + > +/** > + * free_pfhwdev - Free the extended components of PF > + * @pfhwdev: the HW device for PF > + **/ > +static void free_pfhwdev(struct hinic_pfhwdev *pfhwdev) > +{ > +} Please drop these functions, they do nothing and are not used as stubs in any operations table.