On 08/26/2010 11:56 AM, Erik Gilling wrote: > On Wed, Aug 25, 2010 at 3:59 PM, Ryan Mallon <ryan@xxxxxxxxxxxxxxxx> wrote: >> On 08/26/2010 10:03 AM, Erik Gilling wrote: >> >> You sent this just to me, did you mean to post to the list also? > > oops > >>>>> +config FB_TEGRA >>>>> + tristate "Tegra Framebuffer driver" >>>>> + depends on TEGRA_DC && FB = y >>>> >> >>>> >>>> The dump_regs code is very long for a debugging feature. Can it just be >>>> replaced by a for loop which prints the offsets and values of each register? >>> >>> Printing the names of the registers has been invaluable so far. I >>> could move this to a separate file. >> >> Its useful for debugging while developing the driver, but it adds a lot >> of code. Do you expect the register dumping features to be required once >> the driver hits mainline? > > I expect the driver will hit mainline is several phases. This first > one. One where HDMI works. One where DSI works. One which has > rotation. This code is useful for all those phases. ... > They still fill different uses. You can't cat a debugfs file from an > interrupt handler. You can't call dump_regs() from userspace. I'm > still actively using both. I could see taking one out once > development on the driver calms down. Okay, maybe keep both with a comment stating why the debug is still there, and that it is scheduled to be removed once the driver is complete in mainline? ~Ryan -- Bluewater Systems Ltd - ARM Technology Solution Centre Ryan Mallon 5 Amuri Park, 404 Barbadoes St ryan@xxxxxxxxxxxxxxxx PO Box 13 889, Christchurch 8013 http://www.bluewatersys.com New Zealand Phone: +64 3 3779127 Freecall: Australia 1800 148 751 Fax: +64 3 3779135 USA 1800 261 2934 -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html