[alsa-devel] [PATCH v2] drm/i915: set proper N/CTS in modeset
Jani Nikula
jani.nikula at linux.intel.com
Tue Oct 13 16:35:31 CEST 2015
On Tue, 13 Oct 2015, "Yang, Libin" <libin.yang at intel.com> wrote:
> + gfx driver team.
>
> Hi David,
>
>> -----Original Message-----
>> From: David Henningsson [mailto:david.henningsson at canonical.com]
>> Sent: Tuesday, October 13, 2015 9:43 PM
>> To: alsa-devel at alsa-project.org; Yang, Libin
>> Subject: Re: [alsa-devel] [PATCH v2] drm/i915: set proper N/CTS in
>> modeset
>>
>>
>>
>> On 2015-09-25 03:36, libin.yang at intel.com wrote:
>> > @@ -24,8 +24,18 @@
>> > #ifndef _I915_COMPONENT_H_
>> > #define _I915_COMPONENT_H_
>> >
>> > +/* MAX_PORT is the number of port
>> > + * It must be sync with I915_MAX_PORTS defined i915_drv.h
>> > + * 5 should be enough as only HSW, BDW, SKL need such fix.
>> > + */
>> > +#define MAX_PORTS 5
>> > +
>> > struct i915_audio_component {
>> > struct device *dev;
>> > + /**
>> > + * @aud_sample_rate: the array of audio sample rate per port
>> > + */
>> > + int aud_sample_rate[MAX_PORTS];
>>
>> Just a quick question as I was trying to understand the code:
>>
>> aud_sample_rate seems only set and get on the i915 side, never on the
>> hda side.
>>
>> In short, why is this variable here, and not in e g, as a single integer
>> in the intel_digital_port struct?
>>
>> That way we also avoid the possibility of MAX_PORTS becoming out of
>> sync
>> with the i915 side.
>
> Put it in struct i915_audio_component just because it is audio related and
> easy for management.
>
> It seems to put it in intel_digital_port will be easier.
>
> Let's hear the gfx team's comments.
Ack on moving it into struct intel_digital_port.
We had a bit of a hurry to get the original patches in, didn't want to
start bikeshedding at each step of the way...
BR,
Jani.
>
> Regards,
> Libin
>
>>
>> --
>> David Henningsson, Canonical Ltd.
>> https://launchpad.net/~diwic
--
Jani Nikula, Intel Open Source Technology Center
More information about the Alsa-devel
mailing list