On Thu, 23 Jul 2015 08:25:21 +0200,
David Henningsson wrote:
>
>
>
> On 2015-07-23 08:17, David Henningsson wrote:
> > I'm about to go on vacation so it would be good to get some closure
> > here. If you both prefer this setup, how about I remove "struct
> > i915_audio_hotplug_info" for now? We will then have:
> >
> > const struct i915_audio_component_audio_ops {
> > void (*hotplug_notify)(struct hdac_bus *);
> > }
>
> Sorry, it would look like this:
>
> const struct i915_audio_component_audio_ops {
> void (*hotplug_notify)(struct hdac_bus *, int port, int
> port_mst_index);
> }
>
> ...to indicate what port needs updating.
Yes, I think this is simpler.
A remaining question is whether it should be notified to bus or
codec. In the latter case, we need to allow registering the audio
codec after binding the component.
I find the codec can be a bit better, as this is directly targeted
(while for bus you need to look through the codec list). But it's
just a minor difference, and I don't mind so much about this, if there
is any other difficulty by that move.
thanks,
Takashi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
