On 2012/11/23 11:57, Gleydson Soares wrote:
> set ifp->if_baudrate with IF_Gbps() / IF_Mbps().

The change makes sense, but..

> --- if_ste.c  18 Oct 2012 21:44:21 -0000      1.48
> --- if_tl.c   22 Jun 2011 16:44:27 -0000      1.51
> --- if_txp.c  5 Apr 2011 18:01:21 -0000       1.104
> --- if_vr.c   20 Oct 2012 16:12:22 -0000      1.117
> --- if_wb.c   18 Oct 2012 21:44:21 -0000      1.49
> +++ if_ste.c  23 Nov 2012 14:07:41 -0000

...these ones currently have if_baudrate = 10Mbps but you change them to 100.

> -     ifp->if_baudrate = 10000000;
> +     ifp->if_baudrate = IF_Mbps(100);

It looks like 100 is correct but I think I would prefer to fix the values,
and change to IF_Xbps, in separate commits.

> --- if_vge.c  22 Jun 2011 16:44:27 -0000      1.51

(this one is correct, 1000000000 -> 1Gbps).

Reply via email to