[2/5] intel: add disable_ccs_repack to gen_device_info

Submitted by Dongwon Kim on June 27, 2019, 4:54 p.m.

Details

Message ID 20190627165437.20961-2-dongwon.kim@intel.com
State New
Headers show
Series "Series without cover letter" ( rev: 1 ) in Mesa

Not browsing as part of any series.

Commit Message

Dongwon Kim June 27, 2019, 4:54 p.m.
add a new attribute, 'disable_ccs_repack' to gen_device info, which
indicates whether repacking of components in certain pixel formats
before compression needs to be disabled to keep the compatibility
with decompression capability of display controller (gen11+)

Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
---
 src/intel/dev/gen_device_info.c | 3 +++
 src/intel/dev/gen_device_info.h | 1 +
 2 files changed, 4 insertions(+)

Patch hide | download patch | download mbox

diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c
index fec6159fd37..2b5be842a4e 100644
--- a/src/intel/dev/gen_device_info.c
+++ b/src/intel/dev/gen_device_info.c
@@ -958,6 +958,7 @@  static const struct gen_device_info gen_device_info_ehl_4x8 = {
          [MESA_SHADER_GEOMETRY]  = 1032,
       },
    },
+   .disable_ccs_repack = 1,
    .simulator_id = 28,
 };
 
@@ -978,6 +979,7 @@  static const struct gen_device_info gen_device_info_ehl_4x4 = {
          [MESA_SHADER_GEOMETRY]  = 1032,
       },
    },
+   .disable_ccs_repack = 1,
    .num_eu_per_subslice = 4,
    .simulator_id = 28,
 };
@@ -999,6 +1001,7 @@  static const struct gen_device_info gen_device_info_ehl_2x4 = {
          [MESA_SHADER_GEOMETRY]  = 1032,
       },
    },
+   .disable_ccs_repack = 1,
    .num_eu_per_subslice =4,
    .simulator_id = 28,
 };
diff --git a/src/intel/dev/gen_device_info.h b/src/intel/dev/gen_device_info.h
index af13615be2b..4fe937355a7 100644
--- a/src/intel/dev/gen_device_info.h
+++ b/src/intel/dev/gen_device_info.h
@@ -74,6 +74,7 @@  struct gen_device_info
    bool has_surface_tile_offset;
    bool supports_simd16_3src;
    bool has_resource_streamer;
+   bool disable_ccs_repack;
 
    /**
     * \name Intel hardware quirks

Comments

On Thu, Jun 27, 2019 at 9:55 AM Dongwon Kim <dongwon.kim@intel.com> wrote:
>
> add a new attribute, 'disable_ccs_repack' to gen_device info, which
> indicates whether repacking of components in certain pixel formats
> before compression needs to be disabled to keep the compatibility
> with decompression capability of display controller (gen11+)
>
> Signed-off-by: Dongwon Kim <dongwon.kim@intel.com>
> ---
>  src/intel/dev/gen_device_info.c | 3 +++
>  src/intel/dev/gen_device_info.h | 1 +
>  2 files changed, 4 insertions(+)
>
> diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c
> index fec6159fd37..2b5be842a4e 100644
> --- a/src/intel/dev/gen_device_info.c
> +++ b/src/intel/dev/gen_device_info.c
> @@ -958,6 +958,7 @@ static const struct gen_device_info gen_device_info_ehl_4x8 = {
>           [MESA_SHADER_GEOMETRY]  = 1032,
>        },
>     },
> +   .disable_ccs_repack = 1,
disable_ccs_repack = true

Similar change at few other places in this patch.
>     .simulator_id = 28,
>  };
>
> @@ -978,6 +979,7 @@ static const struct gen_device_info gen_device_info_ehl_4x4 = {
>           [MESA_SHADER_GEOMETRY]  = 1032,
>        },
>     },
> +   .disable_ccs_repack = 1,
>     .num_eu_per_subslice = 4,
>     .simulator_id = 28,
>  };
> @@ -999,6 +1001,7 @@ static const struct gen_device_info gen_device_info_ehl_2x4 = {
>           [MESA_SHADER_GEOMETRY]  = 1032,
>        },
>     },
> +   .disable_ccs_repack = 1,
>     .num_eu_per_subslice =4,
>     .simulator_id = 28,
>  };
> diff --git a/src/intel/dev/gen_device_info.h b/src/intel/dev/gen_device_info.h
> index af13615be2b..4fe937355a7 100644
> --- a/src/intel/dev/gen_device_info.h
> +++ b/src/intel/dev/gen_device_info.h
> @@ -74,6 +74,7 @@ struct gen_device_info
>     bool has_surface_tile_offset;
>     bool supports_simd16_3src;
>     bool has_resource_streamer;
> +   bool disable_ccs_repack;
>
>     /**
>      * \name Intel hardware quirks
> --
> 2.17.1
>