Re: [PATCH v10 02/40] i915/snd_hdac: I915 subcomponent for the snd_hdac

Takashi Iwai
2019-02-04T15:09:04Z

On Mon, 04 Feb 2019 16:00:18 +0100,
Daniel Vetter wrote:
>
> On Thu, Jan 31, 2019 at 12:29:18PM +0530, Ramalingam C wrote:
> > From: Daniel Vetter <daniel.vetter@ffwll.ch>
> >
> > Since we need multiple components for I915 for different purposes
> > (Audio & Mei_hdcp), we adopt the subcomponents methodology introduced
> > by the previous patch (mentioned below).
> >
> > Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Date: Mon Jan 28 17:08:20 2019 +0530
> >
> > components: multiple components for a device
> >
> > Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
> > cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > cc: Russell King <rmk+kernel@arm.linux.org.uk>
> > cc: Rafael J. Wysocki <rafael@kernel.org>
> > cc: Jaroslav Kysela <perex@perex.cz>
> > cc: Takashi Iwai <tiwai@suse.com>
> > cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > cc: Jani Nikula <jani.nikula@linux.intel.com>
>
> Takashi, can you pls take a look and ack for merging through drm-intel? We
> can also do a topic branch in case this conflicts with something on the
> audio side.

I'm fine with the change as long as others agree with this API
extension.

Reviewed-by: Takashi Iwai <tiwai@suse.de>

thanks,

Takashi