On Sun, Sep 08, 2019 at 05:35:37PM +0000, Valentin Vidic wrote: > Use timestamp_t for everything and cleanup duplicate code. Wait, how are these "duplicate"? The fields are in different order, don't these refer to things on-disk? Did you test this? > -struct date_time_t { > - u16 Year; > - u16 Month; > - u16 Day; > - u16 Hour; > - u16 Minute; > - u16 Second; > - u16 MilliSecond; > -}; > - > struct part_info_t { > u32 Offset; /* start sector number of the partition */ > u32 Size; /* in sectors */ > @@ -289,6 +279,16 @@ struct file_id_t { > u32 hint_last_clu; > }; > > +struct timestamp_t { > + u16 millisec; /* 0 ~ 999 */ > + u16 sec; /* 0 ~ 59 */ > + u16 min; /* 0 ~ 59 */ > + u16 hour; /* 0 ~ 23 */ > + u16 day; /* 1 ~ 31 */ > + u16 mon; /* 1 ~ 12 */ > + u16 year; /* 0 ~ 127 (since 1980) */ > +}; They really look "backwards" to me, how are these the same? What am I missing? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel