Re: [PATCH v3 5/5] drm/xe/hwmon: Disable memory controller and PCIe temperatures on CRI

["Poosa, Karthik" <karthik.poosa@intel.com>]

On 18-09-2026 15:50, Rodrigo Vivi wrote:
> On Thu, Sep 10, 2026 at 11:57:56PM +0530, Karthik Poosa wrote:
>> Temporarily disable memory controller and PCIe temperature reporting on
>> CRI platforms.
>>   
>> The definitions and interpretation of these temperature sources are not
>> yet sufficiently clear, which may result in inaccurate reporting.
>> Disable them until the hardware interface and corresponding definitions
>> are fully validated.
> I'm wondering if perhaps some has_ flag should be more appropriate,
> but we would also need to think a way that we don't explode that...
>
> so, for now let's move with this
>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

We can omit this.We have got offsets of memory controller and pcie 
sensors mailbox offsets on CRI from HW team.
I shall raise patch for it.

>
>> Signed-off-by: Karthik Poosa <karthik.poosa@intel.com>
>> ---
>>   drivers/gpu/drm/xe/xe_hwmon.c | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/xe/xe_hwmon.c b/drivers/gpu/drm/xe/xe_hwmon.c
>> index e4a210916df2..669bbfe18cea 100644
>> --- a/drivers/gpu/drm/xe/xe_hwmon.c
>> +++ b/drivers/gpu/drm/xe/xe_hwmon.c
>> @@ -1094,8 +1094,14 @@ static bool is_temp_available(struct xe_hwmon *hwmon, int channel)
>>   
>>   		return true;
>>   	case CHANNEL_MCTRL:
>> +		if (hwmon->xe->info.platform >= XE_CRESCENTISLAND)
>> +			return false;
>> +
>>   		return hwmon->temp.count && !get_mc_temp(hwmon, NULL);
>>   	case CHANNEL_PCIE:
>> +		if (hwmon->xe->info.platform >= XE_CRESCENTISLAND)
>> +			return false;
>> +
>>   		return hwmon->temp.count && !get_pcie_temp(hwmon, NULL);
>>   	default:
>>   		return false;
>> -- 
>> 2.25.1
>>