Re: [PATCH 0/6] dmioem: HPE OEM Updates

[Jean Delvare]

On Fri, 07 Apr 2023 15:15:15 +0200, Petter Reinholdtsen wrote:
> [Jean Delvare]
> > I can't see any indentation issue with these patches. Can you
> > elaborate?  
> 
> The existing code seem to use \t\t<space>, while the new code used
> \t\t.  No idea what is wanted nor why the existing code showed up with
> the extra space, but thought it best to ask.

There's a leading space on lines which contain a multi-line C comment
(except the first). That's pretty much an industry standard, to align
the stars and make the comment easier to read.

There are also leading spaces added for alignment purposes on
multi-line conditionals. Other than this, the code is only aligned
using tabs, and Jerry's patches are OK in that respect.

-- 
Jean Delvare
SUSE L3 Support