mbox series

[0/3] xen/arm: Some clean-up found with -Wconversion and -Warith-conversion

Message ID 20230817214356.47174-1-julien@xen.org (mailing list archive)
Headers show
Series xen/arm: Some clean-up found with -Wconversion and -Warith-conversion | expand

Message

Julien Grall Aug. 17, 2023, 9:43 p.m. UTC
From: Julien Grall <jgrall@amazon.com>

Hi all,

This is a small series to fix some of the issues found while playing
with -Wconversion and -Warith-conversion.

There are a lot more but the bulk are in
 - bitmap
 - cpumask
 - nodemask
 - bitops/atomics
 - find_*

Some are not too difficult to address but other there are even
prototype conflicts between arm x86.

Cheers,

Julien Grall (3):
  xen/arm: vmmio: The number of entries cannot be negative
  xen/arm: vgic: Use 'unsigned int' rather than 'int' whenever it is
    possible
  xen/public: arch-arm: All PSR_* defines should be unsigned

 xen/arch/arm/domain.c           |  3 +-
 xen/arch/arm/include/asm/mmio.h |  6 ++--
 xen/arch/arm/include/asm/vgic.h | 17 ++++++-----
 xen/arch/arm/io.c               |  2 +-
 xen/arch/arm/vgic-v2.c          | 14 +++++++--
 xen/arch/arm/vgic-v3.c          |  2 +-
 xen/arch/arm/vgic.c             | 23 ++++++++-------
 xen/arch/arm/vgic/vgic-init.c   |  2 +-
 xen/include/public/arch-arm.h   | 52 ++++++++++++++++-----------------
 9 files changed, 68 insertions(+), 53 deletions(-)

Comments

Julien Grall Aug. 21, 2023, 5:05 p.m. UTC | #1
Hi,

On 17/08/2023 22:43, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
> 
> Hi all,
> 
> This is a small series to fix some of the issues found while playing
> with -Wconversion and -Warith-conversion.
> 
> There are a lot more but the bulk are in
>   - bitmap
>   - cpumask
>   - nodemask
>   - bitops/atomics
>   - find_*
> 
> Some are not too difficult to address but other there are even
> prototype conflicts between arm x86.
> 
> Cheers,
> 
> Julien Grall (3):
>    xen/arm: vmmio: The number of entries cannot be negative
>    xen/arm: vgic: Use 'unsigned int' rather than 'int' whenever it is
>      possible
>    xen/public: arch-arm: All PSR_* defines should be unsigned

I have committed the series.

Cheers,