Hi David,
> ---
> src/device.c | 2 +-
> src/hcid.h | 1 +
> src/main.c | 16 ++++++++++++++++
> src/main.conf | 5 +++++
> 4 files changed, 23 insertions(+), 1 deletion(-)
>
> diff --git a/src/device.c b/src/device.c
> index 4f1af7012..75340cabe 100644
> --- a/src/device.c
> +++ b/src/device.c
> @@ -4986,7 +4986,7 @@ bool device_attach_att(struct btd_device *dev, GIOChannel *io)
> }
> }
>
> - dev->att_mtu = MIN(mtu, BT_ATT_MAX_LE_MTU);
> + dev->att_mtu = MIN(mtu, main_opts.gatt_max_le_mtu);
> attrib = g_attrib_new(io,
> cid == ATT_CID ? BT_ATT_DEFAULT_LE_MTU : dev->att_mtu,
> false);
> diff --git a/src/hcid.h b/src/hcid.h
> index 2c2b89d9c..bcb077dde 100644
> --- a/src/hcid.h
> +++ b/src/hcid.h
> @@ -54,6 +54,7 @@ struct main_opts {
>
> bt_mode_t mode;
> bt_gatt_cache_t gatt_cache;
> + uint16_t gatt_max_le_mtu;
>
> uint8_t min_enc_key_size;
> };
> diff --git a/src/main.c b/src/main.c
> index 7e6af42cd..5e19d68b3 100644
> --- a/src/main.c
> +++ b/src/main.c
> @@ -50,6 +50,7 @@
> #include "log.h"
> #include "backtrace.h"
>
> +#include "shared/att-types.h"
> #include "lib/uuid.h"
> #include "hcid.h"
> #include "sdpd.h"
> @@ -104,6 +105,7 @@ static const char *policy_options[] = {
> static const char *gatt_options[] = {
> "Cache",
> "MinEncKeySize",
> + "MaxLeMtu",
> NULL
> };
I prefer DefaultMTU or something along the lines. Since MaxMaxTransferUnit sounds weird.
Also while at it, I am not going to git blame who added MinEncKeySize needs to rethink it. We don't abbreviate in main.conf. So that should have been MinEncryptionKeySize and it needs to be fixed.
Also ..
# Minimum required Encryption Key Size for accessing secured characteristics.
# Possible values: 0 and 7-16. 0 means don't care.
# Defaults to 0
# MinEncKeySize = 0
It is #MinEncKeySize.. here. The extra space behind # is not used.
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
