Re: [PATCH v5 03/13] drm/xe/hwmon: Fix memory controller thermal data handling

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] xe_hwmon: kernel-doc mismatch due to typo '@vavlue'
--

commit af26cf662405440ebacf68b7cf48c30a800a4da6
Author: Karthik Poosa <karthik.poosa@intel.com>

drm/xe/hwmon: Fix memory controller thermal data handling

This commit fixes memory controller thermal data handling by reading all
required thermal groups in get_mc_temp() and using a u32-aligned buffer for
PCODE accesses to avoid unaligned memory access issues.

> diff --git a/drivers/gpu/drm/xe/xe_hwmon.c b/drivers/gpu/drm/xe/xe_hwmon.c
> --- a/drivers/gpu/drm/xe/xe_hwmon.c
> +++ b/drivers/gpu/drm/xe/xe_hwmon.c
> @@ -143,8 +143,12 @@ struct xe_hwmon_thermal_info {
>  	};
>  	/** @count: no of temperature sensors available for the platform */
>  	u8 count;
> -	/** @value: per-sensor raw mailbox temperature; bit7=sign, bits6:0=magnitude */
> -	u8 value[U8_MAX];
> +	union {
> +		/** @vavlue: per-sensor raw mailbox temperature; bit7=sign, bits6:0=magnitude */

[Severity: Low]
This isn't a bug, but is there a typo in the kernel-doc here? It looks like
it should be @value instead of @vavlue to match the union member name.

> +		u8 value[U8_MAX + 1];
> +		/** @dword: sensor values as dwords, u32-aligned for pcode reads */
> +		u32 dword[DIV_ROUND_UP(U8_MAX + 1, sizeof(u32))];
> +	};
>  	/** @vram_label: vram label names */
>  	char vram_label[MAX_VRAM_CHANNELS][MAX_LABEL_SIZE];
>  };

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260924111156.2628915-1-karthik.poosa@intel.com?part=3