[PATCH 2/2] serial: sh: Tidy up pre-processor directive indentation

Marek Vasut marek.vasut at mailbox.org
Wed Aug 23 13:23:11 CEST 2023


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.


More information about the U-Boot mailing list