Message ID | YTg+csY9wy4mk035@matsya (mailing list archive) |
---|---|
State | Accepted |
Commit | 0aa2516017123a7c35a2c0c35c4dc7727579b8a3 |
Headers | show |
Series | [GIT,PULL] : dmaengine updates for v5.15-rc1 | expand |
On Tue, Sep 7, 2021 at 9:39 PM Vinod Koul <vkoul@kernel.org> wrote: > > Also contains, bus_remove_return_void-5.15 to resolve dependencies That one actually has a commit message with explanations about that branch from Greg. There are other merges that _don't_ have any reason for them. Like randomly merging 5.14-rc5 with no explanation. Or the three random "merge fixes". Please people. For the Nth time. Merges need commit messages that tell people _why_ you merge, and what the code is that you merge. Not just "merge tree X" Linus
The pull request you sent on Wed, 8 Sep 2021 10:09:14 +0530:
> git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git tags/dmaengine-5.15-rc1
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/0aa2516017123a7c35a2c0c35c4dc7727579b8a3
Thank you!
Hi Linus, On 09-09-21, 11:11, Linus Torvalds wrote: > On Tue, Sep 7, 2021 at 9:39 PM Vinod Koul <vkoul@kernel.org> wrote: > > > > Also contains, bus_remove_return_void-5.15 to resolve dependencies > > That one actually has a commit message with explanations about that > branch from Greg. > > There are other merges that _don't_ have any reason for them. Like > randomly merging 5.14-rc5 with no explanation. > > Or the three random "merge fixes". > > Please people. For the Nth time. Merges need commit messages that tell > people _why_ you merge, and what the code is that you merge. Not just > "merge tree X" Agree, the reason to merge was to resolve conflicts b/w fixes and next. I agree we should document these merges better, will make sure that is taken care of in future!