Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230525153452.125789-1-silviu.barbulescu@nxp.com> <168513902178.4400.7735618892742799772.git-patchwork-notify@kernel.org>
In-Reply-To: <168513902178.4400.7735618892742799772.git-patchwork-notify@kernel.org>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Fri, 26 May 2023 15:37:44 -0700
Message-ID: <CABBYNZKD+nL+fan95r8rca8ivg_9Ng=EH8QOk6tfky8VBoLduA@mail.gmail.com>
Subject: Re: [PATCH v2 0/6] Add initial support for BAP broadcast source
To:     patchwork-bot+bluetooth@kernel.org
Cc:     Silviu Florian Barbulescu <silviu.barbulescu@nxp.com>,
        linux-bluetooth@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Silviu,

On Fri, May 26, 2023 at 3:22=E2=80=AFPM <patchwork-bot+bluetooth@kernel.org=
> wrote:
>
> Hello:
>
> This series was applied to bluetooth/bluez.git (master)
> by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:
>
> On Thu, 25 May 2023 18:34:46 +0300 you wrote:
> > This patch adds initial support for BAP broadcast source.
> >
> > The current implementation allows BAP source endpoint registration,
> > media transport creation, transport acquiring and sending broadcast ISO=
 data.
> >
> > Currently, one BIG containing one BIS is supported.
> >
> > [...]
>
> Here is the summary with links:
>   - [v2,1/6] Update Docs for BAP broadcast source
>     https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=3Df808fa065396
>   - [v2,2/6] Add macro definitions for BAP broadcast source support
>     (no matching commit)
>   - [v2,3/6] Check for ISO broadcast support in controller
>     https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=3D97f3386268fa
>   - [v2,4/6] Add support for setsockopt (BT_IO_OPT_BASE)
>     (no matching commit)
>   - [v2,5/6] Update bluetoothctl with support for broadcast source
>     (no matching commit)
>   - [v2,6/6] Add initial support for BAP broadcast source
>     (no matching commit)
>
> You are awesome, thank you!

So Ive pushed changes up to 4/6, the last 2 have a lot of coding style
problems so please fix them, and I'd still recommend you to split
those as much as possible to make it easier to review.


--=20
Luiz Augusto von Dentz