On 28.09.2023 13:11, Simone Ballarin wrote: > On 13/09/23 11:44, Jan Beulich wrote: >> On 12.09.2023 17:38, Simone Ballarin wrote: >>> The xen sources contains violations of MISRA C:2012 Rule 7.2 whose >>> headline states: >>> "A 'u' or 'U' suffix shall be applied to all integer constants >>> that are represented in an unsigned type". >>> >>> Add the 'U' suffix to integers literals with unsigned type. >>> >>> For the sake of uniformity, the following change is made: >>> - add 'U' suffixes to 'mask16' in 'stdvga.c' >>> >>> Signed-off-by: Simone Ballarin <simone.balla...@bugseng.com> >>> Signed-off-by: Gianluca Luparini <gianluca.lupar...@bugseng.com> >> >> Comparing with v5 - are the two tags in the correct order? And is there >> a From: Gianluca missing now? Apart from that I'm now okay with the >> changes presented here. > > The commit is new and it has been obtained splitting a previous commit > made by Luparini. Currently Luparini is not working on that, for this > reason I'm the author of the new commit. > > I thought that the first signed-off tag should be the commit author, > this is the reason why I've changed the order.
If this patch is a the result of splitting an earlier larger one, then the question still is who made the (initial part of the) changes that are now in here. If it was all you, then I don't see why there's a 2nd SoB. If part came from Gianluca's earlier work, then imo the first SoB still ought to be his. Jan
