* Yegor Yefremov <yegor_sub1@xxxxxxxxxxxxxxxx> [120925 01:47]: > On 25.09.2012 10:32, Russell King - ARM Linux wrote: > > On Tue, Sep 25, 2012 at 10:26:30AM +0200, yegorslists@xxxxxxxxxxxxxx wrote: > >> From: Yegor Yefremov <yegorslists@xxxxxxxxxxxxxx> > >> > >> FORCE_MAX_ZONEORDER of 12 is needed to allocation more than 4MB > >> of consistent DMA memory (da8xx frame buffer driver). > > > > Okay, so the patch description says "This needs to be 12 on this platform". > > > >> config FORCE_MAX_ZONEORDER > >> - int "Maximum zone order" if ARCH_SHMOBILE > >> - range 11 64 if ARCH_SHMOBILE > >> + int "Maximum zone order" if ARCH_SHMOBILE || SOC_AM33XX > >> + range 11 64 if ARCH_SHMOBILE || SOC_AM33XX > > > > but you leave it up to the user to select something that may not be > > suitable. Wouldn't _just_ adding: > > > > default "12" if SOC_AM33XX > > > > after the "range", and making no other changes be good enough and match > > what the patch description says? > > You're right. As we don't allocate anything, but increase the possible size, it shouldn't break anything. Tony is it O.K. with you? Sure. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
