Re: [PATCH] Remove unused function drm_agp_bind_pages

2019-07-10 Thread Daniel Vetter
On Mon, Jul 08, 2019 at 10:18:16AM +0200, Sam Ravnborg wrote: > Hi Maya. > > Nice catch - good to remove unused cruft. > > Please prefix the subject with something like "drm/agp: ". > Check "git log" on the same file to pick up the usual way to identify > agp specific changes. > > With this fixe

Re: [PATCH] Remove unused function drm_agp_bind_pages

2019-07-08 Thread Sam Ravnborg
Hi Maya. Nice catch - good to remove unused cruft. Please prefix the subject with something like "drm/agp: ". Check "git log" on the same file to pick up the usual way to identify agp specific changes. With this fixed: Reviewed-by: Sam Ravnborg You could also consider to add a few words in the