On Sat, 25 Dec 2010, Janusz Krzysztofik wrote: [snip] > > Passing the virtual address allows the API to become much more > > flexible. Not only that, it allows it to be used on ARM, rather than > > becoming (as it currently stands) prohibited on ARM. > > > > I believe that putting ioremap() inside this API was the wrong thing > > to do, and moving it outside makes the API much more flexible and > > usable. It's something I still fully support. > > Thanks, this is what I was missing, having my point of view rather my > machine centric, with not much wider experience. I'll quote your > argumentation in next iteration of this patch if required. AFAIU, this patch is similar to the previous two attempts: http://www.spinics.net/lists/linux-sh/msg05482.html and http://thread.gmane.org/gmane.linux.drivers.video-input-infrastructure/22271 but is even more intrusive, because those two previous attempts added new functions, whereas this one is modifying an existing one. Both those two attempts have been NACKed by FUJITA Tomonori, btw, he is not on the otherwise extensive CC list for this patch. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
