[Mesa-dev] [PATCH 12/24] i965/cnl: Add cnl bits in aubinator

[Anuj Phogat]

On Sat, May 13, 2017 at 9:39 AM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> On May 12, 2017 4:41:22 PM Anuj Phogat <anuj.phogat at gmail.com> wrote:
>
>> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
>> ---
>>  src/intel/tools/aubinator.c | 8 +++++---
>>  1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
>> index 53b2a27..d15476d 100644
>> --- a/src/intel/tools/aubinator.c
>> +++ b/src/intel/tools/aubinator.c
>> @@ -970,7 +970,8 @@ struct {
>>     { "bdw", MAKE_GEN(8, 0) },
>>     { "skl", MAKE_GEN(9, 0) },
>>     { "chv", MAKE_GEN(8, 0) },
>> -   { "bxt", MAKE_GEN(9, 0) }
>> +   { "bxt", MAKE_GEN(9, 0) },
>> +   { "cnl", MAKE_GEN(10, 0) }
>
>
> Mind leaving a trailing comma?  That way the next person to add an entry
> doesn't have to change two lines.
>
>>  };
>>
>>  enum {
>> @@ -1138,7 +1139,7 @@ print_help(const char *progname, FILE *file)
>>             "Decode aub file contents from either FILE or the standard
>> input.\n\n"
>>             "A valid --gen option must be provided.\n\n"
>>             "      --help          display this help and exit\n"
>> -           "      --gen=platform  decode for given platform (ivb, byt,
>> hsw, bdw, chv, skl, kbl or bxt)\n"
>> +           "      --gen=platform  decode for given platform (ivb, byt,
>> hsw, bdw, chv, skl, kbl, bxt or cnl)\n"
>>             "      --headers       decode only command headers\n"
>>             "      --color[=WHEN]  colorize the output; WHEN can be 'auto'
>> (default\n"
>>             "                        if omitted), 'always', or 'never'\n"
>> @@ -1166,7 +1167,8 @@ int main(int argc, char *argv[])
>>        { "chv", 0x22B3 }, /* Intel(R) HD Graphics (Cherryview) */
>>        { "skl", 0x1912 }, /* Intel(R) HD Graphics 530 (Skylake GT2) */
>>        { "kbl", 0x591D }, /* Intel(R) Kabylake GT2 */
>> -      { "bxt", 0x0A84 }  /* Intel(R) HD Graphics (Broxton) */
>> +      { "bxt", 0x0A84 },  /* Intel(R) HD Graphics (Broxton) */
>> +      { "cnl", 0x5A52 }  /* Intel(R) HD Graphics (Cannonlake) */
>
>
> Same here.
Made the changes locally.
>
>>     };
>>     const struct option aubinator_opts[] = {
>>        { "help",       no_argument,       (int *) &help,
>> true },
>> --
>> 2.9.3
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
>