8 Apr
2019
8 Apr
'19
5:43 p.m.
- if this is really a generic issue then shouldn't it be fixed for all users of the
iDISP link? Why stop at the HDAudio machine driver?
I will submit other patches for other machine drivers. Which HDAudio machine driver do you mean?
if you plan on updating other drivers, then by all means let's use a common set of helper functions. I see you have follow-up patches on github already, let's avoid copy/paste if possible if the same functionality is needed in multiple places.
- we already have the component model to deal with interaction between
i915 and audio, now we are adding a second layer. That looks clunky.
The component model between i915 and audio is used to communicate between display driver and audio driver. This patch is used to setup the consumer and supplier relationship.
consumer of what? it's very vague for people who haven't really looked into the details.