Re: [v3] net: ti: icssg-prueth: Fix buffer allocation for ICSSG
From: Markus Elfring
Date: Fri Jul 18 2025 - 14:31:22 EST
>> 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