On 11/12/2025 3:52 PM, Konrad Dybcio wrote: > On 11/10/25 5:37 PM, Akhil P Oommen wrote: >> In A6x family (which is a pretty big one), there are separate >> adreno_func definitions for each sub-generations. To streamline the >> identification of the correct struct for a gpu, move it to the >> catalogue and move the gpu_init routine to struct adreno_gpu_funcs. >> >> Signed-off-by: Akhil P Oommen <[email protected]> >> --- > > [...] > > >> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c >> b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c >> index 44df6410bce1..9007a0e82a59 100644 >> --- a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c >> +++ b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c >> @@ -683,7 +683,7 @@ static const struct adreno_info a6xx_gpus[] = { > > Somewhere among this diff, a619_holi needs to have gmu_wrapper funcs
Could you point me to the holi's devicetree? Currently I see only a610 (sm6115) and a702 (qcm2290) uses gmu_wrapper. In the driver, adreno_is_a619_holi() checks for both 619 rev id and the gmu_wrapper. -Akhil > instead > > Konrad
