> + help > + This driver supports Broadcom's Unified MPI3 based Storage & RAID Controllers. Overly long line here. > +#ifndef MPI30_API_H > +#define MPI30_API_H 1 > +#include "mpi30_transport.h" > +#include "mpi30_image.h" > +#include "mpi30_init.h" > +#include "mpi30_ioc.h" > +#endif Just including the four headers where needed directly would make more sense to me than this meta-header. > + * Name: mpi30_image.h We generally do not add comments like this. It does not add any value and gets stale very quickly. > + * Creation Date: 04/02/2018 > + * Version: 03.00.00 No need for this information either. If you important from a specific internal version this is something that should go into the commit log. > +struct _mpi3_comp_image_version { Please drop the leading underscore from the various type names.