[PATCH] radeon: Define SI_TILE_MODE_COLOR_2D_SCANOUT_8BPP for consistency and readability - libdrm

Michel Dänzer michel at daenzer.net
Fri Feb 12 07:32:36 UTC 2016


On 12.02.2016 09:32, Alexandre Demers wrote:
> ---
>  include/drm/radeon_drm.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/drm/radeon_drm.h b/include/drm/radeon_drm.h
> index cd31794..e57640a 100644
> --- a/include/drm/radeon_drm.h
> +++ b/include/drm/radeon_drm.h
> @@ -1026,6 +1026,7 @@ struct drm_radeon_info {
>  #define SI_TILE_MODE_COLOR_2D_16BPP		15
>  #define SI_TILE_MODE_COLOR_2D_32BPP		16
>  #define SI_TILE_MODE_COLOR_2D_64BPP		17
> +#define SI_TILE_MODE_COLOR_2D_SCANOUT_8BPP	10
>  #define SI_TILE_MODE_COLOR_2D_SCANOUT_16BPP	11
>  #define SI_TILE_MODE_COLOR_2D_SCANOUT_32BPP	12
>  #define SI_TILE_MODE_DEPTH_STENCIL_1D		4
> 

The define itself looks good, thanks, but please don't append " -
libdrm" to the subject, or that'll get preserved in the Git shortlog.
The way to indicate the repository a patch is for is in the [PATCH]
prefix, e.g. in this case [PATCH libdrm].


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the dri-devel mailing list