* [PATCH] drm/i915: add kerneldoc for i915_audio_component @ 2015-09-02 6:12 libin.yang 2015-09-02 12:18 ` Daniel Vetter 0 siblings, 1 reply; 8+ messages in thread From: libin.yang @ 2015-09-02 6:12 UTC (permalink / raw) To: intel-gfx, daniel.vetter, jani.nikula, ville.syrjala From: Libin Yang <libin.yang@intel.com> Add the kerneldoc for i915_audio_component in i915_component.h Signed-off-by: Libin Yang <libin.yang@intel.com> --- include/drm/i915_component.h | 39 ++++++++++++++++++++++++--------------- 1 file changed, 24 insertions(+), 15 deletions(-) diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h index 8ad6f1b..187acc8 100644 --- a/include/drm/i915_component.h +++ b/include/drm/i915_component.h @@ -24,23 +24,32 @@ #ifndef _I915_COMPONENT_H_ #define _I915_COMPONENT_H_ +/** + * struct i915_audio_component_ops - callbacks defined in gfx driver + * @owner: the module owner + * @get_power: get the POWER_DOMAIN_AUDIO power well + * @put_power: put the POWER_DOMAIN_AUDIO power well + * @codec_wake_override: Enable/Disable generating the codec wake signal + * @get_cdclk_freq: get the Core Display Clock in KHz + * @sync_audio_rate: set n/cts based on the sample rate + */ +struct i915_audio_component_ops { + struct module *owner; + void (*get_power)(struct device *); + void (*put_power)(struct device *); + void (*codec_wake_override)(struct device *, bool enable); + int (*get_cdclk_freq)(struct device *); + int (*sync_audio_rate)(struct device *, int port, int rate); +}; + +/** + * struct i915_audio_component - used for audio video interaction + * @dev: the device from gfx driver + * @ops: callback for audio driver calling + */ struct i915_audio_component { struct device *dev; - - const struct i915_audio_component_ops { - struct module *owner; - void (*get_power)(struct device *); - void (*put_power)(struct device *); - void (*codec_wake_override)(struct device *, bool enable); - int (*get_cdclk_freq)(struct device *); - /** - * @sync_audio_rate: set n/cts based on the sample rate - * - * Called from audio driver. After audio driver sets the - * sample rate, it will call this function to set n/cts - */ - int (*sync_audio_rate)(struct device *, int port, int rate); - } *ops; + const struct i915_audio_component_ops *ops; }; #endif /* _I915_COMPONENT_H_ */ -- 1.9.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-02 6:12 [PATCH] drm/i915: add kerneldoc for i915_audio_component libin.yang @ 2015-09-02 12:18 ` Daniel Vetter 2015-09-04 1:52 ` Yang, Libin 0 siblings, 1 reply; 8+ messages in thread From: Daniel Vetter @ 2015-09-02 12:18 UTC (permalink / raw) To: libin.yang; +Cc: daniel.vetter, intel-gfx On Wed, Sep 02, 2015 at 02:12:24PM +0800, libin.yang@intel.com wrote: > From: Libin Yang <libin.yang@intel.com> > > Add the kerneldoc for i915_audio_component in i915_component.h > > Signed-off-by: Libin Yang <libin.yang@intel.com> > --- > include/drm/i915_component.h | 39 ++++++++++++++++++++++++--------------- > 1 file changed, 24 insertions(+), 15 deletions(-) > > diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h > index 8ad6f1b..187acc8 100644 > --- a/include/drm/i915_component.h > +++ b/include/drm/i915_component.h > @@ -24,23 +24,32 @@ > #ifndef _I915_COMPONENT_H_ > #define _I915_COMPONENT_H_ > > +/** > + * struct i915_audio_component_ops - callbacks defined in gfx driver > + * @owner: the module owner > + * @get_power: get the POWER_DOMAIN_AUDIO power well > + * @put_power: put the POWER_DOMAIN_AUDIO power well > + * @codec_wake_override: Enable/Disable generating the codec wake signal > + * @get_cdclk_freq: get the Core Display Clock in KHz > + * @sync_audio_rate: set n/cts based on the sample rate > + */ > +struct i915_audio_component_ops { > + struct module *owner; New kerneldoc in 4.3 allows you to split structure documentation up into per-member comments. Especially with vtables I think this makes a lot of sense, since then you have enough space to document where and how exactly a given hook is called (looks, place in the overall sequence). Also please include your stancas in the drm.tmpl docbook template, otherwise it won't be included in the html docs. And finally please add a DOC: overview section which explains at a high level how i915 and hda-intel corporate for hdmi/dp audio. Thanks, Daniel > + void (*get_power)(struct device *); > + void (*put_power)(struct device *); > + void (*codec_wake_override)(struct device *, bool enable); > + int (*get_cdclk_freq)(struct device *); > + int (*sync_audio_rate)(struct device *, int port, int rate); > +}; > + > +/** > + * struct i915_audio_component - used for audio video interaction > + * @dev: the device from gfx driver > + * @ops: callback for audio driver calling > + */ > struct i915_audio_component { > struct device *dev; > - > - const struct i915_audio_component_ops { > - struct module *owner; > - void (*get_power)(struct device *); > - void (*put_power)(struct device *); > - void (*codec_wake_override)(struct device *, bool enable); > - int (*get_cdclk_freq)(struct device *); > - /** > - * @sync_audio_rate: set n/cts based on the sample rate > - * > - * Called from audio driver. After audio driver sets the > - * sample rate, it will call this function to set n/cts > - */ > - int (*sync_audio_rate)(struct device *, int port, int rate); > - } *ops; > + const struct i915_audio_component_ops *ops; > }; > > #endif /* _I915_COMPONENT_H_ */ > -- > 1.9.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-02 12:18 ` Daniel Vetter @ 2015-09-04 1:52 ` Yang, Libin 2015-09-04 6:40 ` Jani Nikula 0 siblings, 1 reply; 8+ messages in thread From: Yang, Libin @ 2015-09-04 1:52 UTC (permalink / raw) To: Daniel Vetter; +Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org > -----Original Message----- > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > Daniel Vetter > Sent: Wednesday, September 02, 2015 8:18 PM > To: Yang, Libin > Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; > jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com > Subject: Re: [PATCH] drm/i915: add kerneldoc for > i915_audio_component > > On Wed, Sep 02, 2015 at 02:12:24PM +0800, libin.yang@intel.com > wrote: > > From: Libin Yang <libin.yang@intel.com> > > > > Add the kerneldoc for i915_audio_component in i915_component.h > > > > Signed-off-by: Libin Yang <libin.yang@intel.com> > > --- > > include/drm/i915_component.h | 39 ++++++++++++++++++++++++-- > ------------- > > 1 file changed, 24 insertions(+), 15 deletions(-) > > > > diff --git a/include/drm/i915_component.h > b/include/drm/i915_component.h > > index 8ad6f1b..187acc8 100644 > > --- a/include/drm/i915_component.h > > +++ b/include/drm/i915_component.h > > @@ -24,23 +24,32 @@ > > #ifndef _I915_COMPONENT_H_ > > #define _I915_COMPONENT_H_ > > > > +/** > > + * struct i915_audio_component_ops - callbacks defined in gfx > driver > > + * @owner: the module owner > > + * @get_power: get the POWER_DOMAIN_AUDIO power well > > + * @put_power: put the POWER_DOMAIN_AUDIO power well > > + * @codec_wake_override: Enable/Disable generating the codec > wake signal > > + * @get_cdclk_freq: get the Core Display Clock in KHz > > + * @sync_audio_rate: set n/cts based on the sample rate > > + */ > > +struct i915_audio_component_ops { > > + struct module *owner; > > New kerneldoc in 4.3 allows you to split structure documentation up > into > per-member comments. Especially with vtables I think this makes a lot > of > sense, since then you have enough space to document where and how > exactly > a given hook is called (looks, place in the overall sequence). > > Also please include your stancas in the drm.tmpl docbook template, > otherwise it won't be included in the html docs. And finally please add OK, I will add it in drm.tmpl. > a > DOC: overview section which explains at a high level how i915 and > hda-intel corporate for hdmi/dp audio. Where the DOC should be located in? > > Thanks, Daniel > > > + void (*get_power)(struct device *); > > + void (*put_power)(struct device *); > > + void (*codec_wake_override)(struct device *, bool enable); > > + int (*get_cdclk_freq)(struct device *); > > + int (*sync_audio_rate)(struct device *, int port, int rate); > > +}; > > + > > +/** > > + * struct i915_audio_component - used for audio video interaction > > + * @dev: the device from gfx driver > > + * @ops: callback for audio driver calling > > + */ > > struct i915_audio_component { > > struct device *dev; > > - > > - const struct i915_audio_component_ops { > > - struct module *owner; > > - void (*get_power)(struct device *); > > - void (*put_power)(struct device *); > > - void (*codec_wake_override)(struct device *, bool > enable); > > - int (*get_cdclk_freq)(struct device *); > > - /** > > - * @sync_audio_rate: set n/cts based on the sample > rate > > - * > > - * Called from audio driver. After audio driver sets the > > - * sample rate, it will call this function to set n/cts > > - */ > > - int (*sync_audio_rate)(struct device *, int port, int > rate); > > - } *ops; > > + const struct i915_audio_component_ops *ops; > > }; > > > > #endif /* _I915_COMPONENT_H_ */ > > -- > > 1.9.1 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-04 1:52 ` Yang, Libin @ 2015-09-04 6:40 ` Jani Nikula 2015-09-04 7:54 ` Daniel Vetter 0 siblings, 1 reply; 8+ messages in thread From: Jani Nikula @ 2015-09-04 6:40 UTC (permalink / raw) To: Yang, Libin, Daniel Vetter Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org On Fri, 04 Sep 2015, "Yang, Libin" <libin.yang@intel.com> wrote: >> -----Original Message----- >> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of >> Daniel Vetter >> Sent: Wednesday, September 02, 2015 8:18 PM >> To: Yang, Libin >> Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; >> jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com >> Subject: Re: [PATCH] drm/i915: add kerneldoc for >> i915_audio_component >> >> On Wed, Sep 02, 2015 at 02:12:24PM +0800, libin.yang@intel.com >> wrote: >> > From: Libin Yang <libin.yang@intel.com> >> > >> > Add the kerneldoc for i915_audio_component in i915_component.h >> > >> > Signed-off-by: Libin Yang <libin.yang@intel.com> >> > --- >> > include/drm/i915_component.h | 39 ++++++++++++++++++++++++-- >> ------------- >> > 1 file changed, 24 insertions(+), 15 deletions(-) >> > >> > diff --git a/include/drm/i915_component.h >> b/include/drm/i915_component.h >> > index 8ad6f1b..187acc8 100644 >> > --- a/include/drm/i915_component.h >> > +++ b/include/drm/i915_component.h >> > @@ -24,23 +24,32 @@ >> > #ifndef _I915_COMPONENT_H_ >> > #define _I915_COMPONENT_H_ >> > >> > +/** >> > + * struct i915_audio_component_ops - callbacks defined in gfx >> driver >> > + * @owner: the module owner >> > + * @get_power: get the POWER_DOMAIN_AUDIO power well >> > + * @put_power: put the POWER_DOMAIN_AUDIO power well >> > + * @codec_wake_override: Enable/Disable generating the codec >> wake signal >> > + * @get_cdclk_freq: get the Core Display Clock in KHz >> > + * @sync_audio_rate: set n/cts based on the sample rate >> > + */ >> > +struct i915_audio_component_ops { >> > + struct module *owner; >> >> New kerneldoc in 4.3 allows you to split structure documentation up >> into >> per-member comments. Especially with vtables I think this makes a lot >> of >> sense, since then you have enough space to document where and how >> exactly >> a given hook is called (looks, place in the overall sequence). >> >> Also please include your stancas in the drm.tmpl docbook template, >> otherwise it won't be included in the html docs. And finally please add > > OK, I will add it in drm.tmpl. > >> a >> DOC: overview section which explains at a high level how i915 and >> hda-intel corporate for hdmi/dp audio. > > Where the DOC should be located in? i915/intel_audio.c already has a "DOC: High Definition Audio over HDMI and Display Port" comment. IMO you could just amend that, as there's already some references to the audio driver. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-04 6:40 ` Jani Nikula @ 2015-09-04 7:54 ` Daniel Vetter 2015-09-09 1:45 ` Yang, Libin 0 siblings, 1 reply; 8+ messages in thread From: Daniel Vetter @ 2015-09-04 7:54 UTC (permalink / raw) To: Jani Nikula; +Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org On Fri, Sep 04, 2015 at 09:40:17AM +0300, Jani Nikula wrote: > On Fri, 04 Sep 2015, "Yang, Libin" <libin.yang@intel.com> wrote: > >> -----Original Message----- > >> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > >> Daniel Vetter > >> Sent: Wednesday, September 02, 2015 8:18 PM > >> To: Yang, Libin > >> Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; > >> jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com > >> Subject: Re: [PATCH] drm/i915: add kerneldoc for > >> i915_audio_component > >> > >> On Wed, Sep 02, 2015 at 02:12:24PM +0800, libin.yang@intel.com > >> wrote: > >> > From: Libin Yang <libin.yang@intel.com> > >> > > >> > Add the kerneldoc for i915_audio_component in i915_component.h > >> > > >> > Signed-off-by: Libin Yang <libin.yang@intel.com> > >> > --- > >> > include/drm/i915_component.h | 39 ++++++++++++++++++++++++-- > >> ------------- > >> > 1 file changed, 24 insertions(+), 15 deletions(-) > >> > > >> > diff --git a/include/drm/i915_component.h > >> b/include/drm/i915_component.h > >> > index 8ad6f1b..187acc8 100644 > >> > --- a/include/drm/i915_component.h > >> > +++ b/include/drm/i915_component.h > >> > @@ -24,23 +24,32 @@ > >> > #ifndef _I915_COMPONENT_H_ > >> > #define _I915_COMPONENT_H_ > >> > > >> > +/** > >> > + * struct i915_audio_component_ops - callbacks defined in gfx > >> driver > >> > + * @owner: the module owner > >> > + * @get_power: get the POWER_DOMAIN_AUDIO power well > >> > + * @put_power: put the POWER_DOMAIN_AUDIO power well > >> > + * @codec_wake_override: Enable/Disable generating the codec > >> wake signal > >> > + * @get_cdclk_freq: get the Core Display Clock in KHz > >> > + * @sync_audio_rate: set n/cts based on the sample rate > >> > + */ > >> > +struct i915_audio_component_ops { > >> > + struct module *owner; > >> > >> New kerneldoc in 4.3 allows you to split structure documentation up > >> into > >> per-member comments. Especially with vtables I think this makes a lot > >> of > >> sense, since then you have enough space to document where and how > >> exactly > >> a given hook is called (looks, place in the overall sequence). > >> > >> Also please include your stancas in the drm.tmpl docbook template, > >> otherwise it won't be included in the html docs. And finally please add > > > > OK, I will add it in drm.tmpl. > > > >> a > >> DOC: overview section which explains at a high level how i915 and > >> hda-intel corporate for hdmi/dp audio. > > > > Where the DOC should be located in? > > i915/intel_audio.c already has a "DOC: High Definition Audio over HDMI > and Display Port" comment. IMO you could just amend that, as there's > already some references to the audio driver. Yeah I think that would be a perfect place. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-04 7:54 ` Daniel Vetter @ 2015-09-09 1:45 ` Yang, Libin 2015-09-09 15:09 ` Daniel Vetter 0 siblings, 1 reply; 8+ messages in thread From: Yang, Libin @ 2015-09-09 1:45 UTC (permalink / raw) To: Daniel Vetter, Jani Nikula Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org Hi Daniel, As Takashi has already accepted the first 3 patches for sync_audio_rate() and the patches are not merged into -nightly branch. If I make a kerneldoc patch based on currently -nightly branch, there will be conflict when you are merging Takashi's branch. What do you think if I make the kerneldoc patch after the sync_audio_rate() patches are merged into the -nightly branch? Regards, Libin > -----Original Message----- > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > Daniel Vetter > Sent: Friday, September 04, 2015 3:55 PM > To: Jani Nikula > Cc: Yang, Libin; Daniel Vetter; intel-gfx@lists.freedesktop.org; > daniel.vetter@ffwll.ch; ville.syrjala@linux.intel.com > Subject: Re: [PATCH] drm/i915: add kerneldoc for > i915_audio_component > > On Fri, Sep 04, 2015 at 09:40:17AM +0300, Jani Nikula wrote: > > On Fri, 04 Sep 2015, "Yang, Libin" <libin.yang@intel.com> wrote: > > >> -----Original Message----- > > >> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > > >> Daniel Vetter > > >> Sent: Wednesday, September 02, 2015 8:18 PM > > >> To: Yang, Libin > > >> Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; > > >> jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com > > >> Subject: Re: [PATCH] drm/i915: add kerneldoc for > > >> i915_audio_component > > >> > > >> On Wed, Sep 02, 2015 at 02:12:24PM +0800, > libin.yang@intel.com > > >> wrote: > > >> > From: Libin Yang <libin.yang@intel.com> > > >> > > > >> > Add the kerneldoc for i915_audio_component in > i915_component.h > > >> > > > >> > Signed-off-by: Libin Yang <libin.yang@intel.com> > > >> > --- > > >> > include/drm/i915_component.h | 39 > ++++++++++++++++++++++++-- > > >> ------------- > > >> > 1 file changed, 24 insertions(+), 15 deletions(-) > > >> > > > >> > diff --git a/include/drm/i915_component.h > > >> b/include/drm/i915_component.h > > >> > index 8ad6f1b..187acc8 100644 > > >> > --- a/include/drm/i915_component.h > > >> > +++ b/include/drm/i915_component.h > > >> > @@ -24,23 +24,32 @@ > > >> > #ifndef _I915_COMPONENT_H_ > > >> > #define _I915_COMPONENT_H_ > > >> > > > >> > +/** > > >> > + * struct i915_audio_component_ops - callbacks defined in gfx > > >> driver > > >> > + * @owner: the module owner > > >> > + * @get_power: get the POWER_DOMAIN_AUDIO power well > > >> > + * @put_power: put the POWER_DOMAIN_AUDIO power well > > >> > + * @codec_wake_override: Enable/Disable generating the > codec > > >> wake signal > > >> > + * @get_cdclk_freq: get the Core Display Clock in KHz > > >> > + * @sync_audio_rate: set n/cts based on the sample rate > > >> > + */ > > >> > +struct i915_audio_component_ops { > > >> > + struct module *owner; > > >> > > >> New kerneldoc in 4.3 allows you to split structure documentation > up > > >> into > > >> per-member comments. Especially with vtables I think this makes > a lot > > >> of > > >> sense, since then you have enough space to document where and > how > > >> exactly > > >> a given hook is called (looks, place in the overall sequence). > > >> > > >> Also please include your stancas in the drm.tmpl docbook > template, > > >> otherwise it won't be included in the html docs. And finally please > add > > > > > > OK, I will add it in drm.tmpl. > > > > > >> a > > >> DOC: overview section which explains at a high level how i915 and > > >> hda-intel corporate for hdmi/dp audio. > > > > > > Where the DOC should be located in? > > > > i915/intel_audio.c already has a "DOC: High Definition Audio over > HDMI > > and Display Port" comment. IMO you could just amend that, as > there's > > already some references to the audio driver. > > Yeah I think that would be a perfect place. > -Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-09 1:45 ` Yang, Libin @ 2015-09-09 15:09 ` Daniel Vetter 2015-09-09 15:10 ` Takashi Iwai 0 siblings, 1 reply; 8+ messages in thread From: Daniel Vetter @ 2015-09-09 15:09 UTC (permalink / raw) To: Yang, Libin Cc: Takashi Iwai, daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org On Wed, Sep 09, 2015 at 01:45:47AM +0000, Yang, Libin wrote: > Hi Daniel, > > As Takashi has already accepted the first 3 patches for > sync_audio_rate() and the patches are not merged > into -nightly branch. If I make a kerneldoc patch > based on currently -nightly branch, there will > be conflict when you are merging Takashi's branch. > > What do you think if I make the kerneldoc patch > after the sync_audio_rate() patches are merged > into the -nightly branch? Hm, I do pull in sound/for-next into drm-intel-nightly ... Are the patches somewhere else? I can easily add another branch to nightly to make things easier. Takashi, which branches should I all pull in to get all the audio stuff into drm-intel-nightly? Thanks, Daniel > > Regards, > Libin > > > > -----Original Message----- > > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > > Daniel Vetter > > Sent: Friday, September 04, 2015 3:55 PM > > To: Jani Nikula > > Cc: Yang, Libin; Daniel Vetter; intel-gfx@lists.freedesktop.org; > > daniel.vetter@ffwll.ch; ville.syrjala@linux.intel.com > > Subject: Re: [PATCH] drm/i915: add kerneldoc for > > i915_audio_component > > > > On Fri, Sep 04, 2015 at 09:40:17AM +0300, Jani Nikula wrote: > > > On Fri, 04 Sep 2015, "Yang, Libin" <libin.yang@intel.com> wrote: > > > >> -----Original Message----- > > > >> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > > > >> Daniel Vetter > > > >> Sent: Wednesday, September 02, 2015 8:18 PM > > > >> To: Yang, Libin > > > >> Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; > > > >> jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com > > > >> Subject: Re: [PATCH] drm/i915: add kerneldoc for > > > >> i915_audio_component > > > >> > > > >> On Wed, Sep 02, 2015 at 02:12:24PM +0800, > > libin.yang@intel.com > > > >> wrote: > > > >> > From: Libin Yang <libin.yang@intel.com> > > > >> > > > > >> > Add the kerneldoc for i915_audio_component in > > i915_component.h > > > >> > > > > >> > Signed-off-by: Libin Yang <libin.yang@intel.com> > > > >> > --- > > > >> > include/drm/i915_component.h | 39 > > ++++++++++++++++++++++++-- > > > >> ------------- > > > >> > 1 file changed, 24 insertions(+), 15 deletions(-) > > > >> > > > > >> > diff --git a/include/drm/i915_component.h > > > >> b/include/drm/i915_component.h > > > >> > index 8ad6f1b..187acc8 100644 > > > >> > --- a/include/drm/i915_component.h > > > >> > +++ b/include/drm/i915_component.h > > > >> > @@ -24,23 +24,32 @@ > > > >> > #ifndef _I915_COMPONENT_H_ > > > >> > #define _I915_COMPONENT_H_ > > > >> > > > > >> > +/** > > > >> > + * struct i915_audio_component_ops - callbacks defined in gfx > > > >> driver > > > >> > + * @owner: the module owner > > > >> > + * @get_power: get the POWER_DOMAIN_AUDIO power well > > > >> > + * @put_power: put the POWER_DOMAIN_AUDIO power well > > > >> > + * @codec_wake_override: Enable/Disable generating the > > codec > > > >> wake signal > > > >> > + * @get_cdclk_freq: get the Core Display Clock in KHz > > > >> > + * @sync_audio_rate: set n/cts based on the sample rate > > > >> > + */ > > > >> > +struct i915_audio_component_ops { > > > >> > + struct module *owner; > > > >> > > > >> New kerneldoc in 4.3 allows you to split structure documentation > > up > > > >> into > > > >> per-member comments. Especially with vtables I think this makes > > a lot > > > >> of > > > >> sense, since then you have enough space to document where and > > how > > > >> exactly > > > >> a given hook is called (looks, place in the overall sequence). > > > >> > > > >> Also please include your stancas in the drm.tmpl docbook > > template, > > > >> otherwise it won't be included in the html docs. And finally please > > add > > > > > > > > OK, I will add it in drm.tmpl. > > > > > > > >> a > > > >> DOC: overview section which explains at a high level how i915 and > > > >> hda-intel corporate for hdmi/dp audio. > > > > > > > > Where the DOC should be located in? > > > > > > i915/intel_audio.c already has a "DOC: High Definition Audio over > > HDMI > > > and Display Port" comment. IMO you could just amend that, as > > there's > > > already some references to the audio driver. > > > > Yeah I think that would be a perfect place. > > -Daniel > > -- > > Daniel Vetter > > Software Engineer, Intel Corporation > > http://blog.ffwll.ch -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: add kerneldoc for i915_audio_component 2015-09-09 15:09 ` Daniel Vetter @ 2015-09-09 15:10 ` Takashi Iwai 0 siblings, 0 replies; 8+ messages in thread From: Takashi Iwai @ 2015-09-09 15:10 UTC (permalink / raw) To: Daniel Vetter Cc: Takashi Iwai, daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org On Wed, 09 Sep 2015 17:09:52 +0200, Daniel Vetter wrote: > > On Wed, Sep 09, 2015 at 01:45:47AM +0000, Yang, Libin wrote: > > Hi Daniel, > > > > As Takashi has already accepted the first 3 patches for > > sync_audio_rate() and the patches are not merged > > into -nightly branch. If I make a kerneldoc patch > > based on currently -nightly branch, there will > > be conflict when you are merging Takashi's branch. > > > > What do you think if I make the kerneldoc patch > > after the sync_audio_rate() patches are merged > > into the -nightly branch? > > Hm, I do pull in sound/for-next into drm-intel-nightly ... Are the patches > somewhere else? I can easily add another branch to nightly to make things > easier. Takashi, which branches should I all pull in to get all the audio > stuff into drm-intel-nightly? No, it's not merged into for-next branch because the 4th patch is missing, and I didn't get your proper ack yet. The three patches are left in topic/drm-sync-audio-rate branch. If these are OK for you, I'll add Acked-by to them before you pulling it (if any). But, still I'm not sure whether we should really merge these partial changes to 4.3. Will the last patch be ready in time? If yes, we can go forward now. thanks, Takashi > > Thanks, Daniel > > > > > Regards, > > Libin > > > > > > > -----Original Message----- > > > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > > > Daniel Vetter > > > Sent: Friday, September 04, 2015 3:55 PM > > > To: Jani Nikula > > > Cc: Yang, Libin; Daniel Vetter; intel-gfx@lists.freedesktop.org; > > > daniel.vetter@ffwll.ch; ville.syrjala@linux.intel.com > > > Subject: Re: [PATCH] drm/i915: add kerneldoc for > > > i915_audio_component > > > > > > On Fri, Sep 04, 2015 at 09:40:17AM +0300, Jani Nikula wrote: > > > > On Fri, 04 Sep 2015, "Yang, Libin" <libin.yang@intel.com> wrote: > > > > >> -----Original Message----- > > > > >> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of > > > > >> Daniel Vetter > > > > >> Sent: Wednesday, September 02, 2015 8:18 PM > > > > >> To: Yang, Libin > > > > >> Cc: intel-gfx@lists.freedesktop.org; daniel.vetter@ffwll.ch; > > > > >> jani.nikula@linux.intel.com; ville.syrjala@linux.intel.com > > > > >> Subject: Re: [PATCH] drm/i915: add kerneldoc for > > > > >> i915_audio_component > > > > >> > > > > >> On Wed, Sep 02, 2015 at 02:12:24PM +0800, > > > libin.yang@intel.com > > > > >> wrote: > > > > >> > From: Libin Yang <libin.yang@intel.com> > > > > >> > > > > > >> > Add the kerneldoc for i915_audio_component in > > > i915_component.h > > > > >> > > > > > >> > Signed-off-by: Libin Yang <libin.yang@intel.com> > > > > >> > --- > > > > >> > include/drm/i915_component.h | 39 > > > ++++++++++++++++++++++++-- > > > > >> ------------- > > > > >> > 1 file changed, 24 insertions(+), 15 deletions(-) > > > > >> > > > > > >> > diff --git a/include/drm/i915_component.h > > > > >> b/include/drm/i915_component.h > > > > >> > index 8ad6f1b..187acc8 100644 > > > > >> > --- a/include/drm/i915_component.h > > > > >> > +++ b/include/drm/i915_component.h > > > > >> > @@ -24,23 +24,32 @@ > > > > >> > #ifndef _I915_COMPONENT_H_ > > > > >> > #define _I915_COMPONENT_H_ > > > > >> > > > > > >> > +/** > > > > >> > + * struct i915_audio_component_ops - callbacks defined in gfx > > > > >> driver > > > > >> > + * @owner: the module owner > > > > >> > + * @get_power: get the POWER_DOMAIN_AUDIO power well > > > > >> > + * @put_power: put the POWER_DOMAIN_AUDIO power well > > > > >> > + * @codec_wake_override: Enable/Disable generating the > > > codec > > > > >> wake signal > > > > >> > + * @get_cdclk_freq: get the Core Display Clock in KHz > > > > >> > + * @sync_audio_rate: set n/cts based on the sample rate > > > > >> > + */ > > > > >> > +struct i915_audio_component_ops { > > > > >> > + struct module *owner; > > > > >> > > > > >> New kerneldoc in 4.3 allows you to split structure documentation > > > up > > > > >> into > > > > >> per-member comments. Especially with vtables I think this makes > > > a lot > > > > >> of > > > > >> sense, since then you have enough space to document where and > > > how > > > > >> exactly > > > > >> a given hook is called (looks, place in the overall sequence). > > > > >> > > > > >> Also please include your stancas in the drm.tmpl docbook > > > template, > > > > >> otherwise it won't be included in the html docs. And finally please > > > add > > > > > > > > > > OK, I will add it in drm.tmpl. > > > > > > > > > >> a > > > > >> DOC: overview section which explains at a high level how i915 and > > > > >> hda-intel corporate for hdmi/dp audio. > > > > > > > > > > Where the DOC should be located in? > > > > > > > > i915/intel_audio.c already has a "DOC: High Definition Audio over > > > HDMI > > > > and Display Port" comment. IMO you could just amend that, as > > > there's > > > > already some references to the audio driver. > > > > > > Yeah I think that would be a perfect place. > > > -Daniel > > > -- > > > Daniel Vetter > > > Software Engineer, Intel Corporation > > > http://blog.ffwll.ch > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-09-09 15:10 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-09-02 6:12 [PATCH] drm/i915: add kerneldoc for i915_audio_component libin.yang 2015-09-02 12:18 ` Daniel Vetter 2015-09-04 1:52 ` Yang, Libin 2015-09-04 6:40 ` Jani Nikula 2015-09-04 7:54 ` Daniel Vetter 2015-09-09 1:45 ` Yang, Libin 2015-09-09 15:09 ` Daniel Vetter 2015-09-09 15:10 ` Takashi Iwai
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox