smsc9220_eth_drv.c; fix warnings from "gcc -Wconversion"

laroche · laroche · commit 5669726263c5 · 2025-04-27T21:18:41.000+02:00
smsc9220_eth_drv.c; fix warnings from "gcc -Wconversion"

Signed-off-by: Florian La Roche &lt;Florian.LaRoche@gmail.com&gt;
diff --git a/source/portable/NetworkInterface/MPS2_AN385/ether_lan9118/smsc9220_eth_drv.c b/source/portable/NetworkInterface/MPS2_AN385/ether_lan9118/smsc9220_eth_drv.c
@@ -958,7 +958,7 @@ enum smsc9220_error_t smsc9220_read_mac_address( const struct smsc9220_eth_dev_t
     mac[ 0 ] = mac_low & 0xFF;
     mac[ 1 ] = ( mac_low >> 8 ) & 0xFF;
     mac[ 2 ] = ( mac_low >> 16 ) & 0xFF;
-    mac[ 3 ] = ( mac_low >> 24 ) & 0xFF;
+    mac[ 3 ] = ( char ) ( ( mac_low >> 24 ) & 0xFF );
     mac[ 4 ] = mac_high & 0xFF;
     mac[ 5 ] = ( mac_high >> 8 ) & 0xFF;
 
@@ -1223,7 +1223,7 @@ uint32_t smsc9220_peek_next_packet_size( const struct
     }
     else
     {
-        rx_status_from_peek = -1;
+        rx_status_from_peek = ( uint32_t ) -1;
     }
 
     return packet_size;

Original file line number	Diff line number	Diff line change
`@@ -958,7 +958,7 @@ enum smsc9220_error_t smsc9220_read_mac_address( const struct smsc9220_eth_dev_t`
`958`	`958`	`mac[ 0 ] = mac_low & 0xFF;`
`959`	`959`	`mac[ 1 ] = ( mac_low >> 8 ) & 0xFF;`
`960`	`960`	`mac[ 2 ] = ( mac_low >> 16 ) & 0xFF;`
`961`		`- mac[ 3 ] = ( mac_low >> 24 ) & 0xFF;`
	`961`	`+ mac[ 3 ] = ( char ) ( ( mac_low >> 24 ) & 0xFF );`
`962`	`962`	`mac[ 4 ] = mac_high & 0xFF;`
`963`	`963`	`mac[ 5 ] = ( mac_high >> 8 ) & 0xFF;`
`964`	`964`
`@@ -1223,7 +1223,7 @@ uint32_t smsc9220_peek_next_packet_size( const struct`
`1223`	`1223`	`}`
`1224`	`1224`	`else`
`1225`	`1225`	`{`
`1226`		`- rx_status_from_peek = -1;`
	`1226`	`+ rx_status_from_peek = ( uint32_t ) -1;`
`1227`	`1227`	`}`
`1228`	`1228`
`1229`	`1229`	`return packet_size;`