On Tue, May 24, 2016 at 10:23 PM, Philipp Zabel <p.zabel at pengutronix.de> wrote: > Am Dienstag, den 24.05.2016, 18:10 +0800 schrieb Liu Ying: >> For normal video modes such as 1280x720p at 60Hz and 1920x1080p at 60Hz, > > I'm very happy to drop all this code if possible, but the stated reason > is not good enough. Rather we should look at the worst case and make > sure that 2 FIFO slots are enough for 180 MPixel/s on i.MX5 and enough > for 240 MPixel/s on i.MX6.
My stated reason is not accurate, perhaps. Sorry. IIRC, when I wrote this patch, I found we use 2 slots for all video modes with the dynamic allocation mechanism. That's why the static one comes up to mind. > According to the current heuristic (total bandwidth = clkrate * 4 pixels > with 8 slots) that should be true. I'm not sure what's the best allocation algorithm is. Hope that the static one can be accepted temporarily and some one may offer a better solution in future. Regards, Liu Ying > > regards > Philipp >
