[U-Boot] [PATCH] e1000: support 64bit physical address
Sergei Temerkhanov
s.temerkhanov at gmail.com
Tue Aug 18 10:47:44 CEST 2015
On Tue, Aug 18, 2015 at 7:18 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Mingkai,
>
> On Mon, Aug 17, 2015 at 2:49 PM, Mingkai Hu <Mingkai.Hu at freescale.com> wrote:
>
> Please add a commit message on this change.
>
>> Signed-off-by: Mingkai Hu <Mingkai.Hu at freescale.com>
>> ---
>> drivers/net/e1000.c | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c
>> index d5d48b1..e816410 100644
>> --- a/drivers/net/e1000.c
>> +++ b/drivers/net/e1000.c
>> @@ -4980,8 +4980,8 @@ e1000_configure_tx(struct e1000_hw *hw)
>> unsigned long tipg, tarc;
>> uint32_t ipgr1, ipgr2;
>>
>> - E1000_WRITE_REG(hw, TDBAL, (unsigned long)tx_base);
>> - E1000_WRITE_REG(hw, TDBAH, 0);
>> + E1000_WRITE_REG(hw, TDBAL, (unsigned long)tx_base & 0xffffffff);
>> + E1000_WRITE_REG(hw, TDBAH, (unsigned long)tx_base >> 32);
>
> Should it be:
>
> E1000_WRITE_REG(hw, TDBAL, (unsigned long)(tx_base & 0xffffffff));
> E1000_WRITE_REG(hw, TDBAH, (unsigned long)(tx_base >> 32));
>
Shouldn't 'unsigned long' be 'uintptr_t' or 'phys_addr_t'?
> ?
>
>>
>> E1000_WRITE_REG(hw, TDLEN, 128);
>>
>> @@ -5124,8 +5124,8 @@ e1000_configure_rx(struct e1000_hw *hw)
>> E1000_WRITE_FLUSH(hw);
>> }
>> /* Setup the Base and Length of the Rx Descriptor Ring */
>> - E1000_WRITE_REG(hw, RDBAL, (unsigned long)rx_base);
>> - E1000_WRITE_REG(hw, RDBAH, 0);
>> + E1000_WRITE_REG(hw, RDBAL, (unsigned long)rx_base & 0xffffffff);
>> + E1000_WRITE_REG(hw, RDBAH, (unsigned long)rx_base >> 32);
>>
>> E1000_WRITE_REG(hw, RDLEN, 128);
>>
>> --
>
> Regards,
> Bin
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
More information about the U-Boot
mailing list