lkml.org 
[lkml]   [2016]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [Intel-wired-lan] [PATCH v2 net-next 0/2] e1000e: fix PTP on e1000_pch_variants
    On Sat, 23 Jul 2016 12:44:32 -0400
    Jarod Wilson <jarod@redhat.com> wrote:

    > This little series factors out the systim sanitization code first, then
    > adds e1000_pch_lpt as a new case in the switch that calls the sanitize
    > function, fixing PTP clock issues I've had reported against an Intel
    > I-218V NIC in an Intel NUC5ik5RYH system.
    >
    > Jarod Wilson (2):
    > e1000e: factor out systim sanitization
    > e1000e: fix PTP on e1000_pch_lpt variants
    >

    Thanks for your patch Jarod, the refactor itself is fine and a good
    idea, and thanks for working on the fix!

    This code should have been using a feature flag, and the alert that
    you're having to add more device IDs to the switch statement makes it
    even more obvious.

    Please see line 406 of e1000e/e1000.h, where the flags are declared,
    add a flag for this workaround (to flags2), and and add some code in the
    e1000_info_tbl entry to set the flag for the appropriate mac(s)

    Then the runtime code should only check the flag, and if any further
    devices require the workaround we just add the flag to that device, or
    if this is init code, just always call the workaround funtion and have
    the function itself make sure the right flags2 is set or return.

    The code has somehow gotten away from this model in some places and any
    new code we add should be doing it the right way.

    Thanks,
    Jesse

    \
     
     \ /
      Last update: 2016-07-25 20:41    [W:2.975 / U:0.540 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site