Stefan Roese wrote: > On Thursday 27 March 2008, Larry Johnson wrote: >>> +- CFG_MEM_TOP_HIDE (PPC only): >>> + If CFG_MEM_TOP_HIDE is defined in the board config header, >>> + this specified memory area will get subtracted from the top >>> + (end) of ram and won't get "touched" at all by U-Boot. By >>> + fixing up gd->ram_size the Linux kernel should gets passed >>> + the now "corrected" memory size and won't touch it either. >>> + This should work for arch/ppc and arch/powerpc. Only Linux >>> + board ports in arch/powerpc with bootwrapper support, that >>> + recalculate the memory size from the SDRAM controller setup >>> + will have to get fixed. >>> [...] >> Hi Stefan, >> >> I also have a nit to pick :-) . It's not clear to me whether the clause >> in the last sentence is meant to be restrictive or non-restrictive. If >> restrictive, I recommend removing the comma before the word "that". If >> non-restrictive, I recommend replacing the word "that" with the word >> "which". > > Thanks. Its sometimes helpful if native English speakers helps out here. I > have to admit that I don't really get the difference of both versions you > mention above. In this sentense I refer to Linux board ports which need > *additional* fixes in Linux. I'll send the changed patch in a few minutes > with this sentence: > > " > Only Linux board > ports in arch/powerpc with bootwrapper support, which recalculate the > memory size from the SDRAM controller setup, will have to get fixed > in Linux additionally. > " > > I hope its clear now.
It's clear now, but unfortunately I don't know enough to tell whether it is also correct :-( . The statement as now written implies that the statement without the clause, i.e., "Only Linux board ports in arch/powerpc with bootwrapper support will have to get fixed in Linux additionally", is also true. If this is not correct (because boards that do not recalculate their memory size do not need to be fixed in Linux), then you would write: "Only Linux board ports in arch/powerpc with bootwrapper support that recalculate the memory size from the SDRAM controller setup will have to get fixed in Linux additionally." BTW, there is no general agreement on whether the word "that" in the preceding sentence is correct. While I prefer it, some believe the word "which" should be used instead. English is such a complicated language, even for native speakers, that I am always amazed at the quality of the postings from contributors who did not learn it as their first language. Best regards, Larry ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users