Message ID | 20220128205915.3995760-1-luiz.dentz@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 0a78117213c4bb85dfebf89b924226883597bc60 |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | pull request: bluetooth 2022-01-28 | expand |
Context | Check | Description |
---|---|---|
netdev/tree_selection | success | Pull request for net |
netdev/build_32bit | success | Errors and warnings before: 42 this patch: 38 |
netdev/build_clang | success | Errors and warnings before: 5 this patch: 0 |
netdev/verify_signedoff | success | Signed-off-by tag matches author and committer |
netdev/verify_fixes | success | Fixes tag looks correct |
netdev/build_allmodconfig_warn | success | Errors and warnings before: 42 this patch: 38 |
On Fri, 28 Jan 2022 12:59:15 -0800 Luiz Augusto von Dentz wrote: > The following changes since commit 8aaaf2f3af2ae212428f4db1af34214225f5cec3: > > Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net (2022-01-09 17:00:17 -0800) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git tags/for-net-next-2022-01-28 > > for you to fetch changes up to 91cb4c19118a19470a9d7d6dbdf64763bbbadcde: > > Bluetooth: Increment management interface revision (2022-01-27 12:35:13 -0800) > > ---------------------------------------------------------------- > bluetooth-next pull request for net-next: > > - Add support for RTL8822C hci_ver 0x08 > - Add support for RTL8852AE part 0bda:2852 > - Fix WBS setting for Intel legacy ROM products > - Enable SCO over I2S ib mt7921s > - Increment management interface revision Thanks for fixing the warnings! :) I presume this is for the net-next given the name of your tree, but a lot of patches here have fixes tags. What's your methodology on separating fixes from new features? I think it may be worth adjusting the filter there and send more stuff earlier to Linus's tree. Especially fixes with the right mix of confidence and impact or pure device ID additions. To be clear - happy to pull this PR as is, I was meaning to ask about this for a while.
Hi Jakub, >> The following changes since commit 8aaaf2f3af2ae212428f4db1af34214225f5cec3: >> >> Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net (2022-01-09 17:00:17 -0800) >> >> are available in the Git repository at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git tags/for-net-next-2022-01-28 >> >> for you to fetch changes up to 91cb4c19118a19470a9d7d6dbdf64763bbbadcde: >> >> Bluetooth: Increment management interface revision (2022-01-27 12:35:13 -0800) >> >> ---------------------------------------------------------------- >> bluetooth-next pull request for net-next: >> >> - Add support for RTL8822C hci_ver 0x08 >> - Add support for RTL8852AE part 0bda:2852 >> - Fix WBS setting for Intel legacy ROM products >> - Enable SCO over I2S ib mt7921s >> - Increment management interface revision > > Thanks for fixing the warnings! :) > > I presume this is for the net-next given the name of your tree, but > a lot of patches here have fixes tags. What's your methodology on > separating fixes from new features? > > I think it may be worth adjusting the filter there and send more > stuff earlier to Linus's tree. Especially fixes with the right mix > of confidence and impact or pure device ID additions. > > To be clear - happy to pull this PR as is, I was meaning to ask about > this for a while. we started to add Fixes: tag whenever you can identify a faulty commit or can track down the original issue. This way we can later easily go back and check. It have to note that a lot of vendor trees cherrypick patches and this helps them picking the right ones. I reviewed the list of patches again, and frankly none of them are super critical to go to Linus right away. So if you don’t mind, please pull. Regards Marcel
On Fri, 28 Jan 2022 23:30:14 +0100 Marcel Holtmann wrote: > > Thanks for fixing the warnings! :) > > > > I presume this is for the net-next given the name of your tree, but > > a lot of patches here have fixes tags. What's your methodology on > > separating fixes from new features? > > > > I think it may be worth adjusting the filter there and send more > > stuff earlier to Linus's tree. Especially fixes with the right mix > > of confidence and impact or pure device ID additions. > > > > To be clear - happy to pull this PR as is, I was meaning to ask about > > this for a while. > > we started to add Fixes: tag whenever you can identify a faulty commit or > can track down the original issue. This way we can later easily go back > and check. It have to note that a lot of vendor trees cherrypick patches > and this helps them picking the right ones. Thumbs up for that! > I reviewed the list of patches again, and frankly none of them are super > critical to go to Linus right away. My concern is that GregKH will start asking us why we hold onto trivial fixes like 5201d23cc8e5 until the merge window, I think this merge window has overflown his patch ID scheme ;) The risk of pushing fixes in early -rc's should be pretty low. But your call at the end of the day! > So if you don’t mind, please pull. Sure thing, done! :)
Hello: This pull request was applied to netdev/net-next.git (master) by Jakub Kicinski <kuba@kernel.org>: On Fri, 28 Jan 2022 12:59:15 -0800 you wrote: > The following changes since commit 8aaaf2f3af2ae212428f4db1af34214225f5cec3: > > Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net (2022-01-09 17:00:17 -0800) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git tags/for-net-next-2022-01-28 > > [...] Here is the summary with links: - pull request: bluetooth 2022-01-28 https://git.kernel.org/netdev/net-next/c/0a78117213c4 You are awesome, thank you!