On Tue, Dec 20, 2016 at 10:25 AM, Ferruh Yigit <ferruh.yi...@intel.com> wrote:
> On 12/19/2016 6:00 PM, Alejandro Lucero wrote: > > I forgot one thing: to update the features file with this new one. > > > > I will wait for your feedback regarding the discussed problem for > > sending another version. > > I think it is good to go, please send updated version. > > OK > > > > <...> > > > > > Sorry, confused. Is it like following: > > > > " > > For new FW, there is no problem, it supports the range between > > 1G - 50G, > > and reports correct current speed. > > > > With old FW, device still can be set to speed range between 1G - > > 50G, > > but it doesn't report current speed correct, and DPDK driver > reports > > back to application as device current speed is 40G, without > really > > knowing the current speed. > > " > > > > > > Yes, that is. Should then I do anything else or the patch is right > > for you now? > > So, this patch is correct. > > But for the previous patch "net/nfp: report link speed using hardware > info", it would be nice to add above information into source as comment > and into commit log. > If you can, would you mind sending a new version of that patch? > > I'll do it. Thanks! > Thanks, > ferruh > > > > <...> >
