[alsa-devel] [PATCH v6 4/4] drm/i915: set proper N/CTS in modeset
Jani Nikula
jani.nikula at linux.intel.com
Mon Oct 19 10:25:22 CEST 2015
On Wed, 02 Sep 2015, Takashi Iwai <tiwai at suse.de> wrote:
> On Wed, 02 Sep 2015 11:02:42 +0200,
> Jani Nikula wrote:
>>
>> >> Nitpick. I'd prefer some sharing with the similar blocks from the
>> >> earlier patch. Also a debug message on n == 0 would be nice; you
>> >> probably didn't notice your audio_config_get_rate() wasn't working
>> >> right
>> >> because this silently fell back to the automatic mode here.
>> >
>> > OK, I will add the msg. As you and Ville are insisting on
>> > sharing code, I will do it in next version.
>>
>> Well, really, I'm fine with having that part duplicated as-is for now,
>> we can fix it later. More important to focus on getting
>> audio_config_get_rate() right.
>>
>> I don't know if you're still targeting v4.3 with this (up to Takashi I
>> guess) we'll really need to wrap this up soon.
>
> I'm in favor of merging this into 4.3, so it'd be appreciated if Libin
> can prepare the fixed version soonish, indeed.
Takashi, just to double check, none of this made it into v4.3 after all?
BR,
Jani.
>
>
> thanks,
>
> Takashi
--
Jani Nikula, Intel Open Source Technology Center
More information about the Alsa-devel
mailing list