Re: [PATCH 7/7][OMAP 3/4+] BRIDGE: set 0 for phys_mempool_* as default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Felipe,

From: "ext Felipe Balbi" <me@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH 7/7][OMAP 3/4+] BRIDGE: set 0 for phys_mempool_* as default
Date: Sat, 13 Sep 2008 11:47:40 +0300

> Hi Hiroshi,
> 
> On Sat, Sep 13, 2008 at 07:48:44AM +0300, Hiroshi DOYU wrote:
> > I think that this configuration name "FB_OMAP_CONSISTENT_DMA_SIZE"
> > sounds a bit strange because it is not specific to framebuffer.
> 
> I agree here, it's really not fb specific.
> 
> > So how about renaming this more generic name like
> > "OMAP_CONSISTENT_DMA_SIZE"? Or making "CONSISTENT_DMA_SIZE" itself
> > configurable in menuconfig?
> 
> For me it sounds like that value would change depending on the board, no?

Hm...this can be considered as a board specific, yes.

> So maybe that size should be set up during board init. Maybe bundled
> inside omapX_globals ??

This sounds reasonable to me. Since FB and IVA are the only modules
which depend on this size so far, expanding flexibility in menuconfig
may no be necessary.

    Hiroshi DOYU

--
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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux