Hi Jani, thanks for looking at it. > > - intel_gt_init_early(&dev_priv->gt, dev_priv); > > + __intel_gt_init_early(&dev_priv->gt, dev_priv); > > Why double underscores here? It looks like it's supposed to be internal > to intel_gt, not to be called by anyone else. I forgot to write two lines in the commit log about this. It's a temporary solution that will go away in the next patch series[*]. The reason for it is because at this point I need to break the early initialization of the gt into two parts. In the specific the '__intel_gt_init_early' assigns the i915 private data and the uncore. It's not pretty, but, knowing what's coming next, it's the change with the smallest impact. > > > > i915_gem_init_early(dev_priv); > > Thank you, Andi [*] https://patchwork.freedesktop.org/patch/464475/?series=97352&rev=1
