Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-30 Thread Thomas Zimmermann
Am 30.08.2018 um 08:45 schrieb Christian König: [...] >> or create their private instance. > > That doesn't sounds good. Drivers should not be allowed to create their > own private instance of that. OK, will be changed in the patchset. Best regards Thomas > > Thanks for doing this, > Christian

Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-29 Thread Christian König
Hi Thomas, Am 30.08.2018 um 08:34 schrieb Thomas Zimmermann: Hi Christian, I just wanted to ask if there's something else required to get this patch set reviewed and landed. I just need to find some time to review them. On top of these two patches, I have a patch set that replaces the drive

Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-29 Thread Thomas Zimmermann
Hi Christian, I just wanted to ask if there's something else required to get this patch set reviewed and landed. On top of these two patches, I have a patch set that replaces the driver-specific global-bo-and-mem combos with a single struct ttm_global structure. It further merges struct drm_globa

Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-13 Thread Thomas Hellstrom
On 08/13/2018 02:28 PM, Thomas Zimmermann wrote: Hi Am 13.08.2018 um 12:33 schrieb Christian König: Yes, please! I had it on my TODO list to clean that up for an eternity. On top of these patches, I have a patch set that provides a single init/release interface for TTM global data. I'll post i

Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-13 Thread Thomas Zimmermann
Hi Am 13.08.2018 um 12:33 schrieb Christian König: > Yes, please! I had it on my TODO list to clean that up for an eternity. On top of these patches, I have a patch set that provides a single init/release interface for TTM global data. I'll post it when the current patches got some feed back. I'

Re: [PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-13 Thread Christian König
Yes, please! I had it on my TODO list to clean that up for an eternity. Actually I never understood why that should be driver work to setup TTM? I mean can't we just have a module_init/module_exit for TTM? Thanks, Christian. Am 13.08.2018 um 12:24 schrieb Thomas Zimmermann: TTM uses global me

[PATCH 0/2] Provide init/release functions for struct ttm_bo_global

2018-08-13 Thread Thomas Zimmermann
TTM uses global memory and BO for backing graphics buffers. These are represented by struct ttm_mem_global and struct ttm_bo_global. Currently, struct ttm_bo_global can only be initialized and released through struct ttm_bo_global_ref. This is a workaround for passing an instance of ttm_mem_global