Re: [PATCH V2 5/5] usb: serial: f81534: fix tx error on some baud rate

From: Johan Hovold
Date: Wed Jan 10 2018 - 03:50:45 EST


On Wed, Jan 10, 2018 at 01:42:32PM +0800, Ji-Ze Hong (Peter Hong) wrote:
> Hi Johan,
>
> Johan Hovold æ 2018/1/9 äå 07:32 åé:
> > On Thu, Jan 04, 2018 at 10:29:21AM +0800, Ji-Ze Hong (Peter Hong) wrote:
> >> + /*
> >> + * We'll make tx frame error when baud rate from 384~500kps. So we'll
> >> + * delay all tx data frame with 1bit.
> >> + */
> >> + port_priv->shadow_clk |= F81534_CLK_TX_DELAY_1BIT;
> >
> > You don't wan't to enable this only for the affected rates?
> >
>
> This bit will control all transmit TX frame always delay 1 bit
> on enabled, but It'll transmit TX frame randomly delay 1 bit on
> disabled.
>
> We had tested it with BurnInTest to check the performance,
> It'll not make the performance regression. So we'll directly add
> it on all baud rate.

Ok, thanks for confirming.

Johan