On Wed, Jul 25, 2018 at 11:16:21PM +0100, John Whitmore wrote: > -typedef struct tag_cmd_pkt_tx_feedback { > +struct cmpk_txfb_t { > /* DWORD 0 */ > u8 element_id; /* Command packet type. */ > u8 length; /* Command packet length. */ > @@ -53,7 +53,7 @@ typedef struct tag_cmd_pkt_tx_feedback { > /* DWORD 5 */ > u16 reserve3; > u16 duration; > -} cmpk_txfb_t; > +}; People use the "_t" to try to denote a "typedef". When converting to just a structure, there is no need to keep the _t anymore at all, so this should just be "struct cmpk_txfb". Or just drop the typedef and use the name for the struct they had here already, "struct tag_cmd_pkt_tx_feedback", that also would work. Same for all of the other patches in this series. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel