On Wed, 9 Dec 2020 at 20:48, Daniel Vetter <daniel@xxxxxxxx> wrote: > > On Wed, Dec 9, 2020 at 5:22 AM Dave Airlie <airlied@xxxxxxxxx> wrote: > > > > From: Dave Airlie <airlied@xxxxxxxxxx> > > > > This just refactors out the fdi code to a separate file. > > > > Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx> > > There's also hsw_fdi_link_train from intel_ddi.c (another fairly big > file), I think that also belongs in here. It's not in the vtable > because it's directly called from the hsw crt encoder. With that this > looks reasonable to me. It does but it's definitely a lot messier to move, it has a lot of links into the ddi code and tables. I'll add another patch to the end that we can look at to decide if there's a cleaner way to do it. Dave. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx