Message ID | 20211001230850.3635543-1-luiz.dentz@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | pull request: bluetooth 2021-10-01 | expand |
Context | Check | Description |
---|---|---|
netdev/tree_selection | success | Not a local patch |
On Fri, 1 Oct 2021 16:08:50 -0700 Luiz Augusto von Dentz wrote: > bluetooth-next pull request for net-next: > > - Add support for MediaTek MT7922 and MT7921 > - Enable support for AOSP extention in Qualcomm WCN399x and Realtek > 8822C/8852A. > - Add initial support for link quality and audio/codec offload. > - Rework of sockets sendmsg to avoid locking issues. > - Add vhci suspend/resume emulation. Lots of missing sign-offs from Marcel: Commit 927ac8da35db ("Bluetooth: set quality report callback for Intel") committer Signed-off-by missing author email: josephsih@chromium.org committer email: marcel@holtmann.org Signed-off-by: Joseph Hwang <josephsih@chromium.org> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit ae7d925b5c04 ("Bluetooth: Support the quality report events") committer Signed-off-by missing author email: josephsih@chromium.org committer email: marcel@holtmann.org Signed-off-by: Joseph Hwang <josephsih@chromium.org> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit 93fb70bc112e ("Bluetooth: refactor set_exp_feature with a feature table") committer Signed-off-by missing author email: josephsih@chromium.org committer email: marcel@holtmann.org Signed-off-by: Joseph Hwang <josephsih@chromium.org> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit 76a56bbd810d ("Bluetooth: btintel: support link statistics telemetry events") committer Signed-off-by missing author email: chethan.tumkur.narayan@intel.com committer email: marcel@holtmann.org Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> Signed-off-by: Kiran K <kiran.k@intel.com> Signed-off-by: Joseph Hwang <josephsih@chromium.org> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit 0331b8e990ed ("Bluetooth: btusb: disable Intel link statistics telemetry events") committer Signed-off-by missing author email: josephsih@chromium.org committer email: marcel@holtmann.org Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> Signed-off-by: Kiran K <kiran.k@intel.com> Signed-off-by: Joseph Hwang <josephsih@chromium.org> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit 81218cbee980 ("Bluetooth: mgmt: Disallow legacy MGMT_OP_READ_LOCAL_OOB_EXT_DATA") committer Signed-off-by missing author email: brian.gix@intel.com committer email: marcel@holtmann.org Signed-off-by: Brian Gix <brian.gix@intel.com> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Commit 0b59e272f932 ("Bluetooth: reorganize functions from hci_sock_sendmsg()") committer Signed-off-by missing author email: penguin-kernel@I-love.SAKURA.ne.jp committer email: marcel@holtmann.org Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> Can this be fixed?
Hi Jakub, On Fri, Oct 1, 2021 at 8:11 PM Jakub Kicinski <kuba@kernel.org> wrote: > > On Fri, 1 Oct 2021 16:08:50 -0700 Luiz Augusto von Dentz wrote: > > bluetooth-next pull request for net-next: > > > > - Add support for MediaTek MT7922 and MT7921 > > - Enable support for AOSP extention in Qualcomm WCN399x and Realtek > > 8822C/8852A. > > - Add initial support for link quality and audio/codec offload. > > - Rework of sockets sendmsg to avoid locking issues. > > - Add vhci suspend/resume emulation. > > Lots of missing sign-offs from Marcel: > > Commit 927ac8da35db ("Bluetooth: set quality report callback for Intel") > committer Signed-off-by missing > author email: josephsih@chromium.org > committer email: marcel@holtmann.org > Signed-off-by: Joseph Hwang <josephsih@chromium.org> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit ae7d925b5c04 ("Bluetooth: Support the quality report events") > committer Signed-off-by missing > author email: josephsih@chromium.org > committer email: marcel@holtmann.org > Signed-off-by: Joseph Hwang <josephsih@chromium.org> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit 93fb70bc112e ("Bluetooth: refactor set_exp_feature with a feature table") > committer Signed-off-by missing > author email: josephsih@chromium.org > committer email: marcel@holtmann.org > Signed-off-by: Joseph Hwang <josephsih@chromium.org> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit 76a56bbd810d ("Bluetooth: btintel: support link statistics telemetry events") > committer Signed-off-by missing > author email: chethan.tumkur.narayan@intel.com > committer email: marcel@holtmann.org > Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> > Signed-off-by: Kiran K <kiran.k@intel.com> > Signed-off-by: Joseph Hwang <josephsih@chromium.org> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit 0331b8e990ed ("Bluetooth: btusb: disable Intel link statistics telemetry events") > committer Signed-off-by missing > author email: josephsih@chromium.org > committer email: marcel@holtmann.org > Signed-off-by: Chethan T N <chethan.tumkur.narayan@intel.com> > Signed-off-by: Kiran K <kiran.k@intel.com> > Signed-off-by: Joseph Hwang <josephsih@chromium.org> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit 81218cbee980 ("Bluetooth: mgmt: Disallow legacy MGMT_OP_READ_LOCAL_OOB_EXT_DATA") > committer Signed-off-by missing > author email: brian.gix@intel.com > committer email: marcel@holtmann.org > Signed-off-by: Brian Gix <brian.gix@intel.com> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > Commit 0b59e272f932 ("Bluetooth: reorganize functions from hci_sock_sendmsg()") > committer Signed-off-by missing > author email: penguin-kernel@I-love.SAKURA.ne.jp > committer email: marcel@holtmann.org > Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> I suspect those fixed and force pushed since I originally applied them given them all have my sign-offs, is this a blocker though?
On Fri, 1 Oct 2021 20:29:54 -0700 Luiz Augusto von Dentz wrote: > On Fri, Oct 1, 2021 at 8:11 PM Jakub Kicinski <kuba@kernel.org> wrote: > > On Fri, 1 Oct 2021 16:08:50 -0700 Luiz Augusto von Dentz wrote: > > > bluetooth-next pull request for net-next: > > > > > > - Add support for MediaTek MT7922 and MT7921 > > > - Enable support for AOSP extention in Qualcomm WCN399x and Realtek > > > 8822C/8852A. > > > - Add initial support for link quality and audio/codec offload. > > > - Rework of sockets sendmsg to avoid locking issues. > > > - Add vhci suspend/resume emulation. > > Commit 0b59e272f932 ("Bluetooth: reorganize functions from hci_sock_sendmsg()") > > committer Signed-off-by missing > > author email: penguin-kernel@I-love.SAKURA.ne.jp > > committer email: marcel@holtmann.org > > Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> > > Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > I suspect those fixed and force pushed since I originally applied them > given them all have my sign-offs, is this a blocker though? I'm not an expert on SoB semantics so not sure if it's a deal breaker. Stephen's checker will definitely notice and send us warnings. I think it's worth the hassle to rebase and resubmit for 7 bad commits.
Hi Jakub, >>>> bluetooth-next pull request for net-next: >>>> >>>> - Add support for MediaTek MT7922 and MT7921 >>>> - Enable support for AOSP extention in Qualcomm WCN399x and Realtek >>>> 8822C/8852A. >>>> - Add initial support for link quality and audio/codec offload. >>>> - Rework of sockets sendmsg to avoid locking issues. >>>> - Add vhci suspend/resume emulation. > >>> Commit 0b59e272f932 ("Bluetooth: reorganize functions from hci_sock_sendmsg()") >>> committer Signed-off-by missing >>> author email: penguin-kernel@I-love.SAKURA.ne.jp >>> committer email: marcel@holtmann.org >>> Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> >>> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> >> >> I suspect those fixed and force pushed since I originally applied them >> given them all have my sign-offs, is this a blocker though? > > I'm not an expert on SoB semantics so not sure if it's a deal breaker. > Stephen's checker will definitely notice and send us warnings. > I think it's worth the hassle to rebase and resubmit for 7 bad commits. at some point, I had to fix one commit via git rebase —interactive. Looks like it then made me committer for all commits. Originally Luiz applied all these patches and my Signed-off-by should be in it. I can fix this for sure and get the right committer back, I am just curious what git rebase did here. Regards Marcel