From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lei Wen Date: Tue, 10 Apr 2012 13:05:13 +0800 Subject: [U-Boot] [PATCH 0/6] add zip command support for uboot In-Reply-To: <201204100037.47663.vapier@gentoo.org> References: <1332690817-31759-1-git-send-email-adrian.wenl@gmail.com> <201204100037.47663.vapier@gentoo.org> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Mike, On Tue, Apr 10, 2012 at 12:37 PM, Mike Frysinger wrote: > On Tuesday 03 April 2012 05:31:09 Lei Wen wrote: >> Hi Mike, >> >> On Tue, Apr 3, 2012 at 3:17 AM, Mike Frysinger wrote: >> > On Tue, Mar 27, 2012 at 04:04, Lei Wen wrote: >> >> On Sun, Mar 25, 2012 at 11:53 PM, ? wrote: >> >>> From: Lei Wen >> >>> >> >>> This patch set add zip command support for uboot. >> >>> The first two patches import deflate and trees functions from zlib >> >>> 1.2.5 without any change. While the third patch did the necessary >> >>> change to make the import file could be built passed in uboot >> >>> environment. >> >> >> >> Any comments to this series? >> > >> > did you forward port the misc fixes/optimization that were done ? ?or >> > did you just drop in the new code ? >> >> The fixes/optimization is already separated in different patch, as the >> 0004 in current series. >> Other file is just import as intact from zlib1.2.5, the 0001 and 0002 >> patch. > > i don't understand what you mean. ?0004 is "lib: zlib: remove the limitation > for cannot using 0 as start" and doesn't look like a fix that was merged > before. > >> > for example, cd514aeb996e2f7aefbe1f78481965d9d074aed4 is pretty important >> >> I see. I try my best to keep the current modification history adding >> to the original zlib code. > > so you have maintained the bug fixes / optimizations ? ?it isn't clear in your > response (at least to me). I see what your concerns... Yes, this change is not included in the zlib's own code, but just added by myself. Without this change, in uboot, we cannot zip from the start of ddr memory, since many platforms, at least from what I saw, their start ddr memory address is from 0. If you are still not comfortable with this patch, I could remove the 004 from patch set. What do you think of this suggestion? > -mike Thanks, Lei