> -----Original Message----- > From: Bart Van Assche [mailto:Bart.VanAssche@xxxxxxx] > Sent: Thursday, December 21, 2017 10:40 AM > To: jejb@xxxxxxxxxxxxxxxxxx; Raghava Aditya Renukunta > <RaghavaAditya.Renukunta@xxxxxxxxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx; > martin.petersen@xxxxxxxxxx > Cc: dl-esc-Aacraid Linux Driver <aacraid@xxxxxxxxxxxxx>; > gpiccoli@xxxxxxxxxxxxxxxxxx; Tom White <tom.white@xxxxxxxxxxxxx>; > Scott Benesh <scott.benesh@xxxxxxxxxxxxx> > Subject: Re: [PATCH 14/29] scsi: aacraid: Move function around to match > existing code > > EXTERNAL EMAIL > > > On Thu, 2017-12-21 at 09:34 -0800, Raghava Aditya Renukunta wrote: > > +/** > > + * aac_get_safw_ciss_luns() Process topology change > > + * @dev: aac_dev structure > > + * @rescan Indicates rescan > > + * > > + * Execute a CISS REPORT PHYS LUNS and process the results into > > + * the current hba_map. > > + */ > > The above function header is not following the kernel-doc syntax: a colon is > missing. Have you tried to build your driver with W=1? That should be > sufficient > with kernel version v4.15-rc1 or later to report kernel-doc header issues. > > Bart. I was not aware of that option I will fix it in the my next version. Regards, Raghava Aditya