> On March 27, 2015, 2:16 p.m., Andreas Hansson wrote: > > Looks great. Thanks for the contribution. > > > > A few minor things: > > > > 1) The summary should start with "dev : " (see > > http://gem5.org/Commit_Access for details) > > 2) Are any regressions affected? (or is this not impacting linux)
this is not impact linux - Ruslan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2705/#review5974 ----------------------------------------------------------- On March 25, 2015, 11:10 a.m., Ruslan Bukin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2705/ > ----------------------------------------------------------- > > (Updated March 25, 2015, 11:10 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Add 32-bit access width for PrimaryTiming register and 16bit for UDMAControl > register as FreeBSD required. > > > Diffs > ----- > > src/dev/ide_ctrl.cc 8f5993cfa916 > > Diff: http://reviews.gem5.org/r/2705/diff/ > > > Testing > ------- > > Test passed OK booting FreeBSD guest. > > > Thanks, > > Ruslan Bukin > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
