Quoting Liwei Song (2020-04-21 10:59:04) > > > On 4/21/20 17:29, Chris Wilson wrote: > > Quoting Liwei Song (2020-04-21 09:41:28) > >> +static const struct intel_device_info intel_cannonlake_info = { > >> + .gen = 0115, > > .gen = 0120 /* 10 */ > > > >> +}; > >> + > >> +static const struct intel_device_info intel_icelake_info = { > >> + .gen = 0116, > > .gen = 0130 /* 11 */ > > > >> +}; > >> + > >> +static const struct intel_device_info intel_elkhartlake_info = { > >> + .gen = 0117, > > .gen = 0131 > > > >> +}; > >> + > >> +static const struct intel_device_info intel_tigerlake_info = { > >> + .gen = 0120, > > .gen = 0140 /* 12 */ > > > > You definitely do not want to feed them through the gen9 assembler. > > Thanks, will modify it in v5, but could you explain more here how we > decide the gen number, I really do not have too much knowledge here, > I use to think it is an increasing number like some other definition. It's octal, with the low octet being a revision number (so we can quickly tell big from little core mostly, but whatever else makes sense for that gen). -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx