Hi, Tony, On Wed, 2010-12-01 at 08:20 +0800, Luck, Tony wrote: > +struct pstore_info { > + unsigned long header_size; > + unsigned long data_size; > + unsigned long footer_size; > + int (*reader)(u64 *id, int *type, void *buf, unsigned long *size); > + int (*writer)(int type, void *buf, unsigned long size); > + int (*eraser)(u64 id); > +}; How about rename these 3 functions to read, write and erase? Best Regards, Huang Ying -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html