[PATCH] drm/radeon/dce6: set correct number of audio pins
2013/12/12 Deucher, Alexander : >> -Original Message- >> From: Rafa? Mi?ecki [mailto:zajec5 at gmail.com] >> Sent: Thursday, December 12, 2013 1:10 PM >> To: Alex Deucher >> Cc: dri-devel; Deucher, Alexander >> Subject: Re: [PATCH] drm/radeon/dce6: set correct number of audio pins >> >> 2013/12/12 Alex Deucher : >> > DCE6.0, 8.x has 6 >> > DCE6.1 has 4 >> > >> > Signed-off-by: Alex Deucher >> > --- >> > drivers/gpu/drm/radeon/dce6_afmt.c | 4 +++- >> > 1 file changed, 3 insertions(+), 1 deletion(-) >> > >> > diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c >> b/drivers/gpu/drm/radeon/dce6_afmt.c >> > index de86493..ab59fd7 100644 >> > --- a/drivers/gpu/drm/radeon/dce6_afmt.c >> > +++ b/drivers/gpu/drm/radeon/dce6_afmt.c >> > @@ -308,7 +308,9 @@ int dce6_audio_init(struct radeon_device *rdev) >> > rdev->audio.enabled = true; >> > >> > if (ASIC_IS_DCE8(rdev)) >> > - rdev->audio.num_pins = 7; >> > + rdev->audio.num_pins = 6; >> > + else if (ASIC_IS_DCE61(rdev)) >> > + rdev->audio.num_pins = 4; >> > else >> > rdev->audio.num_pins = 6; >> >> You don't check for DCE64, so OLAND will be treated as DCE6.0 (since >> there isn't IGP Oland, so it won't be treated as 6.1). >> >> Does 6.4 also have 6 pins? > > Yes. Oland has the same audio chip as the other SI parts. Great. Thanks for the patch! -- Rafa?
[PATCH] drm/radeon/dce6: set correct number of audio pins
2013/12/12 Alex Deucher : > DCE6.0, 8.x has 6 > DCE6.1 has 4 > > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/radeon/dce6_afmt.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c > b/drivers/gpu/drm/radeon/dce6_afmt.c > index de86493..ab59fd7 100644 > --- a/drivers/gpu/drm/radeon/dce6_afmt.c > +++ b/drivers/gpu/drm/radeon/dce6_afmt.c > @@ -308,7 +308,9 @@ int dce6_audio_init(struct radeon_device *rdev) > rdev->audio.enabled = true; > > if (ASIC_IS_DCE8(rdev)) > - rdev->audio.num_pins = 7; > + rdev->audio.num_pins = 6; > + else if (ASIC_IS_DCE61(rdev)) > + rdev->audio.num_pins = 4; > else > rdev->audio.num_pins = 6; You don't check for DCE64, so OLAND will be treated as DCE6.0 (since there isn't IGP Oland, so it won't be treated as 6.1). Does 6.4 also have 6 pins?
[PATCH] drm/radeon/dce6: set correct number of audio pins
> -Original Message- > From: Rafa? Mi?ecki [mailto:zajec5 at gmail.com] > Sent: Thursday, December 12, 2013 1:10 PM > To: Alex Deucher > Cc: dri-devel; Deucher, Alexander > Subject: Re: [PATCH] drm/radeon/dce6: set correct number of audio pins > > 2013/12/12 Alex Deucher : > > DCE6.0, 8.x has 6 > > DCE6.1 has 4 > > > > Signed-off-by: Alex Deucher > > --- > > drivers/gpu/drm/radeon/dce6_afmt.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c > b/drivers/gpu/drm/radeon/dce6_afmt.c > > index de86493..ab59fd7 100644 > > --- a/drivers/gpu/drm/radeon/dce6_afmt.c > > +++ b/drivers/gpu/drm/radeon/dce6_afmt.c > > @@ -308,7 +308,9 @@ int dce6_audio_init(struct radeon_device *rdev) > > rdev->audio.enabled = true; > > > > if (ASIC_IS_DCE8(rdev)) > > - rdev->audio.num_pins = 7; > > + rdev->audio.num_pins = 6; > > + else if (ASIC_IS_DCE61(rdev)) > > + rdev->audio.num_pins = 4; > > else > > rdev->audio.num_pins = 6; > > You don't check for DCE64, so OLAND will be treated as DCE6.0 (since > there isn't IGP Oland, so it won't be treated as 6.1). > > Does 6.4 also have 6 pins? Yes. Oland has the same audio chip as the other SI parts. Alex
[PATCH] drm/radeon/dce6: set correct number of audio pins
DCE6.0, 8.x has 6 DCE6.1 has 4 Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/dce6_afmt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c b/drivers/gpu/drm/radeon/dce6_afmt.c index de86493..ab59fd7 100644 --- a/drivers/gpu/drm/radeon/dce6_afmt.c +++ b/drivers/gpu/drm/radeon/dce6_afmt.c @@ -308,7 +308,9 @@ int dce6_audio_init(struct radeon_device *rdev) rdev->audio.enabled = true; if (ASIC_IS_DCE8(rdev)) - rdev->audio.num_pins = 7; + rdev->audio.num_pins = 6; + else if (ASIC_IS_DCE61(rdev)) + rdev->audio.num_pins = 4; else rdev->audio.num_pins = 6; -- 1.8.3.1