On Tue, 2011-10-11 at 16:24 -0500, Madhuranath N Iyengar wrote: > From: Madhuranath Iyengar <mni@xxxxxxxxxxxxxxx> > > Folks, > > This patch series contains patches below to address the following: > - purge the seven ctio* structs, into two ctio* and two ctio7* structures. > - some naming concerns associated with functions, structures, macros and #defines. > > Please have a look and let us know if you have any concerns. > > Thanks. > > Madhuranath Iyengar (3): > qla2xxx: Purge ctio_common_entry_t struct, and rename ctio_* structs > for ISP 2xxx > qla2xxx: Purge ctio7_common_entry_t struct, rename ctio7_* structs > for ISP 24xx. > qla2xxx: Renamed functions, structures, macros and defines. > Hey Madhu, After some light testing this afternoon, these cleanups appear to be working as expected. I don't personally have strong feelings on the individual naming changes, and am more concerned about what makes Andrew @ Qlogic happy. :) Also, a followup patch to remove typedef usage following Christoph's recommendation for kernel coding style would be useful. Thanks, --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html