[alsa-devel] [PATCH v2 0/3] Make the audio component binding more generic
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Thu Jul 19 15:05:45 CEST 2018
On 7/19/18 12:50 AM, Takashi Iwai wrote:
> On Wed, 18 Jul 2018 22:54:35 +0200,
> Pierre-Louis Bossart wrote:
>>
>>
>>
>> On 07/17/2018 04:26 AM, Takashi Iwai wrote:
>>> Hi,
>>>
>>> this is a preliminiary patch set to convert the existing i915 /
>>> HD-audio component binding to be applicable to other drivers like
>>> radeon / amdgpu. This patchset itself doesn't change the
>>> functionality but only renames and split to a new drm_audio_component
>>> stuff from i915_audio_component.
>>>
>>> The actual usage of the new API will follow once after this one gets
>>> reviewed / accepted. The whole patches (including this patchset) are
>>> found in topic/hda-acomp branch of sound.git tree.
>>>
>>> BTW, since the whole stuff is about the audio binding, I suppose these
>>> will go through sound git tree. Let me know if anyone has concerns.
>> No objections but a slight concern that this will conflict with the
>> HDAudio+DSP patches that I was about to resubmit on top of your
>> topic/hda-core-intel branch. the two series touch the same files so
>> it'd be a miracle if there is no issue.
>> How do you want to deal with this?
>
> Does it conflict severely? If it's trivial, it can be resolved at
> merge time, too. The changes in my patchset are fairly trivial, so it
> shouldn't be too hard.
I was able to make things work by taking your topic/hda-core-intel,
merge it on Mark's for-next, then add my additional changes and these
DRM changes. The last two can be done in any order. But I am getting
some conflicts if I try to apply these DRM changes first, not sure why
git is complaining though, the changes look trivial enough.
So yes it looks possible to deal with the two series in parallel, will
send my update later today.
Thanks
-Pierre
More information about the Alsa-devel
mailing list