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@xxxxxxxxx> Acked-by: Florian Fainelli <florian@xxxxxxxxxxx> > --- > > 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];