Instead of relying on reles of pointer arithmetic (implicit multiplication by the size of pointer type), change the code to retreive address of an array elemet to clarify the intent. Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> --- drivers/net/e1000/main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/e1000/main.c b/drivers/net/e1000/main.c index b728add7f..90b6d6e43 100644 --- a/drivers/net/e1000/main.c +++ b/drivers/net/e1000/main.c @@ -3200,12 +3200,11 @@ static int e1000_sw_init(struct eth_device *edev) static void e1000_fill_rx(struct e1000_hw *hw) { - volatile struct e1000_rx_desc *rd; + volatile struct e1000_rx_desc *rd = &hw->rx_base[hw->rx_tail]; volatile u32 *bla; int i; hw->rx_last = hw->rx_tail; - rd = hw->rx_base + hw->rx_tail; hw->rx_tail = (hw->rx_tail + 1) % 8; bla = (void *)rd; -- 2.20.1 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox