On Mon, Dec 30, 2013 at 4:15 PM, Stephen Boyd <sboyd(a)codeaurora.org> wrote:
> This file doesn't use the clk_reset() API that is exposed in
> mach-msm's mach/clk.h file. Remove the include so that this
> driver can be compiled as part of the multi-platform kernel.
Thanks!
Signed-off-by: Rob Clark <robdclark(a)gmail.com>
> Cc: Rob Clark <robdclark(a)gmail.com>
> Cc: Dave Airlie <airlied(a)redhat.com>
> ---
> drivers/gpu/drm/msm/mdp4/…
[View More]mdp4_dtv_encoder.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/mdp4/mdp4_dtv_encoder.c b/drivers/gpu/drm/msm/mdp4/mdp4_dtv_encoder.c
> index 5e0dcae..3799ccc 100644
> --- a/drivers/gpu/drm/msm/mdp4/mdp4_dtv_encoder.c
> +++ b/drivers/gpu/drm/msm/mdp4/mdp4_dtv_encoder.c
> @@ -15,8 +15,6 @@
> * this program. If not, see <http://www.gnu.org/licenses/>.
> */
>
> -#include <mach/clk.h>
> -
> #include "mdp4_kms.h"
>
> #include "drm_crtc.h"
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> hosted by The Linux Foundation
>
[View Less]
I was reminded on Monday that we'd like to be able to subclass struct
drm_device so we don't have to always juggle between a pointer to a struct
drm_device and its drv_private field to access everything we care about.
It'd be nicer for those two pointers to have the same value to avoid a few
extra instructions pretty everywhere.
Turns out, there's a cheap way to go towards that goal, just make DRM core
allocate enough space for drivers that want to subclass struct drm_device and
have a …
[View More]transition path where dev_private points to the start of the structure.
I've converted i915 to do just that, seems to still boot. Of course, a number
of nice refactoring are possible once dev == dev_priv, but that'd be for follow
up patches on top.
Thoughts?
--
Damien
[View Less]
The S/PDIF input asks for a greater audio clock divider.
Signed-off-by: Jean-Francois Moine <moinejf(a)free.fr>
---
drivers/gpu/drm/i2c/tda998x_drv.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
index d3b3f3a..67d7450 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
+++ b/drivers/gpu/drm/i2c/tda998x_drv.c
@@ -757,6 +757,10 @@ tda998x_configure_audio(struct tda998x_priv *priv,
…
[View More]AIP_CNTRL_0_ACR_MAN);
reg_write(priv, REG_CTS_N, cts_n);
+ /* S/PDIF asks for a large divider */
+ if (priv->audio_type == (AFMT_SPDIF | AUDIO_ACTIVE))
+ adiv = AUDIO_DIV_SERCLK_32;
+
/*
* Audio input somehow depends on HDMI line rate which is
* related to pixclk. Testing showed that modes with pixclk
@@ -764,7 +768,7 @@ tda998x_configure_audio(struct tda998x_priv *priv,
* There is no detailed info in the datasheet, so we just
* assume 100MHz requires larger divider.
*/
- if (mode->clock > 100000)
+ else if (mode->clock > 100000)
adiv = AUDIO_DIV_SERCLK_16;
else
adiv = AUDIO_DIV_SERCLK_8;
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
[View Less]
This patch fixes the ENABLE_SPACE register, the value of which was
inverted.
Signed-off-by: Jean-Francois Moine <moinejf(a)free.fr>
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c
index 8a7d8a2..192ddd2 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
+++ b/drivers/gpu/drm/i2c/tda998x_drv.c
@@ -1069,7 +1069,7 @@ tda998x_encoder_mode_set(struct drm_encoder *…
[View More]encoder,
if (priv->rev == TDA19988) {
/* let incoming pixels fill the active space (if any) */
- reg_write(priv, REG_ENABLE_SPACE, 0x01);
+ reg_write(priv, REG_ENABLE_SPACE, 0x00);
}
}
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
[View Less]
https://bugzilla.kernel.org/show_bug.cgi?id=60858
Bug ID: 60858
Summary: [radeon HD 4570m] Error setting UVD clocks
Product: Drivers
Version: 2.5
Kernel Version: 3.11
Hardware: x86-64
OS: Linux
Tree: Mainline
Status: NEW
Severity: normal
Priority: P1
Component: Video(DRI - non Intel)
Assignee: drivers_video-dri(a)kernel-bugs.osdl.org
Reporter: …
[View More]pinak.ahuja(a)gmail.com
Regression: No
Created attachment 107431
--> https://bugzilla.kernel.org/attachment.cgi?id=107431&action=edit
generated using $dmesg > dmesg.txt
I'm using the dell studio 1555 laptop with radeon HD 4570m graphic card.
Running the stable 3.11 kernel with the latest linux-firmware files. I am not
able to play any video using VDPAU regardless of the codec. During boot up
there are a few errors regarding setting UVD clocks
[ 8.936898] [drm:radeon_uvd_send_upll_ctlreq] *ERROR* Timeout setting UVD
clocks!
[ 9.983683] [drm:radeon_uvd_send_upll_ctlreq] *ERROR* Timeout setting UVD
clocks!
[ 9.983695] [drm:r600_uvd_ib_test] *ERROR* radeon: failed to raise UVD
clocks (-110).
I've attached the full dmesg.
--
You are receiving this mail because:
You are watching the assignee of the bug.
[View Less]