Hi all, Any comments? Thanks. Regards, Libin > -----Original Message----- > From: Yang, Libin > Sent: Thursday, October 01, 2015 5:01 PM > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; daniel.vetter@xxxxxxxx; > jani.nikula@xxxxxxxxxxxxxxx; ville.syrjala@xxxxxxxxxxxxxxx > Cc: Yang, Libin > Subject: [PATCH 3/3] drm/i915: add DOC for i915_component > > From: Libin Yang <libin.yang@xxxxxxxxx> > > Add the DOC for i915_component.h. Explain the struct > i915_audio_component_ops and struct > i915_audio_component_audio_ops > usage. > > Signed-off-by: Libin Yang <libin.yang@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_audio.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_audio.c > b/drivers/gpu/drm/i915/intel_audio.c > index 72d696b..56c2f54 100644 > --- a/drivers/gpu/drm/i915/intel_audio.c > +++ b/drivers/gpu/drm/i915/intel_audio.c > @@ -50,6 +50,11 @@ > * co-operation between the graphics and audio drivers is handled via > audio > * related registers. (The notable exception is the power management, > not > * covered here.) > + * > + * The struct i915_audio_component is used to interact between the > graphics > + * and audio drivers. The struct i915_audio_component_ops *ops in it > is > + * defined in graphics driver and called in audio driver. The > + * struct i915_audio_component_audio_ops *audio_ops is called > from i915 driver. > */ > > static const struct { > -- > 1.9.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx