On Thu, Jan 27, 2011 at 10:19:55AM +0100, Thomas Hellstrom wrote:
> On 01/07/2011 06:11 PM, Konrad Rzeszutek Wilk wrote:
>
> Apart from previously mentioned indentation issues,
> Reviewed-by: Thomas Hellstrom <thellstrom@xxxxxxxxxxxx>
Thank you..
>
.. snip..
> > nouveau_sgdma_populate(struct ttm_backend *be, unsigned long num_pages,
> >- struct page **pages, struct page *dummy_read_page)
> >+ struct page **pages, struct page *dummy_read_page,
> >+ dma_addr_t *dma_addrs)
This is weird, but when I pull this up in 'vim' it looks OK. I think it is
just the emailer playing tricks? (attached a screenshot)
checkpatch.pl which usually throws a fit when it comes to indentations does not
complain here at all.
> > {
> > struct nouveau_sgdma_be *nvbe = (struct nouveau_sgdma_be *)be;
> > struct drm_device *dev = nvbe->dev;
> >diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c
> >index 01c2c73..6f156e9 100644
> >--- a/drivers/gpu/drm/radeon/radeon_ttm.c
> >+++ b/drivers/gpu/drm/radeon/radeon_ttm.c
> >@@ -655,7 +655,8 @@ struct radeon_ttm_backend {
> > static int radeon_ttm_backend_populate(struct ttm_backend *backend,
> > unsigned long num_pages,
> > struct page **pages,
> >- struct page *dummy_read_page)
> >+ struct page *dummy_read_page,
> >+ dma_addr_t *dma_addrs)
Ditto. Looks just fine.
> > {
> > struct radeon_ttm_backend *gtt;
> >
> >diff --git a/drivers/gpu/drm/ttm/ttm_agp_backend.c b/drivers/gpu/drm/ttm/ttm_agp_backend.c
> >index f999e36..1c4a72f 100644
> >--- a/drivers/gpu/drm/ttm/ttm_agp_backend.c
> >+++ b/drivers/gpu/drm/ttm/ttm_agp_backend.c
> >@@ -47,7 +47,8 @@ struct ttm_agp_backend {
> >
> > static int ttm_agp_populate(struct ttm_backend *backend,
> > unsigned long num_pages, struct page **pages,
> >- struct page *dummy_read_page)
> >+ struct page *dummy_read_page,
> >+ dma_addr_t *dma_addrs)
> > {
> > struct ttm_agp_backend *agp_be =
> > container_of(backend, struct ttm_agp_backend, backend);
> >diff --git a/drivers/gpu/drm/ttm/ttm_tt.c b/drivers/gpu/drm/ttm/ttm_tt.c
> >index 0d39001..86d5b17 100644
> >--- a/drivers/gpu/drm/ttm/ttm_tt.c
> >+++ b/drivers/gpu/drm/ttm/ttm_tt.c
> >@@ -169,7 +169,7 @@ int ttm_tt_populate(struct ttm_tt *ttm)
> > }
> >
> > be->func->populate(be, ttm->num_pages, ttm->pages,
> >- ttm->dummy_read_page);
> >+ ttm->dummy_read_page, ttm->dma_address);
> > ttm->state = tt_unbound;
> > return 0;
> > }
> >diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c
> >index 80bc37b..87e43e0 100644
> >--- a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c
> >+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c
> >@@ -102,7 +102,8 @@ struct vmw_ttm_backend {
> >
> > static int vmw_ttm_populate(struct ttm_backend *backend,
> > unsigned long num_pages, struct page **pages,
> >- struct page *dummy_read_page)
> >+ struct page *dummy_read_page,
> >+ dma_addr_t *dma_addrs)
Grrrr... same thing - looks OK?
> > {
> > struct vmw_ttm_backend *vmw_be =
> > container_of(backend, struct vmw_ttm_backend, backend);
> >diff --git a/include/drm/ttm/ttm_bo_driver.h b/include/drm/ttm/ttm_bo_driver.h
> >index 6dc4fcc..ebcd3dd 100644
> >--- a/include/drm/ttm/ttm_bo_driver.h
> >+++ b/include/drm/ttm/ttm_bo_driver.h
> >@@ -50,13 +50,15 @@ struct ttm_backend_func {
> > * @pages: Array of pointers to ttm pages.
> > * @dummy_read_page: Page to be used instead of NULL pages in the
> > * array @pages.
> >+ * @dma_addrs: Array of DMA (bus) address of the ttm pages.
> > *
> > * Populate the backend with ttm pages. Depending on the backend,
> > * it may or may not copy the @pages array.
> > */
> > int (*populate) (struct ttm_backend *backend,
> > unsigned long num_pages, struct page **pages,
> >- struct page *dummy_read_page);
> >+ struct page *dummy_read_page,
> >+ dma_addr_t *dma_addrs);
> > /**
> > * struct ttm_backend_func member clear
> > *
Attachment:
Screenshot-nouveau_sgdma.c (~-ssd-linux-drivers-gpu-drm-nouveau) - GVIM.png
Description: PNG image
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
