Re: [PATCH] drm/vc4: Add runtime PM support to the HDMI encoder driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 14 Apr 2017 11:20:52 -0700
Eric Anholt <eric@xxxxxxxxxx> wrote:

> Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx> writes:
> 
> > The HDMI driver is currently enabling all clks and probe time and keep
> > the power-domain connected to the HDMI encoder enabled.  
> 
> How about "The HDMI driver is currently enabling all clocks at probe
> time and keeps the power domain..."?

Yep.

> 
> > Move all activation code to vc4_hdmi_encoder_enable() and make sure
> > the clks and power domain are released when the HDMI encoder is not used
> > by adding deactivation steps in vc4_hdmi_encoder_disable().
> >
> > Note that the sequencing imposed by the IP requires that we move
> > vc4_hdmi_encoder_mode_set() code into vc4_hdmi_encoder_enable().  
> 
> I'm quite happy to see _mode_set() gone.
> 
> I'm hoping to go through a bunch of mode switching testing with this
> Monday.

Ok, cool. Let me know if you have any problem.

Thanks,

Boris
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux