> -----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. Alex _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel