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

Paul Barker paul.barker.ct at bp.renesas.com
Wed Aug 23 08:33:25 CEST 2023


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0x27F4B3459F002257.asc
Type: application/pgp-keys
Size: 3520 bytes
Desc: OpenPGP public key
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230823/c8a79831/attachment.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230823/c8a79831/attachment.sig>


More information about the U-Boot mailing list