radeon: Define SI_TILE_MODE_COLOR_2D_SCANOUT_8BPP for consistency and readability - libdrm

Commit Message

Alexandre Demers Feb. 12, 2016, 12:32 a.m. UTC

---
 include/drm/radeon_drm.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Michel Dänzer Feb. 12, 2016, 7:32 a.m. UTC | #1

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].

Patch

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