Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1202880 > unrolled thread

[GIT PULL] at91: defconfig for 4.3 #2

Back to article view | Back to linux.kernel

Contents

  [GIT PULL] at91: defconfig for 4.3 #2 Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2015-08-07 18:30 +0200
    Re: [GIT PULL] at91: defconfig for 4.3 #2 Olof Johansson <olof@lixom.net> - 2015-08-13 12:50 +0200
      Re: [GIT PULL] at91: defconfig for 4.3 #2 Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2015-08-13 13:30 +0200
        Re: [GIT PULL] at91: defconfig for 4.3 #2 Olof Johansson <olof@lixom.net> - 2015-08-13 14:30 +0200
          Re: [GIT PULL] at91: defconfig for 4.3 #2 Stephen Rothwell <sfr@canb.auug.org.au> - 2015-08-14 01:30 +0200
            Re: [GIT PULL] at91: defconfig for 4.3 #2 Olof Johansson <olof@lixom.net> - 2015-08-18 23:50 +0200

#1202880 — [GIT PULL] at91: defconfig for 4.3 #2

Arnd, Olof, Kevin,

A little defconfig update. That will probably be all for this cycle.

Thanks, bye,


The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:

  ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2

for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:

  ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)

----------------------------------------------------------------
Second batch of defconfig update for 4.3:
- enable the HLCDC driver for at91sam9
- remove an obsolete config option

----------------------------------------------------------------
Alexandre Belloni (1):
      ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER

Josh Wu (1):
      ARM: at91/defconfig: at91_dt: enable DRM hlcdc support

 arch/arm/configs/at91_dt_defconfig | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1206703

On Fri, Aug 07, 2015 at 06:27:54PM +0200, Alexandre Belloni wrote:
> Arnd, Olof, Kevin,
> 
> A little defconfig update. That will probably be all for this cycle.
> 
> Thanks, bye,
> 
> 
> The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:
> 
>   ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2
> 
> for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:
> 
>   ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)

Nicolas had sent me one patch which you've also included here but it's not in
the pull request ("enable ISI and ov2640 support"). That caused a conflict
here, so to avoid having the patch in the tree twice I instead also directly
applied your patches from this branch instead of merging.

Sorry about that, it started due to Nicolas sending the discrete patch to us.
So it seems like we'll apply at91 defconfig updates directly this release, in
case there are any more. No big deal I hope. :)


-Olof

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1206738

On 13/08/2015 at 12:09:38 +0200, Olof Johansson wrote :
> On Fri, Aug 07, 2015 at 06:27:54PM +0200, Alexandre Belloni wrote:
> > Arnd, Olof, Kevin,
> > 
> > A little defconfig update. That will probably be all for this cycle.
> > 
> > Thanks, bye,
> > 
> > 
> > The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:
> > 
> >   ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2
> > 
> > for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:
> > 
> >   ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)
> 
> Nicolas had sent me one patch which you've also included here but it's not in
> the pull request ("enable ISI and ov2640 support"). That caused a conflict
> here, so to avoid having the patch in the tree twice I instead also directly
> applied your patches from this branch instead of merging.
> 
> Sorry about that, it started due to Nicolas sending the discrete patch to us.
> So it seems like we'll apply at91 defconfig updates directly this release, in
> case there are any more. No big deal I hope. :)
> 

That's not a big deal and there will most probably be all for that cycle
anyway.

However, I'm wondering how I should have done. From the PR, if you get
eff7f41572a645bf14a96a6f844be4f1c88cd9dd..tags/at91-ab-defconfig2, it
correctly excludes "enable ISI and ov2640 support" so I was thinking it
was fine.
Be cause the first patch was taken as a patch, should I have prepared a
branch without it?

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1206773

On Thu, Aug 13, 2015 at 01:21:18PM +0200, Alexandre Belloni wrote:
> On 13/08/2015 at 12:09:38 +0200, Olof Johansson wrote :
> > On Fri, Aug 07, 2015 at 06:27:54PM +0200, Alexandre Belloni wrote:
> > > Arnd, Olof, Kevin,
> > > 
> > > A little defconfig update. That will probably be all for this cycle.
> > > 
> > > Thanks, bye,
> > > 
> > > 
> > > The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:
> > > 
> > >   ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2
> > > 
> > > for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:
> > > 
> > >   ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)
> > 
> > Nicolas had sent me one patch which you've also included here but it's not in
> > the pull request ("enable ISI and ov2640 support"). That caused a conflict
> > here, so to avoid having the patch in the tree twice I instead also directly
> > applied your patches from this branch instead of merging.
> > 
> > Sorry about that, it started due to Nicolas sending the discrete patch to us.
> > So it seems like we'll apply at91 defconfig updates directly this release, in
> > case there are any more. No big deal I hope. :)
> > 
> 
> That's not a big deal and there will most probably be all for that cycle
> anyway.
> 
> However, I'm wondering how I should have done. From the PR, if you get
> eff7f41572a645bf14a96a6f844be4f1c88cd9dd..tags/at91-ab-defconfig2, it
> correctly excludes "enable ISI and ov2640 support" so I was thinking it
> was fine.
> Be cause the first patch was taken as a patch, should I have prepared a
> branch without it?

Yeah, you could have applied the other two if there were no conflicts. If there
were conflicts then it gets complicated, and the best answer would probably
have been to continue applying directly as patches.

This is the drawback of us sometimes applying patches directly -- when we do,
we no longer put them on a branch of their own that you can use as base. We
used to once upon a time but it is quite a bit of overhead for us and the
upside was pretty limited.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1207241

Hi Olof,

On Thu, 13 Aug 2015 14:26:43 +0200 Olof Johansson <olof@lixom.net> wrote:
>
> On Thu, Aug 13, 2015 at 01:21:18PM +0200, Alexandre Belloni wrote:
> > On 13/08/2015 at 12:09:38 +0200, Olof Johansson wrote :
> > > On Fri, Aug 07, 2015 at 06:27:54PM +0200, Alexandre Belloni wrote:
> > > > 
> > > > A little defconfig update. That will probably be all for this cycle.
> > > > 
> > > > Thanks, bye,
> > > > 
> > > > 
> > > > The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:
> > > > 
> > > >   ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)
> > > > 
> > > > are available in the git repository at:
> > > > 
> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2
> > > > 
> > > > for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:
> > > > 
> > > >   ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)
> > > 
> > > Nicolas had sent me one patch which you've also included here but it's not in
> > > the pull request ("enable ISI and ov2640 support"). That caused a conflict
> > > here, so to avoid having the patch in the tree twice I instead also directly
> > > applied your patches from this branch instead of merging.
> > > 
> > > Sorry about that, it started due to Nicolas sending the discrete patch to us.
> > > So it seems like we'll apply at91 defconfig updates directly this release, in
> > > case there are any more. No big deal I hope. :)
> > 
> > That's not a big deal and there will most probably be all for that cycle
> > anyway.
> > 
> > However, I'm wondering how I should have done. From the PR, if you get
> > eff7f41572a645bf14a96a6f844be4f1c88cd9dd..tags/at91-ab-defconfig2, it
> > correctly excludes "enable ISI and ov2640 support" so I was thinking it
> > was fine.
> > Be cause the first patch was taken as a patch, should I have prepared a
> > branch without it?
> 
> Yeah, you could have applied the other two if there were no conflicts. If there
> were conflicts then it gets complicated, and the best answer would probably
> have been to continue applying directly as patches.
> 
> This is the drawback of us sometimes applying patches directly -- when we do,
> we no longer put them on a branch of their own that you can use as base. We
> used to once upon a time but it is quite a bit of overhead for us and the
> upside was pretty limited.

The only thing now is that since the at91 tree is in linux-next as well
as the arm-soc tree, those patches appear twice there and there is a
conflict (easy to fix, but a pain).  The solution here is to update the
at91 tree to be somewhere in the arm-soc tree (probably just reset to
the point where the two trees match in patches but not commits).  This
has the downside that the at91 tree will be rebased which will affect
any development work that is based on that.

If there was just one patch in common, it maybe have been better to
just merge the at91 tree and fix the conflict in the merge.
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1209498

Hi Stephen,

On Fri, Aug 14, 2015 at 1:22 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi Olof,
>
> On Thu, 13 Aug 2015 14:26:43 +0200 Olof Johansson <olof@lixom.net> wrote:
>>
>> On Thu, Aug 13, 2015 at 01:21:18PM +0200, Alexandre Belloni wrote:
>> > On 13/08/2015 at 12:09:38 +0200, Olof Johansson wrote :
>> > > On Fri, Aug 07, 2015 at 06:27:54PM +0200, Alexandre Belloni wrote:
>> > > >
>> > > > A little defconfig update. That will probably be all for this cycle.
>> > > >
>> > > > Thanks, bye,
>> > > >
>> > > >
>> > > > The following changes since commit eff7f41572a645bf14a96a6f844be4f1c88cd9dd:
>> > > >
>> > > >   ARM: at91: at91_dt_defconfig: enable ISI and ov2640 support (2015-07-30 14:17:31 +0200)
>> > > >
>> > > > are available in the git repository at:
>> > > >
>> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git tags/at91-ab-defconfig2
>> > > >
>> > > > for you to fetch changes up to ea7bf603fd494391acc1f42acbfc34260b965c44:
>> > > >
>> > > >   ARM: at91/defconfig: at91_dt: remove ARM_AT91_ETHER (2015-08-07 12:07:50 +0200)
>> > >
>> > > Nicolas had sent me one patch which you've also included here but it's not in
>> > > the pull request ("enable ISI and ov2640 support"). That caused a conflict
>> > > here, so to avoid having the patch in the tree twice I instead also directly
>> > > applied your patches from this branch instead of merging.
>> > >
>> > > Sorry about that, it started due to Nicolas sending the discrete patch to us.
>> > > So it seems like we'll apply at91 defconfig updates directly this release, in
>> > > case there are any more. No big deal I hope. :)
>> >
>> > That's not a big deal and there will most probably be all for that cycle
>> > anyway.
>> >
>> > However, I'm wondering how I should have done. From the PR, if you get
>> > eff7f41572a645bf14a96a6f844be4f1c88cd9dd..tags/at91-ab-defconfig2, it
>> > correctly excludes "enable ISI and ov2640 support" so I was thinking it
>> > was fine.
>> > Be cause the first patch was taken as a patch, should I have prepared a
>> > branch without it?
>>
>> Yeah, you could have applied the other two if there were no conflicts. If there
>> were conflicts then it gets complicated, and the best answer would probably
>> have been to continue applying directly as patches.
>>
>> This is the drawback of us sometimes applying patches directly -- when we do,
>> we no longer put them on a branch of their own that you can use as base. We
>> used to once upon a time but it is quite a bit of overhead for us and the
>> upside was pretty limited.
>
> The only thing now is that since the at91 tree is in linux-next as well
> as the arm-soc tree, those patches appear twice there and there is a
> conflict (easy to fix, but a pain).  The solution here is to update the
> at91 tree to be somewhere in the arm-soc tree (probably just reset to
> the point where the two trees match in patches but not commits).  This
> has the downside that the at91 tree will be rebased which will affect
> any development work that is based on that.
>
> If there was just one patch in common, it maybe have been better to
> just merge the at91 tree and fix the conflict in the merge.

Yeah, this is a somewhat frustrating situation for us. I wonder if we
should essentially tag the downstream ARM platform trees in -next such
that if they conflict with arm-soc, you can drop them for a day if
needed.

It's really useful for us to be able to occasionally adjust a pull
request instead of always merging them exactly as they're presented to
us. It saves a roundtrip to the maintainer for trivial stuff, we can
take care of it and not have to look at it again. We often do send it
back to the maintainers to respin, but in this particular case it
seemed appropriate to just deal with it locally for us.

The downstream users vs rebasing git trees is of course another
aspect. Here I'm mostly relying on subplatform maintainers to know
well how many people actually develop on top of their trees. I think
for most platforms it's a fairly limited use. Interesting enough, I
can't remember last time someone told me they couldn't respin a pull
request to fix something up due to downstream developers (not that we
have _all_ that many of those requests).

The other way to handle this would be to only apply patches directly
and not do merges. Most other high-volume maintainers have exactly
this workflow. We've been able to avoid reverting to that, thankfully
(since we can delegate most of the reviews and patch applications this
way).


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web