On Mon, Jul 28, 2025 at 4:43 PM Rob Clark <[email protected]> wrote:
>
> This is needed to properly interpret some of the sections.
>
> Signed-off-by: Rob Clark <[email protected]>
> ---
> drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
> b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
> index faca2a0243ab..e586577e90de 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c
> @@ -1796,6 +1796,7 @@ static void a7xx_show_shader(struct a6xx_gpu_state_obj
> *obj,
>
> print_name(p, " - type: ", a7xx_statetype_names[block->statetype]);
> print_name(p, " - pipe: ", a7xx_pipe_names[block->pipeid]);
> + drm_printf(p, " - location: %d", block->location);
We should probably at least try to keep it proper YAML by indenting
everything after another level...
>
> for (i = 0; i < block->num_sps; i++) {
> drm_printf(p, " - sp: %d\n", i);
> @@ -1873,6 +1874,7 @@ static void a7xx_show_dbgahb_cluster(struct
> a6xx_gpu_state_obj *obj,
> print_name(p, " - pipe: ", a7xx_pipe_names[dbgahb->pipe_id]);
> print_name(p, " - cluster-name: ",
> a7xx_cluster_names[dbgahb->cluster_id]);
> drm_printf(p, " - context: %d\n", dbgahb->context_id);
> + drm_printf(p, " - location: %d\n", dbgahb->location_id);
> a7xx_show_registers_indented(dbgahb->regs, obj->data, p, 4);
> }
> }
> --
> 2.50.1
>
