On Thu, 2005-03-17 at 22:39 -0500, Theodore Ts'o wrote: > On Thu, Mar 17, 2005 at 10:19:04AM +1100, Benjamin Herrenschmidt wrote: > > You probably want to remove the bit that does > > > > OUTREG(TV_DAC_CNTL, INREG(TV_DAC_CNTL) | 0x07000000); > > > > Or you'll lose TV output :) > > I'm not using TV output, and the original patch stated: > > > > + /* Power down TV DAC, that saves a significant amount of power, > > > + * we'll have something better once we actually have some TVOut > > > + * support > > > + */
Yup, I know, I wrote this bit :) > I suppose I should renable the TV DAC and see how much power it > actually consumes if I enable it. It would seem to me that we should > have a way that we can power down whatever parts of the video chipset > that we're not using. (For example if I don't have anything connected > to the VGA output, it would be good if we could power that down too...) We can power down the internal DAC too, yes, and the TMDS transmitter when no DVI is plugged, etc.. and we can also lower the chip clock :) I do intend to do these things. The problem right now is that the above will break some users who have a BIOS that can set TV-Out. Maybe some sysfs attribute ? At least until I can properly probe all ports including the TV Out (I'm working on that). Ultimately, the driver should be able to properly detect everything that is connected. Ben. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
