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. Thanks, Paul
OpenPGP_0x27F4B3459F002257.asc
Description: OpenPGP public key
OpenPGP_signature
Description: OpenPGP digital signature