[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support
From: |
Hollis Blanchard |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support |
Date: |
Mon, 17 Nov 2008 15:38:39 -0600 |
On Mon, 2008-11-17 at 22:30 +0100, François Revol wrote:
> > - /* TODO: Implement compressed images. */
> > - if (hdr->ih_comp != IH_COMP_NONE) {
> > - fprintf(stderr, "Unable to load compressed u-boot images\
> > n");
> > + /* TODO bzip2 support */
> > + if (hdr->ih_comp == IH_COMP_BZIP2) {
> > + fprintf(stderr, "Unable to load bzip2 compressed u-boot
> > images\n");
>
> Shouldn't it still be testing for !=NONE && !=GZIP ?
> In case the header is incorrect or new compressions are added ?
I don't see what you mean about the header being incorrect, but I agree
on the second point. I'll make this change.
--
Hollis Blanchard
IBM Linux Technology Center
- [Qemu-devel] improvements in uImage loading, Hollis Blanchard, 2008/11/17
- [Qemu-devel] [PATCH 1/4] uImage: free file data after copying into guest memory, Hollis Blanchard, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Jean-Christophe PLAGNIOL-VILLARD, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, François Revol, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Jean-Christophe PLAGNIOL-VILLARD, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, François Revol, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Hollis Blanchard, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Jean-Christophe PLAGNIOL-VILLARD, 2008/11/17
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Hollis Blanchard, 2008/11/18
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Anthony Liguori, 2008/11/18
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Hollis Blanchard, 2008/11/18
- Re: [Qemu-devel] [PATCH 2/4] uImage: implement gzip support, Anthony Liguori, 2008/11/18