Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-28 Thread Bryan Wu
Andrew, could you please add this patch to -mm. As we discussed here, it should be OK for us. Thanks, - Bryan Wu On Fri, 2007-09-21 at 11:32 +0800, David McCullough wrote: > > Jivin Robin Getz lays it down ... > > On Thu 20 Sep 2007 11:03, David McCullough pondered: > > > I would say that (a) i

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread David McCullough
Jivin Robin Getz lays it down ... > On Thu 20 Sep 2007 11:03, David McCullough pondered: > > I would say that (a) is definately not the case. I am sure the BF guys > > will say they have been banging us on the head with changes for a long > > time and getting no where as we considered the changes

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Robin Getz
On Thu 20 Sep 2007 11:03, David McCullough pondered: > I would say that (a) is definately not the case. I am sure the BF guys > will say they have been banging us on the head with changes for a long > time and getting no where as we considered the changes to severe or out > of line. I don't think

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread David McCullough
Jivin Paul Mundt lays it down ... ... > > The other maintainers who have spoken up didn't seem to think this was > > ugly, or an abuse. I'm surprised to hear language like that when > > discussing a patch that adds an if statement, a local variable and one > > parameter in a function call. > >

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Bernd Schmidt
Paul Mundt wrote: This is making API changes where it's convenient for your platform to use this value, and there's no reason to change the API here at all. Your proposed addition of flat_validate_relval is an API change, and very similar in nature to what I've done. A local variable here is t

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Paul Mundt
On Thu, Sep 20, 2007 at 02:04:26PM +0200, Bernd Schmidt wrote: > Paul Mundt wrote: > >I find it a bit disconcerting that blackfin already depends on this > >in-tree without there being any earlier discussion on making these > >changes. > > Parts of the initial submission were picked up (the includ

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Bernd Schmidt
Paul Mundt wrote: I find it a bit disconcerting that blackfin already depends on this in-tree without there being any earlier discussion on making these changes. Parts of the initial submission were picked up (the include/asm directory), other's weren't. Little we can do about that.

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Hirokazu Takata
From: David McCullough <[EMAIL PROTECTED]> Date: Thu, 20 Sep 2007 11:55:25 +1000 > > Jivin Robin Getz lays it down ... > > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > > From: Bernd Schmidt <[EMAIL PROTECTED]> > > > > > > This just adds minimum support for the Blackfin relocations, > > > sinc

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-20 Thread Miles Bader
Robin Getz <[EMAIL PROTECTED]> writes: > BTW - does anyone know Miles Bader's current email address? There isn't one > listed in MAINTAINERS - and he is still listed for the NEC V850? [EMAIL PROTECTED] -Miles -- Everywhere is walking distance if you have the time. -- Steven Wright - To unsubs

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Bryan Wu
On Thu, 2007-09-20 at 14:34 +0800, Bryan Wu wrote: > On Thu, 2007-09-20 at 02:08 -0400, Robin Getz wrote: > > On Wed 19 Sep 2007 23:54, Paul Mundt pondered: > > > On Wed, Sep 19, 2007 at 11:42:53PM -0400, Mike Frysinger wrote: > > > > On 9/19/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > > > > > On T

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Bryan Wu
On Thu, 2007-09-20 at 02:08 -0400, Robin Getz wrote: > On Wed 19 Sep 2007 23:54, Paul Mundt pondered: > > On Wed, Sep 19, 2007 at 11:42:53PM -0400, Mike Frysinger wrote: > > > On 9/19/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > > > > On Thu, Sep 20, 2007 at 11:55:25AM +1000, David McCullough wrote:

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Robin Getz
On Wed 19 Sep 2007 23:54, Paul Mundt pondered: > On Wed, Sep 19, 2007 at 11:42:53PM -0400, Mike Frysinger wrote: > > On 9/19/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > > > On Thu, Sep 20, 2007 at 11:55:25AM +1000, David McCullough wrote: > > > > Jivin Robin Getz lays it down ... > > > > > On Tue 1

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Paul Mundt
On Wed, Sep 19, 2007 at 11:42:53PM -0400, Mike Frysinger wrote: > On 9/19/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > > On Thu, Sep 20, 2007 at 11:55:25AM +1000, David McCullough wrote: > > > Jivin Robin Getz lays it down ... > > > > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > > > > This just

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Mike Frysinger
On 9/19/07, Paul Mundt <[EMAIL PROTECTED]> wrote: > On Thu, Sep 20, 2007 at 11:55:25AM +1000, David McCullough wrote: > > Jivin Robin Getz lays it down ... > > > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > > > This just adds minimum support for the Blackfin relocations, > > > > since we don't

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Paul Mundt
On Thu, Sep 20, 2007 at 11:55:25AM +1000, David McCullough wrote: > Jivin Robin Getz lays it down ... > > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > > This just adds minimum support for the Blackfin relocations, > > > since we don't have enough space in each reloc. The idea > > > is to store

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Robin Getz
On Wed 19 Sep 2007 21:55, David McCullough pondered: > Jivin Robin Getz lays it down ... > > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > > From: Bernd Schmidt <[EMAIL PROTECTED]> > > > > > > This just adds minimum support for the Blackfin relocations, > > > since we don't have enough space in

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread David McCullough
Jivin Robin Getz lays it down ... > On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > > From: Bernd Schmidt <[EMAIL PROTECTED]> > > > > This just adds minimum support for the Blackfin relocations, > > since we don't have enough space in each reloc. The idea > > is to store a value with one relocatio

Re: [PATCH] binfmt_flat: minimum support for theBlackfin relocations

2007-09-19 Thread Robin Getz
On Tue 18 Sep 2007 04:09, Bryan Wu pondered: > From: Bernd Schmidt <[EMAIL PROTECTED]> > > This just adds minimum support for the Blackfin relocations, > since we don't have enough space in each reloc. The idea > is to store a value with one relocation so that subsequent ones can > access it. > >