From: Hayes Wang
...

I should have spotted this before.

>  /* USB_RX_EARLY_AGG */
> -#define EARLY_AGG_SUPPER     0x0e832981
> +#define EARLY_AGG_SUPER      ((((rx_buf_sz - 1522) / 4) << 16) | \
> +     (u32)(CONFIG_RTL8152_EARLY_AGG_TIMEOUT_SUPER <= 0 ? 85 * 125 : \
> +     min(CONFIG_RTL8152_EARLY_AGG_TIMEOUT_SUPER * 125, 0xffff)))
>  #define EARLY_AGG_HIGH               0x0e837a12
>  #define EARLY_AGG_SLOW               0x0e83ffff
> 
> @@ -1978,7 +1980,7 @@ static void r8153_set_rx_agg(struct r8152 *tp)
>                       ocp_write_dword(tp, MCU_TYPE_USB, USB_RX_BUF_TH,
>                                       RX_THR_SUPPER);
>                       ocp_write_dword(tp, MCU_TYPE_USB, USB_RX_EARLY_AGG,
> -                                     EARLY_AGG_SUPPER);
> +                                     EARLY_AGG_SUPER);
>               } else {
>                       ocp_write_dword(tp, MCU_TYPE_USB, USB_RX_BUF_TH,
>                                       RX_THR_HIGH);

It looks as though rx_buf_sz should be a parameter to EARLY_AGG_SUPER.

        David



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to