2013/12/12 Deucher, Alexander <Alexander.Deucher@xxxxxxx>: >> -----Original Message----- >> From: Rafał Miłecki [mailto:zajec5@xxxxxxxxx] >> 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 <alexdeucher@xxxxxxxxx>: >> > DCE6.0, 8.x has 6 >> > DCE6.1 has 4 >> > >> > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> >> > --- >> > 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ł _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
