[Beignet] [PATCH] correct L3 cache settings for baytrail

[Guo, Yejun]

Actually, my original idea is to use static variables inside the function, just changed idea after review function intel_gpgpu_set_L3_gen75

If you don't object, I'll use scalar variables instead of 'array', it will be much more clear.

Thanks
Yejun

-----Original Message-----
From: Song, Ruiling 
Sent: Thursday, May 22, 2014 4:27 PM
To: Guo, Yejun; beignet at lists.freedesktop.org
Cc: Guo, Yejun
Subject: RE: [Beignet] [PATCH] correct L3 cache settings for baytrail

--- a/src/intel/intel_gpgpu.c
+++ b/src/intel/intel_gpgpu.c
@@ -309,14 +309,14 @@ static const uint32_t gpgpu_l3_config_reg1[] = {
   0x00080040, 0x02040040, 0x00800040, 0x01000038,
   0x02000030, 0x01000038, 0x00000038, 0x00000040,
   0x0A140091, 0x09100091, 0x08900091, 0x08900091,
-  0x010000a1
+  0x010000a1, 0x02040001, 0x02040040
 };
 
 static const uint32_t gpgpu_l3_config_reg2[] = {
   0x00000000, 0x00000000, 0x00080410, 0x00080410,
   0x00040410, 0x00040420, 0x00080420, 0x00080020,
   0x00204080, 0x00244890, 0x00284490, 0x002444A0,
-  0x00040810
+  0x00040810, 0x00000000, 0x00000000
 };
 
I would suggest you use another array, don't mixed the configuration, as different HW cannot share the configuration table.