Re: [PATCH V2 1/4] xen/arm: Broadcom 7445D0 ARM cpu support.

[Julien Grall <julien.grall@linaro.org>]

Hello Jon,

On 02/10/2014 23:16, Jon Fraser wrote:
> Relax cpu id mask to cover minor cpu revisions in the SoC versions.
>
> Signed-off-by: Jon Fraser <jfraser@broadcom.com>

Ian and I already acked/reviewed this patch on the previous version.

When you send a new version of a patch, you need to carry the 
acked/reviewed by given on the previous versions. Unless there is big 
change in the patch.

Regards,

> ---
>   xen/arch/arm/arm32/proc-v7.S | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/xen/arch/arm/arm32/proc-v7.S b/xen/arch/arm/arm32/proc-v7.S
> index b0d5343..2f3ff1e 100644
> --- a/xen/arch/arm/arm32/proc-v7.S
> +++ b/xen/arch/arm/arm32/proc-v7.S
> @@ -50,8 +50,8 @@ __v7_ca7mp_proc_info:
>           .section ".init.proc.info", #alloc, #execinstr
>           .type __v7_brahma15mp_proc_info, #object
>   __v7_brahma15mp_proc_info:
> -        .long 0x420F00F2             /* Broadcom Brahma-B15 */
> -        .long 0xFF0FFFFF             /* Mask */
> +        .long 0x420F00F0             /* Broadcom Brahma-B15 */
> +        .long 0xFF0FFFF0             /* Mask */
>           .long brahma15mp_init
>           .long caxx_processor
>           .size __v7_brahma15mp_proc_info, . - __v7_brahma15mp_proc_info
>

-- 
Julien Grall