[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