date:20130613

[Bug 65730] no sound via HDMI for Radeon 6850 mesa driver

2013-06-13 Thread bugzilla-dae...@freedesktop.org

https://bugs.freedesktop.org/show_bug.cgi?id=65730

--- Comment #3 from Alex Deucher  ---
Created attachment 80801
  --> https://bugs.freedesktop.org/attachment.cgi?id=80801=edit
fix AVI infoframe

Does the attached kernel patch along with the radeon.audio=1 help?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/59007bb4/attachment.html>

[Bug 65730] no sound via HDMI for Radeon 6850 mesa driver

2013-06-13 Thread bugzilla-dae...@freedesktop.org

https://bugs.freedesktop.org/show_bug.cgi?id=65730

--- Comment #2 from Jason  ---
Indeed I had already tried that in my /etc/default/grub file.  I edited both
lines and rebooted...
GRUB_CMDLINE_LINUX_DEFAULT="quiet splash radeon.audio=1"
GRUB_CMDLINE_LINUX="radeon.audio=1"

Sound does not work.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/09d1e03e/attachment.html>

[PATCH] drm/edid: Don't print messages regarding stereo or csync by default

2013-06-13 Thread Egbert Eich

drm_mode_detailed() is called quite often, therefore when a monitor
that has a detailed timing mode marked DRM_EDID_PT_STEREO or requiring
composite sync, warning messages will clutter up the kernel log.
Like we already do for incorrect hsync/vsync pluse widths, print these
messages only when KMS debugging is enabled.

Signed-off-by: Egbert Eich 
---
 drivers/gpu/drm/drm_edid.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index e8d17ee..2dc1a60 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -1706,11 +1706,11 @@ static struct drm_display_mode 
*drm_mode_detailed(struct drm_device *dev,
return NULL;

if (pt->misc & DRM_EDID_PT_STEREO) {
-   printk(KERN_WARNING "stereo mode not supported\n");
+   DRM_DEBUG_KMS("stereo mode not supported\n");
return NULL;
}
if (!(pt->misc & DRM_EDID_PT_SEPARATE_SYNC)) {
-   printk(KERN_WARNING "composite sync not supported\n");
+   DRM_DEBUG_KMS("composite sync not supported\n");
}

/* it is incorrect if hsync/vsync width is zero */
-- 
1.8.1.4

[PATCH 1/1] dRM/exynos: exynos_drm_ipp: Remove redundant break statement

2013-06-13 Thread Inki Dae

Applied.

Thanks,
Inki Dae


2013/6/11 Sachin Kamat 

> 'break' after goto statement is redundant. Silences the following
> message:
> drivers/gpu/drm/exynos/exynos_drm_ipp.c:1067 exynos_drm_ipp_check_valid()
> info: ignoring unreachable code.
>
> Signed-off-by: Sachin Kamat 
> ---
>  drivers/gpu/drm/exynos/exynos_drm_ipp.c |1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_ipp.c
> b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
> index 01cb9a0..b1ef8e7 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_ipp.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_ipp.c
> @@ -1064,7 +1064,6 @@ static bool exynos_drm_ipp_check_valid(struct device
> *dev,
> default:
> DRM_ERROR("invalid state.\n");
> goto err_status;
> -   break;
> }
>
> return true;
> --
> 1.7.9.5
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-- next part ------
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/68b0094e/attachment.html>

[PATCH 4/4] ARM/dts: add clocks to hdmi dt node for exynos5250

2013-06-13 Thread Inki Dae

Acked-by: Inki Dae 

Thanks,
Inki Dae


2013/6/11 Rahul Sharma 

> Fix wrong clock numbers in hdmi dt node. Removed hdmiphy
> clock which was a dummy clock earlier and not required now.
> Also added mux clock to change the clock parent.
>
> Signed-off-by: Rahul Sharma 
> ---
>  arch/arm/boot/dts/exynos5250.dtsi |6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos5250.dtsi
> b/arch/arm/boot/dts/exynos5250.dtsi
> index 9d9496a..354e14a 100644
> --- a/arch/arm/boot/dts/exynos5250.dtsi
> +++ b/arch/arm/boot/dts/exynos5250.dtsi
> @@ -604,10 +604,10 @@
> compatible = "samsung,exynos5-hdmi";
> reg = <0x1453 0x7>;
> interrupts = <0 95 0>;
> -   clocks = < 333>, < 136>, < 137>,
> -   < 333>, < 333>;
> +   clocks = < 344>, < 136>, < 137>,
> +   < 157>, < 1024>;
> clock-names = "hdmi", "sclk_hdmi", "sclk_pixel",
> -   "sclk_hdmiphy", "hdmiphy";
> +   "sclk_hdmiphy", "mout_hdmi";
> };
>
> mixer {
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-samsung-soc" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
-- next part --
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/ff56b09d/attachment.html>

[PATCH v2 3/4] gpu: host1x: add runtime pm support for dc

2013-06-13 Thread Thierry Reding

On Thu, Jun 13, 2013 at 03:23:37PM +0530, Mayuresh Kulkarni wrote:
[...]
> diff --git a/drivers/gpu/host1x/drm/dc.c b/drivers/gpu/host1x/drm/dc.c
[...]
> @@ -1128,9 +1129,7 @@ static int tegra_dc_probe(struct platform_device *pdev)
>   return PTR_ERR(dc->clk);
>   }
>  
> - err = clk_prepare_enable(dc->clk);
> - if (err < 0)
> - return err;
> + platform_set_drvdata(pdev, dc);

Why do you move the call to platform_set_drvdata() up here?

Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/f5c476e6/attachment.pgp>

[PATCH 3/4] ARM: dts: Add mixer clocks to mixer node

2013-06-13 Thread Inki Dae

Acked-by: Inki Dae 

Thanks,
Inki Dae


2013/6/11 Rahul Sharma 

> From: Sean Paul 
>
> This patch adds the mixer clocks to the mixer node in the dts file.
>
> Signed-off-by: Sean Paul 
> Signed-off-by: Rahul Sharma 
> ---
>  arch/arm/boot/dts/exynos5250.dtsi |2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5250.dtsi
> b/arch/arm/boot/dts/exynos5250.dtsi
> index 0673524..9d9496a 100644
> --- a/arch/arm/boot/dts/exynos5250.dtsi
> +++ b/arch/arm/boot/dts/exynos5250.dtsi
> @@ -614,6 +614,8 @@
> compatible = "samsung,exynos5-mixer";
> reg = <0x1445 0x1>;
> interrupts = <0 94 0>;
> +   clocks = < 343>, < 136>;
> +   clock-names = "mixer", "sclk_hdmi";
> };
>
> dp-controller {
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-samsung-soc" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
-- next part --
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/47877026/attachment.html>

[PATCH 2/4] drm/exynos: add mout_hdmi clock in hdmi driver to change parent

2013-06-13 Thread Inki Dae

Applied.

Thanks,
Inki Dae

> -Original Message-
> From: Rahul Sharma [mailto:rahul.sharma at samsung.com]
> Sent: Tuesday, June 11, 2013 3:54 PM
> To: linux-samsung-soc at vger.kernel.org;
devicetree-discuss at lists.ozlabs.org;
> dri-devel at lists.freedesktop.org
> Cc: kgene.kim at samsung.com; sw0312.kim at samsung.com; inki.dae at 
> samsung.com;
> seanpaul at chromium.org; joshi at samsung.com; r.sh.open at gmail.com; Rahul
> Sharma
> Subject: [PATCH 2/4] drm/exynos: add mout_hdmi clock in hdmi driver to
> change parent
> 
> HDMI driver needs to configure the mout_hdmi mux clock to change
> the parent between sclk_hdmiphy and sclk_pixel.
> 
> Signed-off-by: Rahul Sharma 
> ---
>  drivers/gpu/drm/exynos/exynos_hdmi.c |   14 ++
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c
> b/drivers/gpu/drm/exynos/exynos_hdmi.c
> index 5a98194..3b5e215 100644
> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c
> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
> @@ -83,6 +83,7 @@ struct hdmi_resources {
>   struct clk  *sclk_pixel;
>   struct clk  *sclk_hdmiphy;
>   struct clk  *hdmiphy;
> + struct clk  *mout_hdmi;
>   struct regulator_bulk_data  *regul_bulk;
>   int regul_count;
>  };
> @@ -1118,7 +1119,7 @@ static void hdmi_v13_mode_apply(struct hdmi_context
> *hdata)
>   }
> 
>   clk_disable_unprepare(hdata->res.sclk_hdmi);
> - clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_hdmiphy);
> + clk_set_parent(hdata->res.mout_hdmi, hdata->res.sclk_hdmiphy);
>   clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* enable HDMI and timing generator */
> @@ -1285,7 +1286,7 @@ static void hdmi_v14_mode_apply(struct hdmi_context
> *hdata)
>   }
> 
>   clk_disable_unprepare(hdata->res.sclk_hdmi);
> - clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_hdmiphy);
> + clk_set_parent(hdata->res.mout_hdmi, hdata->res.sclk_hdmiphy);
>   clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* enable HDMI and timing generator */
> @@ -1311,7 +1312,7 @@ static void hdmiphy_conf_reset(struct hdmi_context
> *hdata)
>   u32 reg;
> 
>   clk_disable_unprepare(hdata->res.sclk_hdmi);
> - clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_pixel);
> + clk_set_parent(hdata->res.mout_hdmi, hdata->res.sclk_pixel);
>   clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* operation mode */
> @@ -1832,8 +1833,13 @@ static int hdmi_resources_init(struct hdmi_context
> *hdata)
>   DRM_ERROR("failed to get clock 'hdmiphy'\n");
>   goto fail;
>   }
> + res->mout_hdmi = devm_clk_get(dev, "mout_hdmi");
> + if (IS_ERR(res->mout_hdmi)) {
> + DRM_ERROR("failed to get clock 'mout_hdmi'\n");
> + goto fail;
> + }
> 
> - clk_set_parent(res->sclk_hdmi, res->sclk_pixel);
> + clk_set_parent(res->mout_hdmi, res->sclk_pixel);
> 
>   res->regul_bulk = devm_kzalloc(dev, ARRAY_SIZE(supply) *
>   sizeof(res->regul_bulk[0]), GFP_KERNEL);
> --
> 1.7.10.4

[PATCH 1/4] drm/exynos: Prepare/Unprepare HDMI subsystem clocks

2013-06-13 Thread Inki Dae

Applied.

Thanks,
Inki Dae

> -Original Message-
> From: Rahul Sharma [mailto:rahul.sharma at samsung.com]
> Sent: Tuesday, June 11, 2013 3:54 PM
> To: linux-samsung-soc at vger.kernel.org;
devicetree-discuss at lists.ozlabs.org;
> dri-devel at lists.freedesktop.org
> Cc: kgene.kim at samsung.com; sw0312.kim at samsung.com; inki.dae at 
> samsung.com;
> seanpaul at chromium.org; joshi at samsung.com; r.sh.open at gmail.com; Rahul
> Sharma
> Subject: [PATCH 1/4] drm/exynos: Prepare/Unprepare HDMI subsystem clocks
> 
> From: Sean Paul 
> 
> Change the clk_enable/clk_disable calls in mixer and hdmi drivers into
> clk_prepare_enable/clk_disable_unprepare, respectively.
> 
> Signed-off-by: Sean Paul 
> Signed-off-by: Rahul Sharma 
> ---
>  drivers/gpu/drm/exynos/exynos_hdmi.c  |   24 
>  drivers/gpu/drm/exynos/exynos_mixer.c |   12 ++--
>  2 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c
> b/drivers/gpu/drm/exynos/exynos_hdmi.c
> index 04255fe..5a98194 100644
> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c
> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
> @@ -1117,9 +1117,9 @@ static void hdmi_v13_mode_apply(struct hdmi_context
> *hdata)
>   hdmi_regs_dump(hdata, "timing apply");
>   }
> 
> - clk_disable(hdata->res.sclk_hdmi);
> + clk_disable_unprepare(hdata->res.sclk_hdmi);
>   clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_hdmiphy);
> - clk_enable(hdata->res.sclk_hdmi);
> + clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* enable HDMI and timing generator */
>   hdmi_reg_writemask(hdata, HDMI_CON_0, ~0, HDMI_EN);
> @@ -1284,9 +1284,9 @@ static void hdmi_v14_mode_apply(struct hdmi_context
> *hdata)
>   hdmi_regs_dump(hdata, "timing apply");
>   }
> 
> - clk_disable(hdata->res.sclk_hdmi);
> + clk_disable_unprepare(hdata->res.sclk_hdmi);
>   clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_hdmiphy);
> - clk_enable(hdata->res.sclk_hdmi);
> + clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* enable HDMI and timing generator */
>   hdmi_reg_writemask(hdata, HDMI_CON_0, ~0, HDMI_EN);
> @@ -1310,9 +1310,9 @@ static void hdmiphy_conf_reset(struct hdmi_context
> *hdata)
>   u8 buffer[2];
>   u32 reg;
> 
> - clk_disable(hdata->res.sclk_hdmi);
> + clk_disable_unprepare(hdata->res.sclk_hdmi);
>   clk_set_parent(hdata->res.sclk_hdmi, hdata->res.sclk_pixel);
> - clk_enable(hdata->res.sclk_hdmi);
> + clk_prepare_enable(hdata->res.sclk_hdmi);
> 
>   /* operation mode */
>   buffer[0] = 0x1f;
> @@ -1701,9 +1701,9 @@ static void hdmi_poweron(struct hdmi_context *hdata)
>   if (regulator_bulk_enable(res->regul_count, res->regul_bulk))
>   DRM_DEBUG_KMS("failed to enable regulator bulk\n");
> 
> - clk_enable(res->hdmiphy);
> - clk_enable(res->hdmi);
> - clk_enable(res->sclk_hdmi);
> + clk_prepare_enable(res->hdmiphy);
> + clk_prepare_enable(res->hdmi);
> + clk_prepare_enable(res->sclk_hdmi);
> 
>   hdmiphy_poweron(hdata);
>  }
> @@ -1726,9 +1726,9 @@ static void hdmi_poweroff(struct hdmi_context
*hdata)
>   hdmiphy_conf_reset(hdata);
>   hdmiphy_poweroff(hdata);
> 
> - clk_disable(res->sclk_hdmi);
> - clk_disable(res->hdmi);
> - clk_disable(res->hdmiphy);
> + clk_disable_unprepare(res->sclk_hdmi);
> + clk_disable_unprepare(res->hdmi);
> + clk_disable_unprepare(res->hdmiphy);
>   regulator_bulk_disable(res->regul_count, res->regul_bulk);
> 
>   mutex_lock(>hdmi_mutex);
> diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c
> b/drivers/gpu/drm/exynos/exynos_mixer.c
> index b0882b3..978894e 100644
> --- a/drivers/gpu/drm/exynos/exynos_mixer.c
> +++ b/drivers/gpu/drm/exynos/exynos_mixer.c
> @@ -900,10 +900,10 @@ static void mixer_poweron(struct mixer_context *ctx)
>   ctx->powered = true;
>   mutex_unlock(>mixer_mutex);
> 
> - clk_enable(res->mixer);
> + clk_prepare_enable(res->mixer);
>   if (ctx->vp_enabled) {
> - clk_enable(res->vp);
> - clk_enable(res->sclk_mixer);
> + clk_prepare_enable(res->vp);
> + clk_prepare_enable(res->sclk_mixer);
>   }
> 
>   mixer_reg_write(res, MXR_INT_EN, ctx->int_en);
> @@ -927,10 +927,10 @@ static void mixer_poweroff(struct mixer_context
*ctx)
> 
>   ctx->int_en = mixer_reg_read(res, MXR_INT_EN);
> 
> - clk_disable(res->mixer);
> + clk_disable_unprepare(res->mixer);
>   if (ctx->vp_enabled) {
> - clk_disable(res->vp);
> - clk_disable(res->sclk_mixer);
> + clk_disable_unprepare(res->vp);
> + clk_disable_unprepare(res->sclk_mixer);
>   }
> 
>   mutex_lock(>mixer_mutex);
> --
> 1.7.10.4

[Bug 65730] no sound via HDMI for Radeon 6850 mesa driver

2013-06-13 Thread bugzilla-dae...@freedesktop.org

https://bugs.freedesktop.org/show_bug.cgi?id=65730

--- Comment #1 from Alex Deucher  ---
You need to boot with radeon.audio=1 on the kernel command line in grub to
enable audio.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL: 
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/89777ca2/attachment.html>

[RFC PATCH] dmabuf-sync: Introduce buffer synchronization framework

2013-06-13 Thread Inki Dae


> +static void dmabuf_sync_timeout_worker(struct work_struct *work)
> +{
> + struct dmabuf_sync *sync = container_of(work, struct dmabuf_sync,
> work);
> + struct dmabuf_sync_object *sobj;
> +
> + mutex_lock(>lock);
> +
> + list_for_each_entry(sobj, >syncs, head) {
> + if (WARN_ON(!sobj->robj))
> + continue;
> +
> + printk(KERN_WARNING "%s: timeout = 0x%x [type = %d, " \
> + "refcnt = %d, locked = %d]\n",
> + sync->name, (u32)sobj->dmabuf,
> + sobj->access_type,
> +
atomic_read(>robj->shared_cnt),
> + sobj->robj->locked);
> +
> + /* unlock only valid sync object. */
> + if (!sobj->robj->locked)
> + continue;
> +
> + if (sobj->robj->shared &&
> + atomic_read(>robj->shared_cnt) > 1) {
> + atomic_dec(>robj->shared_cnt);
> + continue;
> + }
> +
> + ww_mutex_unlock(>robj->lock);
> +
> + if (sobj->access_type & DMA_BUF_ACCESS_READ)
> + printk(KERN_WARNING "%s: r-unlocked = 0x%x\n",
> + sync->name, (u32)sobj->dmabuf);
> + else
> + printk(KERN_WARNING "%s: w-unlocked = 0x%x\n",
> + sync->name, (u32)sobj->dmabuf);
> +
> +#if defined(CONFIG_DEBUG_FS)
> + sync_debugfs_timeout_cnt++;
> +#endif

Oops, unnecessary codes. will remove them.

[RFC 0/2] exynos5250/hdmi: replace dummy hdmiphy clock with pmu reg control

2013-06-13 Thread Inki Dae



> -Original Message-
> From: Sylwester Nawrocki [mailto:s.nawrocki at samsung.com]
> Sent: Thursday, June 13, 2013 5:56 PM
> To: Rahul Sharma
> Cc: Rahul Sharma; Inki Dae; linux-samsung-soc at vger.kernel.org; devicetree-
> discuss at lists.ozlabs.org; DRI mailing list; Kukjin Kim; Seung-Woo Kim;
> Sean Paul; sunil joshi; Kishon Vijay Abraham I; Stephen Warren;
> grant.likely at linaro.org
> Subject: Re: [RFC 0/2] exynos5250/hdmi: replace dummy hdmiphy clock with
> pmu reg control
> 
> Hi,
> 
> On 06/13/2013 06:26 AM, Rahul Sharma wrote:
> > Mr. Dae,
> >
> > Thanks for your valuable inputs.
> >
> > I posted it as RFC because, I also have received comments to register
> > hdmiphy as a clock controller. As we always configure it for specific
> > frequency, hdmi-phy looks similar to a PLL. But it really doesn't
> > belong to that class. Secondly prior to exynos5420, it was a i2c
> > device. I am not sure we can register a I2C device as a clock
> > controller. I wanted to discuss and explore this option here.
> 
> Have you considered using the generic PHY framework for those HDMI
> PHY devices [1] ? I guess we could add a dedicated group of ops for
> video PHYs, similarly as is is done with struct v4l2_subdev_ops. For
> configuring things like the carrier/pixel clock frequency or anything
> what's common across the video PHYs.
> 
> Perhaps you could have a look and see if this framework would be
> useful for HDMI and possibly point out anything what might be missing ?
> 
> I'm not sure it it really solves the issues specific to the Exynos
> HDMI but at least with a generic PHY driver the PHY module would be
> separate from the PHY controller, as often same HDMI DPHY can be used
> with various types of a HDMI controller. So this would allow to not
> duplicate the HDMI PHY drivers in the long-term perspective.

Yeah, at least, it seems that we could use PHY module to control PMU
register, HDMI_PHY_CONTROL. However, PHY module provides only init/on/off
callbacks. As you may know, HDMIPHY needs i2c interfaces to control HDMIPHY
clock. So with PHY module, HDMIPHY driver could enable PMU more generically,
but also has to use existing i2c stuff to control HDMIPHY clock. I had a
quick review to Generic PHY Framework[v6] but I didn't see that the PHY
module could generically support more features such as i2c stuff.

Thanks,
Inki Dae

> 
> [1] https://lkml.org/lkml/2013/4/29/95
> 
> Thanks,
> Sylwester
> 
> > As you said, in parallel, I will align these changes and along with
> > "drm/exynos: hdmi: move hdmiphy related code to hdmiphy driver"
> > series and post them.
> >
> > I hope we should be able to close on one of the above approaches for
> > hdmiphy.
> >
> > regards,
> > Rahul Sharma.
> >
> > On Wed, Jun 12, 2013 at 9:57 AM, Inki Dae  wrote:
> >>
> >> 2013/6/12 Inki Dae 
> >>>
> >>> Hi Rahul,
> >>>
> >>> This patch is important to us. Actually, previous hdmi driver had
> >>> controlled hdmiphy HDMI_PHY_CONTROL as if that were a clock but now
> that
> >>> doesn't exist anymore. So we need to discuss how hdmiphy should be
> handled.
> >>> I konw that you had already posted hdmiphy relevant patch set, [PATCH
> 0/4]
> >>> drm/exynos: hdmi: move hdmiphy related code to hdmiphy driver.
> >>>
> >>> I think we can couple pmu register controlling codes with that patch
> set
> >>> without RFC. Could you update and post them again? like below,
> >>> [PATCH 0/4] drm/exynos: hdmi: move hdmiphy related code to hdmiphy
> driver
> >>> + [RFC 0/2] exynos5250/hdmi: replace dummy hdmiphy clock with pmu reg
> >>> control
> >>>
> >>> And then let's start review :)
> >>
> >> And I think It would be better to move the pmu register controlling
> codes
> >> into hdmiphy driver like drivers/usb/phy/phy-samsung-usb2.c driver
does.
> >>>
> >>> Thanks,
> >>> Inki Dae
> >>>
> >>> 2013/6/11 Rahul Sharma 
> 
>  Previously, hdmiphy is added as a dummy clock in clock file for
>  exynos SoCs. Enable/Disable to this clock, actually toggles the power
>  control bit in PMU, instead of controlling the clock gate.
> 
>  This RFC adds the support to parse hdmiphy control node which is a
> child
>  node to hdmi, and map the pmu register to toggle the power control
> bit.
> 
>  This is based on drm-next branch in Inki Dae's tree.
> 
>  Rahul Sharma (2):
>    drm/exynos: replace dummy hdmiphy clock with pmu register control
>    ARM/dts: add hdmiphy power control pmu register to hdmi dt node
> 
>   arch/arm/boot/dts/exynos5250.dtsi|6 +++
>   drivers/gpu/drm/exynos/exynos_hdmi.c |   69
>  ++
>   drivers/gpu/drm/exynos/regs-hdmi.h   |4 ++
>   3 files changed, 71 insertions(+), 8 deletions(-)
> 
>  --
>  1.7.10.4
> 
> --
> Sylwester Nawrocki
> Samsung R Institute Poland
> Samsung Electronics

[Bug 65730] New: no sound via HDMI for Radeon 6850 mesa driver

2013-06-13 Thread bugzilla-dae...@freedesktop.org

https://bugs.freedesktop.org/show_bug.cgi?id=65730

Priority: medium
Bug ID: 65730
Assignee: dri-devel at lists.freedesktop.org
Summary: no sound via HDMI for Radeon 6850 mesa driver
Severity: normal
Classification: Unclassified
OS: Linux (All)
Reporter: jasontheparamedic at hotmail.com
Hardware: x86-64 (AMD64)
Status: NEW
Version: unspecified
Component: Drivers/DRI/R600
Product: Mesa

Greetings, this is my first bug report so I apologize if it is placed in the
wrong section. I am unable to get sound via HDMI cable using the open source
driver for my Radeon 6850. I have (X)ubuntu 13.04 32bit system, using mesa
9.2, as provided by the xorg-edgers ppa, using all appropriate updates as
passed downstream by the ppa. Using the FGLRX driver does fix the sound issue
along with providing a high level of hardware acceleration, but it does cripple
me being able to play on some emulators due to the well-known buggy nature of
the FGLRX driver. I already made a report on the askubuntu.com forums, but so
far there has been no response. My motherboard's native sound card is broken,
so currently I have my computer speakers plugged into the HD monitor, hence
HDMI sound is my only choice at this time. Here is a link to my askubuntu
question (so as to avoid duplication of effort with providing info).

http://askubuntu.com/questions/306196/no-sound-via-hdmi-using-xubuntu-13-04-32-bit-amd-mesa-9-2

Thanks in advance for any assistance.

--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130613/c0eb837f/attachment.html>

1 2 >

1 - 100 of 119 matches

Mail list logo