Re: [PATCH 00/12] Thumb-2 kernel support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 28 Jul 2008, Russell King - ARM Linux wrote:

> On Mon, Jul 28, 2008 at 01:40:55PM +0100, Catalin Marinas wrote:
> > On Mon, 2008-07-28 at 12:45 +0100, Catalin Marinas wrote:
> > > On Sun, 2008-07-27 at 12:08 +0100, Russell King - ARM Linux wrote:
> > > > On Wed, Jul 23, 2008 at 03:10:45PM -0700, Andrew Morton wrote:
> > > > > Well.  Rather than doing things sequentially we could go parallel. 
> > > > > Russell could say "I'll look at them before 2.6.28 but please put them
> > > > > into linux-next meanwhile".
> > > > 
> > > > There have been some valid but (iirc) non-vocal objections to the
> > > > Thumb-2 support from a few people.  The biggest one is all the
> > > > mess associated with supporting this "unified" assembler stuff.
> > > 
> > > My first implementation of these patches (last year) created a separate
> > > arch/ directory and a lot of people objected to this recommending to
> > > merge it into the existing arch/arm. Once I posted the re-worked
> > > implementation there were no big .
> > 
> > It looks like I didn't finish the above phrase - I meant no big
> > objections to the conditional compilation or someone stating clearly
> > that this is unacceptable.
> 
> And that's partly what I find rather frustrating.  I _know_ that people
> have issues with it, but they haven't raised them.

It happens that I just returned from vacations so couldn't speak up 
before now.

I do have issues with this new and improved unified assembly 
crap^H^H^H^Hsyntax.  And crappy features makes for crappy patches.  
With some additional rounds of review and refinements we might possibly 
make T2 support somewhat tolerable, but I don't think the latest T2 
patch series is there yet.  I'll provide relevant comments in the 
existing threads.


Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux