Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-4411-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Precedence: bulk
MIME-Version: 1.0
References: <20240507143639.539863-1-luiz.dentz@gmail.com> <663a497d.050a0220.99cab.8529@mx.google.com>
 <CABBYNZJ8NDv64Unsa=-j_RJhpCGBBtJf9Vvn2Fq9Vvc9OsHmwg@mail.gmail.com> <CAJNyHpL1sqNxbe_2o1Pjx_BJTka0c5NoanQBfGPU1bYit7dXVQ@mail.gmail.com>
In-Reply-To: <CAJNyHpL1sqNxbe_2o1Pjx_BJTka0c5NoanQBfGPU1bYit7dXVQ@mail.gmail.com>
From: Sungwoo Kim <iam@sung-woo.kim>
Date: Thu, 9 May 2024 07:16:47 +0900
Message-ID: <CAJNyHp+HZF_Vh5c9xmNuFxsc6dMmEboQH2tXx2C5KWCNe1C51A@mail.gmail.com>
Subject: Re: [v2] Bluetooth: HCI: Remove HCI_AMP support
To: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, May 8, 2024 at 5:21=E2=80=AFAM Sungwoo Kim <iam@sung-woo.kim> wrote=
:
>
> Hi Luiz,
>
> On Wed, May 8, 2024 at 4:26=E2=80=AFAM Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
> >
> > Hi Sungwoo,
> >
> > On Tue, May 7, 2024 at 11:32=E2=80=AFAM <bluez.test.bot@gmail.com> wrot=
e:
> > >
> > > This is automated email and please do not reply to this email!
> > >
> > > Dear submitter,
> > >
> > > Thank you for submitting the patches to the linux bluetooth mailing l=
ist.
> > > This is a CI test results with your patch series:
> > > PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=
=3D851233
> >
> > Can you try reproducing the problem with the above change?

This bug is not reproduced under my testing environment.
https://lore.kernel.org/linux-kernel/20240506022035.663102-1-iam@sung-woo.k=
im/

>
> Sure, I will.
>
> For the patch, we can also remove HCI_FLOW_CTL_MODE_PACKET_BASED and
> HCI_FLOW_CTL_MODE_BLOCK_BASED from hci.h as no one is using it.
>
> One question:
> This patch removes some HCI commands and flow control modes that are
> still in the specification v5.4.
> Should we remove these commands and modes from the specification also?
>
> Thanks,
> Sungwoo.