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;
Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.80.23.2.2\))
Subject: Re: [PATCH v3 0/3] Support reading and setting default system
 parameters
From:   Marcel Holtmann <marcel@holtmann.org>
In-Reply-To: <20200611020157.70230-1-alainm@chromium.org>
Date:   Fri, 12 Jun 2020 15:47:16 +0200
Cc:     linux-bluetooth@vger.kernel.org
Content-Transfer-Encoding: 7bit
Message-Id: <17B2F033-19B2-463A-A8D1-B561A5F16AD6@holtmann.org>
References: <20200611020157.70230-1-alainm@chromium.org>
To:     Alain Michaud <alainm@chromium.org>
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Alain,

> This series adds support for reading and setting default system
> parameters from userspace.  In particular, combined with the userspace
> changes, allows platforms to override default system parameters from a
> main.conf file.
> 
> The series has been reviewed and tested on chromeos.
> 
> 
> Changes in v3:
> - Fixed sparse errors
> -Fixing sparse errors
> 
> Changes in v2:
> - Renamed the mgmt.h per Marcel's comments.
> - Addressed Marcel's comments in the implementation.
> 
> Alain Michaud (3):
>  mgmt: read/set system parameter definitions
>  bluetooth:centralize default value initialization.
>  bluetooth: implement read/set default system parameters mgmt
> 
> include/net/bluetooth/hci_core.h |  18 +++
> include/net/bluetooth/mgmt.h     |  30 ++++
> net/bluetooth/Makefile           |   2 +-
> net/bluetooth/hci_conn.c         |  14 +-
> net/bluetooth/hci_core.c         |  14 +-
> net/bluetooth/hci_request.c      |  15 +-
> net/bluetooth/mgmt.c             |   6 +
> net/bluetooth/mgmt_config.c      | 253 +++++++++++++++++++++++++++++++
> net/bluetooth/mgmt_config.h      |  11 ++
> 9 files changed, 341 insertions(+), 22 deletions(-)
> create mode 100644 net/bluetooth/mgmt_config.c
> create mode 100644 net/bluetooth/mgmt_config.h

all 3 patch have been applied with modifications to bluetooth-next tree.

Regards

Marcel