Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Kevin Hilman
Troy Kisky writes: > Reserve channels 0,1,12,13 and > slots 78-109 for dsp use on dm644x. > > Signed-off-by: Troy Kisky > > I've only verified that channels 0, and 78-101 need > reserved, but reserving a little extra seems like > a good idea. I'm not crazy about this hard-coded reservation. Th

RE: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Ring, Chris
ource-boun...@linux.davincidsp.com > ] On Behalf Of Kevin Hilman > Sent: Friday, April 17, 2009 5:02 AM > To: Troy Kisky > Cc: davinci-linux-open-source@linux.davincidsp.com > Subject: Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage > > Troy Kisky writes: > > >

Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Troy Kisky
Kevin Hilman wrote: > Troy Kisky writes: > >> Reserve channels 0,1,12,13 and >> slots 78-109 for dsp use on dm644x. >> >> Signed-off-by: Troy Kisky >> >> I've only verified that channels 0, and 78-101 need >> reserved, but reserving a little extra seems like >> a good idea. > > I'm not crazy ab

Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Troy Kisky
al Message- >> From: davinci-linux-open-source-boun...@linux.davincidsp.com >> [mailto:davinci-linux-open-source-boun...@linux.davincidsp.com >> ] On Behalf Of Kevin Hilman >> Sent: Friday, April 17, 2009 5:02 AM >> To: Troy Kisky >> Cc: davinci-linux-open-sou

RE: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Ring, Chris
..@boundarydevices.com] > Sent: Friday, April 17, 2009 3:17 PM > To: Ring, Chris > Cc: Kevin Hilman; davinci-linux-open-source@linux.davincidsp.com > Subject: Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage > > Ring, Chris wrote: > > It's a bit too much info, but

Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Troy Kisky
Ring, Chris wrote: > It may not have been clear, there's sort of a 2 phase resource management: >1. At build/config time where the resources are statically partitioned - > some for ARM, some for DSP - and given to their respective CPU-specific > resource managers. I think originally we simp

RE: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Ring, Chris
I'm personally not at all against the idea. Chris > -Original Message- > From: Troy Kisky [mailto:troy.ki...@boundarydevices.com] > Sent: Friday, April 17, 2009 4:15 PM > To: Ring, Chris > Cc: Kevin Hilman; davinci-linux-open-source@linux.davincidsp.com > Subjec

Re: [PATCH] ARM: DaVinci: edma: reserve dsp dma usage

2009-04-17 Thread Troy Kisky
Ring, Chris wrote: > Why does this patch get your mpeg4 codec to work - b/c the codec has a > hard-coded channel 0 it needs > (so your patch keeps channel 0 out of Linux's partition)? Yes. And channels 78-101 as well. > > If so, IMHO, that change doesn't belong in the mainline tree - that's a