From mboxrd@z Thu Jan 1 00:00:00 1970 From: Giuseppe CONDORELLI Date: Wed, 29 Jul 2009 10:50:57 +0200 Subject: [U-Boot] [PATCH] zlib: allow 0 as destination pointer In-Reply-To: <20090729071151.641BF832E416@gemini.denx.de> Message-ID: <002901ca1029$b0c3b580$c08182a4@st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi rhabarber1848, please could you test latest two zlib patches I have sent few minutes ago? NOTE: the first one is waiting for approval due to its big size. Thanks, I hope these patches will solve your issue. G. -----Original Message----- From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Wolfgang Denk Sent: Wednesday, July 29, 2009 9:12 AM To: rhabarber1848 at web.de Cc: u-boot at lists.denx.de Subject: Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer Dear rhabarber1848, In message you wrote: > > > Actually I'm waiting for a new patch that integrates that fix. > > please do not commit the zlib-1.2.3 patch in its current state, although > Alessandro's patch fixes on bug there is still the problem that the > WATCHDOG_RESET() calls from gunzip.c > > #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG) > s.outcb = (cb_func)WATCHDOG_RESET; > #else > s.outcb = Z_NULL; > #endif<>/* CONFIG_HW_WATCHDOG */ > > are silently ignored in the new zlib code. My C knowledge is insufficient to > solve this problem. In its current state it will break zlib decompression > on the machines I am taking care of. Agreed - that should be working, too, before we pull that patch into mainline. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Perfection is reached, not when there is no longer anything to add, but when there is no longer anything to take away. - Antoine de Saint-Exupery _______________________________________________ U-Boot mailing list U-Boot at lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot