Re: [PATCH] MIPS: bcm63xx: Fix header_crc comment in bcm963xx_tag.h

[Florian Fainelli <florian@openwrt.org>]

On Friday 08 April 2011 14:32:15 Jonas Gorski wrote:
> The CRC32 actually includes the tag_version.
> 
> Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>

Acked-by: Florian Fainelli <florian@openwrt.org>

> ---
> 
> This was already wrong in the original Broadcom sources (and it still
> seems to be).
> 
>  arch/mips/include/asm/mach-bcm63xx/bcm963xx_tag.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/mips/include/asm/mach-bcm63xx/bcm963xx_tag.h
> b/arch/mips/include/asm/mach-bcm63xx/bcm963xx_tag.h index 5325084..73c499f
> 100644
> --- a/arch/mips/include/asm/mach-bcm63xx/bcm963xx_tag.h
> +++ b/arch/mips/include/asm/mach-bcm63xx/bcm963xx_tag.h
> @@ -88,7 +88,7 @@ struct bcm_tag {
>  	char kernel_crc[CRC_LEN];
>  	/* 228-235: Unused at present */
>  	char reserved1[8];
> -	/* 236-239: CRC32 of header excluding tagVersion */
> +	/* 236-239: CRC32 of header excluding last 20 bytes */
>  	char header_crc[CRC_LEN];
>  	/* 240-255: Unused at present */
>  	char reserved2[16];