From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.151]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7CB4C10E228 for ; Mon, 18 Sep 2023 09:03:06 +0000 (UTC) Message-ID: Date: Mon, 18 Sep 2023 11:01:46 +0200 Content-Language: en-US To: References: <20230915051617.9361-1-sai.gowtham.ch@intel.com> <20230915051617.9361-3-sai.gowtham.ch@intel.com> From: Karolina Stolarek In-Reply-To: <20230915051617.9361-3-sai.gowtham.ch@intel.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Subject: Re: [igt-dev] [PATCH i-g-t 2/4] lib/intel_reg: Add copy commands in the lib List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On 15.09.2023 07:16, sai.gowtham.ch@intel.com wrote: > From: Sai Gowtham Ch > > Add memory copy and set commands to the lib. > > Signed-off-by: Sai Gowtham Ch > --- > lib/intel_reg.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/lib/intel_reg.h b/lib/intel_reg.h > index 3bf3676dc..322aec9fd 100644 > --- a/lib/intel_reg.h > +++ b/lib/intel_reg.h > @@ -2564,6 +2564,10 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. > > #define XY_FAST_COLOR_BLT ((0x2<<29)|(0x44<<22)|0xe) > > +/* RAW memory commands */ > +#define MEM_COPY_CMD ((0x2 << 29)|(0x5a << 22)|0x8) > +#define MEM_SET_CMD ((0x2 << 29)|(0x5b << 22)|0x5) > + This is inserted in the middle of BLT commands and their arguments. Could you move it so it's either before the BLT block or after XY_FAST_COPY_COLOR_DEPTH_128? Many thanks, Karolina > #define XY_FAST_COPY_BLT ((2<<29)|(0x42<<22)|0x8) > /* dword 0 */ > #define XY_FAST_COPY_SRC_TILING_LINEAR (0 << 20)