[PATCH] drm/radeon: remove overzealous warning in hdmi handling

2013-02-12 Thread Jerome Glisse 
On Tue, Feb 12, 2013 at 8:40 AM,   wrote:
> From: Alex Deucher 
>
> hdmi audio works fine.  The warning just confuses users.
>
> fixes:
> https://bugzilla.kernel.org/show_bug.cgi?id=44341
>
> Signed-off-by: Alex Deucher 
Reviewed-by: Jerome Glisse 
> Cc: stable at vger.kernel.org
> ---
>  drivers/gpu/drm/radeon/r600_hdmi.c |1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/r600_hdmi.c 
> b/drivers/gpu/drm/radeon/r600_hdmi.c
> index ff80efe..95970ec 100644
> --- a/drivers/gpu/drm/radeon/r600_hdmi.c
> +++ b/drivers/gpu/drm/radeon/r600_hdmi.c
> @@ -544,7 +544,6 @@ void r600_hdmi_disable(struct drm_encoder *encoder)
>
> /* Called for ATOM_ENCODER_MODE_HDMI only */
> if (!dig || !dig->afmt) {
> -   WARN_ON(1);
> return;
> }
> if (!dig->afmt->enabled)
> --
> 1.7.7.5
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH] drm/radeon: remove overzealous warning in hdmi handling

2013-02-12 Thread alexdeuc...@gmail.com 
From: Alex Deucher 

hdmi audio works fine.  The warning just confuses users.

fixes:
https://bugzilla.kernel.org/show_bug.cgi?id=44341

Signed-off-by: Alex Deucher 
Cc: stable at vger.kernel.org
---
 drivers/gpu/drm/radeon/r600_hdmi.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/drm/radeon/r600_hdmi.c 
b/drivers/gpu/drm/radeon/r600_hdmi.c
index ff80efe..95970ec 100644
--- a/drivers/gpu/drm/radeon/r600_hdmi.c
+++ b/drivers/gpu/drm/radeon/r600_hdmi.c
@@ -544,7 +544,6 @@ void r600_hdmi_disable(struct drm_encoder *encoder)

/* Called for ATOM_ENCODER_MODE_HDMI only */
if (!dig || !dig->afmt) {
-   WARN_ON(1);
return;
}
if (!dig->afmt->enabled)
-- 
1.7.7.5

[PATCH] drm/radeon: remove overzealous warning in hdmi handling

2013-02-12 Thread alexdeucher 
From: Alex Deucher alexander.deuc...@amd.com

hdmi audio works fine.  The warning just confuses users.

fixes:
https://bugzilla.kernel.org/show_bug.cgi?id=44341

Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Cc: sta...@vger.kernel.org
---
 drivers/gpu/drm/radeon/r600_hdmi.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/drm/radeon/r600_hdmi.c 
b/drivers/gpu/drm/radeon/r600_hdmi.c
index ff80efe..95970ec 100644
--- a/drivers/gpu/drm/radeon/r600_hdmi.c
+++ b/drivers/gpu/drm/radeon/r600_hdmi.c
@@ -544,7 +544,6 @@ void r600_hdmi_disable(struct drm_encoder *encoder)
 
/* Called for ATOM_ENCODER_MODE_HDMI only */
if (!dig || !dig-afmt) {
-   WARN_ON(1);
return;
}
if (!dig-afmt-enabled)
-- 
1.7.7.5

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] drm/radeon: remove overzealous warning in hdmi handling

2013-02-12 Thread Jerome Glisse 
On Tue, Feb 12, 2013 at 8:40 AM,  alexdeuc...@gmail.com wrote:
 From: Alex Deucher alexander.deuc...@amd.com

 hdmi audio works fine.  The warning just confuses users.

 fixes:
 https://bugzilla.kernel.org/show_bug.cgi?id=44341

 Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Reviewed-by: Jerome Glisse jgli...@redhat.com
 Cc: sta...@vger.kernel.org
 ---
  drivers/gpu/drm/radeon/r600_hdmi.c |1 -
  1 files changed, 0 insertions(+), 1 deletions(-)

 diff --git a/drivers/gpu/drm/radeon/r600_hdmi.c 
 b/drivers/gpu/drm/radeon/r600_hdmi.c
 index ff80efe..95970ec 100644
 --- a/drivers/gpu/drm/radeon/r600_hdmi.c
 +++ b/drivers/gpu/drm/radeon/r600_hdmi.c
 @@ -544,7 +544,6 @@ void r600_hdmi_disable(struct drm_encoder *encoder)

 /* Called for ATOM_ENCODER_MODE_HDMI only */
 if (!dig || !dig-afmt) {
 -   WARN_ON(1);
 return;
 }
 if (!dig-afmt-enabled)
 --
 1.7.7.5

 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel