[v3] net: ti: icssg-prueth: Fix buffer allocation for ICSSG
Markus Elfring
Markus.Elfring at web.de
Fri Jul 18 11:30:35 PDT 2025
>> Can the usage of the conditional operator be extended another bit
>> (also for the function “prueth_emac_buffer_setup” for example)?
>
> 1. (Subjective) The current code seems fine to me
Under which circumstances would you get into the mood to avoid
duplicate source code further?
> 2. (Objective) Your suggestion features unnecessary parentheses, and
> incorrect indentation and whitespace
Which code formatting would you find more reasonable then instead?
Regards,
Markus
More information about the linux-arm-kernel
mailing list