[PATCH V2 07/10] staging: vchiq_arm: Reduce indentation of service_callback
Dan Carpenter
dan.carpenter at linaro.org
Mon Jun 10 23:04:28 PDT 2024
On Mon, Jun 10, 2024 at 11:02:17PM +0200, Stefan Wahren wrote:
> The service_callback has 5 levels of indentation, which makes it
> hard to read. Reduce this by splitting the code in a new function
> service_single_message() as suggested by Laurent Pinchart.
>
> Signed-off-by: Stefan Wahren <wahrenst at gmx.net>
To be honest, I liked the version of this patch which Laurent didn't
like. Even after pulling this code into a separate function, I'd still
support flipping the conditions around and adding the goto...
Laurent, you really didn't like the goto?
regards,
dan carpenter
More information about the linux-arm-kernel
mailing list