
23 Aug
2023
23 Aug
'23
1:23 p.m.
On 8/23/23 08:33, Paul Barker wrote:
On 22/08/2023 22:40, Marek Vasut wrote:
On 8/22/23 20:10, Paul Barker wrote:
Let's make the indentation of pre-processor macros and conditionals in serial_sh.h consistent before we add to the confusion with a new SoC.
What exactly is this patch doing, adding spaces to ifdeffery ?
I would be tempted to say, can we clean up the ifdeffery instead ? It is some hideous mess.
The header is a mess for sure. I was hoping to make it slightly more easy to follow with this patch.
We can drop this for now if you think it's not a clear improvement. I'll see if I can do some more extensive clean up on this header while I'm adding the RZ/G2L support.
If you can give that at least a try, that would be much appreciated.