Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: - [Critical] Removing the 'brcm,vc4-v3d' compatible string breaks Device Tree ABI compatibility with older Raspberry Pi firmware. --
--- Patch [1]: [PATCH 1/4] drm/vc4: Drop the undocumented brcm,vc4-v3d compatible --- commit 0d9402a3018a535c49f9c7ef653bfa693de83e10 Author: Maíra Canal <[email protected]> drm/vc4: Drop the undocumented brcm,vc4-v3d compatible This commit removes the "brcm,vc4-v3d" compatible string from the VC4 driver's match tables. It notes that the string was never part of the upstream device tree bindings and was only kept for older Raspberry Pi firmware. > diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c > --- a/drivers/gpu/drm/vc4/vc4_drv.c > +++ b/drivers/gpu/drm/vc4/vc4_drv.c > @@ -279,7 +279,6 @@ static const struct of_device_id vc4_dma_range_matches[] > = { > { .compatible = "brcm,bcm2835-hvs" }, > { .compatible = "brcm,bcm2835-v3d" }, > { .compatible = "brcm,cygnus-v3d" }, > - { .compatible = "brcm,vc4-v3d" }, [Severity: Critical] Does removing this compatible string break Device Tree ABI compatibility? The commit message mentions that firmware device trees predating v4.6 (released in 2016) use this string. Since the Linux kernel generally treats Device Tree as a stable ABI, will dropping this cause users who upgrade their kernel but not their boot firmware to lose DMA range matching and 3D hardware acceleration? > {} > }; > > diff --git a/drivers/gpu/drm/vc4/vc4_v3d.c b/drivers/gpu/drm/vc4/vc4_v3d.c > --- a/drivers/gpu/drm/vc4/vc4_v3d.c > +++ b/drivers/gpu/drm/vc4/vc4_v3d.c > @@ -531,7 +531,6 @@ static void vc4_v3d_dev_remove(struct platform_device > *pdev) > const struct of_device_id vc4_v3d_dt_match[] = { > { .compatible = "brcm,bcm2835-v3d" }, > { .compatible = "brcm,cygnus-v3d" }, > - { .compatible = "brcm,vc4-v3d" }, [Severity: Critical] Similarly here in vc4_v3d_dt_match, does removing this prevent the VC4 V3D driver from successfully probing on those older firmware device trees? > {} > }; -- Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=1
