Re: [PATCH] media: staging/ipu7: use guard(mutex) and simplify retry loop in buttress

From: Dan Carpenter

Date: Thu Nov 05 2026 - 02:14:53 EST


On Thu, Nov 05, 2026 at 12:28:32AM +0200, Laurian Iacob wrote:
> Use scope-based guard(mutex) from linux/cleanup.h in
> ipu_buttress_ipc_send_msg() and ipu_buttress_authenticate() to simplify
> error handling and remove redundant goto unlock paths.

Unfortunately, you didn't remove them all.

>
> Additionally, replace the backward goto retry label in
> ipu_buttress_ipc_send_msg() with a standard for (;;) loop to improve
> control flow readability.

I don't like that you got rid of the try: label. How is that related
to locking.

regards,
dan carpenter