From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Doug Anderson <dianders@chromium.org>
Cc: "open list:ARM/Rockchip SoC..."
<linux-rockchip@lists.infradead.org>,
alsa-devel@alsa-project.org,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Andy Yan <andy.yan@rock-chips.com>,
Yakir Yang <ykk@rock-chips.com>,
Fabio Estevam <fabio.estevam@freescale.com>,
Mark Brown <broonie@kernel.org>, Takashi Iwai <tiwai@suse.de>,
Jaroslav Kysela <perex@perex.cz>,
Sascha Hauer <s.hauer@pengutronix.de>,
Jon Nettleton <jon.nettleton@gmail.com>,
David Airlie <airlied@linux.ie>
Subject: Re: [PATCH 6/9] drm: bridge/dw_hdmi: adjust pixel clock values in N calculation
Date: Sat, 5 Sep 2015 09:34:22 +0100 [thread overview]
Message-ID: <20150905083422.GL21084@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAD=FV=WR4iJ57FLh1d46-s9Gu-4wW6NdoROx-kNbbA0zFpUo0w@mail.gmail.com>
On Fri, Sep 04, 2015 at 07:03:11PM -0700, Doug Anderson wrote:
> Then perhaps you shouldn't be using a switch statement. You won't
> catch all values that are within .05% of (25.2 / 1.001).
No.
The clock rates you get ultimately come from the EDID via either the
detailed timing modes or from the CEA mode IDs, which are then looked
up in tables in the DRM EDID parsing code.
Either way, you will end up with 25175 and not 25170 or something
strange based on what the platform does.
--
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
WARNING: multiple messages have this Message-ID (diff)
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Doug Anderson <dianders@chromium.org>
Cc: Fabio Estevam <fabio.estevam@freescale.com>,
alsa-devel@alsa-project.org,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
Jaroslav Kysela <perex@perex.cz>,
"open list:ARM/Rockchip SoC..."
<linux-rockchip@lists.infradead.org>,
Mark Brown <broonie@kernel.org>, Yakir Yang <ykk@rock-chips.com>,
Andy Yan <andy.yan@rock-chips.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 6/9] drm: bridge/dw_hdmi: adjust pixel clock values in N calculation
Date: Sat, 5 Sep 2015 09:34:22 +0100 [thread overview]
Message-ID: <20150905083422.GL21084@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAD=FV=WR4iJ57FLh1d46-s9Gu-4wW6NdoROx-kNbbA0zFpUo0w@mail.gmail.com>
On Fri, Sep 04, 2015 at 07:03:11PM -0700, Doug Anderson wrote:
> Then perhaps you shouldn't be using a switch statement. You won't
> catch all values that are within .05% of (25.2 / 1.001).
No.
The clock rates you get ultimately come from the EDID via either the
detailed timing modes or from the CEA mode IDs, which are then looked
up in tables in the DRM EDID parsing code.
Either way, you will end up with 25175 and not 25170 or something
strange based on what the platform does.
--
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 6/9] drm: bridge/dw_hdmi: adjust pixel clock values in N calculation
Date: Sat, 5 Sep 2015 09:34:22 +0100 [thread overview]
Message-ID: <20150905083422.GL21084@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <CAD=FV=WR4iJ57FLh1d46-s9Gu-4wW6NdoROx-kNbbA0zFpUo0w@mail.gmail.com>
On Fri, Sep 04, 2015 at 07:03:11PM -0700, Doug Anderson wrote:
> Then perhaps you shouldn't be using a switch statement. You won't
> catch all values that are within .05% of (25.2 / 1.001).
No.
The clock rates you get ultimately come from the EDID via either the
detailed timing modes or from the CEA mode IDs, which are then looked
up in tables in the DRM EDID parsing code.
Either way, you will end up with 25175 and not 25170 or something
strange based on what the platform does.
--
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
next prev parent reply other threads:[~2015-09-05 8:34 UTC|newest]
Thread overview: 226+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-08 16:02 [PATCH 00/12] dw-hdmi development Russell King - ARM Linux
2015-08-08 16:02 ` Russell King - ARM Linux
2015-08-08 16:02 ` Russell King - ARM Linux
2015-08-08 16:03 ` [PATCH 01/12] drm: bridge/dw_hdmi: remove pixel repetition setting for all VICs Russell King
2015-08-08 16:03 ` Russell King
2015-08-08 16:03 ` [PATCH 02/12] drm: bridge/dw_hdmi: don't support any pixel doubled modes Russell King
2015-08-08 16:03 ` Russell King
2015-08-08 16:03 ` [PATCH 03/12] gpu: imx: simplify sync polarity setting Russell King
2015-08-08 16:03 ` Russell King
2015-10-06 17:57 ` Fabio Estevam
2015-10-06 17:57 ` Fabio Estevam
2015-10-06 17:57 ` Fabio Estevam
2015-08-08 16:03 ` [PATCH 04/12] gpu: imx: fix support for interlaced modes Russell King
2015-08-08 16:03 ` Russell King
2015-08-27 8:39 ` Philipp Zabel
2015-08-27 8:39 ` Philipp Zabel
2015-08-27 8:39 ` Philipp Zabel
2015-08-27 8:54 ` Russell King - ARM Linux
2015-08-27 8:54 ` Russell King - ARM Linux
2015-08-27 8:54 ` Russell King - ARM Linux
2015-08-27 9:40 ` Philipp Zabel
2015-08-27 9:40 ` Philipp Zabel
2015-08-27 9:40 ` Philipp Zabel
2015-10-06 17:59 ` Fabio Estevam
2015-10-06 17:59 ` Fabio Estevam
2015-10-06 17:59 ` Fabio Estevam
2015-08-08 16:03 ` [PATCH 05/12] drm: bridge/dw_hdmi: add support for interlaced video modes Russell King
2015-08-08 16:03 ` Russell King
2015-08-08 16:03 ` Russell King
2015-10-06 18:00 ` Fabio Estevam
2015-10-06 18:00 ` Fabio Estevam
2015-10-06 18:00 ` Fabio Estevam
2015-08-08 16:03 ` [PATCH 06/12] drm: bridge/dw_hdmi: clean up HDMI vs DVI mode handling Russell King
2015-08-08 16:03 ` Russell King
2015-10-07 3:40 ` Yakir Yang
2015-10-07 3:40 ` Yakir Yang
2015-10-07 3:40 ` Yakir Yang
2015-08-08 16:03 ` [PATCH 07/12] drm: bridge/dw_hdmi: enable audio only if sink supports audio Russell King
2015-08-08 16:03 ` Russell King
2015-10-07 3:42 ` Yakir Yang
2015-10-07 3:42 ` Yakir Yang
2015-10-07 3:42 ` Yakir Yang
2015-08-08 16:04 ` [PATCH 08/12] drm: bridge/dw_hdmi: avoid enabling interface in mode_set Russell King
2015-08-08 16:04 ` Russell King
2015-10-07 3:50 ` Yakir Yang
2015-10-07 3:50 ` Yakir Yang
2015-10-07 9:18 ` Russell King - ARM Linux
2015-10-07 9:18 ` Russell King - ARM Linux
2015-10-07 9:18 ` Russell King - ARM Linux
2015-10-07 9:35 ` Yakir Yang
2015-10-07 9:35 ` Yakir Yang
2015-10-07 9:35 ` Yakir Yang
2015-08-08 16:04 ` [PATCH 09/12] drm: bridge/dw_hdmi: rename dw_hdmi_phy_enable_power() Russell King
2015-08-08 16:04 ` Russell King
2015-08-08 16:04 ` Russell King
2015-10-07 4:00 ` Yakir Yang
2015-08-08 16:04 ` [PATCH 10/12] drm: bridge/dw_hdmi: fix phy enable/disable handling Russell King
2015-08-08 16:04 ` Russell King
2015-08-08 16:04 ` Russell King
2015-10-07 4:05 ` Yakir Yang
2015-10-07 9:48 ` Russell King - ARM Linux
2015-10-07 9:48 ` Russell King - ARM Linux
2015-10-07 9:48 ` Russell King - ARM Linux
2015-10-07 10:40 ` Yakir Yang
2015-10-07 19:17 ` Russell King - ARM Linux
2015-10-07 19:17 ` Russell King - ARM Linux
2015-10-07 19:17 ` Russell King - ARM Linux
2015-10-08 8:35 ` Yakir Yang
2015-10-08 8:35 ` Yakir Yang
2015-10-08 8:35 ` Yakir Yang
2015-08-08 16:04 ` [PATCH 11/12] drm: bridge/dw_hdmi: add connector mode forcing Russell King
2015-08-08 16:04 ` Russell King
2015-08-08 16:04 ` Russell King
2015-10-06 18:01 ` Fabio Estevam
2015-10-06 18:01 ` Fabio Estevam
2015-10-06 18:01 ` Fabio Estevam
2015-08-08 16:04 ` [PATCH 12/12] drm: bridge/dw_hdmi: improve HDMI enable/disable handling Russell King
2015-08-08 16:04 ` Russell King
2015-10-06 18:03 ` Fabio Estevam
2015-10-06 18:03 ` Fabio Estevam
2015-10-06 18:03 ` Fabio Estevam
2015-08-08 16:09 ` [PATCH 0/9] dw-hdmi audio support Russell King - ARM Linux
2015-08-08 16:09 ` Russell King - ARM Linux
2015-08-08 16:09 ` Russell King - ARM Linux
2015-08-08 16:10 ` [PATCH 1/9] drm: bridge/dw_hdmi-ahb-audio: add audio driver Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-08-10 10:05 ` Takashi Iwai
2015-08-10 10:05 ` Takashi Iwai
2015-08-10 10:05 ` Takashi Iwai
2015-08-10 10:39 ` Russell King - ARM Linux
2015-08-10 10:39 ` Russell King - ARM Linux
2015-08-10 10:39 ` Russell King - ARM Linux
2015-08-10 12:23 ` Takashi Iwai
2015-08-10 12:23 ` Takashi Iwai
2015-08-10 12:23 ` Takashi Iwai
2015-08-10 16:49 ` Russell King - ARM Linux
2015-08-10 16:49 ` Russell King - ARM Linux
2015-08-10 16:49 ` Russell King - ARM Linux
2015-08-10 18:16 ` Mark Brown
2015-08-10 18:16 ` Mark Brown
2015-08-14 13:54 ` [PATCH v2 1/9] drm: bridge/dw_hdmi-ahb-audio: add audio driver David Airlie <airlied@linux.ie>, Sascha Hauer <s.hauer@pengutronix.de>, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Jaroslav Kysela <perex@perex.cz>, linux-rockchip@lists.infradead.org, Mark Brown <broonie@kernel.org>, Philipp Zabel <p.zabel@pengutronix.de>, Yakir Yang <ykk@rock-chips.com>, Andy Yan <andy.yan@rock-chips.com>, Jon Nettleton <jon.nettleton@gmail.com>, linux-arm-kernel@lists.infradead.org Russell King
2015-08-14 13:54 ` Russell King
2015-08-14 13:54 ` Russell King
2015-08-14 14:04 ` [PATCH v2 1/9] drm: bridge/dw_hdmi-ahb-audio: add audio driver Russell King
2015-08-14 14:04 ` Russell King
2015-08-14 14:04 ` Russell King
2015-08-14 14:34 ` [alsa-devel] " Takashi Iwai
2015-08-14 14:34 ` Takashi Iwai
2015-08-14 14:34 ` Takashi Iwai
2015-10-06 18:07 ` [PATCH " Fabio Estevam
2015-10-06 18:07 ` Fabio Estevam
2015-10-06 18:07 ` Fabio Estevam
2015-10-06 18:18 ` Russell King - ARM Linux
2015-10-06 18:18 ` Russell King - ARM Linux
2015-10-06 18:18 ` Russell King - ARM Linux
2015-10-06 18:45 ` Fabio Estevam
2015-10-06 18:45 ` Fabio Estevam
2015-10-06 18:45 ` Fabio Estevam
2015-10-06 18:54 ` Russell King - ARM Linux
2015-10-06 18:54 ` Russell King - ARM Linux
2015-10-06 18:54 ` Russell King - ARM Linux
2015-10-06 20:25 ` Fabio Estevam
2015-10-06 20:25 ` Fabio Estevam
2015-10-06 20:25 ` Fabio Estevam
2015-10-09 16:00 ` Russell King - ARM Linux
2015-10-09 16:00 ` Russell King - ARM Linux
2015-10-09 16:00 ` Russell King - ARM Linux
2015-10-09 16:02 ` Fabio Estevam
2015-10-09 16:02 ` Fabio Estevam
2015-10-09 16:02 ` Fabio Estevam
2015-10-09 16:11 ` Russell King - ARM Linux
2015-10-09 16:11 ` Russell King - ARM Linux
2015-10-09 16:11 ` Russell King - ARM Linux
2015-08-08 16:10 ` [PATCH 2/9] drm: bridge/dw_hdmi-ahb-audio: parse ELD from HDMI driver Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` [PATCH 3/9] drm: bridge/dw_hdmi-ahb-audio: basic support for multi-channel PCM audio Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` [PATCH 4/9] drm: bridge/dw_hdmi-ahb-audio: allow larger buffer sizes Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` [PATCH 5/9] drm: bridge/dw_hdmi: avoid being recursive in N calculation Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-09-04 17:50 ` Doug Anderson
2015-09-04 17:50 ` Doug Anderson
2015-09-04 17:50 ` Doug Anderson
2015-08-08 16:10 ` [PATCH 6/9] drm: bridge/dw_hdmi: adjust pixel clock values " Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-09-04 18:21 ` Doug Anderson
2015-09-04 18:21 ` Doug Anderson
2015-09-04 18:21 ` Doug Anderson
2015-09-04 19:48 ` Doug Anderson
2015-09-04 19:48 ` Doug Anderson
2015-09-04 19:48 ` Doug Anderson
2015-09-04 21:24 ` Russell King - ARM Linux
2015-09-04 21:24 ` Russell King - ARM Linux
2015-09-04 21:24 ` Russell King - ARM Linux
2015-09-04 23:50 ` Doug Anderson
2015-09-04 23:50 ` Doug Anderson
2015-09-04 23:50 ` Doug Anderson
2015-09-05 0:27 ` Russell King - ARM Linux
2015-09-05 0:27 ` Russell King - ARM Linux
2015-09-05 0:27 ` Russell King - ARM Linux
2015-09-05 2:03 ` Doug Anderson
2015-09-05 2:03 ` Doug Anderson
2015-09-05 2:03 ` Doug Anderson
2015-09-05 8:31 ` Russell King - ARM Linux
2015-09-05 8:31 ` Russell King - ARM Linux
2015-09-05 8:31 ` Russell King - ARM Linux
2015-09-05 13:46 ` Doug Anderson
2015-09-05 13:46 ` Doug Anderson
2015-09-05 13:46 ` Doug Anderson
2015-09-05 14:01 ` Russell King - ARM Linux
2015-09-05 14:01 ` Russell King - ARM Linux
2015-09-05 14:01 ` Russell King - ARM Linux
2015-09-05 19:44 ` Doug Anderson
2015-09-05 19:44 ` Doug Anderson
2015-09-05 19:44 ` Doug Anderson
2015-09-05 8:34 ` Russell King - ARM Linux [this message]
2015-09-05 8:34 ` Russell King - ARM Linux
2015-09-05 8:34 ` Russell King - ARM Linux
2015-09-05 13:50 ` Doug Anderson
2015-09-05 13:50 ` Doug Anderson
2015-09-05 13:50 ` Doug Anderson
2015-08-08 16:10 ` [PATCH 7/9] drm: bridge/dw_hdmi: remove ratio support from ACR code Russell King
2015-08-08 16:10 ` Russell King
2015-08-08 16:10 ` Russell King
2015-09-04 18:24 ` Doug Anderson
2015-09-04 18:24 ` Doug Anderson
2015-09-04 18:24 ` Doug Anderson
2015-08-08 16:10 ` [PATCH 8/9] drm: bridge/dw_hdmi: replace CTS calculation for the ACR Russell King
2015-08-08 16:10 ` Russell King
2015-09-04 20:00 ` Doug Anderson
2015-09-04 20:00 ` Doug Anderson
2015-09-04 20:00 ` Doug Anderson
2015-08-08 16:10 ` [PATCH 9/9] drm: bridge/dw_hdmi-i2s-audio: add audio driver Russell King
2015-08-08 16:10 ` Russell King
2015-08-10 15:48 ` Russell King - ARM Linux
2015-08-10 15:48 ` Russell King - ARM Linux
2015-08-10 15:48 ` Russell King - ARM Linux
2015-08-10 16:26 ` Yakir Yang
2015-08-10 16:26 ` Yakir Yang
2015-08-10 16:26 ` Yakir Yang
2015-08-27 8:42 ` [PATCH 0/9] dw-hdmi audio support Philipp Zabel
2015-08-27 8:42 ` Philipp Zabel
2015-08-27 8:42 ` Philipp Zabel
2016-01-05 15:40 ` [alsa-devel] " Jean-Michel Hautbois
2016-01-05 15:40 ` Jean-Michel Hautbois
2016-01-05 15:54 ` Fabio Estevam
2016-01-05 15:54 ` Fabio Estevam
2016-01-05 15:54 ` Fabio Estevam
2016-01-05 16:04 ` Russell King - ARM Linux
2016-01-05 16:04 ` Russell King - ARM Linux
2016-01-05 16:04 ` Russell King - ARM Linux
2016-01-07 8:21 ` Jean-Michel Hautbois
2016-01-07 8:21 ` Jean-Michel Hautbois
2016-01-07 8:21 ` Jean-Michel Hautbois
2015-08-10 12:21 ` [PATCH 00/12] dw-hdmi development Thierry Reding
2015-08-10 12:21 ` Thierry Reding
2015-08-10 12:21 ` Thierry Reding
2015-08-18 10:37 ` Russell King - ARM Linux
2015-08-18 10:37 ` Russell King - ARM Linux
2015-08-18 10:37 ` Russell King - ARM Linux
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150905083422.GL21084@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=airlied@linux.ie \
--cc=alsa-devel@alsa-project.org \
--cc=andy.yan@rock-chips.com \
--cc=broonie@kernel.org \
--cc=dianders@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=fabio.estevam@freescale.com \
--cc=jon.nettleton@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=p.zabel@pengutronix.de \
--cc=perex@perex.cz \
--cc=s.hauer@pengutronix.de \
--cc=tiwai@suse.de \
--cc=ykk@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.