Message ID | 20211222141342.56395-1-sudeep.holla@arm.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [GIT,PULL] firmware: arm_scmi: Updates for v5.17 | expand |
Hi again, On Wed, Dec 22, 2021 at 02:13:42PM +0000, Sudeep Holla wrote: > Hi ARM SoC Team, > > Please pull ! > I am aware I sent this pull request just around the same time Arnd finalised the initial set. I have seen emails suggesting that Olof might be planning to merge additional PR, so it would be great if this gets included in that cut. All the changes are in -next since 21st December, so there should be no surprises. Thanks again and sorry for nagging. -- Regards, Sudeep
On Fri, Jan 07, 2022 at 11:37:40AM +0000, Sudeep Holla wrote: > > Hi again, > > On Wed, Dec 22, 2021 at 02:13:42PM +0000, Sudeep Holla wrote: > > Hi ARM SoC Team, > > > > Please pull ! > > > > I am aware I sent this pull request just around the same time Arnd finalised > the initial set. I have seen emails suggesting that Olof might be planning > to merge additional PR, so it would be great if this gets included in > that cut. All the changes are in -next since 21st December, so there should > be no surprises. > > Thanks again and sorry for nagging. Yeah, no worries -- I'd been letting pull requests sitting for a while over the holidays. I've merged this into our late branch now. Thanks, -Olof
On Sat, Jan 8, 2022 at 9:24 PM Olof Johansson <olof@lixom.net> wrote: > On Fri, Jan 07, 2022 at 11:37:40AM +0000, Sudeep Holla wrote: > > On Wed, Dec 22, 2021 at 02:13:42PM +0000, Sudeep Holla wrote: > > > Hi ARM SoC Team, > > > > > > Please pull ! > > > > > > > I am aware I sent this pull request just around the same time Arnd finalised > > the initial set. I have seen emails suggesting that Olof might be planning > > to merge additional PR, so it would be great if this gets included in > > that cut. All the changes are in -next since 21st December, so there should > > be no surprises. > > > > Thanks again and sorry for nagging. > > Yeah, no worries -- I'd been letting pull requests sitting for a while over the > holidays. I've merged this into our late branch now. Hi Sudeep, I'm going through all the stuff that we ended up not getting into v5.17. As you probably already know, this pull request was part of the arm/late branch that missed out, sorry for the mistake on our end. As the contents are still fine, I could offer to either merge the branch again for v5.18, or I can drop it and wait for a new pull request from you, please let me know your preference. If any of the patches are bug fixes that you think should be in v5.17, it would be good to have a separate pull request or patch email for the 5.17 fixes branch. Arnd
On Tue, Feb 08, 2022 at 11:00:43AM +0100, Arnd Bergmann wrote: > On Sat, Jan 8, 2022 at 9:24 PM Olof Johansson <olof@lixom.net> wrote: > > On Fri, Jan 07, 2022 at 11:37:40AM +0000, Sudeep Holla wrote: > > > On Wed, Dec 22, 2021 at 02:13:42PM +0000, Sudeep Holla wrote: > > > > Hi ARM SoC Team, > > > > > > > > Please pull ! > > > > > > > > > > I am aware I sent this pull request just around the same time Arnd finalised > > > the initial set. I have seen emails suggesting that Olof might be planning > > > to merge additional PR, so it would be great if this gets included in > > > that cut. All the changes are in -next since 21st December, so there should > > > be no surprises. > > > > > > Thanks again and sorry for nagging. > > > > Yeah, no worries -- I'd been letting pull requests sitting for a while over the > > holidays. I've merged this into our late branch now. > > Hi Sudeep, > > I'm going through all the stuff that we ended up not getting into v5.17. > As you probably already know, this pull request was part of the arm/late > branch that missed out, sorry for the mistake on our end. > No worries, it was indeed a late PR. > As the contents are still fine, I could offer to either merge the branch again > for v5.18, or I can drop it and wait for a new pull request from you, please > let me know your preference. > Is it OK if I merge this tag with v5.17-rc1 as base and load it with any new v5.18 material on the top ? I just want to avoid rebasing as the same time some of the new material depend on these changes. Let me know if that is fine. I am open to any other suggestions too. > If any of the patches are bug fixes that you think should be in v5.17, > it would be good to have a separate pull request or patch email for the > 5.17 fixes branch. > Sure, I will check again, but AFAIK there were no fixes for existing bugs.
On Tue, Feb 8, 2022 at 11:33 AM Sudeep Holla <sudeep.holla@arm.com> wrote: > On Tue, Feb 08, 2022 at 11:00:43AM +0100, Arnd Bergmann wrote: > > As the contents are still fine, I could offer to either merge the branch again > > for v5.18, or I can drop it and wait for a new pull request from you, please > > let me know your preference. > > > > Is it OK if I merge this tag with v5.17-rc1 as base and load it with any > new v5.18 material on the top ? I just want to avoid rebasing as the same time > some of the new material depend on these changes. Let me know if that > is fine. I am open to any other suggestions too. Yes, I think that is ok, as long as you start a branch from v5.17-rc1 and merge the old contents into the new branch, rather than back-merging the -rc1 tag into your existing branch. This way, the git history still makes sense across the merges. Arnd
On Tue, Feb 08, 2022 at 12:14:25PM +0100, Arnd Bergmann wrote: > On Tue, Feb 8, 2022 at 11:33 AM Sudeep Holla <sudeep.holla@arm.com> wrote: > > On Tue, Feb 08, 2022 at 11:00:43AM +0100, Arnd Bergmann wrote: > > > As the contents are still fine, I could offer to either merge the branch again > > > for v5.18, or I can drop it and wait for a new pull request from you, please > > > let me know your preference. > > > > > > > Is it OK if I merge this tag with v5.17-rc1 as base and load it with any > > new v5.18 material on the top ? I just want to avoid rebasing as the same time > > some of the new material depend on these changes. Let me know if that > > is fine. I am open to any other suggestions too. > > Yes, I think that is ok, as long as you start a branch from v5.17-rc1 and merge > the old contents into the new branch, rather than back-merging the -rc1 tag > into your existing branch. This way, the git history still makes sense > across the merges. Thanks for confirming my understanding.