[Mesa-dev] [PATCH] i965/screen: Allow drirc to set 'allow_rgb10_configs' again.

[Tapani Pälli]

Yeah that works, thanks Mario!

Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

On 11.01.2018 18:00, Mario Kleiner wrote:
> Since setup of ALLOW_RGB10_CONFIGS was moved to i965's own
> brw_config_options.xml, this was hard-coded to false and
> could not be overriden by drirc. Add some parsing into
> i965's private screen->optionCache to enable drirc again.
> 
> Fixes: b391fb26df9f1b ("dri_util: remove ALLOW_RGB10_CONFIGS option (v2)")
> Signed-off-by: Mario Kleiner <mario.kleiner.de at gmail.com>
> Cc: Marek Olšák <marek.olsak at amd.com>
> Cc: Tapani Pälli <tapani.palli at intel.com>
> ---
>   src/mesa/drivers/dri/i965/intel_screen.c | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c
> index 08032c9..eb4b569 100644
> --- a/src/mesa/drivers/dri/i965/intel_screen.c
> +++ b/src/mesa/drivers/dri/i965/intel_screen.c
> @@ -2397,7 +2397,12 @@ __DRIconfig **intelInitScreen2(__DRIscreen *dri_screen)
>         return NULL;
>      }
>      /* parse information in __driConfigOptions */
> -   driParseOptionInfo(&screen->optionCache, brw_config_options.xml);
> +   driOptionCache options;
> +   memset(&options, 0, sizeof(options));
> +
> +   driParseOptionInfo(&options, brw_config_options.xml);
> +   driParseConfigFiles(&screen->optionCache, &options, dri_screen->myNum, "i965");
> +   driDestroyOptionCache(&options);
>   
>      screen->driScrnPriv = dri_screen;
>      dri_screen->driverPrivate = (void *) screen;
>