Re: [PATCH v2 2/7] drm/shmem: Put pages independent of a SG table being set

2019-07-26 Thread Steven Price
On 25/07/2019 02:09, Rob Herring wrote: > If a driver does its own management of pages, the shmem helper object's > pages array could be allocated when a SG table is not. There's not > really any good reason to tie putting pages with having a SG table when > freeing the object, so just put pages i

[PATCH v2 2/7] drm/shmem: Put pages independent of a SG table being set

2019-07-24 Thread Rob Herring
If a driver does its own management of pages, the shmem helper object's pages array could be allocated when a SG table is not. There's not really any good reason to tie putting pages with having a SG table when freeing the object, so just put pages if the pages array is populated. Cc: Maarten Lan