On 1/9/25 13:45, Erkiaga Elorza, Ibai wrote:
[AMD Official Use Only - AMD Internal Distribution Only]
Agree Alex,
I just used /t to stick as much as possible to the current implementation, but I'm open to change to white spaces if Michal thinks it's better.
Regards Ibai
*From:* Alexander Dahl ada@thorsis.com *Sent:* 09 January 2025 07:52 *To:* Erkiaga Elorza, Ibai ibai.erkiaga-elorza@amd.com *Cc:* u-boot@lists.denx.de u-boot@lists.denx.de; Simek, Michal michal.simek@amd.com *Subject:* Re: [U-Boot][PATCH 1/2] fpga: fix alignment on fpga cmd usage info Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
Hello,
Am Wed, Jan 08, 2025 at 02:03:50PM +0000 schrieb Ibai Erkiaga:
The current implementation generates some alignment issues as well as some overlapping when all the fpga command options are enabled. The fix is intended to improve readability of the usage info.
Alignment could indeed improved. One question: the \t is dependent on the terminal, right? Would it be better to use spaces instead? That would also make the code more readable.
Agree. Even other commands are using tabs using spaces would be better.
Thanks, Michal