Imagelibrary/rtems
1
0
Fork 1
mirror of https://gitlab.rtems.org/rtems/rtos/rtems.git synced 2026-02-05 13:11:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

cpukit/dev: Address -Wsign-compare warnings

Browse Source 
This warning occurs when comparing a signed variable to an unsigned one.
This addresses warnings that only occurred on 64-bit targets.  For the
ones which only appeared on 64-bit targets, the cause was frequently
a mismatch when comparing a combination off_t, ssize_t, and int.
This commit is contained in:
Joel Sherrill
2026-01-30 09:16:56 -06:00
parent efe6dd524c
commit 0b7eb103fe
3 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
cpukit/dev/flash/flashdev.c
View File

@@ -328,7 +328,7 @@ static int rtems_flashdev_check_offset_region(
)
{
if ( ( rtems_flashdev_is_region_defined( iop ) ) &&
( offset > rtems_flashdev_get_region_size( flash, iop ) ) ) {
( offset > (int64_t) rtems_flashdev_get_region_size( flash, iop ) ) ) {
rtems_set_errno_and_return_minus_one( EINVAL );
}
return 0;
@@ -506,7 +506,7 @@ static off_t rtems_flashdev_lseek(
}
if ( ( rtems_flashdev_is_region_defined(iop) ) &&
( tmp_offset > rtems_flashdev_get_region_size( flash, iop ) ) ) {
( tmp_offset > (int64_t) rtems_flashdev_get_region_size( flash, iop ) ) ) {
rtems_set_errno_and_return_minus_one( EINVAL );
}

4
cpukit/dev/i2c/eeprom.c
View File

@@ -92,7 +92,7 @@ static ssize_t eeprom_read(
return 0;
}
if (n > avail) {
if (n > (size_t) avail) {
n = (size_t) avail;
}
@@ -154,7 +154,7 @@ static ssize_t eeprom_write(
return 0;
}
if (n > avail) {
if (n > (size_t) avail) {
n = (size_t) avail;
}

4
cpukit/dev/i2c/fpga-i2c-slave.c
View File

@@ -55,7 +55,7 @@ fpga_i2c_slave_read(i2c_dev* base, void* buf, size_t n, off_t offset)
return 0;
}
if (n > avail) {
if ((int64_t)n > avail) {
n = (size_t) avail;
}
@@ -97,7 +97,7 @@ fpga_i2c_slave_write(i2c_dev* base, const void* buf, size_t n, off_t offset)
return 0;
}
if (n > avail) {
if ((int64_t)n > avail) {
n = (size_t) avail;
}