lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20fb9f76-558b-48c8-92b0-9820407bb82f@oss.qualcomm.com>
Date: Thu, 2 Oct 2025 03:00:03 +0530
From: Akhil P Oommen <akhilpo@....qualcomm.com>
To: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
Cc: Rob Clark <robin.clark@....qualcomm.com>,
        Bjorn Andersson <andersson@...nel.org>,
        Konrad Dybcio <konradybcio@...nel.org>, Sean Paul <sean@...rly.run>,
        Dmitry Baryshkov <lumag@...nel.org>,
        Abhinav Kumar
 <abhinav.kumar@...ux.dev>,
        Jessica Zhang <jessica.zhang@....qualcomm.com>,
        Marijn Suijten <marijn.suijten@...ainline.org>,
        David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
        Jonathan Marek <jonathan@...ek.ca>,
        Jordan Crouse
 <jordan@...micpenguin.net>,
        Will Deacon <will@...nel.org>, Robin Murphy <robin.murphy@....com>,
        Joerg Roedel <joro@...tes.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Zimmermann <tzimmermann@...e.de>, linux-arm-msm@...r.kernel.org,
        linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        freedreno@...ts.freedesktop.org, linux-arm-kernel@...ts.infradead.org,
        iommu@...ts.linux.dev, devicetree@...r.kernel.org
Subject: Re: [PATCH 11/17] drm/msm/a8xx: Add support for A8x GMU



On 9/30/2025 1:05 PM, Dmitry Baryshkov wrote:
> On Tue, Sep 30, 2025 at 11:18:16AM +0530, Akhil P Oommen wrote:
>> A8x GMU configuration are very similar to A7x. Unfortunately, there are
>> minor shuffling in the register offsets in the GMU CX register region.
>> Apart from that, there is a new HFI message support to pass table like
>> data. This patch adds support for  perf table using this new HFI
>> message.
>>
>> Apart from that, there is a minor rework in a6xx_gmu_rpmh_arc_votes_init()
>> to simplify handling of MxG to MxA fallback along with the additional
>> calculations for the new dependency vote.
> 
> I'm sorry, I've sent it too early. This looks like a description
> of a not-that-related change which should be split to a separate commit.
> 
>>
>> Signed-off-by: Akhil P Oommen <akhilpo@....qualcomm.com>
>> ---
>>   drivers/gpu/drm/msm/adreno/a6xx_gmu.c             | 161 +++++++++++++++++-----
>>   drivers/gpu/drm/msm/adreno/a6xx_gmu.h             |   5 +-
>>   drivers/gpu/drm/msm/adreno/a6xx_hfi.c             |  53 +++++++
>>   drivers/gpu/drm/msm/adreno/a6xx_hfi.h             |  17 +++
>>   drivers/gpu/drm/msm/adreno/adreno_gpu.h           |   7 +
>>   drivers/gpu/drm/msm/registers/adreno/a6xx_gmu.xml |  48 +++++--
>>   6 files changed, 242 insertions(+), 49 deletions(-)
>>
>> @@ -592,12 +606,16 @@ static void a6xx_gmu_rpmh_init(struct a6xx_gmu *gmu)
>>   	struct a6xx_gpu *a6xx_gpu = container_of(gmu, struct a6xx_gpu, gmu);
>>   	struct adreno_gpu *adreno_gpu = &a6xx_gpu->base;
>>   	struct platform_device *pdev = to_platform_device(gmu->dev);
>> -	void __iomem *pdcptr = a6xx_gmu_get_mmio(pdev, "gmu_pdc", NULL);
>>   	u32 seqmem0_drv0_reg = REG_A6XX_RSCC_SEQ_MEM_0_DRV0;
>>   	void __iomem *seqptr = NULL;
>>   	uint32_t pdc_address_offset;
>> +	void __iomem *pdcptr;
>>   	bool pdc_in_aop = false;
>>   
> 
> A comment would be nice.
> 
>> +	if (adreno_is_a8xx(adreno_gpu))
>> +		return;
>> +
>> +	pdcptr = a6xx_gmu_get_mmio(pdev, "gmu_pdc", NULL);
>>   	if (IS_ERR(pdcptr))
>>   		goto err;
>>   
>> @@ -1489,13 +1523,14 @@ static unsigned int a6xx_gmu_get_arc_level(struct device *dev,
>>   }
>>   
>>   static int a6xx_gmu_rpmh_arc_votes_init(struct device *dev, u32 *votes,
>> -		unsigned long *freqs, int freqs_count, const char *id)
>> +		unsigned long *freqs, int freqs_count,
>> +		const char *pri_id, const char *sec_id)
>>   {
>>   	int i, j;
>>   	const u16 *pri, *sec;
>>   	size_t pri_count, sec_count;
>>   
>> -	pri = cmd_db_read_aux_data(id, &pri_count);
>> +	pri = cmd_db_read_aux_data(pri_id, &pri_count);
> 
> separate commit
> 
>>   	if (IS_ERR(pri))
>>   		return PTR_ERR(pri);
>>   	/*
>> @@ -1506,13 +1541,7 @@ static int a6xx_gmu_rpmh_arc_votes_init(struct device *dev, u32 *votes,
>>   	if (!pri_count)
>>   		return -EINVAL;
>>   
>> -	/*
>> -	 * Some targets have a separate gfx mxc rail. So try to read that first and then fall back
>> -	 * to regular mx rail if it is missing
>> -	 */
>> -	sec = cmd_db_read_aux_data("gmxc.lvl", &sec_count);
>> -	if (IS_ERR(sec) && sec != ERR_PTR(-EPROBE_DEFER))
>> -		sec = cmd_db_read_aux_data("mx.lvl", &sec_count);
>> +	sec = cmd_db_read_aux_data(sec_id, &sec_count);
>>   	if (IS_ERR(sec))
>>   		return PTR_ERR(sec);
>>   
>> @@ -1566,6 +1595,57 @@ static int a6xx_gmu_rpmh_arc_votes_init(struct device *dev, u32 *votes,
>>   	return 0;
>>   }
>>   
>> +static int a6xx_gmu_rpmh_dep_votes_init(struct device *dev, u32 *votes,
>> +		unsigned long *freqs, int freqs_count)
> 
> Definitely a separate commit
> 
>> +{
>> +	const u16 *mx;
>> +	size_t count;
>> +
>> +	mx = cmd_db_read_aux_data("mx.lvl", &count);
>> +	if (IS_ERR(mx))
>> +		return PTR_ERR(mx);
>> +	/*
>> +	 * The data comes back as an array of unsigned shorts so adjust the
>> +	 * count accordingly
>> +	 */
>> +	count >>= 1;
>> +	if (!count)
>> +		return -EINVAL;
>> +
>> +	/* Fix the vote for zero frequency */
>> +	votes[0] = 0xFFFFFFFF;
> 
> lowercase
> 
>> +
>> +	/* Construct a vote for rest of the corners */
>> +	for (int i = 1; i < freqs_count; i++) {
>> +		u8 j, index = 0;
>> +		unsigned int level = a6xx_gmu_get_arc_level(dev, freqs[i]);
>> +
>> +		/* Get the primary index that matches the arc level */
>> +		for (j = 0; j < count; j++) {
>> +			if (mx[j] >= level) {
>> +				index = j;
>> +				break;
>> +			}
>> +		}
>> +
>> +		if (j == count) {
>> +			DRM_DEV_ERROR(dev,
>> +				      "Mx Level %u not found in the RPMh list\n",
>> +				      level);
>> +			DRM_DEV_ERROR(dev, "Available levels:\n");
>> +			for (j = 0; j < count; j++)
>> +				DRM_DEV_ERROR(dev, "  %u\n", mx[j]);
>> +
>> +			return -EINVAL;
>> +		}
>> +
>> +		/* Construct the vote */
>> +		votes[i] = (0x3fff << 14) | (index << 8) | (0xff);
>> +	}
>> +
>> +	return 0;
>> +}
>> +
>>   /*
>>    * The GMU votes with the RPMh for itself and on behalf of the GPU but we need
>>    * to construct the list of votes on the CPU and send it over. Query the RPMh
>> @@ -1580,15 +1660,27 @@ static int a6xx_gmu_rpmh_votes_init(struct a6xx_gmu *gmu)
>>   	struct adreno_gpu *adreno_gpu = &a6xx_gpu->base;
>>   	const struct a6xx_info *info = adreno_gpu->info->a6xx;
>>   	struct msm_gpu *gpu = &adreno_gpu->base;
>> +	const char *sec_id;
>> +	const u16 *gmxc;
>>   	int ret;
>>   
>> +	gmxc = cmd_db_read_aux_data("gmxc.lvl", NULL);
>> +	if (gmxc == ERR_PTR(-EPROBE_DEFER))
>> +		return -EPROBE_DEFER;
>> +
>> +	/* If GMxC is present, prefer that as secondary rail for GX votes */
>> +	sec_id = IS_ERR_OR_NULL(gmxc) ? "mx.lvl" : "gmxc.lvl";
> 
> Can it be NULL?
> 

It seems it cannot be.

>> +
>>   	/* Build the GX votes */
>>   	ret = a6xx_gmu_rpmh_arc_votes_init(&gpu->pdev->dev, gmu->gx_arc_votes,
>> -		gmu->gpu_freqs, gmu->nr_gpu_freqs, "gfx.lvl");
>> +		gmu->gpu_freqs, gmu->nr_gpu_freqs, "gfx.lvl", sec_id);
>>   
>>   	/* Build the CX votes */
>>   	ret |= a6xx_gmu_rpmh_arc_votes_init(gmu->dev, gmu->cx_arc_votes,
>> -		gmu->gmu_freqs, gmu->nr_gmu_freqs, "cx.lvl");
>> +		gmu->gmu_freqs, gmu->nr_gmu_freqs, "cx.lvl", "mx.lvl");
>> +
>> +	ret |= a6xx_gmu_rpmh_dep_votes_init(gmu->dev, gmu->dep_arc_votes,
>> +		gmu->gpu_freqs, gmu->nr_gpu_freqs);
>>   
>>   	/* Build the interconnect votes */
>>   	if (info->bcms && gmu->nr_gpu_bws > 1)
>> @@ -2043,14 +2135,14 @@ int a6xx_gmu_init(struct a6xx_gpu *a6xx_gpu, struct device_node *node)
>>   	 * are otherwise unused by a660.
>>   	 */
>>   	gmu->dummy.size = SZ_4K;
>> -	if (adreno_is_a660_family(adreno_gpu) ||
>> -	    adreno_is_a7xx(adreno_gpu)) {
>> +	if (adreno_is_a660_family(adreno_gpu) || adreno_is_a7xx(adreno_gpu) ||
>> +			adreno_is_a8xx(adreno_gpu)) {
>>   		ret = a6xx_gmu_memory_alloc(gmu, &gmu->debug, SZ_4K * 7,
>>   					    0x60400000, "debug");
>>   		if (ret)
>>   			goto err_memory;
>>   
>> -		gmu->dummy.size = SZ_8K;
>> +		gmu->dummy.size = SZ_16K;
>>   	}
>>   
>>   	/* Allocate memory for the GMU dummy page */
>> @@ -2060,8 +2152,8 @@ int a6xx_gmu_init(struct a6xx_gpu *a6xx_gpu, struct device_node *node)
>>   		goto err_memory;
>>   
>>   	/* Note that a650 family also includes a660 family: */
>> -	if (adreno_is_a650_family(adreno_gpu) ||
>> -	    adreno_is_a7xx(adreno_gpu)) {
>> +	if (adreno_is_a650_family(adreno_gpu) || adreno_is_a7xx(adreno_gpu) ||
>> +			adreno_is_a8xx(adreno_gpu)) {
> 
> Please adjust your editor settings. It would be much easier to read if
> it was:>
> 	if (adreno_is_a650_family(adreno_gpu) ||
> 	    adreno_is_a7xx(adreno_gpu) ||
> 	    adreno_is_a8xx(adreno_gpu)) {
> 

Agree. will update.

> (Adjust it here and in other places)
> 
>>   		ret = a6xx_gmu_memory_alloc(gmu, &gmu->icache,
>>   			SZ_16M - SZ_16K, 0x04000, "icache");
>>   		if (ret)
> 
> [...]
> 
>> @@ -255,11 +256,63 @@ static int a6xx_hfi_send_perf_table_v1(struct a6xx_gmu *gmu)
>>   		NULL, 0);
>>   }
>>   
>> +static int a8xx_hfi_send_perf_table(struct a6xx_gmu *gmu)
>> +{
>> +	unsigned int num_gx_votes = 3, num_cx_votes = 2;
>> +	struct a6xx_hfi_table_entry *entry;
>> +	struct a6xx_hfi_table *tbl;
>> +	int ret, i;
>> +	u32 size;
> 
> Separate commit.
> 
>> +
>> +	size = sizeof(*tbl) +  (2 * sizeof(tbl->entry[0])) +
>> +		(gmu->nr_gpu_freqs * num_gx_votes * sizeof(gmu->gx_arc_votes[0])) +
>> +		(gmu->nr_gmu_freqs * num_cx_votes * sizeof(gmu->cx_arc_votes[0]));
>> +	tbl = devm_kzalloc(gmu->dev, size, GFP_KERNEL);
>> +	tbl->type = HFI_TABLE_GPU_PERF;
>> +
>> +	/* First fill GX votes */
>> +	entry = &tbl->entry[0];
>> +	entry->count = gmu->nr_gpu_freqs;
>> +	entry->stride = num_gx_votes;
>> +
>> +	for (i = 0; i < gmu->nr_gpu_freqs; i++) {
>> +		unsigned int base = i * entry->stride;
>> +
>> +		entry->data[base+0] = gmu->gx_arc_votes[i];
>> +		entry->data[base+1] = gmu->dep_arc_votes[i];
>> +		entry->data[base+2] = gmu->gpu_freqs[i] / 1000;
>> +	}
>> +
>> +	/* Then fill CX votes */
>> +	entry = (struct a6xx_hfi_table_entry *)
>> +		&tbl->entry[0].data[gmu->nr_gpu_freqs * num_gx_votes];
>> +
>> +	entry->count = gmu->nr_gmu_freqs;
>> +	entry->stride = num_cx_votes;
>> +
>> +	for (i = 0; i < gmu->nr_gmu_freqs; i++) {
>> +		unsigned int base = i * entry->stride;
>> +
>> +		entry->data[base] = gmu->cx_arc_votes[i];
>> +		entry->data[base+1] = gmu->gmu_freqs[i] / 1000;
>> +	}
>> +
>> +	ret = a6xx_hfi_send_msg(gmu, HFI_H2F_MSG_TABLE, tbl, size, NULL, 0);
>> +
>> +	devm_kfree(gmu->dev, tbl);
>> +	return ret;
>> +}
>> +
>>   static int a6xx_hfi_send_perf_table(struct a6xx_gmu *gmu)
>>   {
>> +	struct a6xx_gpu *a6xx_gpu = container_of(gmu, struct a6xx_gpu, gmu);
>> +	struct adreno_gpu *adreno_gpu = &a6xx_gpu->base;
>>   	struct a6xx_hfi_msg_perf_table msg = { 0 };
>>   	int i;
>>   
>> +	if (adreno_is_a8xx(adreno_gpu))
>> +		return a8xx_hfi_send_perf_table(gmu);
>> +
>>   	msg.num_gpu_levels = gmu->nr_gpu_freqs;
>>   	msg.num_gmu_levels = gmu->nr_gmu_freqs;
>>   
>> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_hfi.h b/drivers/gpu/drm/msm/adreno/a6xx_hfi.h
>> index 653ef720e2da4d2b0793c0b76e994b6f6dc524c7..e12866110cb8ea0c075b3ae5e4cae679405c4bd1 100644
>> --- a/drivers/gpu/drm/msm/adreno/a6xx_hfi.h
>> +++ b/drivers/gpu/drm/msm/adreno/a6xx_hfi.h
>> @@ -185,6 +185,23 @@ struct a6xx_hfi_msg_core_fw_start {
>>   	u32 handle;
>>   };
>>   
>> +#define HFI_H2F_MSG_TABLE 15
>> +
>> +struct a6xx_hfi_table_entry {
>> +	u32 count;
>> +	u32 stride;
>> +	u32 data[];
>> +};
>> +
>> +struct a6xx_hfi_table {
>> +	u32 header;
>> +	u32 version;
>> +#define HFI_TABLE_BW_VOTE 0
>> +#define HFI_TABLE_GPU_PERF 1
>> +	u32 type;
>> +	struct a6xx_hfi_table_entry entry[];
>> +};
>> +
>>   #define HFI_H2F_MSG_GX_BW_PERF_VOTE 30
>>   
>>   struct a6xx_hfi_gx_bw_perf_vote_cmd {
>> diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.h b/drivers/gpu/drm/msm/adreno/adreno_gpu.h
>> index b27974d97c7512ecae326eb2d22238330d6c52f0..9831401c3bc865b803c2f9759d5e2ffcd79d19f8 100644
>> --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.h
>> +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.h
>> @@ -50,6 +50,8 @@ enum adreno_family {
>>   	ADRENO_7XX_GEN1,  /* a730 family */
>>   	ADRENO_7XX_GEN2,  /* a740 family */
>>   	ADRENO_7XX_GEN3,  /* a750 family */
>> +	ADRENO_8XX_GEN1,  /* a830 family */
>> +	ADRENO_8XX_GEN2,  /* a840 family */
>>   };
>>   
>>   #define ADRENO_QUIRK_TWO_PASS_USE_WFI		BIT(0)
>> @@ -555,6 +557,11 @@ static inline int adreno_is_a7xx(struct adreno_gpu *gpu)
>>   	       adreno_is_a740_family(gpu);
>>   }
>>   
>> +static inline int adreno_is_a8xx(struct adreno_gpu *gpu)
>> +{
>> +	return gpu->info->family >= ADRENO_8XX_GEN1;
>> +}
> 
> This and the register mask updates can go to a separate commit.

Which mask update exactly?

I can split out the hfi table addition and arc table updates into 2 
separate patches.

-Akhil.

> 
>> +
>>   /* Put vm_start above 32b to catch issues with not setting xyz_BASE_HI */
>>   #define ADRENO_VM_START 0x100000000ULL
>>   u64 adreno_private_vm_size(struct msm_gpu *gpu);
>> diff --git a/drivers/gpu/drm/msm/registers/adreno/a6xx_gmu.xml b/drivers/gpu/drm/msm/registers/adreno/a6xx_gmu.xml
>> index 09b8a0b9c0de7615f7e7e6364c198405a498121a..5dce7934056dd6472c368309b4894f0ed4a4d960 100644
>> --- a/drivers/gpu/drm/msm/registers/adreno/a6xx_gmu.xml
>> +++ b/drivers/gpu/drm/msm/registers/adreno/a6xx_gmu.xml
>> @@ -66,10 +66,15 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   	<reg32 offset="0x1f81c" name="GMU_CM3_FW_INIT_RESULT"/>
>>   	<reg32 offset="0x1f82d" name="GMU_CM3_CFG"/>
>>   	<reg32 offset="0x1f840" name="GMU_CX_GMU_POWER_COUNTER_ENABLE"/>
>> +	<reg32 offset="0x1fc10" name="GMU_CX_GMU_POWER_COUNTER_ENABLE" variants="A8XX"/>
>>   	<reg32 offset="0x1f841" name="GMU_CX_GMU_POWER_COUNTER_SELECT_0"/>
>>   	<reg32 offset="0x1f842" name="GMU_CX_GMU_POWER_COUNTER_SELECT_1"/>
>> +	<reg32 offset="0x1fc40" name="GMU_CX_GMU_POWER_COUNTER_SELECT_XOCLK_0" variants="A8XX-"/>
>> +	<reg32 offset="0x1fc41" name="GMU_CX_GMU_POWER_COUNTER_SELECT_XOCLK_1" variants="A8XX-"/>
>>   	<reg32 offset="0x1f844" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_0_L"/>
>> +	<reg32 offset="0x1fca0" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_0_L" variants="A8XX-"/>
>>   	<reg32 offset="0x1f845" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_0_H"/>
>> +	<reg32 offset="0x1fca1" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_0_H" variants="A8XX-"/>
>>   	<reg32 offset="0x1f846" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_1_L"/>
>>   	<reg32 offset="0x1f847" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_1_H"/>
>>   	<reg32 offset="0x1f848" name="GMU_CX_GMU_POWER_COUNTER_XOCLK_2_L"/>
>> @@ -89,7 +94,7 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   	</reg32>
>>   	<reg32 offset="0x1f8c1" name="GMU_PWR_COL_INTER_FRAME_HYST"/>
>>   	<reg32 offset="0x1f8c2" name="GMU_PWR_COL_SPTPRAC_HYST"/>
>> -	<reg32 offset="0x1f8d0" name="GMU_SPTPRAC_PWR_CLK_STATUS">
>> +	<reg32 offset="0x1f8d0" name="GMU_SPTPRAC_PWR_CLK_STATUS" variants="A6XX">
>>   		<bitfield name="SPTPRAC_GDSC_POWERING_OFF" pos="0" type="boolean"/>
>>   		<bitfield name="SPTPRAC_GDSC_POWERING_ON" pos="1" type="boolean"/>
>>   		<bitfield name="SPTPRAC_GDSC_POWER_OFF" pos="2" type="boolean"/>
>> @@ -99,7 +104,11 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   		<bitfield name="GX_HM_GDSC_POWER_OFF" pos="6" type="boolean"/>
>>   		<bitfield name="GX_HM_CLK_OFF" pos="7" type="boolean"/>
>>   	</reg32>
>> -	<reg32 offset="0x1f8d0" name="GMU_SPTPRAC_PWR_CLK_STATUS" variants="A7XX-">
>> +	<reg32 offset="0x1f8d0" name="GMU_SPTPRAC_PWR_CLK_STATUS" variants="A7XX">
>> +		<bitfield name="GX_HM_GDSC_POWER_OFF" pos="0" type="boolean"/>
>> +		<bitfield name="GX_HM_CLK_OFF" pos="1" type="boolean"/>
>> +	</reg32>
>> +	<reg32 offset="0x1f7e8" name="GMU_PWR_CLK_STATUS" variants="A8XX-">
>>   		<bitfield name="GX_HM_GDSC_POWER_OFF" pos="0" type="boolean"/>
>>   		<bitfield name="GX_HM_CLK_OFF" pos="1" type="boolean"/>
>>   	</reg32>
>> @@ -120,9 +129,12 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   		<bitfield name="GFX_MIN_VOTE_ENABLE" pos="15" type="boolean"/>
>>   	</reg32>
>>   	<reg32 offset="0x1f8e9" name="GMU_RPMH_HYST_CTRL"/>
>> -	<reg32 offset="0x1f8ec" name="GPU_GMU_CX_GMU_RPMH_POWER_STATE"/>
>> -	<reg32 offset="0x1f8f0" name="GPU_GMU_CX_GMU_CX_FAL_INTF"/>
>> -	<reg32 offset="0x1f8f1" name="GPU_GMU_CX_GMU_CX_FALNEXT_INTF"/>
>> +	<reg32 offset="0x1f8ec" name="GPU_GMU_CX_GMU_RPMH_POWER_STATE" variants="A6XX"/>
>> +	<reg32 offset="0x1f7e9" name="GPU_GMU_CX_GMU_RPMH_POWER_STATE" variants="A8XX-"/>
>> +	<reg32 offset="0x1f8f0" name="GPU_GMU_CX_GMU_CX_FAL_INTF" variants="A6XX"/>
>> +	<reg32 offset="0x1f7ec" name="GPU_GMU_CX_GMU_CX_FAL_INTF" variants="A8XX-"/>
>> +	<reg32 offset="0x1f8f1" name="GPU_GMU_CX_GMU_CX_FALNEXT_INTF" variants="A6XX"/>
>> +	<reg32 offset="0x1f7ed" name="GPU_GMU_CX_GMU_CX_FALNEXT_INTF" variants="A8XX-"/>
>>   	<reg32 offset="0x1f900" name="GPU_GMU_CX_GMU_PWR_COL_CP_MSG"/>
>>   	<reg32 offset="0x1f901" name="GPU_GMU_CX_GMU_PWR_COL_CP_RESP"/>
>>   	<reg32 offset="0x1f9f0" name="GMU_BOOT_KMD_LM_HANDSHAKE"/>
>> @@ -130,8 +142,10 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   	<reg32 offset="0x1f958" name="GMU_LLM_GLM_SLEEP_STATUS"/>
>>   	<reg32 offset="0x1f888" name="GMU_ALWAYS_ON_COUNTER_L"/>
>>   	<reg32 offset="0x1f889" name="GMU_ALWAYS_ON_COUNTER_H"/>
>> -	<reg32 offset="0x1f8c3" name="GMU_GMU_PWR_COL_KEEPALIVE"/>
>> -	<reg32 offset="0x1f8c4" name="GMU_PWR_COL_PREEMPT_KEEPALIVE"/>
>> +	<reg32 offset="0x1f8c3" name="GMU_GMU_PWR_COL_KEEPALIVE" variants="A6XX-A7XX"/>
>> +	<reg32 offset="0x1f7e4" name="GMU_GMU_PWR_COL_KEEPALIVE" variants="A8XX-"/>
>> +	<reg32 offset="0x1f8c4" name="GMU_PWR_COL_PREEMPT_KEEPALIVE" variants="A6XX-A7XX"/>
>> +	<reg32 offset="0x1f7e5" name="GMU_PWR_COL_PREEMPT_KEEPALIVE" variants="A8XX-"/>
>>   	<reg32 offset="0x1f980" name="GMU_HFI_CTRL_STATUS"/>
>>   	<reg32 offset="0x1f981" name="GMU_HFI_VERSION_INFO"/>
>>   	<reg32 offset="0x1f982" name="GMU_HFI_SFR_ADDR"/>
>> @@ -164,6 +178,14 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   	<reg32 offset="0x1f9cd" name="GMU_GENERAL_8" variants="A7XX"/>
>>   	<reg32 offset="0x1f9ce" name="GMU_GENERAL_9" variants="A7XX"/>
>>   	<reg32 offset="0x1f9cf" name="GMU_GENERAL_10" variants="A7XX"/>
>> +	<reg32 offset="0x1f9c0" name="GMU_GENERAL_0" variants="A8XX"/>
>> +	<reg32 offset="0x1f9c1" name="GMU_GENERAL_1" variants="A8XX"/>
>> +	<reg32 offset="0x1f9c6" name="GMU_GENERAL_6" variants="A8XX"/>
>> +	<reg32 offset="0x1f9c7" name="GMU_GENERAL_7" variants="A8XX"/>
>> +	<reg32 offset="0x1f9c8" name="GMU_GENERAL_8" variants="A8XX"/>
>> +	<reg32 offset="0x1f9c9" name="GMU_GENERAL_9" variants="A8XX"/>
>> +	<reg32 offset="0x1f9ca" name="GMU_GENERAL_10" variants="A8XX"/>
>> +	<reg32 offset="0x1f9cb" name="GMU_GENERAL_11" variants="A8XX"/>
>>   	<reg32 offset="0x1f95d" name="GMU_ISENSE_CTRL"/>
>>   	<reg32 offset="0x23120" name="GPU_CS_ENABLE_REG"/>
>>   	<reg32 offset="0x1f95d" name="GPU_GMU_CX_GMU_ISENSE_CTRL"/>
>> @@ -233,12 +255,12 @@ xsi:schemaLocation="https://gitlab.freedesktop.org/freedreno/ rules-fd.xsd">
>>   	<reg32 offset="0x03ee" name="RSCC_TCS1_DRV0_STATUS"/>
>>   	<reg32 offset="0x0496" name="RSCC_TCS2_DRV0_STATUS"/>
>>   	<reg32 offset="0x053e" name="RSCC_TCS3_DRV0_STATUS"/>
>> -	<reg32 offset="0x05e6" name="RSCC_TCS4_DRV0_STATUS" variants="A7XX"/>
>> -	<reg32 offset="0x068e" name="RSCC_TCS5_DRV0_STATUS" variants="A7XX"/>
>> -	<reg32 offset="0x0736" name="RSCC_TCS6_DRV0_STATUS" variants="A7XX"/>
>> -	<reg32 offset="0x07de" name="RSCC_TCS7_DRV0_STATUS" variants="A7XX"/>
>> -	<reg32 offset="0x0886" name="RSCC_TCS8_DRV0_STATUS" variants="A7XX"/>
>> -	<reg32 offset="0x092e" name="RSCC_TCS9_DRV0_STATUS" variants="A7XX"/>
>> +	<reg32 offset="0x05e6" name="RSCC_TCS4_DRV0_STATUS" variants="A7XX-"/>
>> +	<reg32 offset="0x068e" name="RSCC_TCS5_DRV0_STATUS" variants="A7XX-"/>
>> +	<reg32 offset="0x0736" name="RSCC_TCS6_DRV0_STATUS" variants="A7XX-"/>
>> +	<reg32 offset="0x07de" name="RSCC_TCS7_DRV0_STATUS" variants="A7XX-"/>
>> +	<reg32 offset="0x0886" name="RSCC_TCS8_DRV0_STATUS" variants="A7XX-"/>
>> +	<reg32 offset="0x092e" name="RSCC_TCS9_DRV0_STATUS" variants="A7XX-"/>
>>   </domain>
>>   
>>   </database>
>>
>> -- 
>> 2.51.0
>>
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ