> > > * Add struct mspro_cmdex_argument, argument for> > MS_TPC_EX_SET_CMD> > * Increase size of inline buffer in memstick_request to 32> > because thats the size of registers and someone might need> > to read them> > all. That structure is allocated one per memstick host, so> > its size> > doesn't matter.> > This functional changes are better be joined with the first patch which> actually requires them (these are just 4 lines).Not fully against that, but then what value will that add too?Currently memstick.h host many unused structures for MS IO stuff.And for god's sake it has MS_TPC_EX_SET_CMD declared,so I just add the structure for its argument, whats wrong with doingthat here?

> > > > > * Add comments about few members of memstick_request> > Belongs to the clean-up or a separate commenting patch.Why?

As long as the patch is crearly readable I don't see a reason to addmore patches.