Hi Daniele, > > MTL features a dedicated media engine that operates on its > > independent GT, requiring activation of its specific interrupt > > set. > > > > Enable the necessary interrupts in a single action when the media > > engine is present, bypassing the need to iterate through all the > > GTs. > > > > Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx> > > --- > > Hi, > > > > I'm starting with v0 as this patch is very different from the > > ones proposed recently. > > > > After all the discussions on this patch, I took Matt's suggestion > > since it seemed the most immediate. > > > > However, in the long run, I agree that we should have a > > specific mtl_ function for enabling interrupts. > > > > Thank you Matt and Daniele for your input. > > > > If this patch hasn't missed anything, is it too optimistic to > > expect MTL to boot? :-) > > The GSC engine also has interrupts tied to the media GT and those are > conditional, so that engine won't work with just this patch. The system > might boot because the GSC engine gets disabled if the FW is not there, but > IMO if we want a single function to handle both GTs we need to do it proper > support for the engines and not hack around just the GuC. yeah... we are already having too many things to handle and at this point I don't see any better way to handle this other than using for_each_gt() as it was first sent. Besides, they are different GT's, why not using for_each_gt? Thank you, Daniele, Andi
