From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephan Linz Date: Tue, 26 Jun 2012 18:56:49 +0200 Subject: [U-Boot] [PATCH 2/2] microblaze: Enable ubi support In-Reply-To: References: <1340307771-15204-1-git-send-email-linz@li-pro.net> <1340307771-15204-2-git-send-email-linz@li-pro.net> <4FE7F154.80406@monstr.eu> <20120625072202.238C3202239@gemini.denx.de> <1340644130.4335.43.camel@keto> Message-ID: <1340729809.2767.52.camel@keto> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am Dienstag, den 26.06.2012, 10:52 +0200 schrieb Michal Simek: > 2012/6/25 Stephan Linz : > > Am Montag, den 25.06.2012, 09:22 +0200 schrieb Wolfgang Denk: > >> Dear Michal Simek, > >> > >> In message <4FE7F154.80406@monstr.eu> you wrote: > >> > On 06/21/2012 09:42 PM, Stephan Linz wrote: > >> > > To save memory the UBIFS is disabled by default. > >> > > > >> > > The original patch was introdused with commit: > >> > > 0114da7b06bd47b7f5c3f20a152dd11903b38fba > >> > > >> > This say nothing to me. > >> > Format is Patch name and when sha1. > >> > > >> > Where is this patch? I can't see it in mainline repository? > >> > >> Correct. This commit does not exist in mainline. > > > > Yes, the commit was newer merged into mainline and lives for more than > > one year in Microblaze custodian repo: > > > > http://git.denx.de/?p=u-boot/u-boot-microblaze.git;a=commitdiff;h=0114da7b06bd47b7f5c3f20a152dd11903b38fba > > I have there two branches - master and uboot. uboot follows mainline > branch and master contain one your patch > which was merged to mainline. > That link is correct but it points to nonexisting branch. > That patch was definitely there but that branch was delete quite some time ago. Yes, it was in the net (?) branch that was deleted sometime. > > Wolfgang: How to run git-gc on this repository? > > I am ok to add this to mainline. Please add that two patches together > and resubmit. I'll do so (three patches -- see my other comment). > Also I would prefer to define SYS_MALLOC_LEN based on SIZE. > It means for example > #define CONFIG_SYS_MALLOC_LEN (SIZE *2) > or similar. OK, I'll evaluate the best factor for this multiplication. br, Stephan