[RESEND][PATCH] drm/nouveau: remove unused increment in gm200_flcn_pio_imem_wr

[Danilo Krummrich]

Hi Timur,

Please Cc the relevant maintainers when sending patches.

On 7/26/25 12:17 AM, Timur Tabi wrote:
> The 'tag' parameter is passed by value and is not actually used after
> being incremented, so remove the increment.

Good catch, but the commit message should also mention that the "actual"
increment is in nvkm_falcon_pio_wr(), otherwise it throws up the question if the
increment is missing somewhere else instead.

> Signed-off-by: Timur Tabi <ttabi at nvidia.com>
> ---
>   drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c b/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
> index f0acfaa153d1..7c43397c19e6 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
> @@ -103,7 +103,7 @@ gm200_flcn_pio_imem_wr_init(struct nvkm_falcon *falcon, u8 port, bool sec, u32 i
>   static void
>   gm200_flcn_pio_imem_wr(struct nvkm_falcon *falcon, u8 port, const u8 *img, int len, u16 tag)
>   {
> -	nvkm_falcon_wr32(falcon, 0x188 + (port * 0x10), tag++);
> +	nvkm_falcon_wr32(falcon, 0x188 + (port * 0x10), tag);
>   	while (len >= 4) {
>   		nvkm_falcon_wr32(falcon, 0x184 + (port * 0x10), *(u32 *)img);
>   		img += 4;
> 
> base-commit: 155a3c003e555a7300d156a5252c004c392ec6b0