stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro
@ 2015-09-23  9:10 Tvrtko Ursulin
  2015-09-23 11:36 ` [Intel-gfx] " Ville Syrjälä
  0 siblings, 1 reply; 4+ messages in thread
From: Tvrtko Ursulin @ 2015-09-23  9:10 UTC (permalink / raw)
  To: Intel-gfx
  Cc: Tvrtko Ursulin, Rob Clark, Daniel Stone, Daniel Vetter, dri-devel,
	stable

From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

__u64 should be used instead of u64.

Feature originally added in:

commit e3eb3250d84ef97b766312345774367b6a310db8
Author: Rob Clark <robdclark@gmail.com>
Date:   Thu Feb 5 14:41:52 2015 +0000

    drm: add support for tiled/compressed/etc modifier in addfb2

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Daniel Stone <daniels@collabora.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: dri-devel@lists.freedesktop.org
Cc: stable@vger.kernel.org
---
 include/uapi/drm/drm_fourcc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h
index 8c5e8b91a3cb..0b69a7753558 100644
--- a/include/uapi/drm/drm_fourcc.h
+++ b/include/uapi/drm/drm_fourcc.h
@@ -158,7 +158,7 @@
 /* add more to the end as needed */
 
 #define fourcc_mod_code(vendor, val) \
-	((((u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
+	((((__u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
 
 /*
  * Format Modifier tokens:
-- 
2.5.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro
  2015-09-23  9:10 [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro Tvrtko Ursulin
@ 2015-09-23 11:36 ` Ville Syrjälä
  2015-11-05 12:51   ` Jani Nikula
  0 siblings, 1 reply; 4+ messages in thread
From: Ville Syrjälä @ 2015-09-23 11:36 UTC (permalink / raw)
  To: Tvrtko Ursulin; +Cc: Intel-gfx, Daniel Stone, stable, dri-devel, Daniel Vetter

On Wed, Sep 23, 2015 at 10:10:31AM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> 
> __u64 should be used instead of u64.
> 
> Feature originally added in:
> 
> commit e3eb3250d84ef97b766312345774367b6a310db8
> Author: Rob Clark <robdclark@gmail.com>
> Date:   Thu Feb 5 14:41:52 2015 +0000
> 
>     drm: add support for tiled/compressed/etc modifier in addfb2
> 
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Daniel Stone <daniels@collabora.com>
> Cc: Daniel Vetter <daniel.vetter@intel.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: stable@vger.kernel.org

Reviewed-by: Ville Syrj�l� <ville.syrjala@linux.intel.com>

> ---
>  include/uapi/drm/drm_fourcc.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h
> index 8c5e8b91a3cb..0b69a7753558 100644
> --- a/include/uapi/drm/drm_fourcc.h
> +++ b/include/uapi/drm/drm_fourcc.h
> @@ -158,7 +158,7 @@
>  /* add more to the end as needed */
>  
>  #define fourcc_mod_code(vendor, val) \
> -	((((u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
> +	((((__u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
>  
>  /*
>   * Format Modifier tokens:
> -- 
> 2.5.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrj�l�
Intel OTC

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro
  2015-09-23 11:36 ` [Intel-gfx] " Ville Syrjälä
@ 2015-11-05 12:51   ` Jani Nikula
  2015-11-05 21:12     ` Rob Clark
  0 siblings, 1 reply; 4+ messages in thread
From: Jani Nikula @ 2015-11-05 12:51 UTC (permalink / raw)
  To: Ville Syrjälä, Tvrtko Ursulin
  Cc: dri-devel, Daniel Vetter, Intel-gfx, Daniel Stone, stable

On Wed, 23 Sep 2015, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Wed, Sep 23, 2015 at 10:10:31AM +0100, Tvrtko Ursulin wrote:
>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>> 
>> __u64 should be used instead of u64.
>> 
>> Feature originally added in:
>> 
>> commit e3eb3250d84ef97b766312345774367b6a310db8
>> Author: Rob Clark <robdclark@gmail.com>
>> Date:   Thu Feb 5 14:41:52 2015 +0000
>> 
>>     drm: add support for tiled/compressed/etc modifier in addfb2
>> 
>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>> Cc: Rob Clark <robdclark@gmail.com>
>> Cc: Daniel Stone <daniels@collabora.com>
>> Cc: Daniel Vetter <daniel.vetter@intel.com>
>> Cc: dri-devel@lists.freedesktop.org
>> Cc: stable@vger.kernel.org
>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Pushed to our topic/drm-fixes branch, thanks for the patch and review.

BR,
Jani.

>
>> ---
>>  include/uapi/drm/drm_fourcc.h | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h
>> index 8c5e8b91a3cb..0b69a7753558 100644
>> --- a/include/uapi/drm/drm_fourcc.h
>> +++ b/include/uapi/drm/drm_fourcc.h
>> @@ -158,7 +158,7 @@
>>  /* add more to the end as needed */
>>  
>>  #define fourcc_mod_code(vendor, val) \
>> -	((((u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
>> +	((((__u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
>>  
>>  /*
>>   * Format Modifier tokens:
>> -- 
>> 2.5.1
>> 
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro
  2015-11-05 12:51   ` Jani Nikula
@ 2015-11-05 21:12     ` Rob Clark
  0 siblings, 0 replies; 4+ messages in thread
From: Rob Clark @ 2015-11-05 21:12 UTC (permalink / raw)
  To: Jani Nikula
  Cc: Ville Syrjälä, Tvrtko Ursulin, Daniel Vetter,
	Intel Graphics Development, stable, Daniel Stone,
	dri-devel@lists.freedesktop.org

On Thu, Nov 5, 2015 at 7:51 AM, Jani Nikula <jani.nikula@linux.intel.com> wrote:
> On Wed, 23 Sep 2015, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
>> On Wed, Sep 23, 2015 at 10:10:31AM +0100, Tvrtko Ursulin wrote:
>>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>>
>>> __u64 should be used instead of u64.
>>>
>>> Feature originally added in:
>>>
>>> commit e3eb3250d84ef97b766312345774367b6a310db8
>>> Author: Rob Clark <robdclark@gmail.com>
>>> Date:   Thu Feb 5 14:41:52 2015 +0000
>>>
>>>     drm: add support for tiled/compressed/etc modifier in addfb2
>>>
>>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>> Cc: Rob Clark <robdclark@gmail.com>
>>> Cc: Daniel Stone <daniels@collabora.com>
>>> Cc: Daniel Vetter <daniel.vetter@intel.com>
>>> Cc: dri-devel@lists.freedesktop.org
>>> Cc: stable@vger.kernel.org
>>
>> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Pushed to our topic/drm-fixes branch, thanks for the patch and review.

and I've pushed the libdrm part

BR,
-R

> BR,
> Jani.
>
>>
>>> ---
>>>  include/uapi/drm/drm_fourcc.h | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h
>>> index 8c5e8b91a3cb..0b69a7753558 100644
>>> --- a/include/uapi/drm/drm_fourcc.h
>>> +++ b/include/uapi/drm/drm_fourcc.h
>>> @@ -158,7 +158,7 @@
>>>  /* add more to the end as needed */
>>>
>>>  #define fourcc_mod_code(vendor, val) \
>>> -    ((((u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
>>> +    ((((__u64)DRM_FORMAT_MOD_VENDOR_## vendor) << 56) | (val & 0x00ffffffffffffffULL))
>>>
>>>  /*
>>>   * Format Modifier tokens:
>>> --
>>> 2.5.1
>>>
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx@lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> --
> Jani Nikula, Intel Open Source Technology Center
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-11-05 21:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-23  9:10 [PATCH] drm: Use userspace compatible type in fourcc_mod_code macro Tvrtko Ursulin
2015-09-23 11:36 ` [Intel-gfx] " Ville Syrjälä
2015-11-05 12:51   ` Jani Nikula
2015-11-05 21:12     ` Rob Clark

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).