Re: [patch 00/36] Hexagon: Add support for Qualcomm Hexagon architecture

[Arnd Bergmann]

On Wednesday 17 August 2011 11:34:57 Richard Kuo wrote:
> The following patch set is intended to add support for the Qualcomm Hexagon
> architecture to the Linux kernel.
> 
> The Hexagon architecture is designed to deliver performance with low power
> over a variety of applications.  It has features such as multithreading,
> privilege levels, VLIW, and instructions geared toward efficient signal
> processing.  The port of Linux for Hexagon runs under a hypervisor layer.
> 
> I realize we are not currently in a merge window, but I felt it would be best
> to go ahead and send it out so that the community gets an ample chance to
> review it.  This is a very large patch set, but we've tried to break it up
> into functional pieces to make review easier.

Hi Richard and Linas,

It's good to finally see the source for this, and I have a good feeling
about the quality of the code, so it should not be a problem to resolve
the remaining issues in time to get it into the next merge window.

I have gone over all patches once now and commented about everything
that looked odd or wrong. Most of the issues are really just small
changes. Removing support for the legacy syscalls is an easy change
in the kernel but it might require significant changes in the libc
implementation. It's not a big problem if your libc still requires the
old interfaces at the time the architecture gets merged upstream,
that will just mean that you have to have a small kernel patch to add
them back in while transitioning to the new libc.

As I commented on patch 33, I think the platform specific patches should
not get merged before everything is converted to device tree based
probing. This should also not keep the rest from going in. I think it
would be good to do the conversion together with the ARM-MSM platform
that seems to share a lot of the device specific code, and it would
be bad to end up with duplicate work here or (worse) with incompatible
DT bindings.

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