[PATCH 8/9] drm/panthor: Add i.MX95 support
Florent Tomasin
florent.tomasin at arm.com
Fri Feb 28 03:23:16 PST 2025
Hi Marek and Alexander,
On 28/02/2025 10:21, Alexander Stein wrote:
> Hi Marek,
>
> Am Donnerstag, 27. Februar 2025, 17:58:08 CET schrieb Marek Vasut:
>> The instance of the GPU populated in Freescale i.MX95 is the
>> Mali G310, add support for this variant.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Boris Brezillon <boris.brezillon at collabora.com>
>> Cc: Conor Dooley <conor+dt at kernel.org>
>> Cc: David Airlie <airlied at gmail.com>
>> Cc: Fabio Estevam <festevam at gmail.com>
>> Cc: Krzysztof Kozlowski <krzk+dt at kernel.org>
>> Cc: Liviu Dudau <liviu.dudau at arm.com>
>> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>> Cc: Maxime Ripard <mripard at kernel.org>
>> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
>> Cc: Philipp Zabel <p.zabel at pengutronix.de>
>> Cc: Rob Herring <robh at kernel.org>
>> Cc: Sascha Hauer <s.hauer at pengutronix.de>
>> Cc: Sebastian Reichel <sre at kernel.org>
>> Cc: Shawn Guo <shawnguo at kernel.org>
>> Cc: Simona Vetter <simona at ffwll.ch>
>> Cc: Steven Price <steven.price at arm.com>
>> Cc: Thomas Zimmermann <tzimmermann at suse.de>
>> Cc: devicetree at vger.kernel.org
>> Cc: dri-devel at lists.freedesktop.org
>> Cc: imx at lists.linux.dev
>> Cc: linux-arm-kernel at lists.infradead.org
>> ---
>> drivers/gpu/drm/panthor/panthor_drv.c | 1 +
>> drivers/gpu/drm/panthor/panthor_gpu.c | 1 +
>> 2 files changed, 2 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/panthor/panthor_drv.c b/drivers/gpu/drm/panthor/panthor_drv.c
>> index 06fe46e320738..2504a456d45c4 100644
>> --- a/drivers/gpu/drm/panthor/panthor_drv.c
>> +++ b/drivers/gpu/drm/panthor/panthor_drv.c
>> @@ -1591,6 +1591,7 @@ static struct attribute *panthor_attrs[] = {
>> ATTRIBUTE_GROUPS(panthor);
>>
>> static const struct of_device_id dt_match[] = {
>> + { .compatible = "fsl,imx95-mali" }, /* G310 */
No sure the comment about G310 is needed.
>> { .compatible = "rockchip,rk3588-mali" },
>> { .compatible = "arm,mali-valhall-csf" },
>> {}
>> diff --git a/drivers/gpu/drm/panthor/panthor_gpu.c b/drivers/gpu/drm/panthor/panthor_gpu.c
>> index 0f07ef7d9aea7..2371ab8e50627 100644
>> --- a/drivers/gpu/drm/panthor/panthor_gpu.c
>> +++ b/drivers/gpu/drm/panthor/panthor_gpu.c
>> @@ -67,6 +67,7 @@ struct panthor_model {
>> }
>>
>> static const struct panthor_model gpu_models[] = {
>> + GPU_MODEL(g310, 0, 0), /* NXP i.MX95 */
We can remove the comment about the SoC. This GPU model could be found
in other vendor platforms.
>
> Are you sure about 0, 0? With this I get
>> [drm] mali-unknown id 0xac74 major 0x0 minor 0x0 status 0x1
>
> Using GPU_MODEL(g310, 10, 4) I get:
>> [drm] mali-g310 id 0xac74 major 0x0 minor 0x0 status 0x1
I'd agree it should be GPU_MODEL(g310, 10, 4). Unless the GPU
version has been changed by the vendor?
> Note the output is GPU_VER_MAJOR() but this table is for GPU_ARCH_MAJOR()
> (and the corresponding minor).
>
> Best regards,
> Alexander
>
>> GPU_MODEL(g610, 10, 7),
>> {},
>> };
>>
>
>
Kind regards,
Florent
More information about the linux-arm-kernel
mailing list