Message ID | 1409953475-23130-1-git-send-email-f.fainelli@gmail.com |
---|---|
State | New |
Headers | show |
On Friday 05 September 2014 14:44:35 Florian Fainelli wrote: > Arnd, Olof, > > This pull request contains the basic Broadcom BCM63138 DSL SoC support > that has been posted a few weeks ago. Arnd has reviewed those changes, and > I was expecting Matt to carry these changes for me, but I did not get an > answer so far and I don't want these patches to be held much longer. You still need to take him on Cc at least. It's bad if Matt doesn't reply to your emails, but that's no excuse to send a pull request without giving him a chance to Ack or Nack it. Did you try contacting him on IRC? According to my logs, he was on #armlinux at the time you sent the pull request, although he's marked as idle now. Let's give Matt and Christian a few more days to reply, we can take the pull request if there is no reaction. Arnd
On 09/06/14 07:33, Arnd Bergmann wrote: > On Friday 05 September 2014 14:44:35 Florian Fainelli wrote: >> Arnd, Olof, >> >> This pull request contains the basic Broadcom BCM63138 DSL SoC support >> that has been posted a few weeks ago. Arnd has reviewed those changes, and >> I was expecting Matt to carry these changes for me, but I did not get an >> answer so far and I don't want these patches to be held much longer. > > You still need to take him on Cc at least. It's bad if Matt doesn't reply > to your emails, but that's no excuse to send a pull request without giving > him a chance to Ack or Nack it. > > Did you try contacting him on IRC? According to my logs, he was on #armlinux > at the time you sent the pull request, although he's marked as idle now. I have, but did not get an answer so far. > > Let's give Matt and Christian a few more days to reply, we can take the > pull request if there is no reaction. Works for me. We might want to come up with a different strategy for handling mach-bcm stuff in the future, the BCM_MOBILE code will most likely not be the highest activity. Thanks -- Florian
On Sat, Sep 6, 2014 at 7:33 AM, Arnd Bergmann <arnd@arndb.de> wrote: > On Friday 05 September 2014 14:44:35 Florian Fainelli wrote: >> Arnd, Olof, >> >> This pull request contains the basic Broadcom BCM63138 DSL SoC support >> that has been posted a few weeks ago. Arnd has reviewed those changes, and >> I was expecting Matt to carry these changes for me, but I did not get an >> answer so far and I don't want these patches to be held much longer. > Hi Florian, Apologies for the untimeliness of replies, I've been leaning heavily on Matt to lead mach-bcm maintainership but given recent changes at Broadcom he's found himself pressed for time to do this also. I'm looking at this patchset to pull it, but I don't see any acks on the patches (the git commits in your repo do not have them, and searching through linux-arm-kernel doesn't seem to indicate acks/sign-offs for them). Have they been reviewed ? If not, I'll have to review them prior to pulling. Thanks, csd
On 09/14/14 21:56, Christian Daudt wrote: > On Sat, Sep 6, 2014 at 7:33 AM, Arnd Bergmann <arnd@arndb.de> wrote: >> On Friday 05 September 2014 14:44:35 Florian Fainelli wrote: >>> Arnd, Olof, >>> >>> This pull request contains the basic Broadcom BCM63138 DSL SoC support >>> that has been posted a few weeks ago. Arnd has reviewed those changes, and >>> I was expecting Matt to carry these changes for me, but I did not get an >>> answer so far and I don't want these patches to be held much longer. >> > Hi Florian, > Apologies for the untimeliness of replies, I've been leaning heavily > on Matt to lead mach-bcm maintainership but given recent changes at > Broadcom he's found himself pressed for time to do this also. Based on our earlier conversations on IRC this week, I was expecting you to send an update to the MAINTAINERS file reflecting the fact that we (Brian, Gregory, Marc and myself) should be listed as mach-bcm maintainers as well. > I'm looking at this patchset to pull it, but I don't see any acks on > the patches (the git commits in your repo do not have them, and > searching through linux-arm-kernel doesn't seem to indicate > acks/sign-offs for them). Have they been reviewed ? If not, I'll have > to review them prior to pulling. Arnd has reviewed v2, and v3 addresses his former comments, he has not replied with an Ack though I do not mind waiting for this. Thank you -- Florian
On Sun, Sep 14, 2014 at 10:16 PM, Florian Fainelli <f.fainelli@gmail.com> wrote: > On 09/14/14 21:56, Christian Daudt wrote: >> >> On Sat, Sep 6, 2014 at 7:33 AM, Arnd Bergmann <arnd@arndb.de> wrote: >>> >>> On Friday 05 September 2014 14:44:35 Florian Fainelli wrote: >>>> >>>> Arnd, Olof, >>>> >>>> This pull request contains the basic Broadcom BCM63138 DSL SoC support >>>> that has been posted a few weeks ago. Arnd has reviewed those changes, >>>> and >>>> I was expecting Matt to carry these changes for me, but I did not get an >>>> answer so far and I don't want these patches to be held much longer. >>> >>> >> Hi Florian, >> Apologies for the untimeliness of replies, I've been leaning heavily >> on Matt to lead mach-bcm maintainership but given recent changes at >> Broadcom he's found himself pressed for time to do this also. > > > Based on our earlier conversations on IRC this week, I was expecting you to > send an update to the MAINTAINERS file reflecting the fact that we (Brian, > Gregory, Marc and myself) should be listed as mach-bcm maintainers as well. > Yes, I do plan update the MAINTAINERS to include you. I don't see a need to include 4 more people at this point. That patch is coming. >> I'm looking at this patchset to pull it, but I don't see any acks on >> the patches (the git commits in your repo do not have them, and >> searching through linux-arm-kernel doesn't seem to indicate >> acks/sign-offs for them). Have they been reviewed ? If not, I'll have >> to review them prior to pulling. > > > Arnd has reviewed v2, and v3 addresses his former comments, he has not > replied with an Ack though I do not mind waiting for this. Ok - explicit is better than implicit. Arnd - can you pls confirm your ack to these patches ? Thanks, csd
On Monday 15 September 2014, Christian Daudt wrote: > > > > > > Arnd has reviewed v2, and v3 addresses his former comments, he has not > > replied with an Ack though I do not mind waiting for this. > > Ok - explicit is better than implicit. Arnd - can you pls confirm your > ack to these patches ? I'm glad you asked, because it triggered me to look at the pull request. The patches are all good, feel free to add Acked-by: Arnd Bergmann <arnd@arndb.de> to each patch, or to the merge, whichever works for you. However, the pull request was done wrong, so don't merge this changeset into your branch or we won't pull it from you. The problem is that it is based on top of out for-next branch, which is not meant to be sent upstream to Linus. Instead as you know we expect all pull requests coming in from platform maintainers to be done as topic branches and normall based on top of the previous -rc1 release, or possibly a later -rcX release if you have a strong dependency on a bug fix that went into that release. I'd recommend that Florian rebases from 244a68e90d0775d3e579 to v3.17-rc1 and send a new pull request, but it's also fine if you do the rebase and add your Signed-off-by to the patches in the process. Arnd
On 09/16/2014 09:27 AM, Arnd Bergmann wrote: > On Monday 15 September 2014, Christian Daudt wrote: >>> >>> >>> Arnd has reviewed v2, and v3 addresses his former comments, he has not >>> replied with an Ack though I do not mind waiting for this. >> >> Ok - explicit is better than implicit. Arnd - can you pls confirm your >> ack to these patches ? > > I'm glad you asked, because it triggered me to look at the pull request. > The patches are all good, feel free to add > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > to each patch, or to the merge, whichever works for you. > > However, the pull request was done wrong, so don't merge this changeset > into your branch or we won't pull it from you. > > The problem is that it is based on top of out for-next branch, which is > not meant to be sent upstream to Linus. Instead as you know we expect > all pull requests coming in from platform maintainers to be done as > topic branches and normall based on top of the previous -rc1 release, > or possibly a later -rcX release if you have a strong dependency on > a bug fix that went into that release. > > I'd recommend that Florian rebases from 244a68e90d0775d3e579 to > v3.17-rc1 and send a new pull request, but it's also fine if you > do the rebase and add your Signed-off-by to the patches in the process. I will do this and create a tag for Christian to pull from. Thanks! -- Florian