Hi Daniel, looks good.
On Wed, Jun 23, 2021 at 06:24:56PM +0200, Daniel Vetter wrote: > It's tedious to review this all the time, and my audit showed that > arcpgu actually forgot to set this. > > Make this the default and stop worrying. > > Again I sprinkled WARN_ON_ONCE on top to make sure we don't have > strange combinations of hooks: cleanup_fb without prepare_fb doesn't > make sense, and since simpler drivers are all new they better be GEM > based drivers. > > v2: Warn and bail when it's _not_ a GEM driver (Noralf) > > v3: It's neither ... nor, not not (Sam) > > Cc: Sam Ravnborg <[email protected]> > Cc: Noralf Trønnes <[email protected]> > Acked-by: Noralf Trønnes <[email protected]> > Signed-off-by: Daniel Vetter <[email protected]> > Cc: Maarten Lankhorst <[email protected]> > Cc: Maxime Ripard <[email protected]> > Cc: Thomas Zimmermann <[email protected]> > Cc: David Airlie <[email protected]> > Cc: Daniel Vetter <[email protected]> Acked-by: Sam Ravnborg <[email protected]>
