Hi Mat,

> Signed-off-by: Mat Martineau <[email protected]>
> ---
>  include/net/bluetooth/l2cap.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

this should have a commit message with technical details to explain why
it is a good idea.

> diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h
> index 16e412f..93aba17 100644
> --- a/include/net/bluetooth/l2cap.h
> +++ b/include/net/bluetooth/l2cap.h
> @@ -35,7 +35,7 @@
>  #define L2CAP_DEFAULT_MAX_TX         3
>  #define L2CAP_DEFAULT_RETRANS_TO     2000    /* 2 seconds */
>  #define L2CAP_DEFAULT_MONITOR_TO     12000   /* 12 seconds */
> -#define L2CAP_DEFAULT_MAX_PDU_SIZE   672
> +#define L2CAP_DEFAULT_MAX_PDU_SIZE   1011    /* Sized for 3-DH5 packet */
>  #define L2CAP_DEFAULT_ACK_TO         200
>  #define L2CAP_LOCAL_BUSY_TRIES               12

Also the default PDU size for basic mode should stay with 672. And only
for ERTM move it to 1010. Is this ensured?

And then again here. Is that explained in the commit message?

Regards

Marcel

 


--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to