[Top][All Lists]

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

Re: [Qemu-devel] [PATCH 2/9] Add trustzone support.

From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH 2/9] Add trustzone support.
Date: Tue, 20 Dec 2011 19:38:02 +0000

On 20 December 2011 19:10, Mark Langsdorf <address@hidden> wrote:
> From: address@hidden
> Conflicts:
>        target-arm/cpu.h
>        target-arm/helper.c
> Signed-off-by: Mark Langsdorf <address@hidden>

All sorts of problems here, I'm afraid.
 * you can't submit a patch written by somebody else and without
   their sign-off
 * no git 'conflicts' messages in commit messages!
 * lots of irrelevant changes (like the brace changes)
 * it's clearly not actually an implementation of trustzone
   (just for a start, no register banking, and no monitor mode)

This is a long way from being acceptable upstream.

I have a variant on this patch in qemu-linaro
which at least has fewer irrelevant bits in it but is still
in need of attention. More promisingly,
is a much more complete trustzone implementation, and Johannes
has said he hopes to get it into shape for upstream.

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]