Re: [PATCHv4] drm/sun4i: validate modes for HDMI

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

 



Hi,

On Fri, Dec 15, 2017 at 06:06:32PM +0100, Daniel Vetter wrote:
> On Fri, Dec 15, 2017 at 05:46:19PM +0100, Hans Verkuil wrote:
> > When I connected my cubieboard running 4.15-rc1 to my 4k display I got no picture. Some
> > digging found that there is no check against the upper pixelclock limit of the HDMI
> > output, so X selects a 4kp60 format at 594 MHz, which obviously won't work.
> > 
> > The patch below adds a check for the upper bound of what this hardware can do, and
> > it checks if the requested tmds clock can be obtained.
> > 
> > It also allows for the ± 0.5% pixel clock variation that the HDMI spec permits.
> > 
> > That code is based on commit 22d0be2a557e53a22feb484e8fce255fe09e6ad5 from
> > Jose Abreu for drm/arc.
> > 
> > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> > ---
> > Changes since v3:
> > - Move the mode_valid callback to struct drm_encoder_helper_funcs.
> >   I'm assuming this is the correct struct, since this check is specific to the
> >   hdmi encoder.
> 
> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> 
> I think sun4i is in drm-misc, so feel free to push.

I already pushed the v3. Hans, can you provide an additional patch? on top of your v3?

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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