[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 13:05:45 UTC 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 dri-devel mailing list