[media.git/next] media: verisilicon: Use __set_bit() with local bitmaps

[media-ci]

On Sun May 25 10:11:44 2025 +0200, Christophe JAILLET wrote:
> The 'used' and 'new' bitmaps are local to this function, so there is no
> need to use atomic access because concurrency can not happen.
> 
> Use the non-atomic __set_bit() to save a few cycles.
> 
> Signed-off-by: Christophe JAILLET <christophe.jail...@wanadoo.fr>
> Signed-off-by: Nicolas Dufresne <nicolas.dufre...@collabora.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/verisilicon/hantro_h264.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

---

diff --git a/drivers/media/platform/verisilicon/hantro_h264.c 
b/drivers/media/platform/verisilicon/hantro_h264.c
index 4e9a0ecf5c13..2414782f1eb6 100644
--- a/drivers/media/platform/verisilicon/hantro_h264.c
+++ b/drivers/media/platform/verisilicon/hantro_h264.c
@@ -325,12 +325,12 @@ static void update_dpb(struct hantro_ctx *ctx)
                                continue;
 
                        *cdpb = *ndpb;
-                       set_bit(j, used);
+                       __set_bit(j, used);
                        break;
                }
 
                if (j == ARRAY_SIZE(ctx->h264_dec.dpb))
-                       set_bit(i, new);
+                       __set_bit(i, new);
        }
 
        /* For entries that could not be matched, use remaining free slots. */
@@ -349,7 +349,7 @@ static void update_dpb(struct hantro_ctx *ctx)
 
                cdpb = &ctx->h264_dec.dpb[j];
                *cdpb = *ndpb;
-               set_bit(j, used);
+               __set_bit(j, used);
        }
 }