On Fri, Nov 06, 2015 at 03:08:33PM +0200, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Reading the driver load/unload code leaves one confused as there's > an async_schedule() in the load, but not async_synchronize_full() > in sight. In fact it's hidden inside intel_fbdev.c. So let's move the > async_schedule() into intel_fbdev.c as well so that it's next to the > async_synchronize_full(), which should make the relationship easier > to see. > > Plus this way we won't schedule a nop function call when fbdev is > disabled. And we were passing a pointer to a static inline > function to async_schedule(), which seems rather dubious to me. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
