Re: [PATCH] net:ethernet:intel:Remove outdated fix me comment in the function, gb_acquire_swfw_sync_i210

[nick <xerofoify@gmail.com>]

On 2015-02-08 08:26 PM, Jeff Kirsher wrote:
> On Sun, 2015-02-08 at 17:21 -0800, Jeff Kirsher wrote:
>> On Sun, 2015-02-08 at 00:21 -0500, Nicholas Krause wrote:
>>> Removes the outdated fix me comment in the
>>> function,gb_acquire_swfw_sync_i210
>>> for setting the variables, i and timeout to the intended correct
>>> values for
>>> the function,gb_acquire_swfw_sync_i210 to function correctly. This
>>> comment is
>>> no longer due to these values having been no changed in the last few
>>> years and
>>> no known issues have been found for these variables being set to their
>>> current
>>> values. Due to this the comment can now be removed as the values set
>>> for these
>>> variables are known to be correct after years of no known issues or
>>> bugs related
>>> to their current values.
>>>
>>> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
>>> ---
>>>  drivers/net/ethernet/intel/igb/e1000_i210.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Thank Nick, I will add your patch to my queue.
> 
> Oh and the title should be "igb: Remove outdated ...", not
> "net:ethernet:intel: Remove ..."
> 
Jeff,
Sorry about the title. Good to known for next time. Should I resend or 
will you fix the title for me?
Cheers,
Nick

------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit http://communities.intel.com/community/wired