Sorry for not reviewing this earlier, but what about including the
include file that defines
struct page instead of a separate declaration?
Martin Michlmayr wrote:
> drm/ttm fails to build on MIPS because "struct page" is not known:
> | In file included from drivers/gpu/drm/ttm/ttm_memory.c:28:
> | include/drm/ttm/ttm_memory.h:154: warning: 'struct page' declared inside parameter list
> | include/drm/ttm/ttm_memory.h:154: warning: its scope is only this definition or declaration, which is probably not what you want
> | include/drm/ttm/ttm_memory.h:156: warning: 'struct page' declared inside parameter list
> | drivers/gpu/drm/ttm/ttm_memory.c:540: error: conflicting types for 'ttm_mem_global_alloc_page'
> | include/drm/ttm/ttm_memory.h:154: error: previous declaration of 'ttm_mem_global_alloc_page' was here
> | drivers/gpu/drm/ttm/ttm_memory.c:561: error: conflicting types for 'ttm_mem_global_free_page'
> | include/drm/ttm/ttm_memory.h:156: error: previous declaration of 'ttm_mem_global_free_page' was here
> Signed-off-by: Martin Michlmayr <tbm@...>
> Maybe it wasn't clear when I posted this a week ago but this needs to go
> into 2.6.32.
> --- a/include/drm/ttm/ttm_memory.h 2009-11-11 22:02:56.000000000 +0000
> +++ b/include/drm/ttm/ttm_memory.h 2009-11-11 22:04:23.000000000 +0000
> @@ -143,6 +143,7 @@
> +struct page;
> extern int ttm_mem_global_init(struct ttm_mem_global *glob);
> extern void ttm_mem_global_release(struct ttm_mem_global *glob);
> extern int ttm_mem_global_alloc(struct ttm_mem_global *glob, uint64_t memory,