From: Julien Grall <julien@xen.org>
To: Simone Ballarin <simone.ballarin@bugseng.com>,
xen-devel@lists.xenproject.org
Cc: consulting@bugseng.com,
Gianluca Luparini <gianluca.luparini@bugseng.com>,
Stefano Stabellini <sstabellini@kernel.org>,
Michal Orzel <michal.orzel@amd.com>,
Xenia Ragiadakou <Xenia.Ragiadakou@amd.com>,
Ayan Kumar Halder <ayan.kumar.halder@amd.com>
Subject: Re: [XEN PATCH v2 05/13] xen/device-tree: fix violations of MISRA C:2012 Rule 7.2
Date: Thu, 6 Jul 2023 09:01:50 +0100 [thread overview]
Message-ID: <29756ff8-adcd-5184-c8b1-c619a5a8d3f2@xen.org> (raw)
In-Reply-To: <da656a93e42e5d04e02f62002b1f5ab357c8c16a.1688559115.git.gianluca.luparini@bugseng.com>
Hi,
On 05/07/2023 16:26, Simone Ballarin wrote:
> From: Gianluca Luparini <gianluca.luparini@bugseng.com>
This suggest the author is Gianluca but...
> 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 and also to other
> literals used in the same contexts or near violations, when their positive
> nature is immediately clear. The latter changes are done for the sake of
> uniformity.
>
> Signed-off-by: Simone Ballarin <simone.ballarin@bugseng.com>
... the first signed-off-by is yours. So who is the author?
More generally, there tags are order chronogically. So if Gianluca is
the author then it should be the first Signed-off-by. Then it is yours.
This will be followed-by the Acked-by/Reviewed-by tags.
The same remark goes for the rest of the series.
Cheers,
--
Julien Grall
next prev parent reply other threads:[~2023-07-06 8:02 UTC|newest]
Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-05 15:26 [XEN PATCH v2 00/13] xen: fix violations of MISRA C:2012 Rule 7.2 Simone Ballarin
2023-07-05 15:26 ` [XEN PATCH v2 01/13] x86/cpufreq: " Simone Ballarin
2023-07-05 23:29 ` Stefano Stabellini
2023-07-06 7:44 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 02/13] AMD/IOMMU: " Simone Ballarin
2023-07-05 23:31 ` Stefano Stabellini
2023-07-06 7:48 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 03/13] x86/svm: " Simone Ballarin
2023-07-06 7:52 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 04/13] xen/arm: " Simone Ballarin
2023-07-05 16:27 ` Luca Fancellu
2023-07-05 16:42 ` Julien Grall
2023-07-05 15:26 ` [XEN PATCH v2 05/13] xen/device-tree: " Simone Ballarin
2023-07-05 16:28 ` Luca Fancellu
2023-07-05 23:32 ` Stefano Stabellini
2023-07-06 8:01 ` Julien Grall [this message]
2023-07-05 15:26 ` [XEN PATCH v2 06/13] xen/efi: " Simone Ballarin
2023-07-05 16:34 ` Luca Fancellu
2023-07-05 23:37 ` Stefano Stabellini
2023-07-06 7:55 ` Jan Beulich
2023-07-06 22:14 ` Stefano Stabellini
2023-07-05 15:26 ` [XEN PATCH v2 07/13] x86/vmx: " Simone Ballarin
2023-07-05 23:39 ` Stefano Stabellini
2023-07-06 8:04 ` Jan Beulich
2023-07-06 22:17 ` Stefano Stabellini
2023-07-07 6:44 ` Jan Beulich
2023-07-07 8:18 ` Simone Ballarin
2023-07-05 15:26 ` [XEN PATCH v2 08/13] xen/pci: " Simone Ballarin
2023-07-06 8:05 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 09/13] xen/public: " Simone Ballarin
2023-07-05 15:33 ` Juergen Gross
2023-07-06 7:43 ` Jan Beulich
2023-07-06 7:57 ` Juergen Gross
2023-07-06 8:43 ` Jan Beulich
2023-07-06 9:10 ` Juergen Gross
2023-07-05 23:42 ` Stefano Stabellini
2023-07-05 15:26 ` [XEN PATCH v2 10/13] x86/monitor: " Simone Ballarin
2023-07-05 15:26 ` [XEN PATCH v2 11/13] xen/vpci: " Simone Ballarin
2023-07-06 8:07 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 12/13] xen/x86: " Simone Ballarin
2023-07-06 0:11 ` Stefano Stabellini
2023-07-06 8:26 ` Jan Beulich
2023-07-06 16:08 ` Simone Ballarin
2023-07-06 16:22 ` Jan Beulich
2023-07-07 6:50 ` Simone Ballarin
2023-07-07 7:04 ` Jan Beulich
2023-07-07 8:04 ` Simone Ballarin
2023-07-07 9:46 ` Jan Beulich
2023-07-07 21:53 ` Stefano Stabellini
2023-07-10 16:09 ` Simone Ballarin
2023-07-06 16:22 ` Jan Beulich
2023-07-05 15:26 ` [XEN PATCH v2 13/13] xen: " Simone Ballarin
2023-07-05 16:43 ` Luca Fancellu
2023-07-05 18:42 ` Simone Ballarin
2023-07-05 19:32 ` Simone Ballarin
2023-07-06 8:28 ` Jan Beulich
2023-07-05 23:49 ` Stefano Stabellini
2023-07-06 8:34 ` Jan Beulich
2023-07-06 8:40 ` Jan Beulich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=29756ff8-adcd-5184-c8b1-c619a5a8d3f2@xen.org \
--to=julien@xen.org \
--cc=Xenia.Ragiadakou@amd.com \
--cc=ayan.kumar.halder@amd.com \
--cc=consulting@bugseng.com \
--cc=gianluca.luparini@bugseng.com \
--cc=michal.orzel@amd.com \
--cc=simone.ballarin@bugseng.com \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.