|
From: Richard C. <ric...@gm...> - 2011-12-25 15:31:05
|
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);
> +}
|