Skip to content

Commit

Permalink
Replace size_align_u32() with next_multiple_of()
Browse files Browse the repository at this point in the history
Currently next_multiple_of() is behinged a Feature gate: int_rounding.
See rust-lang/rust#88581
But it seems that this function is stablized in rust 1.73.
See rust-lang/rust#94455

Currently Embassy is still using nightly for many other unstable
features. So I do see an issue to use this function.
  • Loading branch information
vDorst committed Aug 23, 2023
1 parent 4b6045d commit 5f5e3bc
Showing 1 changed file with 5 additions and 25 deletions.
30 changes: 5 additions & 25 deletions embassy-net-adin1110/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,11 +95,6 @@ pub struct ADIN1110<SPI> {
crc: bool,
}

/// Round size up the N u32;
pub(crate) fn size_align_u32(size: u32) -> u32 {
(size + 3) & 0xFFFF_FFFC
}

impl<SPI: SpiDevice> ADIN1110<SPI> {
pub fn new(spi: SPI, crc: bool) -> Self {
Self { spi, crc }
Expand Down Expand Up @@ -192,14 +187,12 @@ impl<SPI: SpiDevice> ADIN1110<SPI> {
let mut tx_buf = Vec::<u8, 16>::new();

// Size of the frame, also includes the appednded header.
let packet_size = self.read_reg(sr::RX_FSIZE).await?;
let packet_size = self.read_reg(sr::RX_FSIZE).await? as usize;

// Packet read of write to the MAC packet buffer must be a multipul of 4!
let read_size = size_align_u32(packet_size);
let read_size = packet_size.next_multiple_of(4);

if packet_size < u32::try_from(FRAME_HEADER_LEN + FSC_LEN).unwrap()
|| read_size > u32::try_from(packet.len()).unwrap()
{
if packet_size < (FRAME_HEADER_LEN + FSC_LEN) || read_size > packet.len() {
return Err(AdinError::PACKET_TOO_BIG);
}

Expand Down Expand Up @@ -836,18 +829,6 @@ mod tests {
spi.done();
}

#[test]
fn align_size() {
assert_eq!(size_align_u32(1), 4);
assert_eq!(size_align_u32(2), 4);
assert_eq!(size_align_u32(3), 4);
assert_eq!(size_align_u32(4), 4);
assert_eq!(size_align_u32(5), 8);
assert_eq!(size_align_u32(6), 8);
assert_eq!(size_align_u32(7), 8);
assert_eq!(size_align_u32(8), 8);
}

// #[test]
// fn write_packet_to_fifo_less_64b_with_crc() {
// // Configure expectations
Expand Down Expand Up @@ -1224,10 +1205,9 @@ mod tests {
// Packet FCS
spi_packet.extend_from_slice(&[147, 149, 213, 68]).unwrap();

let spi_packet_len = u32::try_from(spi_packet.len()).unwrap();

// SPI HEADER Padding of u32
for _ in spi_packet_len..size_align_u32(spi_packet_len) {
let spi_packet_len = spi_packet.len();
for _ in spi_packet_len..spi_packet_len.next_multiple_of(4) {
spi_packet.push(0x00).unwrap();
}

Expand Down

0 comments on commit 5f5e3bc

Please sign in to comment.