Just for the record, reviewing my own patch and pointing out the
mistakes WRT the 82576.

On Tue, Dec 13, 2011 at 04:00:34AM +0100, Richard Cochran wrote:
> +static void igb_82576_systim_write(struct igb_adapter *igb, u64 ns)
> +{
> +     u32 hi, lo;
> +     struct e1000_hw *hw = &igb->hw;
> +
> +     hi = (ns >> 8) & 0xffffffff;
> +     lo = (ns & 0xff) << NS_SHIFT_82576;

Should have been:

        hi = (ns >> 13) & 0xffffffff;
        lo = (ns & 0x1fff) << NS_SHIFT_82576;

> +
> +     wr32(E1000_SYSTIML, lo);
> +     wr32(E1000_SYSTIMH, hi);
> +
> +     igb_overflow_set(igb, ns, hi & SYSTIMH_MSB_82576, OFL_SHIFT_82576);
> +}

------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create 
new or port existing apps to sell to consumers worldwide. Explore the 
Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev
_______________________________________________
E1000-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit 
http://communities.intel.com/community/wired

Reply via email to