Hi, Andrey, On Mon, May 21, 2018 at 11:39:26AM -0700, Andrey Smirnov wrote: > On Mon, May 21, 2018 at 4:54 AM, Peter Mamonov <pmamonov@xxxxxxxxx> wrote: > > Hi, > > > > Here is a second revision of the patchset which fixes various issues with 64 bit > > barebox, primarily MIPS related. IMHO all of these patches are ready for > > application. > > > > Don't think I've seen any comments on: > http://lists.infradead.org/pipermail/barebox/2018-May/032985.html > Right now it's not clear if that thread just slipped though the cracks > or you decided to keep the code as is. Latter is fine with me, I just > want to make sure it is not the former. Sorry for lack of feedback on your proposal. Your solution is actually the proper one. Yet I've decided to keep this change trivial for two reasons: - to make it easier for Sascha to accept it (i.e. no change in driver logic, just fix the problem) - to make this patch self explaining, since it demonstrates the source of the problem: get_drv_data(..., &data->type) overwrites memory beyond data->type member due to mismatch of sizeof(enum pca_type) and sizeof(void *); using argument of a proper type and size fixes the problem. Regards, Peter > > Thanks, > Andrey Smirnov _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
