Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20200615145221.224747-1-alainm@chromium.org>
In-Reply-To: <20200615145221.224747-1-alainm@chromium.org>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Wed, 17 Jun 2020 21:40:56 -0700
Message-ID: <CABBYNZLTorEmM3t5BCWqYLGF6Q4e7kOXo2TTj20YShrFEV3Z0g@mail.gmail.com>
Subject: Re: [BlueZ PATCH v1] main: adding missing connection parameter entries
To:     Alain Michaud <alainm@chromium.org>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Alain,

On Mon, Jun 15, 2020 at 7:55 AM Alain Michaud <alainm@chromium.org> wrote:
>
> This patch adds the missing connection parameter entries for loading
> connection parameters from main.conf
>
> ---
>
>  src/main.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
>
> diff --git a/src/main.c b/src/main.c
> index 77886d58e..ceb839d11 100644
> --- a/src/main.c
> +++ b/src/main.c
> @@ -411,6 +411,22 @@ static void parse_controller_config(GKeyFile *config)
>                   &main_opts.default_params.le_scan_win_connect,
>                   0x0004,
>                   0x4000},
> +               { "LEMinConnectionInterval",
> +                 &main_opts.default_params.le_min_conn_interval,
> +                 0x0006,
> +                 0x0C80},
> +               { "LEMaxConnectionInterval",
> +                 &main_opts.default_params.le_max_conn_interval,
> +                 0x0006,
> +                 0x0C80},
> +               { "LEConnectionLatency",
> +                 &main_opts.default_params.le_conn_latency,
> +                 0x0000,
> +                 0x01F3},
> +               { "LEConnectionSupervisionTimeout",
> +                 &main_opts.default_params.le_conn_lsto,
> +                 0x000A,
> +                 0x0C80},
>                 { "LEAutoconnecttimeout",
>                   &main_opts.default_params.le_autoconnect_timeout,
>                   0x0001,
> --
> 2.27.0.290.gba653c62da-goog

Applied, thanks.

-- 
Luiz Augusto von Dentz