From: "Guzman Lugo, Fernando" <fernando.lugo@ti.com>
To: Felipe Contreras <felipe.contreras@gmail.com>
Cc: "Kanigeri, Hari" <h-kanigeri2@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"ohad@wizery.com" <ohad@wizery.com>,
"hiroshi.doyu@nokia.com" <hiroshi.doyu@nokia.com>,
"ameya.palande@nokia.com" <ameya.palande@nokia.com>,
"felipe.contreras@nokia.com" <felipe.contreras@nokia.com>
Subject: RE: [PATCH 8/9] dspbridge: add map support for big buffers
Date: Wed, 25 Aug 2010 14:42:32 -0500 [thread overview]
Message-ID: <496565EC904933469F292DDA3F1663E602F142E143@dlee06.ent.ti.com> (raw)
In-Reply-To: <AANLkTi=VRROWWZ4x-znXshz9KWNZAaizVFt=M92D3ByW@mail.gmail.com>
> -----Original Message-----
> From: Felipe Contreras [mailto:felipe.contreras@gmail.com]
> Sent: Wednesday, August 25, 2010 2:18 PM
> To: Guzman Lugo, Fernando
> Cc: Kanigeri, Hari; linux-omap@vger.kernel.org;
> linux-kernel@vger.kernel.org; ohad@wizery.com;
> hiroshi.doyu@nokia.com; ameya.palande@nokia.com;
> felipe.contreras@nokia.com
> Subject: Re: [PATCH 8/9] dspbridge: add map support for big buffers
>
> On Wed, Aug 25, 2010 at 7:00 PM, Guzman Lugo, Fernando
> <fernando.lugo@ti.com> wrote:
> >> Or the whole iovmm.c can be copied and just remove the mapping.
> >
> > That is the migration patches do, remove custom bridge
> mapping and use
> > iovmm module instead.
>
> You didn't get me; copy the iovmm.c code into dmm.c, and
> remove the mapping.
It sounds like double work, changing the dmm.c which will disappear.
Also I think what you want is only the change from array to
Linked list. Otherwise copying iovmm.c into dmm.c and adapting
To how dspbridge is at this momment it will need too much changes,
It is better to think like: take memory reservation (linked list)
Implementation from iovmm and map it to dmm.c.
But I don't see the point of doing that, if dmm.c is about to
Dissapear.
Regards,
Fernando.
>
> --
> Felipe Contreras
>
next prev parent reply other threads:[~2010-08-25 19:42 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-01 0:20 [PATCH 0/9] dspbridge: iommu migration Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 1/9] dspbridge: replace iommu custom for opensource implementation Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 2/9] dspbridge: move shared memory iommu maps to tiomap3430.c Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 3/9] dspbridge: rename bridge_brd_mem_map/unmap to a proper name Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 4/9] dspbridge: remove custom mmu code from tiomap3430.c Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 5/9] dspbridge: add mmufault support Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 6/9] dspbridge: remove hw directory Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 7/9] dspbridge: move all iommu related code to a new file Fernando Guzman Lugo
2010-07-01 0:20 ` [PATCH 8/9] dspbridge: add map support for big buffers Fernando Guzman Lugo
2010-07-01 0:21 ` [PATCH 9/9] dspbridge: cleanup bridge_dev_context and cfg_hostres structures Fernando Guzman Lugo
2010-07-01 23:36 ` [PATCH 8/9] dspbridge: add map support for big buffers Kanigeri, Hari
2010-07-02 16:27 ` Guzman Lugo, Fernando
2010-07-02 17:02 ` Kanigeri, Hari
2010-07-02 18:39 ` Guzman Lugo, Fernando
2010-08-24 10:09 ` Felipe Contreras
2010-08-24 16:31 ` Guzman Lugo, Fernando
2010-08-24 17:46 ` Felipe Contreras
2010-08-24 18:44 ` Guzman Lugo, Fernando
2010-08-24 18:55 ` Felipe Contreras
2010-08-24 19:05 ` Guzman Lugo, Fernando
2010-08-24 19:13 ` Felipe Contreras
2010-08-24 19:30 ` Guzman Lugo, Fernando
2010-08-24 21:17 ` Felipe Contreras
2010-08-24 21:55 ` Guzman Lugo, Fernando
2010-08-25 6:05 ` Felipe Contreras
2010-08-25 16:00 ` Guzman Lugo, Fernando
2010-08-25 19:18 ` Felipe Contreras
2010-08-25 19:42 ` Guzman Lugo, Fernando [this message]
2010-08-26 10:04 ` Felipe Contreras
2010-08-26 13:00 ` Guzman Lugo, Fernando
2010-08-26 6:55 ` FUJITA Tomonori
2010-08-26 12:38 ` Guzman Lugo, Fernando
2010-08-26 12:56 ` FUJITA Tomonori
2010-08-26 13:16 ` Guzman Lugo, Fernando
2010-08-26 14:03 ` FUJITA Tomonori
2010-08-26 14:13 ` Guzman Lugo, Fernando
2010-07-02 6:27 ` [PATCH 5/9] dspbridge: add mmufault support Hiroshi DOYU
2010-07-02 16:37 ` Guzman Lugo, Fernando
2010-07-02 6:32 ` [PATCH 1/9] dspbridge: replace iommu custom for opensource implementation Hiroshi DOYU
2010-07-02 16:40 ` Guzman Lugo, Fernando
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=496565EC904933469F292DDA3F1663E602F142E143@dlee06.ent.ti.com \
--to=fernando.lugo@ti.com \
--cc=ameya.palande@nokia.com \
--cc=felipe.contreras@gmail.com \
--cc=felipe.contreras@nokia.com \
--cc=h-kanigeri2@ti.com \
--cc=hiroshi.doyu@nokia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=ohad@wizery.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®