Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1539221

Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check

From Maxime Ripard <maxime.ripard@free-electrons.com>
Newsgroups linux.kernel
Subject Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check
Date 2016-12-09 10:40 +0100
Message-ID <sMpOO-1yx-13@gated-at.bofh.it> (permalink)
References <sH0o2-1XY-11@gated-at.bofh.it> <sLv0e-6YU-21@gated-at.bofh.it> <sLA9z-1Ws-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Wed, Dec 07, 2016 at 10:26:25AM +0800, Chen-Yu Tsai wrote:
> > Some panels require an exact frequency, some have a minimal frequency
> > but no maximum, some have a maximum frequency but no minimal, and I
> > guess most of them deviates by how much exactly they can take (and
> > possibly can take more easily a higher frequency, but are less
> > tolerant if you take a frequency lower than the nominal.
> >
> > And we cannot remove that check entirely, since some bridges will
> > report out of range frequencies for higher modes that we know we
> > cannot reach.
> 
> I believe this should be handled by the bridge driver in the check
> callback?

This doesn't really have anything to do with the bridge itself, it's a
limitation on the encoder. For all we know, the bridge might be able
to operate at the higher resolutions without any issues if the encoder
was able to.

> The callback I'm changing is attached to the connector, which I
> think doesn't get used if you have a bridge instead.  And this only
> checks the pre-registered display modes, such as those specified in
> simple-panel or EDID.

Geeee, I forgot to send that one (and thought I did)... I'll send that
patch next week, but basically, I was creating a mode_valid hook at
the encoder level, and moving the RGB mode_valid hook from the
connector to the encoder (since it really is an encoder limitation).

Maxime

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

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock  frequency check Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-12-06 22:00 +0100
  Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock  frequency check Chen-Yu Tsai <wens@csie.org> - 2016-12-07 03:30 +0100
    Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-12-07 10:50 +0100
      Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock  frequency check Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-12-09 10:00 +0100
    Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock  frequency check Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-12-09 10:40 +0100
  Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check Eric Anholt <eric@anholt.net> - 2016-12-07 20:20 +0100
    Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock frequency check Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2016-12-07 20:40 +0100
    Re: [PATCH RFC] drm/sun4i: rgb: Add 5% tolerance to dot clock  frequency check Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-12-09 09:50 +0100

csiph-web