All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement
@ 2016-04-01 15:02 Alex Deucher
  2016-04-01 16:31 ` Julian Margetson
  2016-04-01 17:55 ` Christian König
  0 siblings, 2 replies; 4+ messages in thread
From: Alex Deucher @ 2016-04-01 15:02 UTC (permalink / raw)
  To: dri-devel; +Cc: Alex Deucher, Thomas Hellstrom, Hans Verkuil, Julian Margetson

Fixes ttm on platforms like PPC460 where the CPU
is in 32-bit mode, but the physical addresses are
>32 bits.

Extracted from a patch by Hans.

Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Julian Margetson <runaway@candw.ms>
Cc: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
 include/drm/ttm/ttm_bo_api.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
index afae231..055a08d 100644
--- a/include/drm/ttm/ttm_bo_api.h
+++ b/include/drm/ttm/ttm_bo_api.h
@@ -92,7 +92,7 @@ struct ttm_placement {
  */
 struct ttm_bus_placement {
 	void		*addr;
-	unsigned long	base;
+	phys_addr_t	base;
 	unsigned long	size;
 	unsigned long	offset;
 	bool		is_iomem;
-- 
2.5.5

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement
  2016-04-01 15:02 [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement Alex Deucher
@ 2016-04-01 16:31 ` Julian Margetson
  2016-04-01 16:42   ` Thomas Hellstrom
  2016-04-01 17:55 ` Christian König
  1 sibling, 1 reply; 4+ messages in thread
From: Julian Margetson @ 2016-04-01 16:31 UTC (permalink / raw)
  To: Alex Deucher, dri-devel; +Cc: Alex Deucher, Thomas Hellstrom, Hans Verkuil

On 4/1/2016 11:02 AM, Alex Deucher wrote:
> Fixes ttm on platforms like PPC460 where the CPU
> is in 32-bit mode, but the physical addresses are
>> 32 bits.
> Extracted from a patch by Hans.
>
> Cc: Thomas Hellstrom <thellstrom@vmware.com>
> Cc: Julian Margetson <runaway@candw.ms>
> Cc: Hans Verkuil <hans.verkuil@cisco.com>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> ---
>   include/drm/ttm/ttm_bo_api.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
> index afae231..055a08d 100644
> --- a/include/drm/ttm/ttm_bo_api.h
> +++ b/include/drm/ttm/ttm_bo_api.h
> @@ -92,7 +92,7 @@ struct ttm_placement {
>    */
>   struct ttm_bus_placement {
>   	void		*addr;
> -	unsigned long	base;
> +	phys_addr_t	base;
>   	unsigned long	size;
>   	unsigned long	offset;
>   	bool		is_iomem;
Tested and working .
Tested-by: Julian Margetson <runaway@candw.ms>

Regards
Julian


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement
  2016-04-01 16:31 ` Julian Margetson
@ 2016-04-01 16:42   ` Thomas Hellstrom
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Hellstrom @ 2016-04-01 16:42 UTC (permalink / raw)
  To: Julian Margetson, Alex Deucher, dri-devel; +Cc: Alex Deucher, Hans Verkuil

Acked-by: Thomas Hellstrom <thellstrom@vmware.com>

On 04/01/2016 06:31 PM, Julian Margetson wrote:
> On 4/1/2016 11:02 AM, Alex Deucher wrote:
>> Fixes ttm on platforms like PPC460 where the CPU
>> is in 32-bit mode, but the physical addresses are
>>> 32 bits.
>> Extracted from a patch by Hans.
>>
>> Cc: Thomas Hellstrom <thellstrom@vmware.com>
>> Cc: Julian Margetson <runaway@candw.ms>
>> Cc: Hans Verkuil <hans.verkuil@cisco.com>
>> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>> ---
>>   include/drm/ttm/ttm_bo_api.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
>> index afae231..055a08d 100644
>> --- a/include/drm/ttm/ttm_bo_api.h
>> +++ b/include/drm/ttm/ttm_bo_api.h
>> @@ -92,7 +92,7 @@ struct ttm_placement {
>>    */
>>   struct ttm_bus_placement {
>>       void        *addr;
>> -    unsigned long    base;
>> +    phys_addr_t    base;
>>       unsigned long    size;
>>       unsigned long    offset;
>>       bool        is_iomem;
> Tested and working .
> Tested-by: Julian Margetson <runaway@candw.ms>
>
> Regards
> Julian
>
>

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement
  2016-04-01 15:02 [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement Alex Deucher
  2016-04-01 16:31 ` Julian Margetson
@ 2016-04-01 17:55 ` Christian König
  1 sibling, 0 replies; 4+ messages in thread
From: Christian König @ 2016-04-01 17:55 UTC (permalink / raw)
  To: Alex Deucher, dri-devel
  Cc: Alex Deucher, Thomas Hellstrom, Hans Verkuil, Julian Margetson

Am 01.04.2016 um 17:02 schrieb Alex Deucher:
> Fixes ttm on platforms like PPC460 where the CPU
> is in 32-bit mode, but the physical addresses are
>> 32 bits.
> Extracted from a patch by Hans.
>
> Cc: Thomas Hellstrom <thellstrom@vmware.com>
> Cc: Julian Margetson <runaway@candw.ms>
> Cc: Hans Verkuil <hans.verkuil@cisco.com>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
>   include/drm/ttm/ttm_bo_api.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
> index afae231..055a08d 100644
> --- a/include/drm/ttm/ttm_bo_api.h
> +++ b/include/drm/ttm/ttm_bo_api.h
> @@ -92,7 +92,7 @@ struct ttm_placement {
>    */
>   struct ttm_bus_placement {
>   	void		*addr;
> -	unsigned long	base;
> +	phys_addr_t	base;
>   	unsigned long	size;
>   	unsigned long	offset;
>   	bool		is_iomem;

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-04-01 17:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-01 15:02 [PATCH] drm/ttm: use phys_addr_t for ttm_bus_placement Alex Deucher
2016-04-01 16:31 ` Julian Margetson
2016-04-01 16:42   ` Thomas Hellstrom
2016-04-01 17:55 ` Christian König

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.