Message ID | 1497785298-13468-10-git-send-email-netanel@amazon.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Sun, Jun 18, 2017 at 02:28:14PM +0300, netanel@amazon.com wrote: > From: Netanel Belgazal <netanel@amazon.com> > > Signed-off-by: Netanel Belgazal <netanel@amazon.com> > --- > drivers/net/ethernet/amazon/ena/ena_com.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/amazon/ena/ena_com.c b/drivers/net/ethernet/amazon/ena/ena_com.c > index f6e1d30523a6..8efb85e25a42 100644 > --- a/drivers/net/ethernet/amazon/ena/ena_com.c > +++ b/drivers/net/ethernet/amazon/ena/ena_com.c > @@ -100,7 +100,7 @@ static inline int ena_com_mem_addr_set(struct ena_com_dev *ena_dev, > } > > ena_addr->mem_addr_low = (u32)addr; > - ena_addr->mem_addr_high = (u64)addr >> 32; > + ena_addr->mem_addr_high = (u16)((u64)addr >> 32); We have macro for getting upper 32 bits - upper_32_bits(addr), the same goes for lower 32 bits. > > return 0; > } > -- > 2.7.4 >
Ack, Will use the proposed functions.
diff --git a/drivers/net/ethernet/amazon/ena/ena_com.c b/drivers/net/ethernet/amazon/ena/ena_com.c index f6e1d30523a6..8efb85e25a42 100644 --- a/drivers/net/ethernet/amazon/ena/ena_com.c +++ b/drivers/net/ethernet/amazon/ena/ena_com.c @@ -100,7 +100,7 @@ static inline int ena_com_mem_addr_set(struct ena_com_dev *ena_dev, } ena_addr->mem_addr_low = (u32)addr; - ena_addr->mem_addr_high = (u64)addr >> 32; + ena_addr->mem_addr_high = (u16)((u64)addr >> 32); return 0; }